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

[Chris Bieneman via Phabricator via llvm-commits]

beanz added a comment.

If you add this to your patch here, the Clang patch should work as I've suggested:

  diff --git a/cmake/modules/LLVM-Config.cmake b/cmake/modules/LLVM-Config.cmake
  index 8eabddc7377..a306e41f26d 100644
  --- a/cmake/modules/LLVM-Config.cmake
  +++ b/cmake/modules/LLVM-Config.cmake
  @@ -244,6 +244,9 @@ function(llvm_map_components_to_libnames out_libs)
             list(APPEND expanded_components "LLVM${t}Info")
           endif()
         endforeach(t)
  +    elseif( c STREQUAL "Polly")
  +      # LLVMPolly is the Polly loadable module target, the static archive is just Polly
  +      list(APPEND expanded_components "${c}")
       else( NOT idx LESS 0 )
         # Canonize the component name:
         string(TOUPPER "${c}" capitalized)

It is unfortunate that Polly doesn't match the naming conventions of other LLVM components, but we have a lot of special case handling for this kind of thing anyways.


Repository:
  rL LLVM

https://reviews.llvm.org/D51984