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

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 20 12:26:29 PDT 2021


spatel added inline comments.


================
Comment at: llvm/test/Transforms/VectorCombine/AArch64/load-extractelement-scalarization.ll:253
 ; Scalarizing the load for multiple constant indices may not be profitable.
 define i32 @load_multiple_extracts_with_constant_idx(<4 x i32>* %x) {
 ; CHECK-LABEL: @load_multiple_extracts_with_constant_idx(
----------------
Let me know if I'm overlooking it, but I don't see a positive test for this pattern (multiple extracts of a single load).
Is there some vector type variant (float?) of this test that will transform for AArch64?
If not, we might get x86 or some other target to trigger on this test as-is.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100273



More information about the llvm-commits mailing list