[PATCH] D33814: CodeGen: Fix ARM cmpxchg64 register fragmentation in fast-regalloc
Quentin Colombet via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Aug 7 10:37:53 PDT 2017
qcolombet requested changes to this revision.
qcolombet added a comment.
This revision now requires changes to proceed.
The approach sounds sensible to me however, I'd like we use the AllocationPriority instead of the size of the register class.
Moreover, please refactor the code (I am thinking helper function) so that the definitions are processed in the same way.
Finally, keep the order of the operand as tie breaker.
PS: If AllocationPriority does not fix the problem, then you'll have to fix the description of the related register class, because size alone is too magic.
More information about the llvm-commits