[PATCH] D78192: Tentative support of compiler extension in llvm-config

[Michael Kruse via Phabricator via llvm-commits]

Meinersbur added inline comments.


================
Comment at: llvm/tools/llvm-config/llvm-config.cpp:134
   }
+  if (Name == "passes") {
+    for (auto const &AvailableExtension : AvailableExtensions) {
----------------
Meinersbur wrote:
> Wasn't the intention to add it only 'all' only? LLVMPasses does not directly depend on statically linked pass extensions.
Thinking about, LLVMPasses seems to be a bag for all passes in other libs, so actually ,might be a good fit. Assuming that when using `llvm-config passes -libs`, other libraries don't just expect the vanilla passes.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78192