[libcxx-commits] [PATCH] D149660: [libc++][AIX] Add OS version to target triple

[Jake Egan via Phabricator via libcxx-commits]

Jake-Egan added inline comments.


================
Comment at: libcxx/utils/ci/run-buildbot:656
+
+    TARGET=$(${CXX} --version | grep 'Target:' | awk '{print $2}')
+    PARAMS="target_triple=${TARGET}"
----------------
ldionne wrote:
> Normally, the target triple is figured out automatically in the CMake code. I think it's `LLVM_DEFAULT_TARGET_TRIPLE`. Is there a reason why that triple isn't inferred properly? If it were, I think this would not be necessary.
We considered adding the AIX version to the triple in the driver, but we don't have a good way to retrieve the AIX version in the driver right now. So, we prefer to do it in the test-suite to avoid any potential problems.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D149660