<div dir="ltr">Adding llvm-commits so that it is saved in the list history.<div>Thanks again for fixing this.</div><div><br></div><div>Kind regards,</div><div>Oleg<br><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Jun 13, 2015 at 1:50 AM, Oleg Ranevskyy <span dir="ltr"><<a href="mailto:llvm.mail.list@gmail.com" target="_blank">llvm.mail.list@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Yes, the tests now compile just fine.<div>Thanks for the patch!</div></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jun 12, 2015 at 9:34 PM, Ahmed Bougacha <span dir="ltr"><<a href="mailto:ahmed.bougacha@gmail.com" target="_blank">ahmed.bougacha@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span>On Mon, Jun 8, 2015 at 10:35 AM, Oleg Ranevskyy<br>
<<a href="mailto:llvm.mail.list@gmail.com" target="_blank">llvm.mail.list@gmail.com</a>> wrote:<br>
> Hi Ahmed,<br>
><br>
> This patch added 3 new compiler-rt tests that include "fp_test.h".<br>
> "fp_test.h" uses the __uint128_t type, which is not available on 32 bits.<br>
> Some other tests also use the header but protect its include with `#if<br>
> __LDBL_MANT_DIG__ == 113`. The added tests don't do this, so compilation<br>
> fails.<br>
><br>
> Would you consider adding similar protection to your tests, please?<br>
<br>
</span>Sure, sorry for the breakage.  I hid the long double helpers in<br>
fp_test.h directly instead, in r239630.  Is that good?<br>
<span><font color="#888888"><br>
-Ahmed<br>
</font></span><div><div><br>
> Thank you.<br>
> Oleg<br>
><br>
><br>
> 12.05.2015 04:39, Ahmed Bougacha пишет:<br>
><br>
> Hi scanon,<br>
><br>
> Mostly uninteresting, except:<br>
><br>
> - in __extendXfYf2, when checking if the number is normal, the current code<br>
> relies on the unsignedness of src_rep_t, which is fine.  Except that when<br>
> sizeof(src_rep_t) < sizeof(int), the result gets promoted to int, the<br>
> signedness of which breaks the comparison.  I added an explicit cast to<br>
> avoid that;  it shouldn't affect other types.<br>
><br>
> - we can't just pass __fp16 around (you even get a clang error), so I made<br>
> uint16_t explicit.<br>
><br>
> - is there a good way to define aliases (for the gnu_*_ieee names) ?  Using<br>
> the attribute, on darwin, clang complains about only weak aliases being<br>
> supported, so I went with the simple def.<br>
><br>
><br>
> I also need to finish writing unit tests in compiler-rt proper (I paused<br>
> when I realized lit+cmake isn't supported).  In the meantime, opinions<br>
> welcome.<br>
><br>
> <a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__reviews.llvm.org_D9693&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=mQ4LZ2PUj9hpadE3cDHZnIdEwhEBrbAstXeMaFoB9tg&m=YdOeAZ2sWs5JefQug0Sqhj4ecWlN5UEDUHCYpmmV8-w&s=wetR1a-72msVM0UdQHyVFIgXEXcCbqzyH4xEgX5NsiA&e=" rel="noreferrer" target="_blank">http://reviews.llvm.org/D9693</a><br>
><br>
> Files:<br>
>   lib/builtins/CMakeLists.txt<br>
>   lib/builtins/extendhfsf2.c<br>
>   lib/builtins/fp_extend.h<br>
>   lib/builtins/fp_extend_impl.inc<br>
>   lib/builtins/fp_trunc.h<br>
>   lib/builtins/truncdfhf2.c<br>
>   lib/builtins/truncsfhf2.c<br>
><br>
> EMAIL PREFERENCES<br>
>   <a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__reviews.llvm.org_settings_panel_emailpreferences_&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=mQ4LZ2PUj9hpadE3cDHZnIdEwhEBrbAstXeMaFoB9tg&m=YdOeAZ2sWs5JefQug0Sqhj4ecWlN5UEDUHCYpmmV8-w&s=HOFZfeG4egQZXnsrInke4x99gW8KQYpC4rEqEEQOku8&e=" rel="noreferrer" target="_blank">http://reviews.llvm.org/settings/panel/emailpreferences/</a><br>
><br>
><br>
><br>
> _______________________________________________<br>
> llvm-commits mailing list<br>
> <a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
><br>
><br>
</div></div></blockquote></div><br></div>
</div></div></blockquote></div><br></div></div></div>