[RFA 0/6] Add -Wunused-but-set-parameter and -Wunused-but-set-variable

Pedro Alves palves@redhat.com
Thu Jul 21 15:18:00 GMT 2016


On 07/21/2016 03:47 PM, Pedro Alves wrote:
> On 07/21/2016 01:15 PM, Pedro Alves wrote:
>> On 07/21/2016 12:56 PM, Pedro Alves wrote:
>>> On 07/21/2016 12:35 PM, Pedro Alves wrote:
>>>> On 07/21/2016 12:10 PM, Yao Qi wrote:
>>>>> On Thu, Jul 21, 2016 at 11:38 AM, Pedro Alves <palves@redhat.com> wrote:
>>>>
>>>>>>
>>>>>> I'd think it'd be acceptable to just build on a couple of the
>>>>>> more common hosts, in the name of forward progress.
>>>>>>
>>>>>
>>>>> Yes, I agree.
>>>>>
>>>>
>>>> I had a mingw-w64 build tree already handy, so I gave it a try.  It
>>>> needs a couple patches.  I'll send them as reply to this email.
>>>
>>> I tried a cross to macOS/OS X/darwin/whatever-it's-called-nowadays
>>> now, with -Wunused-but-set-parameter -Wunused-but-set-variable + -O2,
>>> and no new warnings appeared.
>>
>> djgpp [1] has a few pre-existing build errors due to C++, but after 
>> fixing those, -Wunused-but-set-parameter -Wunused-but-set-variable
>> introduce no new problems.
> 
> AIX (gcc111 on the compile farm) compiles fine.  It fails to link
> due to the stabs overflow problem, but that's a preexisting problem.

No build regressions on

 PPC64 GNU/Linux
 S390 GNU/Linux 
 x64-64 NetBSD

Any host missing?  I think we're good to go.

Thanks,
Pedro Alves



More information about the Gdb-patches mailing list