Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

dsp/benchmark/magnitude_power_uc8_benchmark.c is missing an include for math.h #105

Closed
gtjoseph opened this issue Feb 5, 2021 · 1 comment

Comments

@gtjoseph
Copy link

gtjoseph commented Feb 5, 2021

...which causes lots of errors like...

In file included from dsp/generated/benchmark.c:1249:
dsp/generated/../benchmark/magnitude_power_uc8_benchmark.c: In function ‘starch_magnitude_power_uc8_benchmark’:
dsp/generated/../benchmark/magnitude_power_uc8_benchmark.c:20:36: error: implicit declaration of function ‘cos’ [-Werror=implicit-function-declaration]
   20 |         in[i].I = (uint8_t) (0.9 * cos(degrees * M_PI / 180.0) * 128 + 127.4);
      |                                    ^~~
@mutability
Copy link

Hmm, I think this is being masked by an ordering thing. On a clean checkout this works because another benchmark which does include math.h is included first.

mutability added a commit that referenced this issue Feb 5, 2021
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

No branches or pull requests

2 participants