[libcxx-commits] [PATCH] D88458: [CMake] Cache compiler-rt builtin results

[Petr Hosek via Phabricator via libcxx-commits]

phosek created this revision.
phosek added reviewers: beanz, smeenai, ldionne.
Herald added subscribers: libcxx-commits, Sanitizers, dexonsmith, mgorny, dberris.
Herald added projects: Sanitizers, libc++, libc++abi, libunwind.
Herald added a reviewer: libc++.
Herald added a reviewer: libc++abi.
Herald added a reviewer: libunwind.
phosek requested review of this revision.

There's a lot of duplicated calls to find various compiler-rt libraries.
This change adds caching to avoid repeated Clang invocations. We also
use the same variable name across all runtimes to further reduce
duplication. This change is based on the existing logic already used
by compiler-rt for the same purpose.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D88458

Files:
  compiler-rt/cmake/Modules/HandleCompilerRT.cmake
  libcxx/cmake/Modules/HandleCompilerRT.cmake
  libcxxabi/cmake/Modules/HandleCompilerRT.cmake
  libunwind/cmake/Modules/HandleCompilerRT.cmake

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D88458.294842.patch
Type: text/x-patch
Size: 13763 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/libcxx-commits/attachments/20200929/35d126a9/attachment-0001.bin>