aboutsummaryrefslogtreecommitdiffstats
path: root/powerpc/CBuiltins.ml
diff options
context:
space:
mode:
authorXavier Leroy <xavier.leroy@inria.fr>2016-07-08 15:44:46 +0200
committerXavier Leroy <xavier.leroy@inria.fr>2016-07-08 15:44:46 +0200
commit4c7650c3eaf4dfbe5971864bf084e76f844051ee (patch)
tree32b5deb84cb3fd36f4baada8eed4ce108aed9599 /powerpc/CBuiltins.ml
parente73d5db97cdb22cce2ee479469f62af3c4b6264a (diff)
downloadcompcert-4c7650c3eaf4dfbe5971864bf084e76f844051ee.tar.gz
compcert-4c7650c3eaf4dfbe5971864bf084e76f844051ee.zip
Unwanted partial constant propagation in 64-bit integer arguments to builtins
Here are two examples that cause an internal error in Asmexpand.ml: volatile long long x; void f(unsigned int i) { x = i; } unsigned g(unsigned i) { return __builtin_clzll(i); } The argument "i" to builtin volatile store or __builtin_clzll is turned into a BA_splitlong(BA_int 0, BA <variable i>), which Asmexpand.ml doesn't know how to handle. The fix (in AST.builtin_arg_ok) is to prevent this 'optimization' for all builtins except those of the "OK_all" kind, i.e. __builtin_annot. Regression tests were added and tested on IA32. Need to retest on ARM and PowerPC.
Diffstat (limited to 'powerpc/CBuiltins.ml')
0 files changed, 0 insertions, 0 deletions