[PATCH] D144001: [Metarenamer] Use 'inst' as default name for instructions

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 14 23:44:48 PST 2023


nikic accepted this revision.
nikic added a comment.
This revision is now accepted and ready to land.

LGTM

A thought I had is that we could also use the name of the instruction, producing something like `%phi1 = phi i32 [ ... ]`. This is something I often end up doing when manually cleaning up names. That makes things easier to follow than just numbering all instructions.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D144001/new/

https://reviews.llvm.org/D144001



More information about the llvm-commits mailing list