[PATCH] D157676: [VectorCombine] Enable transform 'foldSingleElementStore' for scalable vector types
Dave Green via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Aug 15 00:37:43 PDT 2023
dmgreen added a comment.
Does this need to be limited to LE? My understanding is that BE and LE vectors would store the elements in the same places, even for scalable vectors. (i.e. it's like an array, where the 0th element ends up first in memory).
================
Comment at: llvm/lib/Transforms/Vectorize/VectorCombine.cpp:47
+static cl::opt<bool>
+ DisableVectorCombine("disable-vector-combine", cl::init(false), cl::Hidden,
+ cl::desc("Disable all vector combine transforms"));
----------------
Can you remove the unrelated formatting changes.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D157676/new/
https://reviews.llvm.org/D157676
More information about the llvm-commits
mailing list