[PATCH] D109213: [RegAllocFast] Clean-up. Remove redundant operations. NFC.

Serguei Katkov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 2 23:24:33 PDT 2021


skatkov added inline comments.


================
Comment at: llvm/lib/CodeGen/RegAllocFast.cpp:1267
       // around as a marker in setPhysReg() to recognize this case here.
       if (MO.getSubReg() != 0) {
         MO.setSubReg(0);
----------------
EarlyClobber's sub regs will be handled here, so no need to do the same while we are processing EarlyClobbers at the end.


================
Comment at: llvm/lib/CodeGen/RegAllocFast.cpp:1312
         continue;
       if (MRI->isReserved(Reg))
         continue;
----------------
We checked the condition here.


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

https://reviews.llvm.org/D109213



More information about the llvm-commits mailing list