[PATCH] D51197: [GISel]: Add missing opcodes for overflow intrinsics

[Amara Emerson via Phabricator via llvm-commits]

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

In https://reviews.llvm.org/D51197#1215516, @igorb wrote:

> In https://reviews.llvm.org/D51197#1215499, @aemerson wrote:
>
> > x86 is still using UADDE, and it's generated only by IRTranslator, so if you do this then x86 will lose support for compiling llvm.uadd.with.overflow intrinsics. @igorb what do you think?
> >
> > Which is exposing a hole in our general GISel testing, where individual changes in passes can result in overall support for some input IR to be lost.
>
>
> Hi,
>  I think currently X86 use G_UADDE  generated by Legalizer  to support 64bit add on 32bit architecture.
>
> Regards,
> Igor


Ah yes, I see it now.


Repository:
  rL LLVM

https://reviews.llvm.org/D51197