[Bug other/59924] New: [4.9 Regression] memory hog / build failure with -Wformat -Werror=format-security

[Bug other/59924] New: [4.9 Regression] memory hog / build failure with -Wformat -Werror=format-security

[doko at gcc dot gnu.org]

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

            Bug ID: 59924
           Summary: [4.9 Regression] memory hog / build failure with
                    -Wformat -Werror=format-security
           Product: gcc
           Version: 4.9.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: other
          Assignee: unassigned at gcc dot gnu.org
          Reporter: doko at gcc dot gnu.org

Created attachment 31936
  --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=31936&action=edit
preprocessed source

seen with trunk 20140122, works with 4.8 branch.  Omitting the "-Wformat
-Werror=format-security" warnings lets the build succeed.

$ gcc -g -O1 -fstack-protector -Wformat -Werror=format-security -Wall -c
matanza.i
[...]
cc1: out of memory allocating 104 bytes after a total of 8836583424 bytes

[Bug tree-optimization/59924] [4.9 Regression] uninit memory hog

[rguenth at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Priority|P3                          |P1
          Component|other                       |tree-optimization

[Bug tree-optimization/59924] [4.9 Regression] uninit memory hog

[rth at gcc dot gnu.org]

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

--- Comment #2 from Richard Henderson <rth at gcc dot gnu.org> ---
Definitely a bug.  We keep pushing and popping the same node
off of the work_list.

[Bug tree-optimization/59924] [4.9 Regression] uninit memory hog

[rth at gcc dot gnu.org]

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

--- Comment #3 from Richard Henderson <rth at gcc dot gnu.org> ---
The memory hog bug is caused by not handling incorrect input:
the variable dist_x in the function mat_object_woozka is
conditionally uninitialized.

A nice bit of meta-circularity, that.

[Bug tree-optimization/59924] [4.9 Regression] uninit memory hog

[rth at gcc dot gnu.org]

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

--- Comment #5 from Richard Henderson <rth at gcc dot gnu.org> ---
Author: rth
Date: Mon Feb  3 23:13:08 2014
New Revision: 207442

URL: http://gcc.gnu.org/viewcvs?rev=207442&root=gcc&view=rev
Log:
PR tree-opt/59924

        * tree-ssa-uninit.c (push_to_worklist): Don't re-push if we've
        already processed this node.
        (normalize_one_pred_1): Pass along mark_set.
        (normalize_one_pred): Create and destroy a pointer_set_t.
        (normalize_one_pred_chain): Likewise.

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

[Bug tree-optimization/59924] [4.9 Regression] uninit memory hog

[rth at gcc dot gnu.org]

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

Richard Henderson <rth at gcc dot gnu.org> changed:

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

--- Comment #6 from Richard Henderson <rth at gcc dot gnu.org> ---
Fixed.

[Bug tree-optimization/59924] [4.9 Regression] uninit memory hog

[jakub at gcc dot gnu.org]

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

--- Comment #7 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
Author: jakub
Date: Tue Feb  4 00:46:42 2014
New Revision: 207446

URL: http://gcc.gnu.org/viewcvs?rev=207446&root=gcc&view=rev
Log:
    PR tree-optimization/59924
    * gcc.dg/pr59924.c: New test.

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