[PATCH] D68360: PR41162 Implement LKK remainder and divisibility algorithms
Roman Lebedev via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sun Oct 6 11:11:12 PDT 2019
lebedev.ri added inline comments.
================
Comment at: llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp:3966
+/// Computation" (LKK)
+SDValue DAGCombiner::foldUREM(SDNode *node) {
+ SDLoc DL(node);
----------------
TG908 wrote:
> lebedev.ri wrote:
> > This isn't folding, it's lowering.
> > This should be in `TargetLowering.{h,cpp}`
> Good point. I'll move it.
While you are at it, please split `srem` part into a new patch.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D68360/new/
https://reviews.llvm.org/D68360
More information about the llvm-commits
mailing list