<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
+  Value *foldFMulConst(Instruction *FMulOrDiv, ConstantFP *C,<br>
+                       Instruction *InsertBefore);<br></blockquote><div><br></div><div style>Formatting.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

+  if (!I || (I->getOpcode() != Instruction::FMul &&<br>
+             I->getOpcode() != Instruction::FDiv)) {<br>
+    return false;<br>
+  }<br>
+<br></blockquote><div><br></div><div style>No need for braces here.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
+Value *InstCombiner::foldFMulConst<br>
+  (Instruction *FMulOrDiv, ConstantFP *C, Instruction *InsertBefore) {<br>
</blockquote><div><br></div><div style>Formatting.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
+        if (C0) {<br>
+          std::swap(C0, C1); std::swap(Opnd0, Opnd1); Swap = true;<br>
+        }<br>
+<br></blockquote><div><br></div><div style>Was going to say no need for braces here, but I missed the Swap = at the end of the line, how about putting that on the next line so it's more clear?</div><div> </div><div style>
-eric</div></div></div></div>