This is an archive of the discontinued LLVM Phabricator instance.

[mlir] Take ValueRange instead of ArrayRef<Value> in StructuredIndexed
ClosedPublic

Authored by ftynse on Sep 4 2020, 1:01 AM.

Details

Summary

This was likely overlooked when ValueRange was first introduced. There is no
reason why StructuredIndexed needs specifically an ArrayRef so use ValueRange
for better type compatibility with the rest of the APIs.

Diff Detail

Event Timeline

ftynse created this revision.Sep 4 2020, 1:01 AM
Herald added a project: Restricted Project. · View Herald TranscriptSep 4 2020, 1:01 AM
ftynse requested review of this revision.Sep 4 2020, 1:01 AM
nicolasvasilache accepted this revision.Sep 4 2020, 8:47 AM
This revision is now accepted and ready to land.Sep 4 2020, 8:47 AM
mehdi_amini accepted this revision.Sep 4 2020, 11:24 AM