[llvm-commits] PATCH: Preserving the 'nsw' flag in the instruction combiner.

Pranav Bhandarkar pranavb at codeaurora.org
Wed Aug 10 20:14:51 PDT 2011


Hi Eli and Nick,

>>A couple notes on the patch:
>>
>>1. When submitting a patch, please include the test in the patch itself.
>>2. Using APInt::sadd_ov and APInt::ssub_ov might simplify the logic here a
bit.

Thanks for your suggestions. I have incorporated your suggestions in the
attached revision of my patch. Please do let me know if this is ok to
commit.

Thanks,
Pranav 
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: MaintainNoSignedWrapPatch_Integrated.txt
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20110810/d1dc47dc/attachment.txt>


More information about the llvm-commits mailing list