This is an archive of the discontinued LLVM Phabricator instance.

[MLIR] Switch the default for usePropertiesForAttributes (NFC)
ClosedPublic

Authored by mehdi_amini on Aug 22 2023, 11:22 PM.

Diff Detail

Event Timeline

mehdi_amini created this revision.Aug 22 2023, 11:22 PM
Herald added a reviewer: rriddle. · View Herald Transcript
Herald added a reviewer: aartbik. · View Herald Transcript
Herald added 1 blocking reviewer(s): jpienaar. · View Herald Transcript
Herald added a reviewer: ftynse. · View Herald Transcript
Herald added a reviewer: aartbik. · View Herald Transcript
Herald added a reviewer: ftynse. · View Herald Transcript
Herald added a reviewer: dcaballe. · View Herald Transcript
Herald added a reviewer: kuhar. · View Herald Transcript
Herald added a reviewer: kuhar. · View Herald Transcript
Herald added a project: Restricted Project. · View Herald Transcript
mehdi_amini requested review of this revision.Aug 22 2023, 11:23 PM
csigg added a comment.Aug 23 2023, 5:58 AM

Would it be possible to land https://reviews.llvm.org/D158608 first?
It seems that even some of mlir's own tests require this (python tests, iirc).

Matt added a subscriber: Matt.Aug 23 2023, 10:48 AM

Seems like we didn't get push back on this, @jpienaar can we land it?

springerm accepted this revision.Aug 28 2023, 7:11 AM
This revision was not accepted when it landed; it landed in state Needs Review.Aug 28 2023, 8:25 PM
This revision was automatically updated to reflect the committed changes.