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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to