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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
