[PATCH] D119199: replace clang LLVM_ENABLE_PLUGINS -> CLANG_PLUGIN_SUPPORT in tests

Simon Tatham via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Feb 11 09:28:21 PST 2022


simon_tatham added inline comments.


================
Comment at: clang/tools/driver/CMakeLists.txt:53
 # Support plugins.
 if(CLANG_PLUGIN_SUPPORT)
   export_executable_symbols_for_plugins(clang)
----------------
I think we've managed to fix our build by changing this line so that it tests
```if(CLANG_PLUGIN_SUPPORT OR LLVM_EXPORT_SYMBOLS_FOR_PLUGINS)```
because that fixes the cmake complaint that you can't link against an executable that didn't have `ENABLE_EXPORTS`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D119199



More information about the cfe-commits mailing list