Skip to content

Conversation

rlcamp
Copy link

@rlcamp rlcamp commented Jan 7, 2020

Proposed solution to #201

@ladyada
Copy link
Member

ladyada commented Jan 7, 2020

aweseomme.. we have a super big favor to ask - could you add an example that 'tests' that the math library is linking? that way when the CMSIS path changes in the future we have a test that will catch it

@rlcamp
Copy link
Author

rlcamp commented Jan 7, 2020

The following will fail to link if the old version of the .a file is found.

#include <arm_math.h>

arm_rfft_fast_instance_f32 plan;

void setup() {
arm_rfft_fast_init_f32(&plan, 256);
}

void loop() {
float in[256] = { 0 }, out[256] = { 0 };
arm_rfft_fast_f32(&plan, in, out, 0);
}

test_cmsis_fast_rfft.ino.txt

@ladyada
Copy link
Member

ladyada commented Jan 7, 2020

ok cool - i added that test and verified it does not pass CI
https://github.com/adafruit/ArduinoCore-samd/runs/376823304
!

@ladyada
Copy link
Member

ladyada commented Jan 7, 2020

can you push a trivial change (extra space in a comment or something) to force a re-run of CI? we cant manually trigger actions yet

Added extra space to force rerun of CI
@ladyada ladyada merged commit 330cb4e into adafruit:master Jan 7, 2020
@ladyada
Copy link
Member

ladyada commented Jan 7, 2020

nice!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants