This is an archive of the discontinued LLVM Phabricator instance.

[bazel] Enable layering_check for MLIR test directory
ClosedPublic

Authored by rupprecht on Jan 25 2022, 4:44 AM.

Details

Summary

This would have enabled me to notice the MLIR test file needed updating too, preventing the test file of 2074eef5db330ab437410bfb617b58ea70f4fbff from being necessary.

layering_check is already enabled in mlir/BUILD.bazel. I don't know why I didn't see the other breakage there.

Diff Detail

Event Timeline

rupprecht created this revision.Jan 25 2022, 4:44 AM
rupprecht requested review of this revision.Jan 25 2022, 4:44 AM
GMNGeoffrey accepted this revision.Jan 25 2022, 9:25 AM
GMNGeoffrey added a subscriber: GMNGeoffrey.

LGTM, thanks

This revision is now accepted and ready to land.Jan 25 2022, 9:25 AM
This revision was automatically updated to reflect the committed changes.