<div dir="ltr">I don't think it is an hard requirement. Simply good to have.<br></div><div class="gmail_extra"><br><div class="gmail_quote">2014-10-30 21:49 GMT-07:00 Hal Finkel <span dir="ltr"><<a href="mailto:hfinkel@anl.gov" target="_blank">hfinkel@anl.gov</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">----- Original Message -----<br>
> From: "deadal nix" <<a href="mailto:deadalnix@gmail.com">deadalnix@gmail.com</a>><br>
> To: "Philip Reames" <<a href="mailto:listmail@philipreames.com">listmail@philipreames.com</a>><br>
> Cc: "Chandler Carruth" <<a href="mailto:chandlerc@google.com">chandlerc@google.com</a>>, "Hal Finkel" <<a href="mailto:hfinkel@anl.gov">hfinkel@anl.gov</a>>, "llvm-commits"<br>
> <<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a>><br>
</span><span class="">> Sent: Thursday, October 30, 2014 11:45:12 PM<br>
> Subject: Re: Add a pass to convert aggregate loads/stores into scalar load stores<br>
><br>
</span><div><div class="h5">> Option 4 - Transform to series of smaller integer loads<br>
> - This appears to be what this patch implements. Not entirely sure<br>
> why this was chosen.<br>
><br>
><br>
> Yes, this is what this diff is doing.<br>
><br>
><br>
> This option allow to keep pointerness of load/stores (it could be<br>
> refined in this patch, but the idea is there). It also avoid to<br>
> generate a bunch of small load when a big load would suffice.<br>
><br>
><br>
> Making a big load of iN (option 2) is also reasonable, but in this<br>
> case, the load won't respect the target alignment's constraints. I<br>
> thought it was a problem so I went for the option 3 instead.<br>
><br>
><br>
> My first approach was option 1, but i was told that doing this in a<br>
> separate pass was a better approach. Option 5 do not really solve<br>
> the problem. It also force to generate an alloca in the first block,<br>
> so it is not possible to work at a basic block level anymore.<br>
<br>
</div></div>Why do you think that the transformation needs to be basic-block local?<br>
<span class="HOEnZb"><font color="#888888"><br>
 -Hal<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
><br>
<br>
--<br>
Hal Finkel<br>
Assistant Computational Scientist<br>
Leadership Computing Facility<br>
Argonne National Laboratory<br>
</div></div></blockquote></div><br></div>