[PATCH] D32277: Replace slow LEA instructions in X86

Lama via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 27 00:18:48 PDT 2017


lsaba added a comment.

In https://reviews.llvm.org/D32277#739023, @skatkov wrote:

> I have not got the conclusion: R13 is a bad register or not?
>  According to what I see in the comment it is not but patch still consider it is?
>  Could you please clarify this?


according to the optimization guide:  "LEA that uses base and index registers where the base is EBP, RBP,or R13" so R13 is bad when there is an index register as well, in  aqjune's example the instruction does not have an index register (leal    8(%r13), %eax )


https://reviews.llvm.org/D32277





More information about the llvm-commits mailing list