serge-sans-paille added a comment.

In D78192#1997698 <https://reviews.llvm.org/D78192#1997698>, @Meinersbur wrote:

> Is this ready to try out?


It is!

> 
> 
> In D78192#1996859 <https://reviews.llvm.org/D78192#1996859>, 
> @serge-sans-paille wrote:
> 
>> Note that this should solve in an elegant (?) way the multiple link errors 
>> found when linking clang with LLVM_DYLIB.
> 
> 
> Aren't these fixed already?

Indeed, but the fix is not generic (it made change to the polly Cmake code), 
and it's not compatible with Polly declaring component dependencies (see 
https://reviews.llvm.org/D78358).
Anyway, that's just a side effect of the cleanup, didn't want to emphazise too 
much on that.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78192



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to