<div dir="ltr">This seems like a better solution to me, but I'd like to hear from someone who actually uses floating point values in LLVM-land if they think this is OK.</div><div class="gmail_extra"><br><br><div class="gmail_quote">
On Tue, Oct 29, 2013 at 4:22 PM, Aaron Ballman <span dir="ltr"><<a href="mailto:aaron@aaronballman.com" target="_blank">aaron@aaronballman.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Here's a patch with an attempt at llvm::huge_valf (I also included<br>
vald and vall for consistency).  It compiles cleanly with VS 2013,<br>
though it's impossible to tell how it affects tests since there is<br>
currently a miscompile that causes most of them to crash.<br>
<span class="HOEnZb"><font color="#888888"><br>
~Aaron<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
On Tue, Oct 29, 2013 at 6:55 PM, Reid Kleckner <<a href="mailto:rnk@google.com">rnk@google.com</a>> wrote:<br>
> On Tue, Oct 29, 2013 at 3:48 PM, Aaron Ballman <<a href="mailto:aaron@aaronballman.com">aaron@aaronballman.com</a>><br>
> wrote:<br>
>><br>
>> There were about 15 warnings in several places from it.<br>
>><br>
>> LiveIntervalAnalysis.cpp<br>
>> RegAllocGreedy.cpp in three places<br>
>> Spiller.cpp<br>
>> LiveInterval.h in two places<br>
>><br>
>> If it was just in a single header file and getting tripped over on<br>
>> include, I would be okay with it, but it's a bit more widely dispersed<br>
>> than that.<br>
>><br>
>> Alternatively, we could make an llvm::huge_valf that does suppress the<br>
>> warning and gets used in place of HUGE_VALF, if that's preferable?<br>
><br>
><br>
> llvm::huge_valf does sound preferable, as annoying as it is not to be able<br>
> to use standard stuff from math.h.<br>
</div></div></blockquote></div><br></div>