[PATCH] D55251: [X86] Enable -x86-experimental-vector-widening-legalization by default.

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 24 12:15:56 PDT 2019


craig.topper marked an inline comment as done.
craig.topper added inline comments.


================
Comment at: llvm/test/CodeGen/X86/widen_cast-3.ll:14
 ; X86-NEXT:    pextrd $2, %xmm0, 8(%eax)
-; X86-NEXT:    movq %xmm0, (%eax)
+; X86-NEXT:    pextrd $1, %xmm0, 4(%eax)
+; X86-NEXT:    movd %xmm0, (%eax)
----------------
xbolva00 wrote:
> Not ideal?
Looks like something pessimistic is happening in type legalization for store widening. This test case used to generate the same code it is with this patch, but was changed in March in r357120. I'll see if we can enhance r357120 to handle this.


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

https://reviews.llvm.org/D55251





More information about the llvm-commits mailing list