[PATCH] D103323: [DAGCombiner] Add support for mulhi const folding in DAGCombiner

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 28 10:18:35 PDT 2021


craig.topper added inline comments.


================
Comment at: llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp:4455
+  if (SDValue C =
+          DAG.FoldConstantArithmetic(ISD::MULHS, SDLoc(N), VT, {N0, N1}))
+    return C;
----------------
Use DL variable that already exists


================
Comment at: llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp:4508
+  if (SDValue C =
+          DAG.FoldConstantArithmetic(ISD::MULHU, SDLoc(N), VT, {N0, N1}))
+    return C;
----------------
Use DL variable that already exists


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D103323/new/

https://reviews.llvm.org/D103323



More information about the llvm-commits mailing list