Re: [testsuite] require arm_little_endian in two tests

2011-10-14 Thread Richard Earnshaw
On 14/10/11 16:21, Joseph S. Myers wrote: On Fri, 14 Oct 2011, Richard Earnshaw wrote: One way we might address this is to redefine our 128-bit vector types as structs of low/high Dwords. Each Dword remains a vector (apart from 64-bit lane types), but the Dword order then matches the ABI

Re: PATCH: PR rtl-optimization/50696: [x32] Unnecessary lea

2011-10-14 Thread Paolo Bonzini
On 10/14/2011 05:36 PM, H.J. Lu wrote: There is a testcase at http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50696 It passes with my patch. Cool, so let's wait for the results of testing. Paolo

Re: [Patch,AVR] Print no-return functions as JMP

2011-10-14 Thread Georg-Johann Lay
Richard Henderson schrieb: On 10/13/2011 11:31 PM, Georg-Johann Lay wrote: Richard Henderson schrieb: On 10/13/2011 12:00 PM, Georg-Johann Lay wrote: What do you propose? o A command line option that is on per default like -mnoreturn-tail-calls or -mjmp-noreturn The command-line-option.

Re: PATCH: PR rtl-optimization/50696: [x32] Unnecessary lea

2011-10-14 Thread H.J. Lu
On Fri, Oct 14, 2011 at 9:23 AM, Paolo Bonzini bonz...@gnu.org wrote: On 10/14/2011 05:36 PM, H.J. Lu wrote: There is a testcase at http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50696 It passes with my patch. Cool, so let's wait for the results of testing. Paolo Here is the complete

Re: [PATCH] 32-byte integer vec_interleave_{high,low}mode

2011-10-14 Thread Richard Henderson
On 10/13/2011 11:16 PM, Jakub Jelinek wrote: 2011-10-14 Jakub Jelinek ja...@redhat.com * config/i386/sse.md (vec_interleave_highmode, vec_interleave_lowmode): Add AVX2 expanders for VI_256 modes. * config/i386/i386.c (expand_vec_perm_interleave3): New function.

Re: [Patch,AVR] Print no-return functions as JMP

2011-10-14 Thread Paolo Bonzini
On 10/14/2011 06:23 PM, Georg-Johann Lay wrote: +@item -mjump-to-noreturn +@opindex mjump-to-noreturn +Use a jump instruction instead of a call instruction when calling a +no-return functions. This option is active if optimization is turned +on and just affects the way a call instruction is

[Patch,AVR,Comitted]: Avoid unwind warning from toplev.c [fix thinko]

2011-10-14 Thread Georg-Johann Lay
Denis Chertykov schrieb: 2011/10/10 Georg-Johann Lay a...@gjlay.de: toplev.c complains about unwind tables currently require a frame pointer for correctness. This patchlet supplies a fix to avoid build warnings/test fails in that it sets flag_omit_frame_pointer to 0 if unwind needs FP.

libobjc/50002: Applied fix to 4.6 branch as well

2011-10-14 Thread Nicola Pero
I applied the following patch to the 4.6 branch to backport the fix for libobjc/50002. It makes sense to backport it to 4.6.x so that it appears in 4.6.2. It really is quite a bug, and the fix is simple/safe (and, the ObjFW guys were particularly keen on it). Thanks Index: class.c

libobjc/49883: Applied fix to 4.6 branch as well

2011-10-14 Thread Nicola Pero
I applied the following patch to backport the fix for libobjc/49883 to GCC 4.6 so that it appears in 4.6.2. This is the clang-related problem that was recently discussed. Again, it's an important fix, but safe, with users (the ObjFW guys) asking for it in 4.6.2, which made total sense, so I

Re: [PATCH, Atom] Fix performance regression with -mtune=atom

2011-10-14 Thread H.J. Lu
On Fri, Oct 14, 2011 at 10:13 AM, Vladimir Yakovlev vbyakov...@gmail.com wrote: Patch is attached. Thanks, Vladimir 2011-10-14  Yakovlev Vladimir  vladimir.b.yakov...@intel.com ^ Should be in .     * gcc/config/i386/i386.c (atom_cost): Changed cost for

[PATCH] Fix typo in Builtin infrastructure change

2011-10-14 Thread Michael Meissner
David pointed out that I had a typo in the AIX code in my builtin changes on October 11th. I've checked this patch in as obvious. 2011-10-14 Michael Meissner meiss...@linux.vnet.ibm.com * config/rs6000/rs6000.c (rs6000_init_builtins): Fix typo in my change on October 11th,

Re: Fix for PR obj-c++/48275 (getter=namespace failing with .mm)

2011-10-14 Thread Nicola Pero
Can I apply this fix to the 4.6 branch as well ? Some users are asking for it to be backported to the 4.6 branch. It's not a regression, because the property implementation is new in GCC 4.6, but it is still a serious bug if you're trying to use properties with ObjC++, as you can't use

C++ PATCH for c++/50507 (NSDMI for const field)

2011-10-14 Thread Jason Merrill
We should check for an initializer before complaining about lack thereof. :) Tested x86_64-pc-linux-gnu, applying to trunk. commit fc150101bb79b3b328263e272a44eecd1083e6cd Author: Jason Merrill ja...@redhat.com Date: Thu Oct 13 21:57:21 2011 -0400 PR c++/50507 * method.c

[C++ Patch] PR 50732

2011-10-14 Thread Paolo Carlini
Hi, submitter complains that, at variance with C++11, __is_base_of doesn't handle an incomplete base type (the first parameter). The reason seems simple: in finish_trait_expr we try to complete *both* types instead of doing it where/when necessary. Tested x86_64-linux. Ok? Thanks, Paolo.

Re: [PATCH] Fix target default on biarch Linux/Sparc

2011-10-14 Thread Eric Botcazou
If you configure a biarch Linux/Sparc compiler defaulting to 32-bit, but give --with-cpu= for a v9 cpu it erroneously turns on 64-bit in TARGET_DEFAULT. PR target/50354 reports the breakage of the opposite case after the change: configuring for sparc64-linux --with-cpu=v8 used to build a

Re: [C++ Patch] PR 50732

2011-10-14 Thread Paolo Carlini
On 10/14/2011 08:23 PM, Paolo Carlini wrote: Hi, submitter complains that, at variance with C++11, __is_base_of doesn't handle an incomplete base type (the first parameter). The reason seems simple: in finish_trait_expr we try to complete *both* types instead of doing it where/when

[PATCH] AVX2 vec_widen_[su]mult_{hi,lo}*, sdot_prod* and udot_prod* (take 2)

2011-10-14 Thread Jakub Jelinek
On Fri, Oct 14, 2011 at 09:21:15AM -0700, Richard Henderson wrote: So what you're doing here is the low-part permutation: 0 4 1 5 2 6 3 7 followed by a shift to get 4 . 5 . 6 . 7 . But you need to load a 256-bit constant from memory to get it. Right. I wonder if it

Re: [C++ Patch] PR 50732

2011-10-14 Thread Paolo Carlini
On 10/14/2011 08:30 PM, Paolo Carlini wrote: On 10/14/2011 08:23 PM, Paolo Carlini wrote: Hi, submitter complains that, at variance with C++11, __is_base_of doesn't handle an incomplete base type (the first parameter). The reason seems simple: in finish_trait_expr we try to complete *both*

Re: [Patch,AVR] Print no-return functions as JMP

2011-10-14 Thread Paolo Bonzini
On Fri, Oct 14, 2011 at 19:19, Georg-Johann Lay a...@gjlay.de wrote: Paolo Bonzini schrieb: On 10/14/2011 06:23 PM, Georg-Johann Lay wrote: +@item -mjump-to-noreturn +@opindex mjump-to-noreturn +Use a jump instruction instead of a call instruction when calling a +no-return functions.  This

Re: [Patch,AVR] Print no-return functions as JMP

2011-10-14 Thread Pedro Alves
On Friday 14 October 2011 18:19:00, Georg-Johann Lay wrote: Paolo Bonzini schrieb: On 10/14/2011 06:23 PM, Georg-Johann Lay wrote: +@item -mjump-to-noreturn +@opindex mjump-to-noreturn +Use a jump instruction instead of a call instruction when calling a +no-return functions. This

Re: C++ PATCH for c++/50614 (ICE with NSDMI and -fcompare-debug)

2011-10-14 Thread Jason Merrill
On 10/13/2011 05:22 PM, Jason Merrill wrote: + DECL_INITIAL (r) = error_mark_node; While working on 50507 I noticed that we check for error_mark_node in walk_field_subobs, so I'm changing the placeholder to void_zero_node. Tested x86_64-pc-linux-gnu, applying to trunk. commit

Re: [C++ Patch] PR 50732

2011-10-14 Thread Jason Merrill
How about using complete_type_or_else? Jason

Re: [PATCH] AVX2 vec_widen_[su]mult_{hi,lo}*, sdot_prod* and udot_prod* (take 2)

2011-10-14 Thread Richard Henderson
On 10/14/2011 11:34 AM, Jakub Jelinek wrote: 2011-10-14 Jakub Jelinek ja...@redhat.com * config/i386/sse.md (vec_widen_smult_hi_v8hi, vec_widen_smult_lo_v8hi, vec_widen_umult_hi_v8hi, vec_widen_umult_lo_v8hi): Macroize using VI2_AVX2 mode iterator and any_extend

Re: [rs6000] Enable scalar shifts of vectors

2011-10-14 Thread Michael Meissner
shift_right_vect,.-.L.shift_right_vect .comm c,4096,32 .comm b,4096,32 .comm a,4096,32 .ident GCC: (GNU) 4.7.0 20111014 (experimental) .file foo.c .section.toc,aw .section.text .section.toc,aw .LC1

Re: [PATCH] Fix target default on biarch Linux/Sparc

2011-10-14 Thread David Miller
From: Eric Botcazou ebotca...@adacore.com Date: Fri, 14 Oct 2011 20:27:03 +0200 If you configure a biarch Linux/Sparc compiler defaulting to 32-bit, but give --with-cpu= for a v9 cpu it erroneously turns on 64-bit in TARGET_DEFAULT. PR target/50354 reports the breakage of the opposite case

RE: Intrinsics for N2965: Type traits and base classes

2011-10-14 Thread Michael Spertus
Redo test to run at compile-time per Jason's suggestion Index: libstdc++-v3/include/tr2/type_traits === --- libstdc++-v3/include/tr2/type_traits(revision 0) +++ libstdc++-v3/include/tr2/type_traits(revision 0) @@ -0,0

Re: Intrinsics for N2965: Type traits and base classes

2011-10-14 Thread Jason Merrill
Looks good, thanks. I'll let Benjamin check this in. Jason

RE: Intrinsics for N2965: Type traits and base classes

2011-10-14 Thread Michael Spertus
:) -Original Message- From: Jason Merrill [mailto:ja...@redhat.com] Sent: Friday, October 14, 2011 2:41 PM To: Michael Spertus Cc: Benjamin Kosnik; Jonathan Wakely; gcc-patches@gcc.gnu.org; libstd...@gcc.gnu.org Subject: Re: Intrinsics for N2965: Type traits and base classes

Re: [rs6000, spu] Add vec_perm named pattern

2011-10-14 Thread Michael Meissner
On Wed, Oct 12, 2011 at 03:42:12PM -0700, Richard Henderson wrote: The generic support for vector permutation will allow for automatic lowering to V*QImode, so all we need to add to support for these targets is the single V16QI pattern that represents the base permutation insn. I'm not

Re: [Patch,AVR] Fix PR46278, Take #3

2011-10-14 Thread Gerald Pfeifer
This patch breaks bootstrap on FreeBSD 8 and FreeBSD 9 and everyone else using makeinfo 4.8 (when the minimum we require right now is version 4.7). Hence I went ahead and applied to simple patch below. If you'd like to address this differently, happy to see a different approach. Testing a

Re: [Patch,AVR] Print no-return functions as JMP

2011-10-14 Thread Gerald Pfeifer
+@item -mjump-to-noreturn +@opindex mjump-to-noreturn +Use a jump instruction instead of a call instruction when calling a +no-return functions. This option is active if optimization is turned +on and just affects the way a call instruction is printed out. Would emit be better here than

Re: [PATCH] Fix target default on biarch Linux/Sparc

2011-10-14 Thread Eric Botcazou
If one wants a 32-bit default compiler, they should build for the sparc-linux target. And this is absolutely trivial to make happen in the environments where this is supposedly a problem. I have criticized so many times this combination in the past, while Jakub and also you IIRC were

RE: Question about default_elf_asm_named_section function

2011-10-14 Thread Iyer, Balaji V
Attached, please find a patch fixing this issue. 2011-10-14 Balaji V. Iyer balaji.v.i...@intel.com * varasm.c (default_elf_asm_named_section): Removed ATTRIBUTE_UNUSED tag before decl. Thanks, Balaji V. Iyer. -Original Message- From: Ian Lance Taylor

Re: [Patch,AVR] Fix PR46278, Take #3

2011-10-14 Thread Georg-Johann Lay
Gerald Pfeifer schrieb: This patch breaks bootstrap on FreeBSD 8 and FreeBSD 9 and everyone else using makeinfo 4.8 (when the minimum we require right now is version 4.7). Hence I went ahead and applied to simple patch below. If you'd like to address this differently, happy to see a different

Re: [PATCH] Fix target default on biarch Linux/Sparc

2011-10-14 Thread David Miller
From: Eric Botcazou ebotca...@adacore.com Date: Fri, 14 Oct 2011 22:31:53 +0200 If one wants a 32-bit default compiler, they should build for the sparc-linux target. And this is absolutely trivial to make happen in the environments where this is supposedly a problem. I have criticized so

Re: [PATCH 3/7] Emit macro expansion related diagnostics

2011-10-14 Thread Jason Merrill
On 10/13/2011 01:12 PM, Dodji Seketeli wrote: + while (true) +{ + if (!linemap_macro_expansion_map_p (map0) + || !linemap_macro_expansion_map_p (map1) + || map0 == map1) + break; I'd put the test in the condition, but if you find it clearer this way, I guess

Re: [patch, Fortran] Change -std=f2008tr to f2008ts, update *.texi status and TR29113-TS29113

2011-10-14 Thread Tobias Burnus
*ping* http://gcc.gnu.org/ml/fortran/2011-10/msg00073.html On 12.10.2011 15:57, Tobias Burnus wrote: Hello all, this patch does two things: a) It updates the Fortran 2003 and TR/TS 29113 status in the GNU Fortran manual. b) It changes all references to Technical Report 29113 to Technical

Re: [rs6000, spu] Add vec_perm named pattern

2011-10-14 Thread Richard Henderson
On 10/14/2011 12:53 PM, Michael Meissner wrote: I would rather change altivec_vperm_mode to use the new name (and also altivec_vperm_mode_uns). But I can live with a wrapper function for now. As I said, I'm leaving the cleanup of the old patterns to port maintainers. If we are adding permute

Re: [google] support for building Linux kernel with FDO (issue4523061)

2011-10-14 Thread vulcansh
I found this thread through a Google search so I didn't have much context. Can you point me to the google/main source repo for gcc and the kernel? Are there plans to migrate this patch to the gcc and Linux mainline? Sorry, but I have never used the PGO option in icc. Steve Xinliang David Li

Re: [C++ Patch] PR 50732

2011-10-14 Thread Paolo Carlini
On 10/14/2011 09:08 PM, Jason Merrill wrote: How about using complete_type_or_else? The CPTK_IS_BASE_OF case becomes much simpler indeed, thanks. For the unary traits, though, I don't see an advantage in using it, because in some cases in check_trait_type we don't want to error out even when

Re: [C++ Patch] PR 50732

2011-10-14 Thread Jason Merrill
On 10/14/2011 05:58 PM, Paolo Carlini wrote: On 10/14/2011 09:08 PM, Jason Merrill wrote: How about using complete_type_or_else? The CPTK_IS_BASE_OF case becomes much simpler indeed, thanks. For the unary traits, though, I don't see an advantage in using it, because in some cases in

[pph] Fix merging for namespaces. (issue5280048)

2011-10-14 Thread Lawrence Crowl
Factor pph_out_binding_level and pph_in_binding_level into three routines each: one to handle bindings that merge; one to handle those that do not; and one to handle the parts common to both. Streaming the global namespace now uses the mergeable version of binding level streaming. At present we

Re: [Patch,AVR] Print no-return functions as JMP

2011-10-14 Thread Richard Kenner
+@item -mjump-to-noreturn +@opindex mjump-to-noreturn +Use a jump instruction instead of a call instruction when calling a +no-return functions. This option is active if optimization is turned +on and just affects the way a call instruction is printed out. Would emit be better here than

Re: [C++ Patch] PR 50732

2011-10-14 Thread Paolo Carlini
On 10/15/2011 12:20 AM, Jason Merrill wrote: That should work. Excellent. Then we can do something like the below, a great improvement. I'm finishing testing it (already past g++.dg), Ok if it passes? Thanks, Paolo. /// /cp 2011-10-14 Paolo Carlini

Re: [PATCH] Fix target default on biarch Linux/Sparc

2011-10-14 Thread Eric Botcazou
Agreed, please install this patch if you haven't already. Thanks, done after successfully bootstrapping on SPARC64/Linux. -- Eric Botcazou

Re: [pph] Make libcpp symbol validation a warning (issue5235061)

2011-10-14 Thread Gabriel Charette
Yes, I understand that. But when the second 2.pph is skipped when reading foo.pph, the reading of its line_table is also skipped (as foo.pph doesn't contain the line_table information for 2.h, 2.pph does and adds it when its included as a child, but if it's skipped, the line_table info for 2.h

Re: [SH] PR 49263 - underutilized TST #imm, R0 instruction

2011-10-14 Thread Kaz Kojima
Oleg Endo oleg.e...@t-online.de wrote: the attached patch is the same as the last proposed patch in the PR but with some fixed formatting and comments. Hope it's fine like that. Tested against trunk rev 179778 with make -k -j4 check RUNTESTFLAGS=--target_board=sh-sim

Re: Fix for PR obj-c++/48275 (getter=namespace failing with .mm)

2011-10-14 Thread Mike Stump
On Oct 14, 2011, at 10:51 AM, Nicola Pero wrote: Can I apply this fix to the 4.6 branch as well ? OK to commit to the 4.6 branch ? Ok.

<    1   2   3