From: Diego Novillo <dnovillo@google.com>
To: Jeffrey Yasskin <jyasskin@google.com>
Cc: reply@codereview.appspotmail.com, gcc-patches@gcc.gnu.org
Subject: Re: [google] Merge r173574 to google/gcc-4_6 to fix an incompatibility between C++98 and C++0x (issue4592057)
Date: Tue, 14 Jun 2011 19:59:00 -0000 [thread overview]
Message-ID: <BANLkTinO3+ZM-PBjv9FRPXxCOFufngOWDA@mail.gmail.com> (raw)
In-Reply-To: <20110614184508.2E50B1695C7@enki.mtv.corp.google.com>
On Tue, Jun 14, 2011 at 14:45, Jeffrey Yasskin <jyasskin@google.com> wrote:
> In C++0x mode, without this patch, calls to a user-defined trunc() function with an argument in namespace std and
> a parameter type that has an implicit conversion from the argument's type, cause infinite recursion in std::trunc().
>
> This patch also includes http://gcc.gnu.org/viewcvs/trunk/libstdc%2B%2B-v3/testsuite/26_numerics/headers/cmath/overloads_c%2B%2B0x_neg.cc?view=markup&pathrev=173574 and http://gcc.gnu.org/viewcvs/trunk/libstdc%2B%2B-v3/testsuite/tr1/8_c_compatibility/cmath/overloads_neg.cc?view=markup&pathrev=173574, but `svn diff` didn't capture them.
Yeah, svn diff never picks up added files. Silly thing.
>
> Tested with `make check-c++` on x86_64-unknown-linux-gnu.
>
> 2011-06-14 Jeffrey Yasskin <jyasskin@google.com>
>
> Merge r173574 to google/gcc-4_6.
> * include/c_global/cmath (acosh, asinh, atanh, cbrt, copysign,
> erf, erfc, exp2, expm1, fdim, fma, fmax, hypot, ilogb, lgamma,
> llrint, llround, log1p, log2, logb, lrint, lround, nearbyint,
> nextafter, nexttoward, remainder, remquo, rint, round, scalbln,
> scalbn, tgamma, trunc): Use __enable_if on the return type.
> * include/tr1/cmath: Likewise.
> * testsuite/26_numerics/headers/cmath/overloads_c++0x_neg.cc: New.
> * testsuite/tr1/8_c_compatibility/cmath/overloads_neg.cc: Likewise.
>
Any reason not to put this in google/main?
Diego.
next prev parent reply other threads:[~2011-06-14 19:38 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-06-14 19:04 Jeffrey Yasskin
2011-06-14 19:59 ` Diego Novillo [this message]
2011-06-14 20:13 ` Jeffrey Yasskin
2011-06-14 20:27 ` Diego Novillo
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=BANLkTinO3+ZM-PBjv9FRPXxCOFufngOWDA@mail.gmail.com \
--to=dnovillo@google.com \
--cc=gcc-patches@gcc.gnu.org \
--cc=jyasskin@google.com \
--cc=reply@codereview.appspotmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).