[PATCH] D139644: [InlineAdvisor] Allow loading advisors as plugins

Nico Weber via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Dec 18 04:39:35 PST 2022


thakis added inline comments.


================
Comment at: llvm/unittests/Analysis/CMakeLists.txt:19
 
+# If plugins are disabled, the PluginInlineAdvisorTest will disable itself at runtime. 
+# Otherwise, reconfiguring with plugins disabled will leave behind a stale executable.
----------------
mtrofin wrote:
> Use `include/llvm/Config/llvm-config.h.cmake` instead define the preprocessor macro.
Also, why did 7d2c1150d31bb remove the comment together with the define? The comment seems useful independent of the define.


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

https://reviews.llvm.org/D139644



More information about the llvm-commits mailing list