<div dir="ltr"><div><br></div>Thanks Renato.<div><br></div><div>I was confused myself since I haven't touched this part of the code for a while.</div><div><br></div><div><div><div>       // way is semantically identical, but fast-isel and the optimizer</div>
<div>       // generally likes scalar values better than FCAs.</div></div><div>Should we try to investigate if this patch changes performance for AAPCS-VFP?</div></div><div><br></div><div><div>+      // We cannot do this for functions using the AAPCS calling convention,</div>
<div>+      // as structures are treated differently by that calling convention.</div></div><div>Is it possible to be more informative, maybe explain how [why] structures are treated differently?</div><div><br></div><div>
<div>-      if (llvm::StructType *st = dyn_cast<llvm::StructType>(argType)) {</div><div>+      llvm::StructType *st = dyn_cast<llvm::StructType>(argType);</div><div>+      if (st && !isAAPCSVFP(FI, getTarget())) {</div>
</div><div>Since the patch is trying to solve issues for HA, is it beneficial to only disable this for HA, instead of for all structs?<br></div><div><br></div><div>If there is no performance issue with this patch, we should try to have a corresponding testing case in the llvm part of the patch, for each testing case that currently exists in clang for HA (i.e the testing cases that are updated in the clang part of the patch).</div>
<div><br></div><div>Thanks for working on this, Oliver.</div><div><br></div><div>Manman</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Mar 17, 2014 at 2:43 PM, Renato Golin <span dir="ltr"><<a href="mailto:renato.golin@linaro.org" target="_blank">renato.golin@linaro.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
  Manman,<br>
<br>
  Looks at the discussion in D3082.<br>
<br>
  --renato<br>
<br>
<a href="http://llvm-reviews.chandlerc.com/D3083" target="_blank">http://llvm-reviews.chandlerc.com/D3083</a><br>
<div class="HOEnZb"><div class="h5">_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
</div></div></blockquote></div><br></div>