breaking cris-elf build with "Re: [RTL] move rtl diagnostics out of toplev.h"

On 1 July 2010 01:07, Hans-Peter Nilsson <hp@bitrange.com> wrote:
> On Thu, 1 Jul 2010, Manuel López-Ibáñez wrote:
>> On 1 July 2010 00:05, Hans-Peter Nilsson <hp@bitrange.com> wrote:
>> > libbackend.a(recog.o): In function `constrain_operands':
>> > /tmp/hpautotest-gcc1/gcc/gcc/recog.c:2616: undefined reference
>> > to `exact_log2'
>> > libbackend.a(recog.o): In function `asm_operand_ok':
>> > /tmp/hpautotest-gcc1/gcc/gcc/recog.c:1756: undefined reference
>> > to `exact_log2'
>> > libbackend.a(reload.o): In function `find_reloads':
>> > /tmp/hpautotest-gcc1/gcc/gcc/reload.c:3363: undefined reference
>> > to `exact_log2'
>> > collect2: ld returned 1 exit status
>> >
>> > with "warning: implicit declaration of function 'exact_log2'"
>> > when compiling reload.c and recog.c, probably due to exact_log2
>> > being used in CONST_OK_FOR_CONSTRAINT_P.
>> >
>> > I'm lost in this new scheme of moving includes around, but you
>> > should know.  What's the best course of action besides moving to
>> > constraints.md?
>>
>> For every file that produces such error, just include toplev.h
>>
>> #include "toplev.h" /* exact_log2 */
>>
>> after everything else. I will do it for you if you give me the list of
>> files affected.
>
> Thanks, the list seen in the quoted buildlog above is complete.
>
> brgds, H-P

Richard, can I commit the following patch to fix the breakage or you
want me to wait until the merge is over?

2010-06-30  Manuel López-Ibáñez  <manu@gcc.gnu.org>

	* reload.c: Include toplev.h.
	* recog.c:  Likewise.
	* Makefile.in: Adjust dependencies.

BTW, is there a rule against targets including the headers that they
need. Can .md files include headers?

Manuel.

On Wed, Jun 30, 2010 at 4:17 PM, Manuel López-Ibáñez
<lopezibanez@gmail.com> wrote:
> BTW, is there a rule against targets including the headers that they
> need. Can .md files include headers?

rs6000 uses exact_log2 in its constraints.md.  This works because
genpreds.c creates a source file which includes toplev.h.  See
genpreds.c:1340.  I think someone should go through those includes
also.

Thanks,
Andrew Pinski

On 1 July 2010 01:23, Andrew Pinski <pinskia@gmail.com> wrote:
> On Wed, Jun 30, 2010 at 4:17 PM, Manuel López-Ibáñez
> <lopezibanez@gmail.com> wrote:
>> BTW, is there a rule against targets including the headers that they
>> need. Can .md files include headers?
>
> rs6000 uses exact_log2 in its constraints.md.  This works because
> genpreds.c creates a source file which includes toplev.h.  See
> genpreds.c:1340.  I think someone should go through those includes
> also.

But I didn't touch that file and toplev.h still contains exact_log2,
so what is going on?

Manuel.

On Wed, Jun 30, 2010 at 4:29 PM, Manuel López-Ibáñez
<lopezibanez@gmail.com> wrote:
> On 1 July 2010 01:23, Andrew Pinski <pinskia@gmail.com> wrote:
>> On Wed, Jun 30, 2010 at 4:17 PM, Manuel López-Ibáñez
>> <lopezibanez@gmail.com> wrote:
>>> BTW, is there a rule against targets including the headers that they
>>> need. Can .md files include headers?
>>
>> rs6000 uses exact_log2 in its constraints.md.  This works because
>> genpreds.c creates a source file which includes toplev.h.  See
>> genpreds.c:1340.  I think someone should go through those includes
>> also.
>
> But I didn't touch that file and toplev.h still contains exact_log2,
> so what is going on?

Easy, Cris has not be converted over to use define_constraint yet.

Thanks,
Andrew Pinski

On Thu, 1 Jul 2010, Manuel López-Ibáñez wrote:
> Richard, can I commit the following patch to fix the breakage or you
> want me to wait until the merge is over?

Richi ok'd this on IRC, so I verified that it fixed the
build problem and committed it on behalf of you.  Thanks.

> 2010-06-30  Manuel López-Ibáñez  <manu@gcc.gnu.org>
>
> 	* reload.c: Include toplev.h.
> 	* recog.c:  Likewise.
> 	* Makefile.in: Adjust dependencies.

brgds, H-P