lmi
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [lmi] [PATCH] Better support for non-x87 platforms


From: Greg Chicares
Subject: Re: [lmi] [PATCH] Better support for non-x87 platforms
Date: Thu, 5 Jan 2017 15:20:13 +0000
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Icedove/45.4.0

On 2017-01-05 12:58, Vadim Zeitlin wrote:
> On Thu, 5 Jan 2017 03:47:48 +0000 Greg Chicares <address@hidden> wrote:
[...]
> GC> BTW, how is it that we can do things like this:
> GC> +    int z = fegetround();
> GC> when the only applicable header we include is <cfenv>?
> GC> Shouldn't 'std::' be required?
> 
>  Yes, it really like it should -- even though in practice it isn't (because
> this code compiles...) with gcc, probably because it inadvertently pulls in
> fenv.h.

I wouldn't like to imagine that every compiler we might want to use would
duplicate this mistake; it's better to fix it now.

>  It's a trivial change, so I won't propose to make a patch for it, but it
> should still be changed.

I'm already working on it.




reply via email to

[Prev in Thread] Current Thread [Next in Thread]