[PATCH] D19246: Do not rename a tied operand in AggressiveAntiDepBreaker

Hal Finkel via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 26 10:06:31 PDT 2016


hfinkel added a comment.

> The anti-dependence breaking code should not attempt to rename an operand that is tied. Otherwise, the two operand will end up with different registers.


This does not seem quite true. It should either not rename them or rename them together. AggressiveAntiDepBreaker has logic which should handle tied operands. See AggressiveAntiDepBreaker::GetPassthruRegs and its various callers. If this logic is broken, then we should fix it.


http://reviews.llvm.org/D19246





More information about the llvm-commits mailing list