<div dir="ltr">LGTM for branch, thanks!<br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Aug 14, 2015 at 3:52 AM, Daniel Sanders <span dir="ltr"><<a href="mailto:Daniel.Sanders@imgtec.com" target="_blank">Daniel.Sanders@imgtec.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Well that seemed to work. Hans and Richard are still CC'd when the list sends the email back to me as long as they are at the front of the CC list *shrug*. I've mentioned it to Tanya.<br>
<span class="im HOEnZb"><br>
> -----Original Message-----<br>
> From: Daniel Sanders<br>
</span><div class="HOEnZb"><div class="h5">> Sent: 14 August 2015 11:27<br>
> To: Daniel Sanders<br>
> Cc: Hans Wennborg; Richard Smith; Jaydeep Patil; Mohit Bhakkad; Sagar<br>
> Thakur; <a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a><br>
> Subject: RE: [compiler-rt] r244646 - [ubsan][mips] Revise r243384 to avoid<br>
> special casing big-endian mips.<br>
><br>
> ??? Hans and Richard disappeared from the CC list again. Trying one more<br>
> time but this time at the other end of the CC list.<br>
><br>
> > -----Original Message-----<br>
> > From: llvm-commits [mailto:<a href="mailto:llvm-commits-bounces@lists.llvm.org">llvm-commits-bounces@lists.llvm.org</a>] On<br>
> Behalf<br>
> > Of Daniel Sanders via llvm-commits<br>
> > Sent: 14 August 2015 11:12<br>
> > Cc: Jaydeep Patil; Mohit Bhakkad; <a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>; Sagar<br>
> Thakur<br>
> > Subject: RE: [compiler-rt] r244646 - [ubsan][mips] Revise r243384 to avoid<br>
> > special casing big-endian mips.<br>
> ><br>
> > Let's try that again, this time with the correct list CC'd. If Outlook is going to<br>
> > make me re-add the list manually on each reply, I guess I'll have to remove<br>
> > the old list from my autocomplete :-).<br>
> ><br>
> > > -----Original Message-----<br>
> > > From: Daniel Sanders<br>
> > > Sent: 14 August 2015 11:08<br>
> > > To: Daniel Sanders<br>
> > > Cc: Sagar Thakur; Mohit Bhakkad; Jaydeep Patil; llvm-<br>
> > <a href="mailto:commits@cs.uiuc.edu">commits@cs.uiuc.edu</a>;<br>
> > > Hans Wennborg; Richard Smith<br>
> > > Subject: RE: [compiler-rt] r244646 - [ubsan][mips] Revise r243384 to avoid<br>
> > > special casing big-endian mips.<br>
> > ><br>
> > > That's odd. Richard and Hans are listed as recipients in my sent mail but<br>
> > don't<br>
> > > show up the mail I received from llvm-commits. Attempting to re-add<br>
> > them.<br>
> > ><br>
> > > > -----Original Message-----<br>
> > > > From: llvm-commits [mailto:<a href="mailto:llvm-commits-bounces@lists.llvm.org">llvm-commits-bounces@lists.llvm.org</a>] On<br>
> > > Behalf<br>
> > > > Of Daniel Sanders via llvm-commits<br>
> > > > Sent: 11 August 2015 19:46<br>
> > > > To: <a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a><br>
> > > > Cc: Sagar Thakur; Mohit Bhakkad; Jaydeep Patil<br>
> > > > Subject: RE: [compiler-rt] r244646 - [ubsan][mips] Revise r243384 to<br>
> avoid<br>
> > > > special casing big-endian mips.<br>
> > > ><br>
> > > > This is the revision to r243384 that was requested in its merge request<br>
> at<br>
> > > > <a href="http://reviews.llvm.org/D11448" rel="noreferrer" target="_blank">http://reviews.llvm.org/D11448</a>. I've tested it on a big-endian MIPS<br>
> > system<br>
> > > > as well as on my x86_64 system and it works correctly for me. I'm just<br>
> > > waiting<br>
> > > > for it to work it's way through the buildbots.<br>
> > > ><br>
> > > > Richard: Assuming the buildbots are ok, is this ok to merge to LLVM<br>
> 3.7.0?<br>
> > > > ________________________________________<br>
> > > > From: llvm-commits [<a href="mailto:llvm-commits-bounces@lists.llvm.org">llvm-commits-bounces@lists.llvm.org</a>] on behalf<br>
> of<br>
> > > > Daniel Sanders via llvm-commits [<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>]<br>
> > > > Sent: 11 August 2015 19:40<br>
> > > > To: <a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a><br>
> > > > Subject: [compiler-rt] r244646 - [ubsan][mips] Revise r243384 to avoid<br>
> > > special<br>
> > > > casing big-endian mips.<br>
> > > ><br>
> > > > Author: dsanders<br>
> > > > Date: Tue Aug 11 13:40:02 2015<br>
> > > > New Revision: 244646<br>
> > > ><br>
> > > > URL: <a href="http://llvm.org/viewvc/llvm-project?rev=244646&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=244646&view=rev</a><br>
> > > > Log:<br>
> > > > [ubsan][mips] Revise r243384 to avoid special casing big-endian mips.<br>
> > > ><br>
> > > > Account for the case when uptr is 32-bit instead of trying to fix this case<br>
> > > > using the little endian path.<br>
> > > ><br>
> > > > Modified:<br>
> > > > compiler-rt/trunk/lib/ubsan/ubsan_value.cc<br>
> > > ><br>
> > > > Modified: compiler-rt/trunk/lib/ubsan/ubsan_value.cc<br>
> > > > URL: <a href="http://llvm.org/viewvc/llvm-project/compiler-" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/compiler-</a><br>
> > > ><br>
> > ><br>
> ><br>
> rt/trunk/lib/ubsan/ubsan_value.cc?rev=244646&r1=244645&r2=244646&vie<br>
> > > > w=diff<br>
> > > ><br>
> > ><br>
> ><br>
> ==========================================================<br>
> > > > ====================<br>
> > > > --- compiler-rt/trunk/lib/ubsan/ubsan_value.cc (original)<br>
> > > > +++ compiler-rt/trunk/lib/ubsan/ubsan_value.cc Tue Aug 11 13:40:02<br>
> > 2015<br>
> > > > @@ -83,10 +83,11 @@ FloatMax Value::getFloatValue() const {<br>
> > > > #endif<br>
> > > > case 32: {<br>
> > > > float Value;<br>
> > > > -#if defined(__BIG_ENDIAN__) && !defined(__mips__)<br>
> > > > - // For big endian the float value is in the second 4 bytes<br>
> > > > - // instead of the first 4 bytes.<br>
> > > > - internal_memcpy(&Value, ((const char*)&Val)+4, 4);<br>
> > > > +#if defined(__BIG_ENDIAN__)<br>
> > > > + // For big endian the float value is in the last 4 bytes.<br>
> > > > + // On some targets we may only have 4 bytes so we count<br>
> > backwards<br>
> > > > from<br>
> > > > + // the end of Val to account for both the 32-bit and 64-bit cases.<br>
> > > > + internal_memcpy(&Value, ((const char*)(&Val + 1)) - 4, 4);<br>
> > > > #else<br>
> > > > internal_memcpy(&Value, &Val, 4);<br>
> > > > #endif<br>
> > > ><br>
> > > ><br>
> > > > _______________________________________________<br>
> > > > llvm-commits mailing list<br>
> > > > <a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a><br>
> > > > <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
> > > > _______________________________________________<br>
> > > > llvm-commits mailing list<br>
> > > > <a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a><br>
> > > > <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
> > _______________________________________________<br>
> > llvm-commits mailing list<br>
> > <a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a><br>
> > <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</div></div></blockquote></div><br></div></div>