[PATCH] D13732: Combining DIV+REM->DIVREM doesn't belong in LegalizeDAG; move it over into DAGCombiner.

John Brawn via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 15 08:44:34 PDT 2015


john.brawn added a subscriber: john.brawn.
john.brawn added a comment.

>   I'm going to commit the three as a single patch, but I'm splitting them out for ease of reviewing.


Personally I find it easier to review http://reviews.llvm.org/D13733 plus this patch as a single patch, given that that is just modifying changes made by this patch.

This+http://reviews.llvm.org/D13733 looks generally OK to me, though I'm not a DAGCombiner expert so wait until someone who is also gives the OK.


http://reviews.llvm.org/D13732





More information about the llvm-commits mailing list