Richard Henderson <r...@redhat.com> writes:

> On 04/27/2011 11:13 AM, Anatoly Sokolov wrote:
>>         * config/sparc/sparc.h (PRINT_OPERAND, PRINT_OPERAND_ADDRESS,
>>         PRINT_OPERAND_PUNCT_VALID_P): Remove.
>>         * config/sparc/sparc-protos.h (print_operand): Remove declaration.
>>         * config/sparc/sparc.c (TARGET_PRINT_OPERAND_PUNCT_VALID_P,
>>         TARGET_PRINT_OPERAND, TARGET_PRINT_OPERAND_ADDRESS): Define.
>>         (print_operand): Rename to...
>>         (sparc_print_operand): ...this. Make static. Adjust
>>         sparc_print_operand function call.
>>         (sparc_print_operand_punct_valid_p, sparc_print_operand_address): New
>>         functions.
>
> Ok, except, 
>
>> +  if(code == '#'
>
> Space before (, re-indenting the rest of the condition to match.

This patch broke Solaris 2/SPARC bootstrap which still uses
print_operand in sparc/sol2.h (ASM_OUTPUT_CALL).  A bootstrap with the
obvious fix is currently running.

What is so hard about running grep when removing/renaming symbols???

        Rainer

-- 
-----------------------------------------------------------------------------
Rainer Orth, Center for Biotechnology, Bielefeld University

Reply via email to