public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug c/16450] New: Nested function inlining bug in 3.4.x
@ 2004-07-09 12:53 jakub at gcc dot gnu dot org
  2004-07-09 16:48 ` [Bug c/16450] " pinskia at gcc dot gnu dot org
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: jakub at gcc dot gnu dot org @ 2004-07-09 12:53 UTC (permalink / raw)
  To: gcc-bugs

/* { dg-do compile } */
/* { dg-options "-O2" } */

int foo (int x)
{
  int i = x;
  inline __attribute__((always_inline)) int bar (void)
  {
    return i;
  }
  return bar ();
}

/* { dg-final { scan-assembler-not "bar.0" } } */

fails in 3.4.x, doesn't fail in 3.3.x or on the trunk.
This is quite serious bug, as it extremely bloats e.g. glibc dynamic linker.
A nested function body is emitted even when it is never used.

-- 
           Summary: Nested function inlining bug in 3.4.x
           Product: gcc
           Version: 3.4.1
            Status: UNCONFIRMED
          Severity: normal
          Priority: P2
         Component: c
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: jakub at gcc dot gnu dot org
                CC: gcc-bugs at gcc dot gnu dot org


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


^ permalink raw reply	[flat|nested] 4+ messages in thread

* [Bug c/16450] Nested function inlining bug in 3.4.x
  2004-07-09 12:53 [Bug c/16450] New: Nested function inlining bug in 3.4.x jakub at gcc dot gnu dot org
@ 2004-07-09 16:48 ` pinskia at gcc dot gnu dot org
  2004-07-21  7:37 ` cvs-commit at gcc dot gnu dot org
  2004-07-21  7:39 ` cvs-commit at gcc dot gnu dot org
  2 siblings, 0 replies; 4+ messages in thread
From: pinskia at gcc dot gnu dot org @ 2004-07-09 16:48 UTC (permalink / raw)
  To: gcc-bugs


------- Additional Comments From pinskia at gcc dot gnu dot org  2004-07-09 16:48 -------
This is a dup of bug 15345, which was already decided by Mark it will not be fixed for 3.4.x at all.

*** This bug has been marked as a duplicate of 15345 ***

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


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


^ permalink raw reply	[flat|nested] 4+ messages in thread

* [Bug c/16450] Nested function inlining bug in 3.4.x
  2004-07-09 12:53 [Bug c/16450] New: Nested function inlining bug in 3.4.x jakub at gcc dot gnu dot org
  2004-07-09 16:48 ` [Bug c/16450] " pinskia at gcc dot gnu dot org
@ 2004-07-21  7:37 ` cvs-commit at gcc dot gnu dot org
  2004-07-21  7:39 ` cvs-commit at gcc dot gnu dot org
  2 siblings, 0 replies; 4+ messages in thread
From: cvs-commit at gcc dot gnu dot org @ 2004-07-21  7:37 UTC (permalink / raw)
  To: gcc-bugs


------- Additional Comments From cvs-commit at gcc dot gnu dot org  2004-07-21 07:37 -------
Subject: Bug 16450

CVSROOT:	/cvs/gcc
Module name:	gcc
Branch: 	gcc-3_4-branch
Changes by:	jakub@gcc.gnu.org	2004-07-21 07:37:18

Modified files:
	gcc            : ChangeLog tree-optimize.c toplev.c 
	gcc/testsuite  : ChangeLog 
Added files:
	gcc/testsuite/gcc.dg/torture: nested-fn-1.c 

Log message:
	PR middle-end/15345
	PR c/16450
	* toplev.c (rest_of_handle_inlining): Set DECL_DEFER_OUTPUT on C
	nested functions as well.
	* tree-optimize.c (tree_rest_of_compilation): Don't clear decl rtls
	for deferred nested inlines.
	
	* gcc.dg/torture/nested-fn-1.c: New test.

Patches:
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/ChangeLog.diff?cvsroot=gcc&only_with_tag=gcc-3_4-branch&r1=2.2326.2.563&r2=2.2326.2.564
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/tree-optimize.c.diff?cvsroot=gcc&only_with_tag=gcc-3_4-branch&r1=2.8.8.5&r2=2.8.8.6
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/toplev.c.diff?cvsroot=gcc&only_with_tag=gcc-3_4-branch&r1=1.863.4.10&r2=1.863.4.11
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/testsuite/ChangeLog.diff?cvsroot=gcc&only_with_tag=gcc-3_4-branch&r1=1.3389.2.231&r2=1.3389.2.232
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/testsuite/gcc.dg/torture/nested-fn-1.c.diff?cvsroot=gcc&only_with_tag=gcc-3_4-branch&r1=NONE&r2=1.1.2.1



-- 


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


^ permalink raw reply	[flat|nested] 4+ messages in thread

* [Bug c/16450] Nested function inlining bug in 3.4.x
  2004-07-09 12:53 [Bug c/16450] New: Nested function inlining bug in 3.4.x jakub at gcc dot gnu dot org
  2004-07-09 16:48 ` [Bug c/16450] " pinskia at gcc dot gnu dot org
  2004-07-21  7:37 ` cvs-commit at gcc dot gnu dot org
@ 2004-07-21  7:39 ` cvs-commit at gcc dot gnu dot org
  2 siblings, 0 replies; 4+ messages in thread
From: cvs-commit at gcc dot gnu dot org @ 2004-07-21  7:39 UTC (permalink / raw)
  To: gcc-bugs


------- Additional Comments From cvs-commit at gcc dot gnu dot org  2004-07-21 07:39 -------
Subject: Bug 16450

CVSROOT:	/cvs/gcc
Module name:	gcc
Changes by:	jakub@gcc.gnu.org	2004-07-21 07:39:09

Modified files:
	gcc/testsuite  : ChangeLog 
Added files:
	gcc/testsuite/gcc.dg/torture: nested-fn-1.c 

Log message:
	PR middle-end/15345
	PR c/16450
	* gcc.dg/torture/nested-fn-1.c: New test.

Patches:
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/testsuite/ChangeLog.diff?cvsroot=gcc&r1=1.4029&r2=1.4030
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/testsuite/gcc.dg/torture/nested-fn-1.c.diff?cvsroot=gcc&r1=1.1&r2=1.2



-- 


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


^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2004-07-21  7:39 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2004-07-09 12:53 [Bug c/16450] New: Nested function inlining bug in 3.4.x jakub at gcc dot gnu dot org
2004-07-09 16:48 ` [Bug c/16450] " pinskia at gcc dot gnu dot org
2004-07-21  7:37 ` cvs-commit at gcc dot gnu dot org
2004-07-21  7:39 ` cvs-commit at gcc dot gnu dot org

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).