<div dir="ltr">Hi Tim,<div><br></div><div>Thanks. I've made the changes you've asked and as you LGTM'd it on IRC, have committed it with revisions r208192, r208193 and r208194.</div><div><br></div><div>Cheers,</div>
<div>James</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On 2 May 2014 16:46, Tim Northover <span dir="ltr"><<a href="mailto:t.p.northover@gmail.com" target="_blank">t.p.northover@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi James,<br>
<br>
>From 0001:<br>
<br>
+      uint32_t OpSize = Flags.isByVal() ? Flags.getByValSize()*8 :<br>
+                                          VA.getLocVT().getSizeInBits();<br>
+      OpSize = (OpSize + 7) / 8;<br>
+<br>
+      // FIXME: This works on big-endian for composite byvals, which<br>
are the common<br>
+      // case. It should also work for fundamental types too.<br>
+      uint32_t BEAlign = 0;<br>
+      if (OpSize < 8 && !Subtarget->isLittleEndian() && !Flags.isByVal())<br>
<br>
Since OpSize is only ever used when !isByVal, this could probably be<br>
simplified, possibly to "if (!Subtarget->isLittleEndian() &&<br>
!Flags.isByVal() && VA.getLocVT().getSizeInBits() < 64" (well,<br>
strictly, <= 56, but I'm unconvinced that there's an actual difference<br>
or that 56 is more correct if there *is*).<br>
<br>
>From 0003:<br>
<br>
+def : Pat<(v1i64 (bitconvert GPR64:$Xn)), (COPY_TO_REGCLASS<br>
GPR64:$Xn, FPR64)>; // b)<br>
<br>
Is that comment (etc) meaningful?<br>
<br>
+def : Pat<(f128 (bitconvert (v4i32 FPR128:$src))), (f128 (EXTv16i8<br>
(REV64v4i32 FPR128:$src), (REV64v4i32 FPR128:$src), (i32 8)))>;<br>
<br>
I think the long .td lines should probably be split now.<br>
<br>
I'd also suggest squashing 0003, 0004 & 0005 into a single commit.<br>
<br>
Other than that, it's looking good I think.<br>
<br>
Cheers.<br>
<span class="HOEnZb"><font color="#888888"><br>
Tim.<br>
</font></span><div class="HOEnZb"><div class="h5">_______________________________________________<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>