bruno accepted this revision.
bruno added a comment.
This revision is now accepted and ready to land.
LGTM
================
Comment at: lib/Frontend/InitHeaderSearch.cpp:474
+ if (Lang.CPlusPlus && !Lang.AsmPreprocessor &&
+ HSOpts.UseStandardCXXIncludes && HSOpts.UseStandardSystemIncludes) {
if (HSOpts.UseLibcxx) {
----------------
arphaman wrote:
> bruno wrote:
> > Since you added `HSOpts.UseStandardCXXIncludes` to the condition, would it
> > be worth adding a test case for triggering this option when
> > `!Lang.AsmPreprocessor` evals to true?
> This option was already there. I only added '&& !Lang.AsmPreprocessor'. Or do
> you mean something else?
Oh, right!
Repository:
rC Clang
https://reviews.llvm.org/D48736
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
