[PATCH] D75819: [PowerPC][NFC] Rename instruction formats in PPCInstrPrefix.td

Amy Kwan via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 9 16:12:41 PDT 2020


amyk marked an inline comment as done.
amyk added inline comments.


================
Comment at: llvm/lib/Target/PowerPC/PPCInstrPrefix.td:46
 
-class MLS_DForm_R_D34_RTA5<bits<6> opcode, dag OOL, dag IOL, string asmstr,
-                           InstrItinClass itin, list<dag> pattern>
+class MLS_DForm_R_SI34_FRS5<bits<6> opcode, dag OOL, dag IOL, string asmstr,
+                            InstrItinClass itin, list<dag> pattern>
----------------
Note that I've changed this format to have `SI34_FRS5`. I was originally going to change just `D34` to `SI34` but then the instruction format name would be `MLS_DForm_R_SI34_RTA5`. This would not be possible, as there already exists an instruction format with this name (below the current instruction format).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75819





More information about the llvm-commits mailing list