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

Clement Courbet via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 5 06:59:47 PST 2018


courbet added a comment.

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

> I just looked over the codegen changes so far, but I want to add some more knowledgeable x86 hackers to have a look too. There are 2 concerns:
>
> 1. Are there any known uarch problems with overlapping loads?
> 2. Are there any known uarch problems with unaligned accesses (either scalar or SSE)?


FYI we're already using overlapping loads for memcpy lowering: https://godbolt.org/z/9iSE3g


Repository:
  rL LLVM

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

https://reviews.llvm.org/D55263





More information about the llvm-commits mailing list