<div dir="ltr"><div>Hi Nadav,</div><div><br></div><div>Thanks for the review!</div><div><br></div>On 29 January 2013 18:46, Nadav Rotem <span dir="ltr"><<a href="mailto:nrotem@apple.com" target="_blank">nrotem@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"><div style="word-wrap:break-word">

<div>The changes to ARMTargetMachine.h are unrelated to the cost model. Lets commit them in a separate patch. <br></div></div></blockquote><div><br></div><div>That's necessary so I can build the TLI via TM->getTargetLowering(), which would have return a generic TargetLowering object, not an ARM one, without it.</div>

<div><br></div><div> </div><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"><div style="word-wrap:break-word">

<div></div><div>The code in ARMTTI::getCastInstrCos looks good.<br></div><div><br></div><div>+                                 ISD, DstTy.getSimpleVT(), SrcTy.getSimpleVT());</div><div><br></div><div>Did we pass the 80-col ?  I am not sure. </div>

</div></blockquote><div><br></div><div>By 1 !! ;) Fixing...</div><div><br></div><div><br></div><div><br></div><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">

<div style="word-wrap:break-word"><div>In your test cases you execute both LLC and OPT. If you are checking the LLC generates the right pattern then this test should be in tests/CodeGen/ARM/.  Can you make the tests smaller ?  You can write a two-line function in LL that takes the arguments and performs the operation on it. </div>

</div></blockquote><div><br></div><div style>I'll use Arnold's snippets...</div><div><br></div><div>cheers,</div><div>--renato</div><div><br></div></div></div></div>