]> git.ipfire.org Git - thirdparty/u-boot.git/commit
Makefile: Use -fno-strict-aliasing globally
authorBin Meng <bmeng.cn@gmail.com>
Mon, 17 Sep 2018 05:35:28 +0000 (22:35 -0700)
committerTom Rini <trini@konsulko.com>
Wed, 26 Sep 2018 01:50:34 +0000 (21:50 -0400)
commit5bc11922f1f8e0bfb27e8e260c7971405535c257
tree07abdd51a12f9a5fb376631423f2fe0372f5843f
parentc1523c65fc61d9ec8e779cbfb31314e94f500a3e
Makefile: Use -fno-strict-aliasing globally

The -fstrict-aliasing option is implicitly enabled at levels -O2,
-O3, -Os by GCC. This option allows the compiler to assume the
strictest aliasing rules applicable to the language being compiled.
For example, the practice of reading from a different union member
than the one most recently written to (called "type-punning") is
common. In this case, "type-punning" only works if the memory is
accessed through the union type, but might not work by taking the
address, casting the resulting pointer and dereferencing the result,
which is an undefined behavior per the "strict aliasing rules".

GCC's -Wstrict-aliasing (included in -Wall) option does not catch
all cases, but does attempt to catch the more common pitfalls. So
there are cases that GCC does not report but the codes are violating
the "strict aliasing rules".

Given lots of codes that may be written to rely on "type-punning",
and Linux kernel disables it by -fno-strict-aliasing globally, since
U-Boot currently does this on nds32/riscv/x86 builds only, extend
this for all architecture builds.

Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
Reviewed-by: Tom Rini <trini@konsulko.com>
Reviewed-by: Simon Glass <sjg@chromium.org>
Makefile