[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