[PATCH] D62050: [Analysis] Only run plugins tests if plugins are actually enabled

Don Hinton via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon May 20 19:45:09 PDT 2019


hintonda added a comment.

In D62050#1509400 <https://reviews.llvm.org/D62050#1509400>, @vitalybuka wrote:

> In D62050#1509384 <https://reviews.llvm.org/D62050#1509384>, @vitalybuka wrote:
>
> > This also breaks "ninja check-cfi-and-supported" on clean build (run cmake in empty directory)
>
>
> I've fixed this with DLLVM_BUILD_LLVM_DYLIB=ON but not sure if this is the right approach long-term.


I have a patch that only reverts the changes to HandleLLVMOptions.cmake, D62154 <https://reviews.llvm.org/D62154>, and restores previous behavior.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D62050





More information about the cfe-commits mailing list