[PATCH] D57516: [PatternMatch] add special-case uaddo matching for increment-by-one

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 31 14:31:30 PST 2019


spatel added a comment.

In D57516#1379520 <https://reviews.llvm.org/D57516#1379520>, @craig.topper wrote:

> In D57516#1379515 <https://reviews.llvm.org/D57516#1379515>, @RKSimon wrote:
>
> > @craig.topper Are you OK with the INC - ADD changes?
>
>
> Yeah. I imagine it won't be too hard to add a combine to recover it.


Thanks - my first hack at that doesn't account for all the test diffs, but I haven't messed around with those EFLAGS combines much before, so I might've misplaced the fold.
There's still a question about the Hexagon test regardless of whether we're ok with the x86 diffs.


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

https://reviews.llvm.org/D57516





More information about the llvm-commits mailing list