[PATCH] D15195: PR4941: Add support for -fno-builtin-foo options.

Bob Wilson via cfe-commits cfe-commits at lists.llvm.org
Mon Dec 14 17:35:24 PST 2015


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





More information about the cfe-commits mailing list