[PATCH] D51986: Fixes for `LLVM_LINK_LLVM_DYLIB` && Polly.

Chris Bieneman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Sep 12 17:11:35 PDT 2018


beanz added a comment.

I agree that changing the naming scheme probably makes sense, but I also think that you can probably make a fairly well contained change to the LLVM CMake module that maps components to libnames to special case Polly and make this all work.

I’d really like to see us avoid needing special casing for `LLVM_LINK_LLVM_DYLIB` as much as possible because it really shouldn’t matter to the tool where the LLVM library functionality comes from as long as it is there.


Repository:
  rC Clang

https://reviews.llvm.org/D51986





More information about the cfe-commits mailing list