[Mlir-commits] [llvm] [mlir] adds missing header, removes Bazel unnecessary dependency (PR #110932)

Jordan Rupprecht llvmlistbot at llvm.org
Wed Oct 2 15:08:42 PDT 2024


https://github.com/rupprecht commented:

MLIR seems to be passing at trunk, at least on buildkite. What issue does this fix?

I would expect the cstdint to fix a CppHeaderCompile issue (being able to compile a standalone header), and adding the MaskableOpInterface would fix a layering check. Why is VectorOpsIncGen being removed though?

Otherwise, this kind of change seems fine. LGTM with a tiny bit of context added to the PR description.

https://github.com/llvm/llvm-project/pull/110932


More information about the Mlir-commits mailing list