[PATCH] D35700: DAGCombiner: Extend reduceBuildVecToTrunc to handle non-zero offset

Zvi Rackover via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 24 15:19:36 PDT 2017


zvi added inline comments.


================
Comment at: test/CodeGen/X86/shuffle-strided-with-offset-256.ll:39
+; AVX512F-NEXT:    vpmovsxwd %ymm0, %zmm0
+; AVX512F-NEXT:    vpmovdb %zmm0, %xmm0
 ; AVX512F-NEXT:    vmovdqa %xmm0, (%rsi)
----------------
zvi wrote:
> guyblank wrote:
> > should this be 
> > 
> > ```
> > vpmovdb %zmm0, (%rsi)
> > ```
> > 
> > same for AVX512VL
> Look like a follow-up work needed here.
Ah you were referring to the regalloc issue that's been reported in pr32719


https://reviews.llvm.org/D35700





More information about the llvm-commits mailing list