[PATCH] D55263: [CodeGen][ExpandMemcmp] Add an option for allowing overlapping loads.

[John Reagan via Phabricator via llvm-commits]

JohnReagan added inline comments.


================
Comment at: lib/Target/X86/X86TargetTransformInfo.cpp:2903
+    // All GPR loads can be unaligned, and vector loads too starting form SSE2.
+    Options.AllowOverlappingLoads = true;
     return Options;
----------------
Should this be guarded with hasSSE2()?  Does it makes sense for -no-sse compiles?


Repository:
  rL LLVM

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

https://reviews.llvm.org/D55263