<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On 4 February 2014 00:20, Stepan Dyatkovskiy <span dir="ltr"><<a href="mailto:stpworld@narod.ru" target="_blank">stpworld@narod.ru</a>></span> wrote:<br>
<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>Hi all,</div><div> </div><div>Jan,</div><div>Its proper fix, IMHO. But may be its better to check for alignments we exactly know we support? I mean:</div>
<div>+ if (NumGPRs && (Align == 8 || Align == 16)) && ...</div><div> </div><div>Otherwise you actually could just put the next line:</div><div>+ if (NumGPRs && (Align > 4)) && ...</div></blockquote>
<div><br></div><div>OK, I've changed it to use "Align > 4".</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>Something stopped me from doing that last time. I need Renato and Chandler opinion.</div><div>What strategy should be used here? "Align > 4" or "(Align == 8 || Align == 16)"? Too small point details of course..</div>
<div> </div><div>Comments in method mentioned only 8 bytes alignment:</div><div><div class="">    // Add padding for part of param recovered from GPRs, so<br></div>    // its last byte must be at address K*8 - 1.</div><div>
That should be fixed with something like that:</div><div>-    // its last byte must be at address K*8 - 1.</div><div>+   // For example, if Align == 8, its last byte must be at address K*8 - 1.</div><div> </div><div>The same comments fix should be done in ARMMachineFunctionInfo.h, with "StByValParamsPadding" field.</div>
</blockquote><div><br></div><div>OK, I've changed the comments as above.  I've updated <a href="http://llvm-reviews.chandlerc.com/D2677">http://llvm-reviews.chandlerc.com/D2677</a>.</div><div><br></div><div>I've also changed the commit message to reflect that this wasn't really a bug, just a case that the code didn't handle because it didn't need to (since before the change, only 4 and 8 byte alignments were supported).</div>
<div><br></div><div>Cheers,</div><div>Mark</div><div><br></div></div></div></div>