[PATCH] D32916: [DAGCombine] (addcarry 0, 0, X) -> (ext/trunc X)

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 19 10:41:57 PDT 2017

Previous message: [PATCH] D32916: [DAGCombine] (addcarry 0, 0, X) -> (ext/trunc X)
Next message: [PATCH] D32916: [DAGCombine] (addcarry 0, 0, X) -> (ext/trunc X)
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

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

I agree that we shouldn't gate this patch waiting for a reply to the i1024 question. If you're confident that the code in those super-wide cases is still correct, then LGTM.


https://reviews.llvm.org/D32916

Previous message: [PATCH] D32916: [DAGCombine] (addcarry 0, 0, X) -> (ext/trunc X)
Next message: [PATCH] D32916: [DAGCombine] (addcarry 0, 0, X) -> (ext/trunc X)
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

More information about the llvm-commits mailing list