[PATCH] D103419: [VectorCombine] Fix alignment in single element store
Roman Lebedev via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Jun 8 03:17:07 PDT 2021
lebedev.ri added a comment.
Thanks.
This looks fine to me now.
Can anyone spot any issues with the new alignment logic? @fhahn @spatel?
================
Comment at: llvm/test/Transforms/VectorCombine/load-insert-store.ll:142
+define void @insert_store_nonconst_large_alignment(<4 x i32>* %q, i32 zeroext %s, i32 %idx) {
+; CHECK-LABEL: @insert_store_nonconst_large_alignment(
----------------
I think we still want those two tests i suggested,
they demonstrate that we don't increase alignment from the maximal one allowed.
Please precommit the tests.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D103419/new/
https://reviews.llvm.org/D103419
More information about the llvm-commits
mailing list