[Bug c++/71482] Add -Wglobal-constructors warning option

2024-05-05 Thread egallager at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71482

--- Comment #9 from Eric Gallager  ---
(In reply to Jonathan Wakely from comment #8)
> (In reply to Eric Gallager from comment #6)
> > Another reason this warning might be wanted: name mangling and demangling of
> > global constructors has been buggy for awhile now; see bug 54254
> 
> Looks like that's just a problem demangling the symbol name to print it in a
> human-readable form. What's buggy about the mangling?

Well, I guess I was just remembering that "mangler dogfooding" proposal that
would have added a checking option to ensure that every mangling of a symbol
name that the mangler emits can also be demangled by the demangler... that
didn't go in, did it? If it had, this would be an example of something that
might trip it up...

[Bug c++/71482] Add -Wglobal-constructors warning option

2024-05-04 Thread redi at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71482

--- Comment #8 from Jonathan Wakely  ---
(In reply to Eric Gallager from comment #6)
> Another reason this warning might be wanted: name mangling and demangling of
> global constructors has been buggy for awhile now; see bug 54254

Looks like that's just a problem demangling the symbol name to print it in a
human-readable form. What's buggy about the mangling?

[Bug c++/71482] Add -Wglobal-constructors warning option

2024-05-04 Thread egallager at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71482

Eric Gallager  changed:

   What|Removed |Added

   See Also||https://gcc.gnu.org/bugzill
   ||a/show_bug.cgi?id=2474,
   ||https://gcc.gnu.org/bugzill
   ||a/show_bug.cgi?id=56009

--- Comment #7 from Eric Gallager  ---
(In reply to Eric Gallager from comment #6)
> Another reason this warning might be wanted: name mangling and demangling of
> global constructors has been buggy for awhile now; see bug 54254

Some more bugs about global constructors/destructors that might lead one to
want this warning: bug 2474 and bug 56009

[Bug c++/71482] Add -Wglobal-constructors warning option

2024-04-03 Thread egallager at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71482

Eric Gallager  changed:

   What|Removed |Added

   See Also||https://gcc.gnu.org/bugzill
   ||a/show_bug.cgi?id=54254

--- Comment #6 from Eric Gallager  ---
Another reason this warning might be wanted: name mangling and demangling of
global constructors has been buggy for awhile now; see bug 54254

[Bug c++/71482] Add -Wglobal-constructors warning option

2021-07-24 Thread i at maskray dot me via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71482

Fangrui Song  changed:

   What|Removed |Added

 CC||i at maskray dot me

--- Comment #5 from Fangrui Song  ---
Still wanted.

In llvm-project, compiler-rt is using -Wglobal-constructors. llvm/lib/Support
will use it as well.

[Bug c++/71482] Add -Wglobal-constructors warning option

2019-05-31 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71482

--- Comment #4 from Marek Polacek  ---
(In reply to Eric Gallager from comment #3)
> (In reply to Marek Polacek from comment #2)
> > Patch: https://gcc.gnu.org/ml/gcc-patches/2019-05/msg01860.html
> 
> Does Sean have a bugzilla account?

I don't see it.  Probably not yet.

[Bug c++/71482] Add -Wglobal-constructors warning option

2019-05-31 Thread egallager at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71482

Eric Gallager  changed:

   What|Removed |Added

 CC||msebor at gcc dot gnu.org

--- Comment #3 from Eric Gallager  ---
(In reply to Marek Polacek from comment #2)
> Patch: https://gcc.gnu.org/ml/gcc-patches/2019-05/msg01860.html

Does Sean have a bugzilla account?

[Bug c++/71482] Add -Wglobal-constructors warning option

2019-05-30 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71482

Marek Polacek  changed:

   What|Removed |Added

 CC||mpolacek at gcc dot gnu.org

--- Comment #2 from Marek Polacek  ---
Patch: https://gcc.gnu.org/ml/gcc-patches/2019-05/msg01860.html

[Bug c++/71482] Add -Wglobal-constructors warning option

2016-06-10 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71482

Richard Biener  changed:

   What|Removed |Added

 Status|UNCONFIRMED |NEW
   Last reconfirmed||2016-06-10
 Ever confirmed|0   |1

--- Comment #1 from Richard Biener  ---
Confirmed.