[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 mailing list