Re: [patch testsuite g++.dg]: Reflect ABI change for windows native targets about bitfield layout in structures

2011-12-27 Thread Kai Tietz
Ping 2011/12/16 Dave Korn : > On 16/12/2011 09:01, Kai Tietz wrote: >> 2011/12/15 Dave Korn: > >>> { dg-options "-mno-align-double" { target i?86-*-cygwin* i?86-*-mingw* } } >>> { dg-additional-options "-mno-ms-bitfields" { target i?86-*-mingw* } } >>> >>> ... so that MinGW gets both and Cygwin on

Re: [patch testsuite g++.old-deja]: Fix some testcases for mingw targets

2011-12-27 Thread Kai Tietz
Ping 2011/12/15 Dave Korn : > On 15/12/2011 17:44, Mike Stump wrote: >> On Dec 15, 2011, at 1:43 AM, Kai Tietz wrote: >>> This patch takes care that we are using for operator new/delete >>> replacement test static version on mingw-targets.  As the shared (DLL) >>> version isn't able to have operat

Re: [C++ Patch] for c++/23211

2011-12-27 Thread Jason Merrill
OK, thanks. Jason

Re: [C++ Patch] for c++/23211

2011-12-27 Thread Fabien Chêne
2011/12/27 Jason Merrill : > On 12/27/2011 02:12 PM, Fabien Chêne wrote: >> >> -  if (!scope_dependent_p) >> +  if (!dependent_scope_p (scope)) > > I was thinking to change the line > >>  scope_dependent_p = dependent_type_p (scope); > > to use dependent_scope_p instead of dependent_type_p. OK, I

Go patch committed: No func, map or slice comparisons

2011-12-27 Thread Ian Lance Taylor
The Go language was changed to prohibit comparing values of func, map, or slice type, except for comparisons to nil. This patch implements that in the Go frontend. The patch includes some testsuite updates. Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline. Ian

Go patch committed: Tweak debug info

2011-12-27 Thread Ian Lance Taylor
This patch to the gcc-specific part of the Go frontend tweaks the debug info in a couple of ways. For a named struct or array type, this uses build_distinct_type_copy rather than build_variant_type_copy. Using build_variant_type_copy caused trouble because it wound up causing the main variant to

Re: [C++ Patch] for c++/23211

2011-12-27 Thread Jason Merrill
On 12/27/2011 02:12 PM, Fabien Chêne wrote: - if (!scope_dependent_p) + if (!dependent_scope_p (scope)) I was thinking to change the line scope_dependent_p = dependent_type_p (scope); to use dependent_scope_p instead of dependent_type_p. Jason

Re: [C++ Patch] for c++/23211

2011-12-27 Thread Fabien Chêne
2011/12/21 Jason Merrill : > This seems problematic to me; it could be that a dependent scope ends up > matching a non-dependent base in the end, i.e. > > struct A { int x; }; > > template > struct B: A > { >  using T::x; > }; > > B b; Very nice indeed, I hadn't thought of that. > this code is s

Re: -finstrument-functions leads to unsats to _mm instrinsic wrappers

2011-12-27 Thread Xinliang David Li
Ping .. On Fri, Dec 9, 2011 at 9:42 AM, Xinliang David Li wrote: > Ping .. > > On Wed, Dec 7, 2011 at 4:01 PM, Xinliang David Li wrote: >> Build the test case in the patch file with -finstrument-functions, the >> link will fail with unsat. The problem is gcc instruments the >> artificial wrapper

Re: [google] Patch to enable efficient function level instrumentation (issue 5416043)

2011-12-27 Thread Xinliang David Li
Ok for google branches when tests are done. Update ChangeLog file properly. David On Tue, Dec 20, 2011 at 1:55 PM, Harshit Chopra wrote: > > > On Fri, Dec 16, 2011 at 3:10 PM, wrote: >> >> Ok, Cary's explanation makes sense. Please update the comments to make >> it clearer. >> >> >> >> http://c

Re: [patch] Flag-controlled type conversions/promotions

2011-12-27 Thread Steve Kargl
On Tue, Dec 27, 2011 at 12:52:19PM +0100, Dominique Dhumieres wrote: > > -freal-4-real-8 is not equivalent to -fdefault-real-8 and > > -fdefault-double-8. > > > > -freal-4-real-8 interprets any 4-byte real type, whether it is a > > default real type or explicitly declared as 4-byte, as a 8-byte d

Fix PR rtl-optimization/51667

2011-12-27 Thread Eric Botcazou
. (find_removable_extensions): Create and destroy the definition map. (find_and_remove_re): Return void. Change 'long' variables to 'int'. Do not deal with is_insn_merge_attempted. 2011-12-27 Eric Botcazou * gcc.c-torture/execute/20111227-1.c: New test. -- Eric Botcazo

Re: [patch] Flag-controlled type conversions/promotions

2011-12-27 Thread Zydrunas Gimbutas
On Tue, Dec 27, 2011 at 6:52 AM, Dominique Dhumieres wrote: >> -freal-4-real-8 is not equivalent to -fdefault-real-8 and -fdefault-double-8. >> >> -freal-4-real-8 interprets any 4-byte real type, whether it is a >> default real type or explicitly declared as 4-byte, as a 8-byte double >> precision

Re: [Patch] Adjust diag-scans in vect-tests to fix fails on AVX/AVX2

2011-12-27 Thread Dominique Dhumieres
The patch at http://gcc.gnu.org/ml/gcc-patches/2011-12/msg01600/vec-tests-avx2_fixes-7.patch fixes the XPASS, tested on powerpc-apple-darwin9 and x86_64-apple-darwin10. Thanks, Dominique

PING: Re: [PATCH] small build fix

2011-12-27 Thread Robert Millan
El 19 de desembre de 2011 21:04, Robert Millan ha escrit: > This small patch fixes a build regression on GNU/kFreeBSD. Ping! -- Robert Millan

Re: [patch] Flag-controlled type conversions/promotions

2011-12-27 Thread Dominique Dhumieres
> -freal-4-real-8 is not equivalent to -fdefault-real-8 and -fdefault-double-8. > > -freal-4-real-8 interprets any 4-byte real type, whether it is a > default real type or explicitly declared as 4-byte, as a 8-byte double > precision type, and that applies to all variables, functions and > constan

PR middle-end/48641 (tree-ssa-threadupdate messing up profile)

2011-12-27 Thread Jan Hubicka
Hi this patch solves problem with negative frequency that is result of overflow in profile updating code. The testcase shows quite pathological case where profile is misguessed and the mistake is propagated across the CFG by jump threading. I can't of think of much better solution than adding a c

[PATCH, alpha]: Use __sync* functions in libjava locks.h

2011-12-27 Thread Uros Bizjak
Hello! No functional change. 2011-12-27 Uros Bizjak PR libgcj/49193 * sysdep/alpha/locks.h (compare_and_swap): Call __sync_bool_compare_and_swap. (release_set): Call __sync_synchronize. Tested on alphaeev68-pc-linux-gnu, committed to mainline SVN. Uros. Index