This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
[C++ Patch / RFC] PR 52432
- From: Paolo Carlini <paolo dot carlini at oracle dot com>
- To: "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
- Cc: Jason Merrill <jason at redhat dot com>
- Date: Wed, 19 Sep 2012 17:55:36 +0200
- Subject: [C++ Patch / RFC] PR 52432
Hi all, Jason,
the large testcase attached by Jon to this PR triggers an "Internal
compiler error: Error reporting routines re-entered." from
-fdump-tree-gimple. The immediate cause seems obvious: in one place in
tsubst_copy_and_build we are calling unqualified_name_lookup_error
unconditionally, thus irrespective of complain. Changing that indeed
avoids the ICE.
Then I come to the existing testcase which doesn't pass as-is: it's the
testcase added by Jason as part of fixing 50075, which, as analyzed by
Jason himself, was clearly about an endless recursion. Currently,
however, we error out with that unqualified_name_lookup_error, once, and
we don't mention the recursion in the error message. With the patchlet
applied, the diagnostics actually shows the recursion, and we error out
few lines above in tsubst_copy_and_build, with the error messages koenig
lookup related. That makes some sense to me, but the testcase needs
tweaking.
Thanks,
Paolo.
////////////////////////
Attachment:
patch_52432_draft
Description: Text document