[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