[PATCH] D29528: [DAGCombiner] Push truncate through adde when the carry isn't used.

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 3 18:32:53 PST 2017


arsenm added inline comments.


================
Comment at: lib/CodeGen/SelectionDAG/DAGCombiner.cpp:7781
   }
+
   // fold (truncate (load x)) -> (smaller load x)
----------------
Extra whitespace change


================
Comment at: lib/CodeGen/SelectionDAG/DAGCombiner.cpp:7870-7872
+  if (N0.getOpcode() == ISD::ADDE &&
+      N0.hasOneUse() &&
+      !N0.getNode()->hasAnyUseOfValue(1)) {
----------------
Needs a legality check for the resulting adde?


https://reviews.llvm.org/D29528





More information about the llvm-commits mailing list