search

xcore / xcommon

Common application framework for XMOS software
Other
23 stars 7 forks source link

Don't add -fno-strict-aliasing by default to C flags #10

Closed rlsosborne closed 12 years ago

rlsosborne commented 12 years ago

module_xcommon/build/Makefile.common1 contains:

EXTRA_C_FLAGS += -fno-strict-aliasing

This forces the C compiler to be more conservative when optimizing. It seems wrong to do this globally - It should be the responsibility of apps that need the option to their own makefiles.

ghost commented 12 years ago

I'm not sure why this was in there in the first place. With the current compiler this does not make any difference so it is safe (and correct) to remove it.

I've already pushed the change for this.