[PATCH] D88396: [X86] Replace movaps with movups when avx is enabled.

LuoYuanke via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 30 00:50:37 PDT 2020


LuoYuanke added a comment.

> Compiling for SSE this code will likely use the memory form of addps which will fault on the misalignment. I know this patch only targets AVX.
>
> I don’t think you can motivate this change by showing what code you want to accept if the code would crash when compiled with the default SSE2 target.

Yes, this patch only targets AVX.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D88396



More information about the llvm-commits mailing list