[Bug c++/57524] New: internal compiler error on dump translation unit

[Bug c++/57524] New: internal compiler error on dump translation unit

[JamesMikeDuPont at googlemail dot com]

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

            Bug ID: 57524
           Summary: internal compiler error on dump translation unit
           Product: gcc
           Version: 4.7.2
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: c++
          Assignee: unassigned at gcc dot gnu.org
          Reporter: JamesMikeDuPont at googlemail dot com

using the debian sid standard : GNU C++ (Debian 4.7.2-5) version 4.7.2
(x86_64-linux-gnu) compiled by GNU C version 4.7.2, GMP version 5.0.5, MPFR
version 3.1.0-p10, MPC version 0.9 warning: MPFR header version 3.1.0-p10
differs from library version 3.1.1-p2. GGC heuristics: --param
ggc-min-expand=100 --param ggc-min-heapsize=131072

Compiling boost spirit.qi code,
Here is the one that creates the error.

Invocation is :
/usr/lib/gcc/x86_64-linux-gnu/4.7/cc1plus -std=gnu++11 -fdump-translation-unit
treedumpparse.ii

the output is:

(this is the last of the function print )
 std::size_t boost::detail::variant::variant_hasher::operator()(const T&) const

the error is :
In file included from /usr/include/boost/variant/variant.hpp:35:0,
                 from /usr/include/boost/spirit/home/support/info.hpp:14,
                 from /usr/include/boost/spirit/home/qi/domain.hpp:16,
                 from /usr/include/boost/spirit/home/qi/meta_compiler.hpp:15,
                 from /usr/include/boost/spirit/home/qi/action/action.hpp:14,
                 from /usr/include/boost/spirit/home/qi/action.hpp:14,
                 from /usr/include/boost/spirit/home/qi.hpp:14,
                 from /usr/include/boost/spirit/include/qi.hpp:16,
                 from treedumpparse.cpp:6:
/usr/include/boost/variant/detail/hash_variant.hpp:32:33: internal compiler
error: in timevar_start, at timevar.c:344
Please submit a full bug report,
with preprocessed source if appropriate.
See <file:///usr/share/doc/gcc-4.7/README.Bugs> for instructions.

This can be reproduced with this one include :
#include "/usr/include/boost/variant/detail/hash_variant.hpp"


I am trying to reduce the problem, but it is difficult.

[Bug c++/57524] internal compiler error on dump translation unit

[JamesMikeDuPont at googlemail dot com]

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

--- Comment #1 from James Michael DuPont <JamesMikeDuPont at googlemail dot com> ---
I tried to reduce the input size but it is not producing any results.

namespace std {
  typedef long unsigned int size_t;    
};
namespace boost { 
  namespace detail {
    struct is_static_visitor_tag { };
    typedef void static_visitor_default_return;
  } 
  template <typename R = ::boost::detail::static_visitor_default_return>
  class static_visitor    : public detail::is_static_visitor_tag {
  public:
    typedef R result_type;
  protected:
    static_visitor() { }    ~static_visitor() { }
  };}
namespace boost
{
  template <class T> struct hash;
  template <class T> void hash_combine(std::size_t& seed, T const& v);
  template <class It> std::size_t hash_range(It, It);
  template <class It> void hash_range(std::size_t&, It, It);
}
namespace boost {
  namespace detail { namespace variant {
      struct variant_hasher: public boost::static_visitor<std::size_t> {
    template <class T>
    std::size_t operator()(T const& val) const {
      using namespace boost;
      hash<T> hasher;
      return hasher(val);
    }      };    }  }}

[Bug c++/57524] internal compiler error on dump translation unit

[JamesMikeDuPont at googlemail dot com]

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

--- Comment #2 from James Michael DuPont <JamesMikeDuPont at googlemail dot com> ---
Here is a small change to work around the crash :

namespace boost {

    namespace detail { namespace variant {
        struct variant_hasher: public //boost::
    static_visitor<std::size_t> {
            template <class T>
            std::size_t operator()(T const& val) const {
          //using namespace boost;
                hash<T> hasher;
                return hasher(val);
            }
        };
    }}
}

It is the uneeded namespace usage that causes the problem.

[Bug c++/57524] internal compiler error on dump translation unit

[paolo.carlini at oracle dot com]

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

Paolo Carlini <paolo.carlini at oracle dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |WAITING
   Last reconfirmed|                            |2013-06-05
     Ever confirmed|0                           |1

--- Comment #3 from Paolo Carlini <paolo.carlini at oracle dot com> ---
I'm still not seeing in attach a self contained (minimized) reproducer.

[Bug c++/57524] internal compiler error on dump translation unit

[JamesMikeDuPont at googlemail dot com]

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

--- Comment #5 from James Michael DuPont <JamesMikeDuPont at googlemail dot com> ---
Sorry, It seems that the original attachment was not applied, I have bzip2ed it
so now it fits.
thanks
mike

[Bug c++/57524] internal compiler error on dump translation unit

[JamesMikeDuPont at googlemail dot com]

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

--- Comment #4 from James Michael DuPont <JamesMikeDuPont at googlemail dot com> ---
Created attachment 30259
  --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=30259&action=edit
test case

this is the minimal test case that I have.
by applying the small namespace changes to the end it will go away.

[Bug c++/57524] internal compiler error on dump translation unit

[JamesMikeDuPont at googlemail dot com]

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

--- Comment #6 from James Michael DuPont <JamesMikeDuPont at googlemail dot com> ---
I have filed a small cosmetic bug in boost about this change to the namespace
usage :
https://svn.boost.org/trac/boost/ticket/8651

[Bug c++/57524] internal compiler error on dump translation unit

[paolo.carlini at oracle dot com]

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

Paolo Carlini <paolo.carlini at oracle dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|WAITING                     |UNCONFIRMED
     Ever confirmed|1                           |0

--- Comment #7 from Paolo Carlini <paolo.carlini at oracle dot com> ---
I can't reproduce the issue anywhere. Eg, not on stock FSF 4.7.2, not on 4.8 or
current mainline.

Can somebody reproduce?

[Bug c++/57524] internal compiler error on dump translation unit

[JamesMikeDuPont at googlemail dot com]

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

--- Comment #8 from James Michael DuPont <JamesMikeDuPont at googlemail dot com> ---
Tested like this:
/usr/lib/gcc/x86_64-linux-gnu/4.7/cc1plus -std=gnu++11 -fdump-translation-unit
testvariant_min.ii

[Bug c++/57524] internal compiler error on dump translation unit

[trippels at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |trippels at gcc dot gnu.org
      Known to fail|                            |4.9.0

--- Comment #11 from Markus Trippelsdorf <trippels at gcc dot gnu.org> ---
This still happens with 4.9:

markus@x4 more % cat test.ii
namespace detail {
namespace indirect_traits {}
using namespace indirect_traits;
void fn1() {
using namespace detail;
}
}

markus@x4 more % g++ -ftime-report -c test.ii
test.ii: In function ‘void detail::fn1()’:
test.ii:5:17: internal compiler error: in timevar_start, at timevar.c:343
 using namespace detail;
                 ^
>From gcc-bugs-return-438898-listarch-gcc-bugs=gcc.gnu.org@gcc.gnu.org Thu Jan 02 10:23:03 2014
Return-Path: <gcc-bugs-return-438898-listarch-gcc-bugs=gcc.gnu.org@gcc.gnu.org>
Delivered-To: listarch-gcc-bugs@gcc.gnu.org
Received: (qmail 11384 invoked by alias); 2 Jan 2014 10:23:02 -0000
Mailing-List: contact gcc-bugs-help@gcc.gnu.org; run by ezmlm
Precedence: bulk
List-Id: <gcc-bugs.gcc.gnu.org>
List-Archive: <http://gcc.gnu.org/ml/gcc-bugs/>
List-Post: <mailto:gcc-bugs@gcc.gnu.org>
List-Help: <mailto:gcc-bugs-help@gcc.gnu.org>
Sender: gcc-bugs-owner@gcc.gnu.org
Delivered-To: mailing list gcc-bugs@gcc.gnu.org
Received: (qmail 11346 invoked by uid 48); 2 Jan 2014 10:22:58 -0000
From: "mpolacek at gcc dot gnu.org" <gcc-bugzilla@gcc.gnu.org>
To: gcc-bugs@gcc.gnu.org
Subject: [Bug c++/59638] [c++1y] ICE with pointer to function that has auto as parameter
Date: Thu, 02 Jan 2014 10:23:00 -0000
X-Bugzilla-Reason: CC
X-Bugzilla-Type: changed
X-Bugzilla-Watch-Reason: None
X-Bugzilla-Product: gcc
X-Bugzilla-Component: c++
X-Bugzilla-Version: 4.9.0
X-Bugzilla-Keywords: error-recovery, ice-on-invalid-code
X-Bugzilla-Severity: normal
X-Bugzilla-Who: mpolacek at gcc dot gnu.org
X-Bugzilla-Status: NEW
X-Bugzilla-Priority: P3
X-Bugzilla-Assigned-To: unassigned at gcc dot gnu.org
X-Bugzilla-Target-Milestone: 4.9.0
X-Bugzilla-Flags:
X-Bugzilla-Changed-Fields: bug_status cf_reconfirmed_on cc target_milestone everconfirmed
Message-ID: <bug-59638-4-X5cswdKKY4@http.gcc.gnu.org/bugzilla/>
In-Reply-To: <bug-59638-4@http.gcc.gnu.org/bugzilla/>
References: <bug-59638-4@http.gcc.gnu.org/bugzilla/>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 7bit
X-Bugzilla-URL: http://gcc.gnu.org/bugzilla/
Auto-Submitted: auto-generated
MIME-Version: 1.0
X-SW-Source: 2014-01/txt/msg00040.txt.bz2
Content-length: 614

http://gcc.gnu.org/bugzilla/show_bug.cgi?idY638

Marek Polacek <mpolacek at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
   Last reconfirmed|                            |2014-01-02
                 CC|                            |mpolacek at gcc dot gnu.org
   Target Milestone|---                         |4.9.0
     Ever confirmed|0                           |1

--- Comment #2 from Marek Polacek <mpolacek at gcc dot gnu.org> ---
Confirmed.

[Bug c++/57524] [4.7/4.8/4.9 Regression] ICE: in timevar_start, at timevar.c:343

[trippels at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |4.7.4

[Bug c++/57524] [4.7/4.8/4.9 Regression] ICE: in timevar_start, at timevar.c:343

[paolo.carlini at oracle dot com]

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

Paolo Carlini <paolo.carlini at oracle dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
           Assignee|unassigned at gcc dot gnu.org      |paolo.carlini at oracle dot com

--- Comment #12 from Paolo Carlini <paolo.carlini at oracle dot com> ---
As far as I can see, it's just matter of using timevar_cond_start instead of
timevar_start.

[Bug c++/57524] [4.7/4.8/4.9 Regression] ICE: in timevar_start, at timevar.c:343

[paolo at gcc dot gnu.org]

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

--- Comment #13 from paolo at gcc dot gnu.org <paolo at gcc dot gnu.org> ---
Author: paolo
Date: Fri Jan 24 15:45:14 2014
New Revision: 207047

URL: http://gcc.gnu.org/viewcvs?rev=207047&root=gcc&view=rev
Log:
/cp
2014-01-24  Paolo Carlini  <paolo.carlini@oracle.com>

    PR c++/57524
    * name-lookup.c (push_using_directive): Use timevar_cond_start.

/testsuite
2014-01-24  Paolo Carlini  <paolo.carlini@oracle.com>

    PR c++/57524
    * g++.dg/ext/timevar2.C: New.

Added:
    trunk/gcc/testsuite/g++.dg/ext/timevar2.C
Modified:
    trunk/gcc/cp/ChangeLog
    trunk/gcc/cp/name-lookup.c
    trunk/gcc/testsuite/ChangeLog

[Bug c++/57524] [4.7/4.8/4.9 Regression] ICE: in timevar_start, at timevar.c:343

[paolo at gcc dot gnu.org]

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

--- Comment #14 from paolo at gcc dot gnu.org <paolo at gcc dot gnu.org> ---
Author: paolo
Date: Fri Jan 24 15:53:07 2014
New Revision: 207048

URL: http://gcc.gnu.org/viewcvs?rev=207048&root=gcc&view=rev
Log:
/cp
2014-01-24  Paolo Carlini  <paolo.carlini@oracle.com>

    PR c++/57524
    * name-lookup.c (push_using_directive): Use timevar_cond_start.

/testsuite
2014-01-24  Paolo Carlini  <paolo.carlini@oracle.com>

    PR c++/57524
    * g++.dg/ext/timevar2.C: New.

Added:
    branches/gcc-4_8-branch/gcc/testsuite/g++.dg/ext/timevar2.C
Modified:
    branches/gcc-4_8-branch/gcc/cp/ChangeLog
    branches/gcc-4_8-branch/gcc/cp/name-lookup.c
    branches/gcc-4_8-branch/gcc/testsuite/ChangeLog

[Bug c++/57524] [4.7 Regression] ICE: in timevar_start, at timevar.c:343

[paolo.carlini at oracle dot com]

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

Paolo Carlini <paolo.carlini at oracle dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|---                         |FIXED
           Assignee|paolo.carlini at oracle dot com    |unassigned at gcc dot gnu.org
   Target Milestone|4.7.4                       |4.8.3
            Summary|[4.7/4.8/4.9 Regression]    |[4.7 Regression] ICE: in
                   |ICE: in timevar_start, at   |timevar_start, at
                   |timevar.c:343               |timevar.c:343

--- Comment #15 from Paolo Carlini <paolo.carlini at oracle dot com> ---
Fixed mainline and 4.8.3.