[PATCH] D99565: [X86] Support replacing aligned vector moves with unaligned moves when avx is enabled.
Craig Topper via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Apr 8 15:40:20 PDT 2021
craig.topper added a comment.
I think I wouldn't mind if we just didn't emit aligned loads/store instructions for AVX/AVX512 from isel and other places in the compiler in the first place. As noted, if the load gets folded the alignment check doesn't happen. That would reduce the size of the isel tables and remove branches, reducing complexity of the compiler. Adding a new step and a command line to undo the earlier decision increases complexity.
The counter argument to that is that the alignment check has found bugs in the vectorizer on more than one occasion that I know of.
Sergey, remind me, does icc always emit unaligned loads/stores? Is there any option to control it?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D99565/new/
https://reviews.llvm.org/D99565
More information about the cfe-commits
mailing list