[PATCH] D52504: [DAGCombiner] Div/rem folds

[Simon Pilgrim via Phabricator via llvm-commits]

RKSimon added a reviewer: spatel.
RKSimon added inline comments.


================
Comment at: lib/CodeGen/SelectionDAG/DAGCombiner.cpp:3112
   // division-by-zero or remainder-by-zero, so assume the divisor is 1.
   // Similarly, if we're zero-extending a boolean divisor, then assume it's a 1.
 
----------------
You're ignoring this comment - consult simplifyDivRem inside InstructionSimplify.cpp which shows the issue


https://reviews.llvm.org/D52504