[llvm-dev] RFC: Import of Integer Set Library into LLVM source tree

Chris Lattner via llvm-dev llvm-dev at lists.llvm.org
Thu Jan 18 20:47:16 PST 2018



> On Jan 18, 2018, at 6:02 AM, Michael Kruse <llvmdev at meinersbur.de> wrote:
> 
> 2018-01-18 6:40 GMT+01:00 Chris Lattner <clattner at nondot.org>:
>> Great, I think that that would be a fine approach: you can have the cmake logic detect which version of isl is installed and fail if it is the wrong version.  This would address my concern.
> 
>> The motivation explained in the original email seemed to hinge around the fact that Polly’s unit tests depend on accidental details of ISL that change across releases.  If you can fix that, then it seems more plausible to unbundle it.
>> 
>> If you can’t fix that, and can’t replace ISL, then the best approach seems to have CMake detect and reject incompatible versions.
> 
> This would require buildbot admins to regularly install new versions of isl.

It seems like you can choose how frequently to update.  Updating every 6 to 12 months doesn’t seem overly onerous, and still gives the benefit of updates from the upstream project.

-Chris



More information about the llvm-dev mailing list