[patch] Cleanup diagnostic.h in the gimple passes

[Steven Bosscher]

On Wed, Jun 2, 2010 at 4:47 PM, H.J. Lu <hjl.tools@gmail.com> wrote:
> On Wed, Jun 2, 2010 at 7:44 AM, Steven Bosscher <stevenb.gcc@gmail.com> wrote:
>> On Wed, Jun 2, 2010 at 4:37 PM, H.J. Lu <hjl.tools@gmail.com> wrote:
>>> This also caused:
>>>
>>> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44379
>>
>> This also caused annoyance with incomplete bug reports.
>>
>> Target?
>
>
> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44379
>
> said:
>
> "On Linux/ia64, revision 160132 gave
>
> /export/gnu/import/svn/gcc-test/bld/gcc/../../src-trunk/gcc/tree-sra.c:2033:
> undefined reference to `MOVE_RATIO'
> libbackend.a(tree-inline.o): In function `estimate_move_cost':
> /export/gnu/import/svn/gcc-test/bld/gcc/../../src-trunk/gcc/tree-inline.c:3168:
> undefined reference to `MOVE_RATIO'
> collect2: ld returned 1 exit status
> make[6]: *** [cc1-dummy] Error 1
>
> Revision 160116 is OK."
>
> Which part isn't clear?

I expect the target in the "Target" field of the bugzilla report.

Problem here, is that a target (ia64) doesn't give a definition for a
target macro, and the default target macro is defined in expr.h.
Ideally the solution would be to hookize these target macros, but that
will take a little while to complete.

Would it be OK for everyone if I move the default macro definitions to
target.h with the commitment to hookize them before the end of stage
1?

Ciao!
Steven