Let the toolchain set the CC variable

smuellerDD / acvpparser

ACVP Parser for invocation of cryptographic implementations using the ACVP JSON test vectors

https://www.chronox.de/acvpparser

Other

36 stars 27 forks source link

Let the toolchain set the CC variable #67

Closed andreasaxis closed 1 year ago

andreasaxis commented 1 year ago

Lets the toolchain set the CC variable (compiler).

smuellerDD commented 1 year ago

Am Freitag, 8. September 2023, 11:24:02 CEST schrieb Andreas Bank:

Hi Andreas,

Can you please be a bit more specific which compiler warnings you see? I have not seen any warnings with my compilers and frankly, simply changing variable names does not look like it addresses the root cause of compiler warnings.

I am fine with the Makefile patch, though.

Ciao Stephan

andreasaxis commented 1 year ago

We use -Wshadow which results in warnings in two places. I have removed this commit from the merge request and have it as a local patch instead.