[PATCH] D112380: [PowerPC] Enforce side effects to FPSCR read/set instructions

Jinsong Ji via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Oct 24 19:55:48 PDT 2021


jsji added inline comments.


================
Comment at: llvm/lib/Target/PowerPC/PPCInstrInfo.td:3113
   let isCodeGenOnly = 1 in
   def MTFSFb  : XFLForm<63, 711, (outs), (ins i32imm:$FM, f8rc:$rT),
                         "mtfsf $FM, $rT", IIC_IntMTFSB0,
----------------
`SETFLM` is expanded into `MTFSF`, why setting `MTFSFb` only here?

How about `MTFSFIb`, `MTFSFI`, `MTFSF` etc? 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D112380



More information about the llvm-commits mailing list