[PATCH] D31908: [AntiDepBreaker] Don't rename callee saved register restore instruction

Krzysztof Parzyszek via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 28 15:42:48 PDT 2017


kparzysz added a comment.

In https://reviews.llvm.org/D31908#741348, @MatzeB wrote:

> In any case as far as I understand it this patch is about not renaming one of the restored registers (so we do not suddenly restore the callee save into some unrelated register that happens to be free). So it's not about liveness but about which registers are legal to rename/recolor.


Then what's the problematic situation?  It was exactly my understanding that we renamed a callee-saved register defined by the restore instruction.  What else could be the problem?


https://reviews.llvm.org/D31908





More information about the llvm-commits mailing list