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

LuoYuanke via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 29 23:39:19 PDT 2020


LuoYuanke added a comment.

If the address is random, during validation the address is aligned and sanitizers tool doesn't notice it. When run in real time, it crashes randomly. There is no harm to replace movaps with movups, and it can avoid some crash issue. Is it doable to add an option to let user choose movups or movaps?


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