[PATCH] D48736: [frontend] Don't include the C++ stdlib for -x assembler-with-cpp
Bruno Cardoso Lopes via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Jun 28 14:58:31 PDT 2018
bruno added inline comments.
================
Comment at: lib/Frontend/InitHeaderSearch.cpp:474
+ if (Lang.CPlusPlus && !Lang.AsmPreprocessor &&
+ HSOpts.UseStandardCXXIncludes && HSOpts.UseStandardSystemIncludes) {
if (HSOpts.UseLibcxx) {
----------------
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?
Repository:
rC Clang
https://reviews.llvm.org/D48736
More information about the cfe-commits
mailing list