[Bug lto/47247] Linker plugin specification makes it difficult to handle COMDATs

2011-02-16 Thread ccoutant at google dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47247 --- Comment #20 from ccoutant at google dot com 2011-02-16 23:35:28 UTC --- > I have created a "small" test of the symbol table problem. It is in > > http://people.mozilla.com/~respindola/test.tar.xz > > The test is f

[Bug rtl-optimization/57451] Incorrect debug ranges emitted for -freorder-blocks-and-partition -g

2013-08-09 Thread ccoutant at google dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57451 --- Comment #7 from ccoutant at google dot com --- > Index: final.c > === > --- final.c (revision 201461) > +++ final.c (working copy) > @@ -1560,6 +1560,16 @@ ch

[Bug rtl-optimization/57451] Incorrect debug ranges emitted for -freorder-blocks-and-partition -g

2013-08-12 Thread ccoutant at google dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57451 --- Comment #9 from ccoutant at google dot com --- >>> + if (!active_insn_p (insn)) >>> +continue; >> >> I'm not clear on why this is needed. Is it because after the >> change_scope, insn will

[Bug other/46770] Replace .ctors/.dtors with .init_array/.fini_array on targets supporting them

2012-04-17 Thread ccoutant at google dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46770 --- Comment #81 from ccoutant at google dot com 2012-04-17 18:52:11 UTC --- >> As Paul noted, this is a moot point in practice for .ctors, since GCC emits >> only a single .ctors entry per TU, but it could be significant for assembl

[Bug other/46770] Replace .ctors/.dtors with .init_array/.fini_array on targets supporting them

2012-04-17 Thread ccoutant at google dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46770 --- Comment #83 from ccoutant at google dot com 2012-04-17 20:10:07 UTC --- >> Didn't I just do that? > > Let me ask it again: > > The proposed --reverse-init-array switch will only reverse the order across > translatio

[Bug other/46770] Replace .ctors/.dtors with .init_array/.fini_array on targets supporting them

2012-04-17 Thread ccoutant at google dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46770 --- Comment #86 from ccoutant at google dot com 2012-04-17 21:09:15 UTC --- > I have seen codes like: > > void (*const init_array []) (void) >     __attribute__ ((section (".init_array"), aligned (sizeof (void * = >

[Bug other/46770] Replace .ctors/.dtors with .init_array/.fini_array on targets supporting them

2012-04-17 Thread ccoutant at google dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46770 --- Comment #87 from ccoutant at google dot com 2012-04-17 21:52:12 UTC --- > Just as a quick reminder, the reversed ctor execution order is big performance > problem for C++ Apps inlcuding Mozilla and Chrome ;) > So whatever we do

[Bug debug/55794] FAIL: g++.dg/debug/dwarf2/non-virtual-thunk.C -std=gnu++98 and -std=gnu++11

2014-04-01 Thread ccoutant at google dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55794 --- Comment #7 from ccoutant at google dot com --- > but the change is no longer in the current 4.9 code. Ah, right. See PR 54499 and this thread: http://gcc.gnu.org/ml/gcc-patches/2012-11/msg00706.html -cary

[Bug debug/61013] [4.9/4.10 Regression] Option parsing difference between < 4.9 and 4.9

2014-04-30 Thread ccoutant at google dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=61013 --- Comment #10 from ccoutant at google dot com --- >> So, my preference would be to revert to the 4.8 and older behavior, or if >> there really is consensus that -g1 -g should mean -g2 rather than -g1, at >> least change it so

[Bug debug/61013] [4.9/4.10 Regression] Option parsing difference between < 4.9 and 4.9

2014-05-14 Thread ccoutant at google dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61013 --- Comment #12 from ccoutant at google dot com --- > FWIW this regresses a few gdb tests. It's easy to fix the > gdb test suite, but if this is going to be fixed before the > next gcc release, I'd rather not bother. Any

[Bug other/63504] [5 Regression] Issues found by --enable-checking=valgrind

2015-01-27 Thread ccoutant at google dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63504 --- Comment #14 from ccoutant at google dot com --- > But then there is (question mainly on Cary) the .debug_types checksumming: > > case dw_val_class_const_double: > CHECKSUM_ULEB128 (DW_FORM_block); > CHECKSUM_

[Bug debug/46123] [4.5/4.6 Regression] ICE: in output_aranges, at dwarf2out.c:11531 with -feliminate-dwarf2-dups -g

2010-11-18 Thread ccoutant at google dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46123 --- Comment #4 from ccoutant at google dot com 2010-11-19 02:17:27 UTC --- > Ugh, this is very ugly.  gen_subprogram_die sometimes decides to reuse old_die > which was DW_AT_declaration and can be deeply nested in type children, which &g