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

Unit TestsFailed

TimeTest
10 msLLVM.Bindings/Go::go.test
Script: -- : 'RUN: at line 1'; llvm-go test llvm.org/llvm/bindings/go/llvm

Event Timeline

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

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.