[Bug middle-end/43513] The stack pointer is adjusted twice

[Bug middle-end/43513] The stack pointer is adjusted twice

[rguenth at gcc dot gnu.org]

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

Richard Guenther <rguenth at gcc dot gnu.org> changed:

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

--- Comment #5 from Richard Guenther <rguenth at gcc dot gnu.org> 2011-07-27 10:56:13 UTC ---
dup.

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

[Bug middle-end/43513] The stack pointer is adjusted twice

[vries at gcc dot gnu.org]

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

--- Comment #6 from vries at gcc dot gnu.org 2011-07-30 12:37:50 UTC ---
Author: vries
Date: Sat Jul 30 12:37:47 2011
New Revision: 176959

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=176959
Log:
2011-07-30  Tom de Vries  <tom@codesourcery.com>

    PR middle-end/43513
    * tree-ssa-dce.c (ref_may_be_aliased): Add assert.
    (propagate_necessity): Handle WITH_SIZE_EXPR call arg.

Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/tree-ssa-dce.c

[Bug middle-end/43513] The stack pointer is adjusted twice

[vries at gcc dot gnu.org]

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

--- Comment #7 from vries at gcc dot gnu.org 2011-07-31 10:02:15 UTC ---
Author: vries
Date: Sun Jul 31 10:02:12 2011
New Revision: 176968

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=176968
Log:
2011-07-31  Tom de Vries  <tom@codesourcery.com>

    PR middle-end/43513
    * testsuite/libmudflap.c/fail31-frag.c: Adapt testcase to prevent
    folding of alloca.

Modified:
    trunk/libmudflap/ChangeLog
    trunk/libmudflap/testsuite/libmudflap.c/fail31-frag.c

[Bug middle-end/43513] The stack pointer is adjusted twice

[vries at gcc dot gnu.org]

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

--- Comment #8 from vries at gcc dot gnu.org 2011-07-31 10:05:11 UTC ---
Author: vries
Date: Sun Jul 31 10:05:07 2011
New Revision: 176969

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=176969
Log:
2011-07-31  Tom de Vries  <tom@codesourcery.com>

    PR middle-end/43513
    * tree-ssa-loop-ivopts.c (may_be_unaligned_p): Use max of
    get_object_alignment and TYPE_ALIGN.

Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/tree-ssa-loop-ivopts.c

[Bug middle-end/43513] The stack pointer is adjusted twice

[vries at gcc dot gnu.org]

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

--- Comment #9 from vries at gcc dot gnu.org 2011-08-31 07:04:31 UTC ---
Author: vries
Date: Wed Aug 31 07:04:25 2011
New Revision: 178353

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=178353
Log:
2011-08-31  Tom de Vries  <tom@codesourcery.com>

    PR middle-end/43513
    * Makefile.in (tree-ssa-ccp.o): Add $(PARAMS_H) to rule.
    * tree-ssa-ccp.c (params.h): Include.
    (fold_builtin_alloca_for_var): New function.
    (ccp_fold_stmt): Use fold_builtin_alloca_for_var.

Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/Makefile.in
    trunk/gcc/tree-ssa-ccp.c

[Bug middle-end/43513] The stack pointer is adjusted twice

[vries at gcc dot gnu.org]

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

--- Comment #10 from vries at gcc dot gnu.org 2011-08-31 07:06:04 UTC ---
Author: vries
Date: Wed Aug 31 07:05:59 2011
New Revision: 178354

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=178354
Log:
2011-08-31  Tom de Vries  <tom@codesourcery.com>

    PR middle-end/43513
    * gcc.dg/pr43513.c: New test.

Added:
    trunk/gcc/testsuite/gcc.dg/pr43513.c
Modified:
    trunk/gcc/testsuite/ChangeLog

[Bug middle-end/43513] The stack pointer is adjusted twice

[vries at gcc dot gnu.org]

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

vries at gcc dot gnu.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |vries at gcc dot gnu.org
         Resolution|DUPLICATE                   |FIXED
         AssignedTo|unassigned at gcc dot       |vries at gcc dot gnu.org
                   |gnu.org                     |

--- Comment #11 from vries at gcc dot gnu.org 2011-08-31 08:56:42 UTC ---
Patch and test-case checked in, marking fixed.

[Bug middle-end/43513] The stack pointer is adjusted twice

[rearnsha at gcc dot gnu dot org]

------- Comment #4 from rearnsha at gcc dot gnu dot org  2010-03-26 09:05 -------
This is a case where we have a 'variable length array' declaration but where
the variable is really const and well-known at compile time.  However, we still
end up with it being 'variably sized' at rtl expand time.

I think it would be much better if the generic optimizers could spot this case
and remove it entirely, turning it into a natural array declartion.  Then we
could eliminate even more from this function (for example, it doesn't really
need a frame pointer).

So changing to 'middle-end'.


-- 

rearnsha at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |rearnsha at gcc dot gnu dot
                   |                            |org
           Severity|normal                      |enhancement
          Component|target                      |middle-end


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