[PATCH] D145169: FastRegAlloc: Fix implicit operands not rewritten

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 3 04:48:18 PST 2023


arsenm added a comment.

Processing all uses of a register at once makes sense to me



================
Comment at: llvm/lib/CodeGen/RegAllocFast.cpp:1410
+      useVirtReg(MI, I, Reg);
+      ChangedOperands = true;
+    }
----------------
Don't you only need to maybe repeat if you saw a kill use?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D145169



More information about the llvm-commits mailing list