[llvm-commits] request to fix commit r116145

pageexec at freemail.hu pageexec at freemail.hu
Mon Oct 11 17:22:09 PDT 2010


this commit:

r116145 | abramo | 2010-10-09 06:51:06 +0200 (Sat, 09 Oct 2010) | 1 line

Fixed __builtin_constant_p return type.

made a mistake:

-BUILTIN(__builtin_constant_p, "Us.", "nc")
+BUILTIN(__builtin_constant_p, "is.", "nc")

in that it made the first argument of the builtin 'short' whereas it
could be anything (compiling the linux kernel will show how badly this
manifests ;), so i think the correct line should be simply:

+BUILTIN(__builtin_constant_p, "i.", "nc")

cheers,

  PaX Team




More information about the llvm-commits mailing list