[cfe-commits] r141803 - in /cfe/trunk: include/clang/Basic/Builtins.def include/clang/Basic/Builtins.h lib/AST/DeclPrinter.cpp lib/CodeGen/CGCall.cpp lib/Sema/SemaDecl.cpp test/Analysis/security-syntax-checks.m test/CodeGen/function-attributes.c

Rafael Ávila de Espíndola rafael.espindola at gmail.com
Mon Oct 17 12:04:55 PDT 2011


> Agreed. Getting GCC to adopt any kind of changes can be non-trivial
> though.

CC richi, he is the one that replied to my questions on IRC.

In summary, I am OK with a patch that

* Fixes the signature in builtins.def or
* Removes it *if* you also email the gcc list and we don't get a 
pushback there.

I am not OK with introducing what looks like a silent miscompilation.

>
> Joerg

Cheers,
Rafael



More information about the cfe-commits mailing list