[libcxx-commits] [PATCH] D119554: [compiler-rt] Force ABI to libcxxabi when building cxustom libc++

[Petr Hosek via Phabricator via libcxx-commits]

phosek accepted this revision.
phosek added a comment.
This revision is now accepted and ready to land.

LGTM



================
Comment at: compiler-rt/cmake/Modules/AddCompilerRT.cmake:638-642
+               -DLIBCXX_CXX_ABI=libcxxabi
                -DLIBCXXABI_ENABLE_SHARED=OFF
                -DLIBCXXABI_HERMETIC_STATIC_LIBRARY=ON
                -DLIBCXXABI_INCLUDE_TESTS=OFF
                -DLIBCXX_ENABLE_EXPERIMENTAL_LIBRARY=OFF
----------------
Nit: can you move it down so that all libc++ options are kept together?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D119554