[PATCH] D99565: [X86] Support replacing aligned vector moves with unaligned moves when avx is enabled.

Mon Apr 12 01:46:51 PDT 2021

LiuChen3 added a comment.

In D99565#2678073 <https://reviews.llvm.org/D99565#2678073>, @craig.topper wrote:

> 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.

Can I understand that if we implement it in isel, you will no longer oppose this patch?

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99565