<div dir="ltr">Hal, Bill,<div><br></div><div>Thanks for the review and suggestions! Committed in r219441.</div><div><br></div><div>I'll create a new patch with the new move instructions. Creating the new feature sounds like a good idea!</div><div><br></div><div>Thanks again,</div><div>Samuel</div>







</div><div class="gmail_extra"><br><div class="gmail_quote">2014-10-09 14:57 GMT-04:00 Bill Schmidt <span dir="ltr"><<a href="mailto:wschmidt@linux.vnet.ibm.com" target="_blank">wschmidt@linux.vnet.ibm.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Thu, 2014-10-09 at 14:16 -0400, Samuel F Antao wrote:<br>
> Hi all,<br>
><br>
><br>
> The current implementation of GPR->FPR register moves uses a stack<br>
> slot. This mechanism writes a double word and reads a word. In<br>
> big-endian the load address must be displaced by 4-bytes in order to<br>
> get the right value. In little endian this is no longer required. This<br>
> patch fixes the issue and adds LE regression tests to<br>
> fast-isel-conversion which currently expose this problem.<br>
><br>
</span>Thanks, Samuel!  LGTM.<br>
<span class=""><br>
> This move can be done using the VSX capabilities that are currently<br>
> deactivated due to some other problems. I understand Bill and Bill are<br>
> fixing some of the VSX issues in FastISel, but this patch seems to be<br>
> orthogonal to that. I can contribute another implementation of the<br>
> moves using these instructions, which I've already implemented, once<br>
> it is okay to activate VSX for little endian.<br>
><br>
</span>Well, we will need to add a feature for ISA 2.07 (thanks for that fix,<br>
btw) vector support before the direct move instructions can be used.<br>
Currently FeatureVSX implies only ISA 2.06 support, which is all that's<br>
been (partially) implemented at this point.  Probably something like<br>
FeatureP8Vector to denote the changes to both VMX and VSX instructions<br>
for 2.07.<br>
<br>
Bill<br>
><br>
> Thanks!<br>
> Samuel<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
</div></div></blockquote></div><br></div>