[PATCH] D36645: [X86] When handling addcarry intrinsic, create the flag result with the correct type so we don't crash if we use a memory instruction

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Aug 12 12:56:28 PDT 2017


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

In https://reviews.llvm.org/D36645#839910, @craig.topper wrote:

> Looks like it fixes PR33229. Probably will fix PR24943, PR22571 as well.


LGTM with additional test cases that covers all of these as well.

I agree this should probably be merged into 5.0.


https://reviews.llvm.org/D36645





More information about the llvm-commits mailing list