[PATCH] Update to Cmake for VS 2013

Reid Kleckner rnk at google.com
Tue Oct 29 15:55:18 PDT 2013


On Tue, Oct 29, 2013 at 3:48 PM, Aaron Ballman <aaron at aaronballman.com>wrote:

> There were about 15 warnings in several places from it.
>
> LiveIntervalAnalysis.cpp
> RegAllocGreedy.cpp in three places
> Spiller.cpp
> LiveInterval.h in two places
>
> If it was just in a single header file and getting tripped over on
> include, I would be okay with it, but it's a bit more widely dispersed
> than that.
>
> Alternatively, we could make an llvm::huge_valf that does suppress the
> warning and gets used in place of HUGE_VALF, if that's preferable?
>

llvm::huge_valf does sound preferable, as annoying as it is not to be able
to use standard stuff from math.h.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20131029/a30c5289/attachment.html>


More information about the llvm-commits mailing list