Re: PATCH RFA: New configure option --with-native-system-header-dir

2011-10-10 Thread DJ Delorie
> + yes|no) AC_MSG_ERROR([bad value ${withval} given for > --with-native-system-header-dir]) ;; > + /*) ;; > + *) AC_MSG_ERROR([--with-native-system-header-dir argument ${withval} must > be an absolute directory]) ;; This doesn't handle DOS-style absolute paths (like c:/Users). Not sure if we

Re: Force aliases to be output when cgraph decides so

2011-10-10 Thread David Edelsohn
Honza, I filed PR50689. After your change, the alias is moved from near the end of the file to just before the definition. AIX assembler is a single pass assembler that does not allow forward references. - David On Sun, Sep 25, 2011 at 6:27 AM, Jan Hubicka wrote: >> Jan, >> >> This patch caus

Re: [v3] cxxabi.h vs. unwind-cxx.h

2011-10-10 Thread Benjamin Kosnik
Here's a patch that works for both C and C++. tested x86/linux -benjamin 2011-10-10 Benjamin Kosnik PR libstdc++/49818 * config/abi/pre/gnu.ver (CXXABI_1.3.6): Add symbols. * testsuite/util/testsuite_abi.cc: Same. * libsupc++/unwind-cxx.h: Move required eh API... * libsupc++/cxxabi.h:

[patch] remove empty directory in libgomp

2011-10-10 Thread Matthias Klose
2011-10-10 Matthias Klose * config/posix95: Remove empty directory. The last files in config/posix95 were removed in r177568. Comitted as obvious. Matthias

Re: Force aliases to be output when cgraph decides so

2011-10-10 Thread Jan Hubicka
> Honza, > > I filed PR50689. After your change, the alias is moved from near the > end of the file to just before the definition. AIX assembler is a > single pass assembler that does not allow forward references. I see, I guess the easy bugfix is to make the aliases output afterwards. I will

PR c++/30195

2011-10-10 Thread Fabien Chêne
2011/9/23 Jason Merrill : > On 09/22/2011 05:11 PM, Fabien Chêne wrote: >> >> 2011/9/22 Jason Merrill: > >>> I don't, it just seemed strange to handle functions differently from >>> other >>> decls here.  But when I look more closely I see that we're in >>> lookup_field_1, which isn't interested in

Re: [Patch] PR c++/26256

2011-10-10 Thread Fabien Chêne
Hi, 2011/9/26 Jason Merrill : > On 09/25/2011 05:06 PM, Fabien Chêne wrote: >> >> +  else if ((using_decl = strip_using_decl (member)) != member) > >> +  /* If it is a using decl, use its underlying decl.  */ >> +  type_decl = strip_using_decl (type_decl); > >> -      if (DECL_NAME (field) == name

[testsuite] modify powerpc test for hard_float target, skip powerpc/warn-[12].c for soft-float

2011-10-10 Thread Janis Johnson
Tests gcc.target/powerpc/warn-[12].c fail for soft-float multilibs with the unexpected warning "-mvsx requires hardware floating point [enabled by default]". This patch skips those tests for soft-float multilibs and modifies the powerpc check for a soft-float effective target to return true for ei

Patch committed: Correct demangling of qualified function types

2011-10-10 Thread Ian Lance Taylor
PR 48665 points out that qualified function types are not correctly demangled. These types can only appear as template parameters. This patch fixes the problem. Bootstrapped and ran C++ testsuite on x86_64-unknown-linux-gnu. Committed to mainline. Ian 2011-10-10 Ian Lance Taylor

[PATCH] [Annotalysis] Bugfix for spurious thread safety warnings with shared mutexes

2011-10-10 Thread Delesley Hutchins
This patch fixes an error where Annotalysis generates bogus warnings when a shared lock is released in a function that has a "universal lock" -- typically produced when gcc cannot parse a lock expression. Bootstrapped and passed gcc regression testsuite on x86_64-unknown-linux-gnu. Okay for googl

Re: [RFC] Slightly fix up vgather* patterns

2011-10-10 Thread Richard Henderson
On 10/08/2011 08:43 AM, Jakub Jelinek wrote: > (define_expand "avx2_gathersi" > - [(set (match_operand:VEC_GATHER_MODE 0 "register_operand" "") > - (unspec:VEC_GATHER_MODE > - [(match_operand:VEC_GATHER_MODE 1 "register_operand" "") > -(match_operand: 2 "memory_operand" "") > -

Re: [PATCH, testsuite, i386] FMA3 testcases + typo fix in MD

2011-10-10 Thread Uros Bizjak
On Thu, Oct 6, 2011 at 3:48 PM, Kirill Yukhin wrote: >> >> BTW, don't you also need "-mfmpath=sse" in dg-options? >> > > According to doc/invoke.texi > ... > @itemx -mfma > ... > These options will enable GCC to use these extended instructions in > generated code, even without @option{-mfpmath=sse

Re: [PATCH, testsuite, i386] FMA3 testcases + typo fix in MD

2011-10-10 Thread Uros Bizjak
On Mon, Oct 10, 2011 at 10:56 PM, Uros Bizjak wrote: >>> BTW, don't you also need "-mfmpath=sse" in dg-options? >>> >> >> According to doc/invoke.texi >> ... >> @itemx -mfma >> ... >> These options will enable GCC to use these extended instructions in >> generated code, even without @option{-mfpm

Re: fix for c++/44473, mangling of decimal types, checked in

2011-10-10 Thread Peter Bergner
On Wed, 2011-10-05 at 11:40 -0500, Peter Bergner wrote: > On Fri, 2011-09-30 at 10:37 -0700, Janis Johnson wrote: > > Patch http://gcc.gnu.org/ml/gcc-patches/2010-12/msg00625.html was > > approved by Jason last December but I never got around to checking > > it in. Paolo Carlini said in PR44473 th

Re: fix for c++/44473, mangling of decimal types, checked in

2011-10-10 Thread Janis Johnson
On 10/10/2011 02:13 PM, Peter Bergner wrote: > On Wed, 2011-10-05 at 11:40 -0500, Peter Bergner wrote: >> On Fri, 2011-09-30 at 10:37 -0700, Janis Johnson wrote: >>> Patch http://gcc.gnu.org/ml/gcc-patches/2010-12/msg00625.html was >>> approved by Jason last December but I never got around to check

Re: fix for c++/44473, mangling of decimal types, checked in

2011-10-10 Thread Peter Bergner
On Mon, 2011-10-10 at 14:18 -0700, Janis Johnson wrote: > Sorry, I didn't see that note about backports, but then I'm way behind > on gcc mail. Heh, no problem. I can sympathize! > Peter, would you be able to test this patch for a backport? I'm kind of > swamped right now, and decimal float i

[gimplefe][patch] A bugfix for a missed symbol

2011-10-10 Thread Sandeep Soni
Ketaki had pointed a minor bug a long time back. This patch fixes it. Index: parser.c === --- parser.c(revision 174754) +++ parser.c(working copy) @@ -882,6 +882,7 @@ switch (code) { case INTEGER_TYPE: + gl_c

[gimplefe][patch] The symbol table for declarations

2011-10-10 Thread Sandeep Soni
Hi, The following patch is a basic attempt to build a symbol table that stores the names of all the declarations made in the input file. Index: gcc/gimple/parser.c === --- gcc/gimple/parser.c (revision 174754) +++ gcc/gimple/parser.c

[PATCH, testsuite]: Close and unlink test file before exit

2011-10-10 Thread Uros Bizjak
Hello! Remove another orphan file from the testsuite directory. 2011-10-11 Uros Bizjak * lib/target-supports.exp (check_effective_target_fd_truncate): Close and unlink test file before exit. Tested on x86_64-pc-linux-gnu, committed to mainline SVN. Uros. Index: lib/target-su

Re: [google] record compiler options to .note sections

2011-10-10 Thread Xinliang David Li
Ok for google branches. 1) document the difference of this option with -grecord-gcc-switches (this one only record codegen related options, and recorded in debug section), and with -frecord-gcc-switches? 2) may be better to use option name: -frecord-gcc-switches-in-object thanks, David On Sun,

Re: [C++ Patch / RFC] PR 33067

2011-10-10 Thread Joseph S. Myers
On Mon, 10 Oct 2011, Paolo Carlini wrote: > + // The fraction 643/2136 approximates log10(2) to 7 significant digits. Whatever the conclusion on the approach to take, note that we don't use C++ comments in GCC at present. -- Joseph S. Myers jos...@codesourcery.com

Re: PATCH RFA: New configure option --with-native-system-header-dir

2011-10-10 Thread Joseph S. Myers
On Sat, 8 Oct 2011, Ian Lance Taylor wrote: > This patch implements this proposal. Only lightly tested so far. How > does this look if testing succeeds? OK in the absence of build system maintainer objections within 24 hours. -- Joseph S. Myers jos...@codesourcery.com

Re: [patch] Don't try to reload match_operator

2011-10-10 Thread Hans-Peter Nilsson
On Mon, 19 Sep 2011, Ulrich Weigand wrote: > Richard Earnshaw wrote: > > On 19/09/11 15:14, Ulrich Weigand wrote: > > > So it seems to me that for match_operator operands, the > > > goal_alternative_win flag should always be true ... > > > > > > Can you find out why this isn't true in your case? >

Re: Intrinsics for N2965: Type traits and base classes

2011-10-10 Thread Benjamin Kosnik
This is looking pretty good, from the libstdc++ side. This latest round of gcc hacking fixes the previous testsuite fixes, so once you get the gcc bits OK'd by Jason you can check in. > I'd still like to see some testcases for the intrinsic, independent > of the library. Seems like some simple t

Re: [C++ Patch] PR 50660

2011-10-10 Thread Jason Merrill
On 10/10/2011 12:44 AM, Paolo Carlini wrote: If I just do this (I hope it's what you had in mind): - tree t = non_reference (totype); + tree t = totype; /*non_reference (totype); */ variadic111.C:16:22: warning: converting ‘false’ to pointer type for argument 3 of ‘void S::f(Args1 ..., Args2&&

Re: [C++ Patch] PR 50660

2011-10-10 Thread Paolo Carlini
On 10/11/2011 01:46 AM, Jason Merrill wrote: On 10/10/2011 12:44 AM, Paolo Carlini wrote: If I just do this (I hope it's what you had in mind): - tree t = non_reference (totype); + tree t = totype; /*non_reference (totype); */ variadic111.C:16:22: warning: converting ‘false’ to pointer type fo

Re: [google] record compiler options to .note sections

2011-10-10 Thread Cary Coutant
> Ok for google branches. > > 1) document the difference of this option with -grecord-gcc-switches > (this one only record codegen related options, and recorded in debug > section), and with -frecord-gcc-switches? > 2) may be better to use option name: -frecord-gcc-switches-in-object Sections whos

[patches] several gcc.target/powerpc tests require hard_float

2011-10-10 Thread Janis Johnson
This patch skips several Power-specific tests if hard_float support isn't available. OK for trunk? 2011-10-10 Janis Johnson * gcc.target/powerpc/ppc-fma-3.c: Require hard_float. * gcc.target/powerpc/ppc-fma-4.c: Likewise. * gcc.target/powerpc/ppc-fma-5.c: Likewise.

Re: [C++ Patch] PR 50660

2011-10-10 Thread Jason Merrill
On 10/11/2011 01:10 AM, Paolo Carlini wrote: Great, this works (and I even learned something ;) Shall I commit it when testing finishes? Please. Jason

Re: [PATCH, testsuite]: Close and unlink test file before exit

2011-10-10 Thread Hans-Peter Nilsson
On Tue, 11 Oct 2011, Uros Bizjak wrote: > Hello! > > Remove another orphan file from the testsuite directory. > > 2011-10-11 Uros Bizjak > > * lib/target-supports.exp (check_effective_target_fd_truncate): > Close and unlink test file before exit. (can't quote attachment...) Or s

Re: [C++ Patch / RFC] PR 33067

2011-10-10 Thread Jason Merrill
On 10/10/2011 12:40 PM, Paolo Carlini wrote: + // The fraction 643/2136 approximates log10(2) to 7 significant digits. + int max_digits10 = 2 + (is_decimal ? fmt->p : fmt->p * 643L / 2136); Please cite N1822 in the comment and convert it to C syntax. OK with that change. Jason

Re: [patches] several gcc.target/powerpc tests require hard_float

2011-10-10 Thread Joseph S. Myers
On Mon, 10 Oct 2011, Janis Johnson wrote: > This patch skips several Power-specific tests if hard_float support > isn't available. OK for trunk? It looks like these are testing for particular instructions using FPRs and so powerpc_fprs is more appropriate; you don't want to match e500v2 hard f

Re: [google] record compiler options to .note sections

2011-10-10 Thread Dehao Chen
How about .gnu.switches.text.quote_paths? Thanks, Dehao On Tue, Oct 11, 2011 at 8:42 AM, Cary Coutant wrote: >> Ok for google branches. >> >> 1) document the difference of this option with -grecord-gcc-switches >> (this one only record codegen related options, and recorded in debug >> section),

Re: Fix for PR libobjc/49883 ("clang + gcc 4.6 runtime = broken") and a small related clang fix

2011-10-10 Thread Mike Stump
On Oct 9, 2011, at 3:30 AM, Nicola Pero wrote: > Unfortunately, the report was correct in that clang is producing incorrect > code and > abusing the higher bits of the class->info field to store some other > information. The clang folks are pretty responsive. I'd always give them a chance to `f

Re: [wwwdocs] gcc-4.6/porting_to.html

2011-10-10 Thread Gerald Pfeifer
Hi Benjamin, On Wed, 16 Mar 2011, Benjamin Kosnik wrote: > Needs some more work, here's a rough draft. I realized this one hasn't made it in, but is really nice. I made a number of minor edits (typos, markup, simplifying headings,... among others). What do you think -- should we include this?

Re: PATCH RFA: New configure option --with-native-system-header-dir

2011-10-10 Thread Paolo Bonzini
On 10/09/2011 08:18 AM, Ian Lance Taylor wrote: +#undef NATIVE_HEADER_HEADER_COMPONENT +#define NATIVE_SYSTEM_HEADER_COMPONENT "MINGW" Typo (I think), otherwise okay. Paolo

<    1   2