tiramisu icon indicating copy to clipboard operation
tiramisu copied to clipboard

tiramisu aligned with llvm 6? Bring Halide up to date.

Open mikeseven opened this issue 6 years ago • 4 comments

Tiramisu has dependencies with LLVM/Clang 5 because Halide branch is outdated. The same could be said with ISL to be updated to 0.20.

Any idea when this updated alignment would occur?

mikeseven avatar Nov 17 '18 17:11 mikeseven

well, that was straightforward. I'll update my repo to submit a PR asap.

mikeseven avatar Nov 18 '18 07:11 mikeseven

The distributed Tiramisu backend requires a specific branch of Halide but the other backends will work on up to date Halide without a problem.

@jrayzero and @psuriana is there anything in the distributed backend that still depends on the private branch ? Are we ready to move the public up-to-date Halide ?

rbaghdadi avatar Nov 18 '18 09:11 rbaghdadi

@mikeseven does the public Halide and LLVM 6.0 work fine for you ? Do all tests succeed ?

rbaghdadi avatar Nov 18 '18 09:11 rbaghdadi

yes all tests pass though I didn't build/test with MPI.

The changes I did are in tiramisu_codegen_halide.cpp for things like: Halide::Internal::Max::make(op0, op1, True); changed to Halide::Internal::Max::make(op0, op1);

For ISL, it is just updating to new version 0.20. And LLVM/CLANG updating to release_60.

Before doing that, for Halide, I didn't see differences with @jrayzero tiramisu_64bit branch and Halide code when the clone was done around December last year. I may have missed something for distributed Tiramisu?

mikeseven avatar Nov 28 '18 07:11 mikeseven