This is an archive of the discontinued LLVM Phabricator instance.

[mlir][bazel] Fix missing dependency in TransformOpsPyFiles.
AbandonedPublic

Authored by ingomueller-net on Jul 20 2023, 9:45 AM.

Details

Reviewers
ftynse
Summary

This is necessary due to a new file introduced in
https://reviews.llvm.org/D155717 but the problem only showed up in some
bazel-based build downstream.

Diff Detail

Event Timeline

Herald added a project: Restricted Project. · View Herald TranscriptJul 20 2023, 9:45 AM
ingomueller-net requested review of this revision.Jul 20 2023, 9:45 AM
ftynse accepted this revision.Jul 21 2023, 12:59 AM

Feel free to commit bazel fixes directly.

This revision is now accepted and ready to land.Jul 21 2023, 12:59 AM
ingomueller-net abandoned this revision.Jul 21 2023, 2:31 AM

This was fixed by https://github.com/llvm/llvm-project/commit/e6c62a2d in the end, which was more elaborate that the one line from this patch.