[PATCH] D130919: [MRI] isConstantPhysReg should also check if the register is clobbered by a RegMask

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 23 13:27:32 PDT 2022


arsenm added inline comments.


================
Comment at: llvm/lib/CodeGen/RegUsageInfoPropagate.cpp:147-149
+  if (Changed)
+    MF.getRegInfo().recollectUsedPhysRegMask();
+
----------------
Won't this be reflected in the regmask updates (BTW I think this pass needs to be changed to directly modify the regmasks in the instruction instead of the analysis)?


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

https://reviews.llvm.org/D130919



More information about the llvm-commits mailing list