Bernd Schmidt <ber...@codesourcery.com> writes:
> On 11/22/2013 09:15 AM, Richard Sandiford wrote:
>> I have a patch to upgrade most genrecog warnings into errors. This patch
>> fixes one for bfin about {push,pop}_multiple_operation not being defined
>> at the .md level.
>>
>> Tested by building bfin-elf with the warnings turned to errors, and by
>> comparing the before and after assembly output at -O2 for gcc.c-torture,
>> gcc.dg and g++.dg. OK to install?
>
>> +extern int push_multiple_operation_p (rtx);
>> +extern int pop_multiple_operation_p (rtx);
>
> I was once told that functions with side effects shouldn't be called _p,
> and these ones set up some global variables. But you can check it in
> either this way or with a different name.
Thanks. How about analyze_*_multiple_operation?
Richard
