[PATCH] D60993: [X86] Initial cleanups on the FixupLEAs pass. Separate Atom LEA creation from other LEA optimizations.

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 23 13:18:06 PDT 2019


RKSimon added inline comments.


================
Comment at: llvm/test/CodeGen/X86/lsr-static-addr.ll:40
+; ATOM-NEXT:    movsd %xmm1, A(,%rax,8)
+; ATOM-NEXT:    leaq 1(%rax), %rax
+; ATOM-NEXT:    cmpq %rax, %rdi
----------------
craig.topper wrote:
> Forgot to mention. This change is specifically because the OptIncDec code was creating an INC here and preventing Atom from visiting it due to the continue.
Please can you regenerate this against trunk?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D60993/new/

https://reviews.llvm.org/D60993





More information about the llvm-commits mailing list