[PATCH] D109078: [clang][driver][AIX] Add system libc++ header paths to driver

Zarko Todorovski via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Sep 10 07:41:50 PDT 2021


ZarkoCA added inline comments.


================
Comment at: clang/test/Driver/aix-toolchain-include.cpp:31
+// CHECK-INTERNAL-INCLUDE:      "-isysroot" "[[SYSROOT:[^"]+]]"
+// CHECK-INTERNAL-INCLUDE-CXX:  "-internal-isystem" "[[SYSROOT]]/opt/IBM/openxlCSDK/include/c++/v1"
+// CHECK-INTERNAL-INCLUDE-CXX:  "-D__LIBC_NO_CPP_MATH_OVERLOADS__"
----------------
It looks like this test is failing the pre-merge check on the x86 bot because this variable isn't captured correctly.
>From what I can see, I think you can fix this by capturing it in the RUN line instead. 

Eg. I found this `test/Driver/darwin-header-search-libcxx.cpp` and https://llvm.org/docs/CommandGuide/FileCheck.html#cmdoption-filecheck-d-var.



Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D109078/new/

https://reviews.llvm.org/D109078



More information about the cfe-commits mailing list