[PATCH] D30401: Refactor the multiply-accumulate combines to act on ARMISD::ADD[CE] nodes, instead of the generic ISD::ADD[CE].

Sam Parker via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 1 01:28:36 PST 2017


samparker added inline comments.


================
Comment at: lib/Target/ARM/ARMISelDAGToDAG.cpp:3098
   case ARMISD::UMLAL:{
     // UMAAL is similar to UMLAL but it adds two 32-bit values to the
     // 64-bit multiplication result.
----------------
Its great you moved this to DAGCombine, but just remove the comment too.


https://reviews.llvm.org/D30401





More information about the llvm-commits mailing list