<div dir="ltr">Those regression tests seem really large. Is there any way you could reduce them?<div><br></div><div><div>+/// If \param [in] BB has more than one predecessor that is a conditional</div><div>+/// branch, attempt to use parallel and/or for the branch condtion. \returns</div>
<div>+/// true on success.</div><div><br></div><div style>Spelling: s/condtion/condition/.</div><div style>Also, typically doxygen commands are put on a line of their own instead of inline with prose (I'm not sure that doxygen even accepts them inline like you have them here).</div>
<div style><br></div><div style><div>+/// Before:</div><div>+///   ......</div><div>+///   %cmp10 = fcmp une float %tmp1, %tmp2</div><div>+///   br i1 %cmp1, label %if.then, label %lor.rhs</div><div><br></div><div style>You want \code + \endcode <<a href="http://www.stack.nl/~dimitri/doxygen/manual/commands.html#cmdcode">http://www.stack.nl/~dimitri/doxygen/manual/commands.html#cmdcode</a>> enclosing this code display, so that doxygen will format it correctly.</div>
<div style><br></div><div style><div>+      if (UCond || !PP || (Preds.count(PP) == 0) ||</div><div>+`        (PTI->getNumSuccessors() != 1) ||</div><div>+`        Pred->hasAddressTaken())</div><div>+        return false;</div>
<div><br></div><div style>Please don't use hard tabs. Your patch also introduces a bunch of trailing whitespace. Also, there are a number of formatting issues; I recommend using clang-format on the regions of code that you are adding to ensure they comply with our preferred code formatting.</div>
</div></div><div><br></div><div style>-- Sean Silva</div></div></div>