This is the mail archive of the gcc@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: Problem with 447.dealII in spec2006 because of r240707


I'd +1 vote to send them a patch. I've had to do this for other
compilers. If you need a hand, I can give you some tips on how to do
that and also where to check if this has already been fixed.

Thanks

On Tue, Oct 4, 2016 at 11:43 PM, Jeff Law <law@redhat.com> wrote:
> On 10/04/2016 09:41 AM, Marek Polacek wrote:
>>
>> On Tue, Oct 04, 2016 at 08:38:00AM -0700, Andrew Pinski wrote:
>>>
>>> On Tue, Oct 4, 2016 at 8:33 AM, Bill Seurer <seurer@linux.vnet.ibm.com>
>>> wrote:
>>>>
>>>> parameter_handler.cc: In member function 'double
>>>> ParameterHandler::get_double(const string&) const':
>>>> parameter_handler.cc:777:28: error: ISO C++ forbids comparison between
>>>> pointer and integer [-fpermissive]
>>>>    AssertThrow ((s.c_str()!='\0') || (*endptr == '\0'),
>>>>                             ^
>>>> With the recent revision r240707 comparing a pointer with \0 became an
>>>> error.  Unfortunately this is used in several spots in the test case
>>>> 447.dealII in spec2006 (one example above).  There doesn't appear to be
>>>> a
>>>> way to disable this error check and we're not supposed to change the
>>>> spec
>>>> test cases.  Any ideas on how to work around this?
>>>
>>>
>>> Did you try -fpermissive ?  Because that seems like it was listed ...
>>
>>
>> -fpermissive ought to help, but really best would be to use NULL instead
>> of '\0'...
>
> And report the issue to SPEC.  They sometimes are willing to fix SPEC when
> the code is clearly wrong.
>
> jeff


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