Page MenuHomePhabricator

Fix MLIR build when the NVPTX target isn't configured
ClosedPublic

Authored by mehdi_amini on Feb 12 2020, 2:12 AM.

Diff Detail

Event Timeline

mehdi_amini created this revision.Feb 12 2020, 2:12 AM
herhut added inline comments.
mlir/tools/mlir-opt/CMakeLists.txt
78–79

Does the mlir-cuda-runner CMakefile need this, as well? Or can this be hidden as a transitive dependency of MLIRGPUtoCUDATransforms?

Fix build with bfd

ftynse accepted this revision.Feb 12 2020, 3:45 AM
This revision was not accepted when it landed; it landed in state Needs Review.Feb 12 2020, 4:41 AM
This revision was automatically updated to reflect the committed changes.

It looks like similar changes may be needed to mlir-cuda-runner and, potentially, mlir-cpu-runner.

https://reviews.llvm.org/D74480 for the mlir-cuda-runner changes. I had to include many dependencies that I assume should come with the JitRunner. Not sure what the overall goal is but this makes things build.