This is an archive of the discontinued LLVM Phabricator instance.

[MLIR][SPIRV] Rename `spv._address_of` to `spv.mlir.addressof`
ClosedPublic

Authored by ergawy on Nov 17 2020, 3:29 AM.

Details

Summary

This commit does the renaming mentioned in the title in order to bring
spv dialect closer to the MLIR naming conventions.

Diff Detail

Event Timeline

ergawy created this revision.Nov 17 2020, 3:29 AM
ergawy requested review of this revision.Nov 17 2020, 3:29 AM
ergawy added a comment.EditedNov 17 2020, 3:33 AM

I think this should be actually renamed to spv.mlir.addressof to be truly consistent the style used in the llvm dialect. Are there any objections to this? If not, I will do as such.

ergawy updated this revision to Diff 305737.Nov 17 2020, 4:29 AM

Rename to addressof instead of address_of.

ergawy retitled this revision from [MLIR][SPIRV] Rename `spv._address_of` to `spv.mlir.address_of` to [MLIR][SPIRV] Rename `spv._address_of` to `spv.mlir.addressof`.Nov 17 2020, 4:29 AM
ergawy updated this revision to Diff 305739.EditedNov 17 2020, 4:33 AM

Fix mistake due to batch editing using sed.

ergawy updated this revision to Diff 305741.Nov 17 2020, 4:37 AM

Fix mistake due to batch editing using sed.

Harbormaster completed remote builds in B79092: Diff 305741.
antiagainst accepted this revision.Nov 17 2020, 8:43 AM

Awesome, thanks for addressing this!!

This revision is now accepted and ready to land.Nov 17 2020, 8:43 AM