[PATCH] D35067: [x86, CGP] increase memcmp() expansion up to 4 load pairs

Clement Courbet via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 18 06:25:17 PDT 2017


courbet accepted this revision.
courbet added a comment.
This revision is now accepted and ready to land.

In https://reviews.llvm.org/D35067#812789, @RKSimon wrote:

> Commandeering as @spatel is on holiday.
>
> Does anyone have any reservations about accepting this ?


As mentioned before I'm excited by that change and I'm convinced it's the right approach. I'm not sure about all the implications though so It'd be great if other reviewers could take a look too.


https://reviews.llvm.org/D35067





More information about the llvm-commits mailing list