[PATCH] D13757: [x86] promote 'add nsw' to a wider type to allow more combines

Quentin Colombet via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 15 17:47:00 PDT 2015


qcolombet added a comment.

Hi Sanjay

> the simplicity and reach of the x86 isel change was too good for me to pass up. And so if we're not doing any x86 harm with this patch, I would like to get this checked in and continue investigating how to do these transforms more generally.


Sounds good to me!

LGTM once you've addressed Zia's concern.

Cheers,
-Quentin


http://reviews.llvm.org/D13757





More information about the llvm-commits mailing list