This is an archive of the discontinued LLVM Phabricator instance.

Fix compiler extension in standalone mode
ClosedPublic

Authored by serge-sans-paille on Feb 18 2020, 2:18 AM.

Details

Summary

Use a dedicated cmake file to store the extension configured within LLVM.

This patch is related to https://reviews.llvm.org/D74602

Diff Detail

Event Timeline

Herald added projects: Restricted Project, Restricted Project. · View Herald TranscriptFeb 18 2020, 2:18 AM

I still need to test many configurations, I'll report here.

Tested without standalone build with polly linked statically and dynamically with success.

The Windows pre-merge check seem to have failed?

llvm/cmake/modules/AddLLVM.cmake
886

Change to [GEN_CONFIG]

llvm/cmake/modules/LLVMConfig.cmake.in
115–116

[nit] unrelated whitespace change

Take @Meinersbur review into account.

serge-sans-paille marked 2 inline comments as done.Feb 18 2020, 11:03 PM
Keruspe accepted this revision.Feb 19 2020, 4:21 AM

Works fine with 10.0.0rc2 and related patches applied, thanks!

This revision is now accepted and ready to land.Feb 19 2020, 4:21 AM
This revision was automatically updated to reflect the committed changes.