This is an archive of the discontinued LLVM Phabricator instance.

[Polly] Compile ISL into its own library
ClosedPublic

Authored by Meinersbur on Sep 11 2015, 11:51 AM.

Details

Summary

Refactor all ISL-related cmake build instructions into its own CMakeLists.txt and build as a separate library.

This is useful to apply ISL-related build flags to ISL only and not to Polly's files. Also, it the separation of both projects becomes clearer.

Proposed name of the library is Polly_isl. It is not "isl" to avoid mix-up with potentially installed libisl.{a|so}.

Tested configurations:

  • Windows with cmake 3.2
  • Ubuntu with cmake 3.0.2
  • Ubuntu with cmake 3.0.2 BUILD_SHARED_LIBS on
  • Ubuntu with cmake 2.8.12.2 (LLVM minimum version)
  • Ubuntu out-of-LLVM-tree

Diff Detail

Event Timeline

Meinersbur retitled this revision from to [Polly] Compile ISL into its own library.
Meinersbur updated this object.
Meinersbur added reviewers: grosser, jdoerfert.
Meinersbur added a project: Restricted Project.
Meinersbur added subscribers: llvm-commits, pollydev.
grosser edited edge metadata.Sep 23 2015, 2:15 AM
grosser added a subscriber: grosser.

Sorry for the delay. This looks good to me.

(I assume the autoconf build remains unaffected by this)

Also, this seems to be a good step towards making the isl tests work
within the context of Polly.

Thanks,
Tobias

This revision was automatically updated to reflect the committed changes.