bob.wilson added a comment.

Regarding the FIXME in lib/Frontend/CompilerInvocation.cpp: I agree with Hal 
that you can remove that. We used to complain about unsupported -fno-builtin-* 
options (and until now they have *all* been unsupported), but in r191434, 
Rafael changed clang to silently ignore those options, with the explanation 
that it matches gcc's behavior. Assuming that gcc has not changed in that 
regard, it makes sense that we should continue to ignore -fno-builtin-* for 
unsupported options.


http://reviews.llvm.org/D15195



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to