Page MenuHomePhabricator

[MLIR][Shape] Canonicalize subsequent `size_to_index` and `index_to_size`
ClosedPublic

Authored by frgossen on Thu, Jun 18, 4:38 AM.

Details

Summary

Eliminate the subsequent applications of size_to_index and index_to_size.

Depends On D82082

Diff Detail

Event Timeline

frgossen created this revision.Thu, Jun 18, 4:38 AM
Herald added a reviewer: silvas. · View Herald Transcript
Herald added a project: Restricted Project. · View Herald Transcript
silvas accepted this revision.Thu, Jun 18, 9:48 AM

Great! Thanks for these nicely factored patches!

nit: You may want to update the title before landing it

rriddle retitled this revision from [MLIR][Shape] Ass canonicalization pattern for `shape.index_to_size` to [MLIR][Shape] Add canonicalization pattern for `shape.index_to_size`.Thu, Jun 18, 10:10 AM
frgossen retitled this revision from [MLIR][Shape] Add canonicalization pattern for `shape.index_to_size` to [MLIR][Shape] Canonicalize subsequent `index_to_size` and `size_to_index`.Fri, Jun 19, 4:22 AM
frgossen retitled this revision from [MLIR][Shape] Canonicalize subsequent `index_to_size` and `size_to_index` to [MLIR][Shape] Canonicalize subsequent `size_to_index` and `index_to_size`.Fri, Jun 19, 4:29 AM
jpienaar accepted this revision.Tue, Jun 23, 2:06 PM
This revision is now accepted and ready to land.Tue, Jun 23, 2:06 PM
frgossen updated this revision to Diff 273244.Thu, Jun 25, 1:21 AM

Remove trailing white line

frgossen updated this revision to Diff 273293.Thu, Jun 25, 3:46 AM

White line

This revision was automatically updated to reflect the committed changes.