[PATCH] D32277: Replace slow LEA instructions in X86

Lama via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 25 01:53:58 PDT 2017


lsaba added a comment.

In https://reviews.llvm.org/D32277#735719, @RKSimon wrote:

> https://reviews.llvm.org/D32352 is looking at more aggressive conversion of IMUL to multiple LEA instructions.


Thanks for notifying, the patch does not contain any 3-Operand LEAs as far as I can see, the only case we need to be careful with is for 2-Operand LEA with RBP/R13/EBP as a base register, since this is determined only after RA, I am thinking it's better to let my patch fix those cases rather than preventing that patch from running on the problematic targets


https://reviews.llvm.org/D32277





More information about the llvm-commits mailing list