[PATCH] D78192: Support compiler extensions as a normal component

serge via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 23 01:02:23 PDT 2020


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





More information about the llvm-commits mailing list