[Bug tree-optimization/46621] New: gimple.h includes tm.h

[Bug tree-optimization/46621] New: gimple.h includes tm.h

[amylaar at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46621

           Summary: gimple.h includes tm.h
           Product: gcc
           Version: 4.6.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: tree-optimization
        AssignedTo: unassigned@gcc.gnu.org
        ReportedBy: amylaar@gcc.gnu.org
            Blocks: 46489

[Bug tree-optimization/46621] gimple.h includes tm.h

[amylaar at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46621

Jorn Wolfgang Rennecke <amylaar at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Blocks|                            |46495

--- Comment #1 from Jorn Wolfgang Rennecke <amylaar at gcc dot gnu.org> 2010-11-24 09:14:22 UTC ---
If target.h and function.h are fixed not to include tm.h, but gimple.h
continues to include tm.h, ppc-elf build of ipa-split.o breaks, because tm.h
ends up being included after function.h, and rs6000.h declares a variable
called frame_pointer_needed.
With the frame_pointer_needed macro from tm.h defined, this causes a syntax
error.

Fixing gimple.h (removing tm.h and hard-reg-set.h includes) first tests out OK.

[Bug tree-optimization/46621] gimple.h includes tm.h

[amylaar at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46621

--- Comment #2 from Jorn Wolfgang Rennecke <amylaar at gcc dot gnu.org> 2010-11-29 00:50:09 UTC ---
Author: amylaar
Date: Mon Nov 29 00:50:04 2010
New Revision: 167230

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=167230
Log:
    PR tree-optimization/46621
    * gimple.h: Don't include tm.h or hard-reg-set.h .
    * Makefile.in (GIMPLE_H): Remove $(TM_H).

Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/Makefile.in
    trunk/gcc/gimple.h

[Bug tree-optimization/46621] gimple.h includes tm.h

[amylaar at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46621

Jorn Wolfgang Rennecke <amylaar at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|                            |FIXED

--- Comment #3 from Jorn Wolfgang Rennecke <amylaar at gcc dot gnu.org> 2011-06-25 14:40:15 UTC ---
Fixed in r167230