[PATCH] define TARGET_PRINTF_POINTER_FORMAT for powerpc-linux (77837)

Martin Sebor msebor@gmail.com
Tue Oct 4 02:11:00 GMT 2016


On 10/03/2016 07:10 PM, Segher Boessenkool wrote:
> On Mon, Oct 03, 2016 at 05:30:35PM -0600, Martin Sebor wrote:
>> The attached patch adds definitions of TARGET_PRINTF_POINTER_FORMAT
>> to the rs6000 pair of linux.h and linux64.h headers, analogous to
>> the config/linux.h header.  This appears to be necessary since
>> unlike most other back ends, the rs6000 back end doesn't include
>> the latter linux.h.
>>
>> The patch fixes bug 77837 - missing -Wformat-length warning for %p
>> with null argument on powerpc64.
>>
>> Thanks
>> Martin
>
>> PR target/77837 - missing -Wformat-length warning for %p with null argument on powerpc64
>>
>> gcc/ChangeLog:
>> 2016-10-03  Martin Sebor  <msebor@redhat.com>
>>
>> 	PR target/77837
>> 	* config/rs6000/linux.h (TARGET_PRINTF_POINTER_FORMAT): Define.
>> 	* config/rs6000/linux64.h (TARGET_PRINTF_POINTER_FORMAT): Likewise.
>
> Okay for trunk, thanks!
>
>
> Segher
>
>
> p.s. You forgot to cc: the maintainers, and the email subject doesn't
> start with "rs6000" or similar either, I found this mail by accident...

Thanks and my bad.  I thought Bill was one of them/you.  I'll
remember to CC you and David in the future.  Better yet, with
my increasingly volatile memory, I might write a script to do
it for me.

Martin



More information about the Gcc-patches mailing list