[Bug ipa/65005] New: [5 Regression] FAIL: c-c++-common/torture/builtin-arith-overflow-12.c

[Bug ipa/65005] New: [5 Regression] FAIL: c-c++-common/torture/builtin-arith-overflow-12.c

[hjl.tools at gmail dot com]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65005

            Bug ID: 65005
           Summary: [5 Regression] FAIL:
                    c-c++-common/torture/builtin-arith-overflow-12.c
           Product: gcc
           Version: 5.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: ipa
          Assignee: unassigned at gcc dot gnu.org
          Reporter: hjl.tools at gmail dot com
                CC: hubicka at ucw dot cz

On Linux/x86-32, r220548 caused:

FAIL: c-c++-common/torture/builtin-arith-overflow-12.c   -O2 -flto
-fuse-linker-plugin -fno-fat-lto-objects  execution test
FAIL: c-c++-common/torture/builtin-arith-overflow-13.c   -O2 -flto
-fuse-linker-plugin -fno-fat-lto-objects  execution test
FAIL: c-c++-common/torture/builtin-arith-overflow-14.c   -O2 -flto
-fuse-linker-plugin -fno-fat-lto-objects  execution test
FAIL: c-c++-common/torture/builtin-arith-overflow-15.c   -O2 -flto
-fuse-linker-plugin -fno-fat-lto-objects  execution test
FAIL: c-c++-common/torture/builtin-arith-overflow-16.c   -O2 -flto
-fuse-linker-plugin -fno-fat-lto-objects  execution test

[Bug ipa/65005] [5 Regression] FAIL: c-c++-common/torture/builtin-arith-overflow-12.c

[hubicka at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65005

Jan Hubicka <hubicka at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |hubicka at gcc dot gnu.org

--- Comment #1 from Jan Hubicka <hubicka at gcc dot gnu.org> ---
OK, given that this is one of few testcases big enough to be partitioned, I
suppose we finally have a reproducer to the way we partition aliases.
What we do currently for alias defined in other unit is to turn it into
external function that makes all local propagation go wrong; we also get wrong
alias information and equality folding..

[Bug ipa/65005] [5 Regression] FAIL: c-c++-common/torture/builtin-arith-overflow-12.c

[hubicka at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65005

--- Comment #2 from Jan Hubicka <hubicka at gcc dot gnu.org> ---
Indeed, it is the problem with aliases not being streamed.  i386.c check
can_change_signature of alias and not function itself.  I am working on the
partitioner fix.

[Bug ipa/65005] [5 Regression] FAIL: c-c++-common/torture/builtin-arith-overflow-12.c

[hubicka at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65005

--- Comment #3 from Jan Hubicka <hubicka at gcc dot gnu.org> ---
Author: hubicka
Date: Wed Feb 11 09:11:06 2015
New Revision: 220608

URL: https://gcc.gnu.org/viewcvs?rev=220608&root=gcc&view=rev
Log:
    PR ipa/65005
    * ipa-visibility.c (cgraph_node::non_local_p): Turn into static
    function.
    * symtab.c (symtab_node::verify_base): Remove check that non-definitions
    have no comdat group.
    * lto-cgraph.c (lto_output_node): Always output thunk and alias info.
    (lto_output_varpool_node): Always output alias info.
    (output_refs): Output refs of boundary aliases, too.
    (compute_ltrans_boundary): Add alias and thunk target into boundaries.
    (output_symtab): Output call eges in thunks in boundary.
    (get_alias_symbol): Remove.
    (input_node, input_varpool_node): Do not special case weakrefs.
    * ipa.c (symbol_table::remove_unreachable_nodes): Do not remove
    alias and thunks targets in the boundary; do not take removed symbols
    from their comdat groups.
    * cgraph.c (cgraph_node::local_info): Look through aliases and thunks.
    (cgraph_node::global_info): Remove.
    (cgraph_node::rtl_info): Look through aliases and thunks.
    * cgrpah.h (global_info): Remove.
    (non_local_p): Remove.


Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/cgraph.c
    trunk/gcc/cgraph.h
    trunk/gcc/ipa-visibility.c
    trunk/gcc/ipa.c
    trunk/gcc/lto-cgraph.c
    trunk/gcc/symtab.c

[Bug ipa/65005] [5 Regression] FAIL: c-c++-common/torture/builtin-arith-overflow-12.c

[rguenth at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65005

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |wrong-code
           Priority|P3                          |P1
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|---                         |FIXED
   Target Milestone|---                         |5.0

--- Comment #4 from Richard Biener <rguenth at gcc dot gnu.org> ---
Fixed(?).