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

[Nico Weber via Phabricator via llvm-commits]

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