[Bug c++/77489] Invalid mangling of static local variables

2017-01-18 Thread trippels at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77489 --- Comment #9 from Markus Trippelsdorf --- Author: trippels Date: Wed Jan 18 15:49:15 2017 New Revision: 244580 URL: https://gcc.gnu.org/viewcvs?rev=244580&root=gcc&view=rev Log: PR c++/77489 -- Reorganize abi warning check PR c++/7748

[Bug c++/77489] Invalid mangling of static local variables

2017-01-18 Thread trippels at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77489 Markus Trippelsdorf changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[Bug c++/77489] Invalid mangling of static local variables

2017-01-18 Thread trippels at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77489 --- Comment #7 from Markus Trippelsdorf --- Author: trippels Date: Wed Jan 18 08:40:05 2017 New Revision: 244566 URL: https://gcc.gnu.org/viewcvs?rev=244566&root=gcc&view=rev Log: Fix PR77489 -- mangling of discriminator >= 1 libiberty:

[Bug c++/77489] Invalid mangling of static local variables

2017-01-10 Thread jason at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77489 --- Comment #6 from Jason Merrill --- (In reply to Markus Trippelsdorf from comment #2) > Untested patch: This looks good, except that the mangling change needs to depend on -fabi-version, and in the demangling change open braces go on a new lin

[Bug c++/77489] Invalid mangling of static local variables

2016-09-05 Thread trippels at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77489 --- Comment #5 from Markus Trippelsdorf --- The intel compiler uses an idiosyncratic approach: markus@x4 tmp % icpc testcase.cc -S -c -o - .file "testcase.cc" .text ..TXTST0: # -- Begin _Z3foov .text .align1

[Bug c++/77489] Invalid mangling of static local variables

2016-09-05 Thread trippels at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77489 --- Comment #4 from Markus Trippelsdorf --- Yeah, keeping the old behavior in the demangler would go against the raison d'etre of the ABI change. David's example _ZZ1gvEN1Y1fEZ1gvE1X_11C would then never demangle correctly. BTW there is another

[Bug c++/77489] Invalid mangling of static local variables

2016-09-05 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77489 --- Comment #3 from Jakub Jelinek --- Seems the ABI changed in 2009 here, Jan 30 2009 version only has the _ part there, while Feb 25 2010 already has the current wording. I found: http://sourcerytools.com/pipermail/cxx-abi-dev/2009-June/002170

[Bug c++/77489] Invalid mangling of static local variables

2016-09-05 Thread trippels at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77489 --- Comment #2 from Markus Trippelsdorf --- Untested patch: diff --git a/gcc/cp/mangle.c b/gcc/cp/mangle.c index bd23260c0883..322f52944f41 100644 --- a/gcc/cp/mangle.c +++ b/gcc/cp/mangle.c @@ -1896,7 +1896,8 @@ discriminator_for_string_literal

[Bug c++/77489] Invalid mangling of static local variables

2016-09-05 Thread trippels at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77489 Markus Trippelsdorf changed: What|Removed |Added Keywords||wrong-code Status|UNCO