[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