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

Matthias Braun via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 28 15:27:29 PDT 2017


MatzeB added a comment.

In https://reviews.llvm.org/D31908#741347, @kparzysz wrote:

> In https://reviews.llvm.org/D31908#741342, @MatzeB wrote:
>
> > All callee saves? That would be new to me. I think we only do that for callee saves that are saved/restored *somewhere* in the function, the rest is considered "pristine" and not explicitely mentioned in the live-in lists.
>
>
> Yes, that's what I meant---the ones that were actually clobbered in the function.


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.


https://reviews.llvm.org/D31908





More information about the llvm-commits mailing list