[PATCH] D149146: [TailDuplicator] Don't constrain register classes due to debug instructions

Mikael Holmén via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 25 05:21:58 PDT 2023


uabelho marked an inline comment as done.
uabelho added inline comments.


================
Comment at: llvm/lib/CodeGen/TailDuplicator.cpp:434
+            // rather than constraining the register class further.
+            if (!NewMI.isDebugInstr())
+              ConstrRC = MRI->constrainRegClass(VI->second.Reg, OrigRC);
----------------
foad wrote:
> Just a cosmetic thing, I'd prefer you swap the "if" and "else" cases so you don't have an "else" after an already negated condition.
Sure, done!


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

https://reviews.llvm.org/D149146



More information about the llvm-commits mailing list