[PATCH] D140496: [TailDuplicator] Fix old bugs in TailDuplicator::duplicateInstruction

Krzysztof Parzyszek via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 3 10:08:12 PST 2023


kparzysz added inline comments.


================
Comment at: llvm/lib/CodeGen/TailDuplicator.cpp:454
               NewRC = OrigRC;
             Register NewReg = MRI->createVirtualRegister(NewRC);
             BuildMI(*PredBB, NewMI, NewMI.getDebugLoc(),
----------------
Maybe we can just use `OrigRC` here, instead of trying to come up with a new class?  `OrigRC` is guaranteed to be valid, since that's what the original code used.



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D140496



More information about the llvm-commits mailing list