This is an archive of the discontinued LLVM Phabricator instance.

[mlir] Add missing dependency to Bazel build fails.
AbandonedPublic

Authored by ingomueller-net on Mar 28 2023, 2:29 AM.

Details

Reviewers
ftynse
Summary

The dependency was introduced in the CMake files in
https://reviews.llvm.org/D146959.

Diff Detail

Event Timeline

ingomueller-net requested review of this revision.Mar 28 2023, 2:29 AM
ftynse accepted this revision.Mar 28 2023, 2:30 AM
ftynse added a subscriber: ftynse.

Feel free to commit such changes directly.

This revision is now accepted and ready to land.Mar 28 2023, 2:30 AM
ftynse added inline comments.Mar 28 2023, 2:30 AM
utils/bazel/llvm-project-overlay/mlir/test/BUILD.bazel
522–524

Nit: keep sorted alphabetically.

Sorting dependencies alphabetically.

ingomueller-net abandoned this revision.Mar 28 2023, 3:03 AM

Abanding since cf115f30 fixed the problem in the meantime.