This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [patch] gcse.c: Fix the header ordering.
Kazu Hirata <kazu@cs.umass.edu> writes:
>> I think rtl.h should include insn-config.
>>
>> Note in the not to distant past such nested included were discouraged but
>> it would appear that that position has changed because grepping for
>> "#include" in *.h files within the gcc directory gets quite a few hits.
>
> Discouraged for simplicity? Or for a little more specific reason?
>
> I just put insn-config.h before rtl.h in combine.c, the generated code
> seems to get slightly better. (That is, CC0_P in combine.c is now
> triggered.)
rtl.h is supposed to be machine independent - that is, it should have
no dependencies on tm.h or any insn-*.h. The CC0_P macro needs to be
moved to a more appropriate location.
zw