[PATCH] D71170: [PowerPC] [NFC] Cleanup xxpermdi peephole optimization

[Jinsong Ji via Phabricator via llvm-commits]

jsji added inline comments.


================
Comment at: llvm/lib/Target/PowerPC/PPCMIPeephole.cpp:350
+
+        if (TrueReg1 != TrueReg2 || Register::isPhysicalRegister(TrueReg1))
+          break;
----------------
 Looks like this might have a problem with `Stack slots`? We used to only handle virtual registers,
but now, we might also let throughg stack slots (and Not a register), which might cause build break?

``` 
  //   0           Not a register, can be used as a sentinel.
  //   [1;2^30)    Physical registers assigned by TableGen.
  //   [2^30;2^31) Stack slots. (Rarely used.)
  //   [2^31;2^32) Virtual registers assigned by MachineRegisterInfo.```


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71170