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

Hal Finkel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 4 07:35:27 PDT 2017


hfinkel added a comment.

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

> In https://reviews.llvm.org/D31908#745274, @MatzeB wrote:
>
> > Yeah I like that way of modeling things explicitely. Unfortunately that is not how most llvm targets work today.
>
>
> Would it be reasonable to implement that for all targets?


Could we check this in the verifier? One advantage this might have is that, for blocks ending in unreachable, we can easily realize that we don't need the restoration code.


https://reviews.llvm.org/D31908





More information about the llvm-commits mailing list