Add MLIR package to llvm 17#280572
Conversation
lxsameer
force-pushed
the
mlir-17
branch
2 times, most recently
from
475e114 to
917015c
Compare
There was a problem hiding this comment.
Are those cmake options case-sensitive?
There was a problem hiding this comment.
| src = runCommand "${pname}-src-${version}" {} '' | |
| src = runCommand "mlir-src-${version}" {} '' |
There was a problem hiding this comment.
I followed the same pattern from other packages.
There was a problem hiding this comment.
| sourceRoot = "${src.name}/${pname}"; | |
| sourceRoot = "${src.name}/mlir"; |
ofborg
bot
requested review from
Ericson2314,
alyssais,
dtzWill,
lovek323,
rrbutani and
sternenseemann
ofborg
bot
added
10.rebuild-darwin: 1-10
|
Can it be backported to 23.11 too? Thanks! |
|
Gentle ping |
|
I rather to keep it here, since it needs a lot of testing and moving to a stable branch right now might not be a good idea |
There was a problem hiding this comment.
Are you sure you need this? My recollection is the CURRENT / RUNTIME stuff is fine to keep as-is.
There was a problem hiding this comment.
I don't know to be honest, I followed the same pattern from other llvm components, I'll try again without this and if it works I'll remove the change. Otherwise, I'll keep it and mark it as resolved
There was a problem hiding this comment.
OK sounds good.
Yes some of the other patches are a bit stale as opposed to what I actually upstreamed, and some things maybe were intentionally not upstreamed because it turned out they weren't needed.
There was a problem hiding this comment.
I removed this patch and built mlir. While it builds, but programs that target mlir fail to find it. To be exact, cmake fails to find mlir-tblgen but with this patch the same program compiles with no issue. So I think we need it.
There was a problem hiding this comment.
Can you write a test for this? OK if it is in a follow-up PR.
Co-authored-by: John Ericson <git@JohnEricson.me>
Co-authored-by: John Ericson <git@JohnEricson.me>
|
OK great this looks good! Can you add it to |
|
I'll add another PR for the git version and another one for tests. |
|
Great, thanks! |
|
Successfully created backport PR for |
| "-DLLVM_INSTALL_TOOLCHAIN_ONLY=OFF" | ||
| "-DMLIR_TOOLS_INSTALL_DIR=${placeholder "out"}/bin/" | ||
| "-DLLVM_ENABLE_IDE=OFF" | ||
| "-DLLD_INSTALL_PACKAGE_DIR=${placeholder "out"}/lib/cmake/mlir" |
There was a problem hiding this comment.
Probably this should be MLIR_INSTALL_PACKAGE_DIR not LLD_INSTALL_PACKAGE_DIR?
(Thanks for working on this!)
|
|
|
It is available as |
|
Oh sorry! |
rrbutani
added
the
6.topic: llvm/clang
|
Backport failed for Please cherry-pick the changes locally and resolve any conflicts. git fetch origin release-23.11
git worktree add -d .worktree/backport-280572-to-release-23.11 origin/release-23.11
cd .worktree/backport-280572-to-release-23.11
git switch --create backport-280572-to-release-23.11
git cherry-pick -x 262ed9bd904310a33653904a56546a15f88e971a 777f8c6a12e3d2c7a3732618accfc29395432be0 da2ccc5719c79b28f4bacd20c20ce897e87f605b |
7 participants
This PR adds the mlir package to llvm 17.
Things done
nix.conf? (See Nix manual)sandbox = relaxedsandbox = truenix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/)Add a 👍 reaction to pull requests you find important.