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

[Clement Courbet via Phabricator via llvm-commits]

courbet added a comment.

In D55263#1329984 <https://reviews.llvm.org/D55263#1329984>, @spatel wrote:

> Unrelated diffs got uploaded?


Weird. They are not in my local commit. I guess it's a git-svn weirdness.



================
Comment at: lib/Target/X86/X86TargetTransformInfo.cpp:2892-2893
     TTI::MemCmpExpansionOptions Options;
     // TODO: enable AVX512 when the DAG is ready.
     // if (ST->hasAVX512()) Options.LoadSizes.push_back(64);
     if (ST->hasAVX2()) Options.LoadSizes.push_back(32);
----------------
spatel wrote:
> Independent of this patch, but I think this can be enabled. See:
> rL342989
Great, I'll look at it in a followup patch.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D55263