[Bug c++/16166] -Weffc++ finer granularity

2012-05-29 Thread redi at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=16166 --- Comment #8 from Jonathan Wakely 2012-05-29 23:21:23 UTC --- I would keep the patches separate.

[Bug c++/53494] [4.7/4.8 Regression] ICE with invalid initializer list

2012-05-29 Thread redi at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53494 --- Comment #16 from Jonathan Wakely 2012-05-29 23:34:14 UTC --- (In reply to comment #14) > struct pair > { > pair(const char*, int) { } > }; > > struct array_p > { > pair data[1]; > }; > > array_p a = { { "smile", 1 } }; > > Here we hav

[Bug c++/53494] [4.7/4.8 Regression] ICE with invalid initializer list

2012-05-29 Thread redi at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53494 --- Comment #17 from Jonathan Wakely 2012-05-29 23:36:02 UTC --- clang version 3.2 (trunk 155804) also rejects it: t.cc:11:18: error: no viable conversion from 'const char [6]' to 'pair'

[Bug debug/53453] darwin linker expects both AT_name and AT_comp_dir debug notes

2012-05-29 Thread mrs at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53453 --- Comment #15 from mrs at gcc dot gnu.org 2012-05-29 23:44:13 UTC --- Author: mrs Date: Tue May 29 23:44:09 2012 New Revision: 187994 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=187994 Log: 2012-05-29 Jack Howarth PR debug/53

[Bug c++/10200] Weird clash with same names in different scopes

2012-05-29 Thread eatmyshortz at gmail dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=10200 Matt Giuca changed: What|Removed |Added CC||eatmyshortz at gmail dot |

[Bug debug/53453] darwin linker expects both AT_name and AT_comp_dir debug notes

2012-05-29 Thread mrs at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53453 m...@gcc.gnu.org changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED Known to work|

[Bug c++/53356] [4.7/4.8 Regression] ICE in verify_gimple_stmt, at tree-cfg.c:4258

2012-05-29 Thread hjl.tools at gmail dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53356 H.J. Lu changed: What|Removed |Added CC||jason at redhat dot com --- Comment #2 from H.J

[Bug c++/53524] New: [4.7/4.8 Regression] Bogus and unsupressable enum comparison warning

2012-05-29 Thread ppluzhnikov at google dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53524 Bug #: 53524 Summary: [4.7/4.8 Regression] Bogus and unsupressable enum comparison warning Classification: Unclassified Product: gcc Version: unknown Status: UNCONFIR

[Bug c++/53524] [4.7/4.8 Regression] Bogus and unsupressable enum comparison warning

2012-05-29 Thread pinskia at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53524 --- Comment #1 from Andrew Pinski 2012-05-30 01:53:24 UTC --- >Further, the warning doesn't get suppressed even under -Wno-enum-compare. That is because it is not a compare; it is a conditional expression. Also I think the warning is correct bu

[Bug c++/53524] [4.7/4.8 Regression] Bogus and unsupressable enum comparison warning

2012-05-29 Thread jyasskin at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53524 Jeffrey Yasskin changed: What|Removed |Added CC||jyasskin at gcc dot gnu.org --- Comment

[Bug target/52911] [4.5/4.6/4.7/4.8 Regression] gcc 4.7.0 (ppc32 e500mc) when compile a c file, after a lot of time, gcc failed and internal compiler error occurs.

2012-05-29 Thread zuogang at huawei dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52911 --- Comment #4 from zuogang 2012-05-30 02:59:22 UTC --- (In reply to comment #3) > The testcase needs reduction. 1: save the attached file into test-gcc47-c.c,then cd the folder contain the C file; 2: gcc -Wall -W -Wshadow -Isrc/headers -O3 -fun

[Bug preprocessor/53525] New: Performance regression due to enabling track-macro-expansion

2012-05-29 Thread jimis at gmx dot net
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53525 Bug #: 53525 Summary: Performance regression due to enabling track-macro-expansion Classification: Unclassified Product: gcc Version: 4.8.0 Status: UNCONFIRMED

[Bug preprocessor/53525] Performance regression due to enabling track-macro-expansion

2012-05-29 Thread jimis at gmx dot net
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53525 jimis changed: What|Removed |Added CC||abel at gcc dot gnu.org, |

[Bug c++/53524] [4.7/4.8 Regression] Bogus and unsupressable enum comparison warning

2012-05-29 Thread crowl at google dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53524 Lawrence Crowl changed: What|Removed |Added CC||crowl at google dot com --- Comment #3 f

[Bug preprocessor/53525] Performance regression due to enabling track-macro-expansion

2012-05-29 Thread jimis at gmx dot net
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53525 --- Comment #2 from jimis 2012-05-30 04:44:54 UTC --- According to valgrind major overhead is due to numerous calls of line-map.c:linemap_line_start() that actually allocate new line_maps. This happens because we are resetting the max_column_hint

[Bug preprocessor/53525] Performance regression due to enabling track-macro-expansion

2012-05-29 Thread jimis at gmx dot net
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53525 --- Comment #3 from jimis 2012-05-30 04:52:20 UTC --- Another simple one that my eye caught but does not effect performance. Generally I don't get many things in macro.c, but am I correct to assume that the following stands? === modified file 'l

[Bug preprocessor/53525] Performance regression due to enabling track-macro-expansion

2012-05-29 Thread jimis at gmx dot net
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53525 --- Comment #4 from jimis 2012-05-30 05:23:54 UTC --- Another hotspot higlighted by valgrind is the multitude of malloc/free() calls in comparison to the past. I'm attaching a slightly more intrusive patch that uses obstacks to allocate some of th

[Bug preprocessor/53525] Performance regression due to enabling track-macro-expansion

2012-05-29 Thread jimis at gmx dot net
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53525 --- Comment #5 from jimis 2012-05-30 05:28:31 UTC --- Created attachment 27520 --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=27520 In macro.c:collect_args() use obstacks for virt_locs instead of malloc/realloc vectors.

[Bug preprocessor/53525] Performance regression due to enabling track-macro-expansion

2012-05-29 Thread jimis at gmx dot net
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53525 --- Comment #6 from jimis 2012-05-30 05:31:03 UTC --- Created attachment 27521 --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=27521 Add some new obstack macros in libiberty.h.

[Bug preprocessor/53525] Performance regression due to enabling track-macro-expansion

2012-05-29 Thread jimis at gmx dot net
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53525 --- Comment #7 from jimis 2012-05-30 06:01:23 UTC --- Now time for the most intrusive and problematic patches. I tried moving all virt_locs, expanded, expanded_virt_locs to obstacks for allocation. After many failures to work with obstacks as they

[Bug preprocessor/53525] Performance regression due to enabling track-macro-expansion

2012-05-29 Thread jimis at gmx dot net
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53525 --- Comment #8 from jimis 2012-05-30 06:06:16 UTC --- Created attachment 27522 --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=27522 Introduce obstack_{mark,release} functions.

[Bug preprocessor/53525] Performance regression due to enabling track-macro-expansion

2012-05-29 Thread jimis at gmx dot net
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53525 --- Comment #9 from jimis 2012-05-30 06:10:38 UTC --- Created attachment 27523 --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=27523 Move all location/expansion vectors to obstacks. Warning MEMLEAKS!

[Bug preprocessor/53525] Performance regression due to enabling track-macro-expansion

2012-05-29 Thread jimis at gmx dot net
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53525 --- Comment #10 from jimis 2012-05-30 06:23:56 UTC --- Here is how this last patch (macro4) compares to trunk (TME) and to completely disabling track-macro-expansion (noTME): time M instr noTME 0.744s 2081 TME0.785s

[Bug c++/53494] [4.7/4.8 Regression] ICE with invalid initializer list

2012-05-29 Thread daniel.kruegler at googlemail dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53494 --- Comment #18 from Daniel Krügler 2012-05-30 06:25:27 UTC --- (In reply to comment #16) > "If the initializer-list begins with a left brace," > > which it does > > "then the succeeding comma-separated list of initializer-clauses initializes >

[Bug c++/53356] [4.7/4.8 Regression] ICE in verify_gimple_stmt, at tree-cfg.c:4258

2012-05-29 Thread jakub at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53356 --- Comment #3 from Jakub Jelinek 2012-05-30 06:47:13 UTC --- Created attachment 27524 --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=27524 gcc48-pr53356.patch Untested fix. As the TARGET_EXPR for which the gimplifier wants to add a clobbe

[Bug rtl-optimization/53519] [4.7/4.8 Regression] ice in do_SUBST, at combine.c:707

2012-05-29 Thread jakub at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53519 --- Comment #3 from Jakub Jelinek 2012-05-30 06:53:53 UTC --- Author: jakub Date: Wed May 30 06:53:46 2012 New Revision: 188001 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=188001 Log: PR rtl-optimization/53519 * combine.c (simp

[Bug rtl-optimization/53519] [4.7/4.8 Regression] ice in do_SUBST, at combine.c:707

2012-05-29 Thread jakub at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53519 --- Comment #4 from Jakub Jelinek 2012-05-30 06:58:05 UTC --- Author: jakub Date: Wed May 30 06:58:00 2012 New Revision: 188002 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=188002 Log: PR rtl-optimization/53519 * combine.c (simp

[Bug rtl-optimization/53519] [4.7/4.8 Regression] ice in do_SUBST, at combine.c:707

2012-05-29 Thread jakub at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53519 Jakub Jelinek changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|

<    1   2