This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [patch] libstdc++/60637 Fix C++98 std::signbit<long double>

From: Marc Glisse <marc dot glisse at inria dot fr>
To: Jonathan Wakely <jwakely at redhat dot com>
Cc: libstdc++ at gcc dot gnu dot org, gcc-patches at gcc dot gnu dot org
Date: Mon, 18 Jan 2016 21:24:04 +0100 (CET)
Subject: Re: [patch] libstdc++/60637 Fix C++98 std::signbit<long double>
Authentication-results: sourceware.org; auth=none
References: <20160118162856 dot GY15084 at redhat dot com>
Reply-to: libstdc++ at gcc dot gnu dot org

On Mon, 18 Jan 2016, Jonathan Wakely wrote:

This fixes PR60637 by using the appropriate built-in for the size of
the argument type. In Bugzilla Marc asked why we don't just use the
same code as for C++11, but I want to make this less intrusive change
on the branches (trunk is already OK anyway).

That's fine for the branch indeed, but I don't see why we couldn't removethat illegal signbit declaration on trunk in C++98 mode. It works but itis still wrong to have it. Now that C++14 is the default, explicitlyasking for C++98 looks like a clear sign to me that people don't want suchextensions.


--
Marc Glisse

Follow-Ups:
- Re: [patch] libstdc++/60637 Fix C++98 std::signbit<long double>
  - From: Jonathan Wakely

References:
- [patch] libstdc++/60637 Fix C++98 std::signbit<long double>
  - From: Jonathan Wakely

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]