[PATCH] D28119: [InstCombine][X86] Add DemandedElts support for PMULDQ/PMULUDQ instructions

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 26 15:20:00 PST 2016


RKSimon marked 2 inline comments as done.
RKSimon added inline comments.


================
Comment at: lib/Transforms/InstCombine/InstCombineSimplifyDemanded.cpp:1459
+                                        Depth + 1);
+      if (TmpV) { II->setArgOperand(1, TmpV); MadeChange = true; }
+
----------------
craig.topper wrote:
> Why is this code formatted differently than the same code for ArgOperand 0?
Because for some reason I used clang-format on one and kept to the formatting for the rest of the file for the other.....


Repository:
  rL LLVM

https://reviews.llvm.org/D28119





More information about the llvm-commits mailing list