<div dir="ltr">On 5 June 2013 16:24, Mark Lacey <span dir="ltr"><<a href="mailto:mark.lacey@apple.com" target="_blank">mark.lacey@apple.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

Can someone please review, and commit if everything looks good?<br></blockquote><div><br></div><div>+static bool CanMergeValues(Value* First, Value* Second) {<br></div><div><br></div><div>+static Value* ChooseMergedValue(Value* First, Value* Second)  {<br>

</div><div><br></div><div style>This file is overwhelmingly star-on-the right style. Please use "Value *" in place of "Value* " for consistency.</div><div style><br></div><div style>+  assert(CanMergeValues(First, Second));</div>

<div style><br></div><div style>Please add a " && message" to this assert.</div><div style><br></div><div style><div>--- /dev/null</div><div>+++ b/test/Transforms/SimplifyCFG/UndefPHIEdgeBlockMerge.ll</div>

<div><br></div><div style>Please merge your test in with 2008-05-16-PHIBlockMerge.ll (and remove the date from its name, date-named tests are deprecated). Possibly also merge 2009-01-18-PHIPropCrash.ll in as well.</div><div style>

<br></div><div style>Nick</div><div style><br></div></div></div></div></div>