[PATCH] D64090: [Codegen][X86][AArch64][ARM][PowerPC] Inc-of-add vs sub-of-not (PR42457)

Hal Finkel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 3 09:39:21 PDT 2019


hfinkel added a comment.

In D64090#1567920 <https://reviews.llvm.org/D64090#1567920>, @lebedev.ri wrote:

> In D64090#1567527 <https://reviews.llvm.org/D64090#1567527>, @efriedma wrote:
>
> > LGTM
>
>
> Thank you for the review.
>
> PPC hook may also need tuning, but i'm not sure as to the incantation needed.


Nothing in the test changes sticks out to me. Go ahead and we'll need to see if we see any performance regressions.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D64090





More information about the llvm-commits mailing list