[PATCH] D136672: [ExpandMemCmp][AArch64] Add a new option PreferCmpToExpand in inMemCmpExpansionOptions and enable on AArch64

Dave Green via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 25 03:18:27 PDT 2022


dmgreen added a reviewer: Allen.
dmgreen added a comment.

Is this the same as https://reviews.llvm.org/D136244? Looks like there are two tickets for the same issue.

They both solve it differently. D136244 <https://reviews.llvm.org/D136244> looks like it is more general - handling more cases then just bcmp. But this might optimize larger bcmp's? I think I would expect them to become a series of `cmp;ccmp;ccmp;ccmp` when optimized properly though.


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

https://reviews.llvm.org/D136672



More information about the llvm-commits mailing list