This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [C++ patch] PR 65858
- From: Paolo Carlini <paolo dot carlini at oracle dot com>
- To: Prathamesh Kulkarni <prathamesh dot kulkarni at linaro dot org>, gcc Patches <gcc-patches at gcc dot gnu dot org>, Jason Merrill <jason at redhat dot com>
- Date: Thu, 30 Apr 2015 19:51:22 +0200
- Subject: Re: [C++ patch] PR 65858
- Authentication-results: sourceware.org; auth=none
- References: <CAAgBjMkvPPYvL=kLGe8oJ6843jsEL2BZdYDEtWQwXZOGnKyfww at mail dot gmail dot com>
Hi,
On 04/30/2015 06:36 PM, Prathamesh Kulkarni wrote:
Hi,
The attached patch fixes ICE in PR65858.
For the test-case:
int x { 0.5 };
Eventually, we have to add to the testsuite the testcase too (the code
is simpler but equivalent to what we added for 65801, -flto makes all
the difference). And a fix goes in gcc-5-branch too.
Anyway, your proposal makes sense to me (this couple of issues is really
weird, essentially we end up using a pedwarn instead of a plain error
*only* to print [-Wnarrowing] and tell people that the diagnostic can be
suppressed!?!) but we want to ear Jason.
Paolo.