search

ARM-software / optimized-routines

Optimized implementations of various library functions for ARM architecture processors
Other
601 stars 96 forks source link

is it deliberate that you need math/ on the include path to build the math tests? #53

Open enh-google opened 1 year ago

enh-google commented 1 year ago

is it deliberate that you need math/ on the include path to build the math tests?

i'm guessing these should all have the leading test/ removed?

math/test/mathbench.c:#include "test/mathbench_wrappers.h"
math/test/mathbench.c:#include "test/mathbench_funcs.h"
math/test/ulp.c:#include "test/ulp_wrappers.h"
math/test/ulp.c:#include "test/ulp_funcs.h"

(worked around in the Android build for now by just adding math/ to the include path.)

nsz-arm commented 1 year ago

it is because now math/test/foo.h is copied into build/include/test/foo.h and build/include is in the include path but build/include/test is not.

this is a bit weird, but the reason is to allow pl/math/test/foo.h to override math/test/foo.h when building test tools. i think we may need a nicer way to do this.