[PATCH] D132478: [runtimes][NFC] Colocate handling of LLVM_ENABLE_PROJECTS and LLVM_ENABLE_RUNTIMES

Louis Dionne via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 23 15:40:15 PDT 2022


ldionne added a comment.

In D132478#3743644 <https://reviews.llvm.org/D132478#3743644>, @Ericson2314 wrote:

> So this is in preparation for `LLVM_DEFAULT_RUNTIMES` in `llvm/CMakeLists.txt` doing a `runtimes/` instead of `llvm/runtimes/` build (among other changes)?

No, that doesn't change. We currently have two places where we define the set of things valid inside `LLVM_ENABLE_RUNTIMES`: `<root>/runtimes/CMakeLists.txt` and `<root>/llvm/runtimes/CMakeLists.txt`. This makes the two of them consistent, and also moves the one in `<root>/llvm/runtimes/CMakeLists.txt` to `<root>/llvm/CMakeLists.txt` so it can be close to the place where we define `LLVM_ENABLE_PROJECTS`, since the two are strongly related.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D132478



More information about the llvm-commits mailing list