[llvm-dev] compiler-rt builtins on MSVC 2019

[David Major via llvm-dev]

Since that file already includes float.h, I wonder whether the ymath.h
is needed at all. I tried removing it on my MSVC 2017 machine and the
build didn't complain. Could you try the same with 2019?

On Thu, Apr 4, 2019 at 10:12 AM Andreas Jonson via llvm-dev
<llvm-dev at lists.llvm.org> wrote:
>
> Hi,
>
>
>
> compiler-rt builtins currently doesn't build on MSVC 2019,
>
> I the problem is that compiler-rt\lib\builtins\int_math.h includes the header ymath.h.
>
> according to eg. https://docs.microsoft.com/en-us/cpp/c-runtime-library/reference/finite-finitef?view=vs-2019 the header to include is float.h
>
> also the ymath.h file contains the comment /* ymath.h internal header */ so probably shall not be used.
>
>
>
> I do not know how compiler-rt works only tried to compile rustc that is using compiler-rt
>
> How shall I go forward with this problem?
>
>
>
> BR/Andreas Jonson
>
>
>
> _______________________________________________
> LLVM Developers mailing list
> llvm-dev at lists.llvm.org
> https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev