[Bug c++/65309] New: [Regression] Executes wrong function inside an anonymous namespace on runtime

[Bug c++/65309] New: [Regression] Executes wrong function inside an anonymous namespace on runtime

[albert.astals at canonical dot com]

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

            Bug ID: 65309
           Summary: [Regression] Executes wrong function inside an
                    anonymous namespace on runtime
           Product: gcc
           Version: 4.9.2
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: c++
          Assignee: unassigned at gcc dot gnu.org
          Reporter: albert.astals at canonical dot com

Created attachment 34947
  --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=34947&action=edit
tar.gz containing files to reproduce

To be honest the summary is probably wrong, not much of a compiler expert
myself

I have this code that when compiled with gcc 4.9.2 and -g fails but works with
-O2 (also works with 4.8.2 whenever the flags used)

Fails when printing:
a.cpp
a.cpp

Works if printing:
a.cpp
b.cpp

This is affecting Qt since this is reduced test case from Qt sources

$ gcc -v
Using built-in specs.
COLLECT_GCC=gcc
COLLECT_LTO_WRAPPER=/usr/lib/gcc/x86_64-linux-gnu/4.9/lto-wrapper
Target: x86_64-linux-gnu
Configured with: ../src/configure -v --with-pkgversion='Ubuntu 4.9.2-10ubuntu7'
--with-bugurl=file:///usr/share/doc/gcc-4.9/README.Bugs
--enable-languages=c,c++,java,go,d,fortran,objc,obj-c++ --prefix=/usr
--program-suffix=-4.9 --enable-shared --enable-linker-build-id
--libexecdir=/usr/lib --without-included-gettext --enable-threads=posix
--with-gxx-include-dir=/usr/include/c++/4.9 --libdir=/usr/lib --enable-nls
--with-sysroot=/ --enable-clocale=gnu --enable-libstdcxx-debug
--enable-libstdcxx-time=yes --enable-gnu-unique-object --disable-vtable-verify
--enable-plugin --with-system-zlib --disable-browser-plugin
--enable-java-awt=gtk --enable-gtk-cairo
--with-java-home=/usr/lib/jvm/java-1.5.0-gcj-4.9-amd64/jre --enable-java-home
--with-jvm-root-dir=/usr/lib/jvm/java-1.5.0-gcj-4.9-amd64
--with-jvm-jar-dir=/usr/lib/jvm-exports/java-1.5.0-gcj-4.9-amd64
--with-arch-directory=amd64 --with-ecj-jar=/usr/share/java/eclipse-ecj.jar
--enable-objc-gc --enable-multiarch --disable-werror --with-arch-32=i686
--with-abi=m64 --with-multilib-list=m32,m64,mx32 --enable-multilib
--with-tune=generic --enable-checking=release --build=x86_64-linux-gnu
--host=x86_64-linux-gnu --target=x86_64-linux-gnu
Thread model: posix
gcc version 4.9.2 (Ubuntu 4.9.2-10ubuntu7)

[Bug c++/65309] [Regression] Executes wrong function inside an anonymous namespace on runtime

[albert.astals at canonical dot com]

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

--- Comment #1 from Albert Astals Cid <albert.astals at canonical dot com> ---
Works fine with

$ gcc -v
Using built-in specs.
COLLECT_GCC=gcc
COLLECT_LTO_WRAPPER=/usr/lib/gcc/i686-linux-gnu/4.8/lto-wrapper
Target: i686-linux-gnu
Configured with: ../src/configure -v --with-pkgversion='Ubuntu 4.8.2-19ubuntu1'
--with-bugurl=file:///usr/share/doc/gcc-4.8/README.Bugs
--enable-languages=c,c++,java,go,d,fortran,objc,obj-c++ --prefix=/usr
--program-suffix=-4.8 --enable-shared --enable-linker-build-id
--libexecdir=/usr/lib --without-included-gettext --enable-threads=posix
--with-gxx-include-dir=/usr/include/c++/4.8 --libdir=/usr/lib --enable-nls
--with-sysroot=/ --enable-clocale=gnu --enable-libstdcxx-debug
--enable-libstdcxx-time=yes --enable-gnu-unique-object --disable-libmudflap
--enable-plugin --with-system-zlib --disable-browser-plugin
--enable-java-awt=gtk --enable-gtk-cairo
--with-java-home=/usr/lib/jvm/java-1.5.0-gcj-4.8-i386/jre --enable-java-home
--with-jvm-root-dir=/usr/lib/jvm/java-1.5.0-gcj-4.8-i386
--with-jvm-jar-dir=/usr/lib/jvm-exports/java-1.5.0-gcj-4.8-i386
--with-arch-directory=i386 --with-ecj-jar=/usr/share/java/eclipse-ecj.jar
--enable-objc-gc --enable-targets=all --enable-multiarch --disable-werror
--with-arch-32=i686 --with-multilib-list=m32,m64,mx32 --with-tune=generic
--enable-checking=release --build=i686-linux-gnu --host=i686-linux-gnu
--target=i686-linux-gnu
Thread model: posix
gcc version 4.8.2 (Ubuntu 4.8.2-19ubuntu1)

[Bug c++/65309] [Regression] Executes wrong function inside an anonymous namespace on runtime

[trippels at gcc dot gnu.org]

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

--- Comment #4 from Markus Trippelsdorf <trippels at gcc dot gnu.org> ---
Does it also happens with -fuse-ld=gold?

[Bug c++/65309] [Regression] Executes wrong function inside an anonymous namespace on runtime

[s at ecloud dot org]

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

ecloud <s at ecloud dot org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |s at ecloud dot org

--- Comment #5 from ecloud <s at ecloud dot org> ---
I can also reproduce this on Arch linux.

Yes it still fails if I change the Makefile to set

LINK          = g++ -fuse-ld=gold

However changing

CXX           = g++ -O2

makes it pass.

$ gcc -v
Using built-in specs.
COLLECT_GCC=gcc
COLLECT_LTO_WRAPPER=/usr/lib/gcc/x86_64-unknown-linux-gnu/4.9.2/lto-wrapper
Target: x86_64-unknown-linux-gnu
Configured with: /build/gcc-multilib/src/gcc-4.9-20150204/configure
--prefix=/usr --libdir=/usr/lib --libexecdir=/usr/lib --mandir=/usr/share/man
--infodir=/usr/share/info --with-bugurl=https://bugs.archlinux.org/
--enable-languages=c,c++,ada,fortran,go,lto,objc,obj-c++ --enable-shared
--enable-threads=posix --with-system-zlib --enable-__cxa_atexit
--disable-libunwind-exceptions --enable-clocale=gnu --disable-libstdcxx-pch
--disable-libssp --enable-gnu-unique-object --enable-linker-build-id
--enable-cloog-backend=isl --enable-lto --enable-plugin
--enable-install-libiberty --with-linker-hash-style=gnu --enable-multilib
--disable-werror --enable-checking=release
Thread model: posix
gcc version 4.9.2 20150204 (prerelease) (GCC)

[Bug c++/65309] [Regression] Executes wrong function inside an anonymous namespace on runtime

[albert.astals at canonical dot com]

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

--- Comment #6 from Albert Astals Cid <albert.astals at canonical dot com> ---
Same as ecloud, also fails if using gold.

[Bug c++/65309] [Regression] Executes wrong function inside an anonymous namespace on runtime

[trippels at gcc dot gnu.org]

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

Markus Trippelsdorf <trippels at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |wrong-code
             Status|UNCONFIRMED                 |NEW
   Last reconfirmed|                            |2015-03-04
   Target Milestone|---                         |4.9.3
     Ever confirmed|0                           |1

--- Comment #7 from Markus Trippelsdorf <trippels at gcc dot gnu.org> ---
Confirmed on 4.9 branch. I was using 4.9 release first. 
Bisecting.

[Bug c++/65309] [Regression] Executes wrong function inside an anonymous namespace on runtime

[gcc at bobbyperu dot info]

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

Sergio Martins <gcc at bobbyperu dot info> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |gcc at bobbyperu dot info

--- Comment #8 from Sergio Martins <gcc at bobbyperu dot info> ---
gcc-4.9.2 $ nm gccbug  | c++filt  | grep operator
00000000004008ea W GlobalStatic<Bar, (anonymous
namespace)::innerFunction()>::operator()()

00000000004008ea W GlobalStatic<Bar, (anonymous
namespace)::innerFunction()>::operator()()

gcc-4.8.4 $ nm gccbug  | c++filt  | grep operator
00000000004008cc t GlobalStatic<Bar, (anonymous
namespace)::innerFunction()>::operator()()
00000000004009a4 t GlobalStatic<Bar, (anonymous
namespace)::innerFunction()>::operator()()

With 4.9.2 the address of both operator()() is the same

[Bug c++/65309] [4.9 Regression] Executes wrong function inside an anonymous namespace on runtime

[trippels at gcc dot gnu.org]

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

Markus Trippelsdorf <trippels at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |jason at gcc dot gnu.org
            Summary|[Regression] Executes wrong |[4.9 Regression] Executes
                   |function inside an          |wrong function inside an
                   |anonymous namespace on      |anonymous namespace on
                   |runtime                     |runtime

--- Comment #9 from Markus Trippelsdorf <trippels at gcc dot gnu.org> ---
Started with r219305.

[Bug c++/65309] [4.9/5 Regression] Executes wrong function inside an anonymous namespace on runtime

[rguenth at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Priority|P3                          |P1
      Known to work|                            |4.8.4, 4.9.2
            Summary|[4.9 Regression] Executes   |[4.9/5 Regression] Executes
                   |wrong function inside an    |wrong function inside an
                   |anonymous namespace on      |anonymous namespace on
                   |runtime                     |runtime
      Known to fail|                            |4.9.3, 5.0

--- Comment #10 from Richard Biener <rguenth at gcc dot gnu.org> ---
Confirmed also on trunk.  P1 because of a regression on the branch that wasn't
released yet.

[Bug c++/65309] [4.9 Regression] Executes wrong function inside an anonymous namespace on runtime

[trippels at gcc dot gnu.org]

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

Markus Trippelsdorf <trippels at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Known to work|                            |5.0
            Summary|[4.9/5 Regression] Executes |[4.9 Regression] Executes
                   |wrong function inside an    |wrong function inside an
                   |anonymous namespace on      |anonymous namespace on
                   |runtime                     |runtime
      Known to fail|5.0                         |

--- Comment #11 from Markus Trippelsdorf <trippels at gcc dot gnu.org> ---
The issue was fixed on trunk by r220991.

[Bug c++/65309] [4.9 Regression] Executes wrong function inside an anonymous namespace on runtime

[jason at gcc dot gnu.org]

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

Jason Merrill <jason at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
           Assignee|unassigned at gcc dot gnu.org      |jason at gcc dot gnu.org

[Bug c++/65309] [4.9 Regression] Executes wrong function inside an anonymous namespace on runtime

[jason at gcc dot gnu.org]

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

--- Comment #13 from Jason Merrill <jason at gcc dot gnu.org> ---
Author: jason
Date: Wed Mar  4 18:13:44 2015
New Revision: 221192

URL: https://gcc.gnu.org/viewcvs?rev=221192&root=gcc&view=rev
Log:
    PR c++/65209
    PR c++/65309
    * decl2.c (constrain_visibility_for_template): Handle reference
    arguments.

Added:
    branches/gcc-4_9-branch/gcc/testsuite/g++.dg/abi/anon4.C
Modified:
    branches/gcc-4_9-branch/gcc/cp/ChangeLog
    branches/gcc-4_9-branch/gcc/cp/decl2.c

[Bug c++/65309] [4.9 Regression] Executes wrong function inside an anonymous namespace on runtime

[jason at gcc dot gnu.org]

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

Jason Merrill <jason at gcc dot gnu.org> changed:

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

--- Comment #12 from Jason Merrill <jason at gcc dot gnu.org> ---
Fixed for 4.9.3.