[PATCH] D48736: [frontend] Don't include the C++ stdlib for -x assembler-with-cpp

[Bruno Cardoso Lopes via Phabricator via cfe-commits]

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