Re: [bfin] Avoid genrecog warning

2013-11-22 Thread Bernd Schmidt
On 11/22/2013 12:50 PM, Richard Sandiford wrote: > Bernd Schmidt writes: >> On 11/22/2013 09:15 AM, Richard Sandiford wrote: >>> +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 _

Re: [bfin] Avoid genrecog warning

2013-11-22 Thread Richard Sandiford
Bernd Schmidt 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 warning

Re: [bfin] Avoid genrecog warning

2013-11-22 Thread Bernd Schmidt
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 > c

[bfin] Avoid genrecog warning

2013-11-22 Thread Richard Sandiford
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.