[PATCH] D100273: [VectorCombine] Scalarize vector load/extract.

[Florian Hahn via Phabricator via llvm-commits]

fhahn added inline comments.


================
Comment at: llvm/test/Transforms/VectorCombine/X86/load-inseltpoison.ll:645
   %gep = getelementptr inbounds <2 x i16>, <2 x i16>* %p, i64 1
   %l = load <2 x i16>, <2 x i16>* %gep, align 8
   %s = extractelement <2 x i16> %l, i32 0
----------------
RKSimon wrote:
> The arg says its align 1m, but here we're saying align 8?
Hm that's indeed odd. I *think* the align of the load would take precedence and `align 1 dereferenceable(16)` are probably not needed for the test. But I think we need to choose a conservative alignment for the scalarized load, that's why `align 1` is used for the scalar load.

Does that answer your question?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100273