Re: [PATCH] Add extra location information - PR43486

2012-09-18 Thread Joseph S. Myers
On Tue, 18 Sep 2012, Ian Lance Taylor wrote: > On Tue, Sep 18, 2012 at 3:58 AM, Arnaud Charlet wrote: > > > > Since this issue is more general, I have split my changes and introduced a > > new > > tentative switch called -fextra-slocs, which is the subject of this email. > > Sorry for picking o

Re: [patch] Fix PR rtl-optimization/54290

2012-09-18 Thread Bernd Schmidt
On 09/17/2012 07:32 PM, Ulrich Weigand wrote: > In any case, the change in the condition you noticed was introduced by a > recent > patch by Bernd: http://gcc.gnu.org/ml/gcc-patches/2012-08/msg00171.html > > It seems that we ought to use a similar test to what Bernd introduced in > gen_reload, th

Re: Use conditional casting with symtab_node

2012-09-18 Thread Lawrence Crowl
On 9/18/12, Richard Guenther wrote: > On Sep 18, 2012 Lawrence Crowl wrote: > > * cgraph.h (varpool_node): Rename to varpool_node_for_tree. > > Sure it should be varpool_node_for_decl, if any. > Or varpool_node_from_decl (grep for what is more common) Grep says _for_decl wins over _from_decl by

Re: [patch, fortran] Fix an issue found by Coverity scan

2012-09-18 Thread Thomas Koenig
Am 18.09.2012 20:46, schrieb Thomas Koenig: Hello world, pretty self-explanatory. No test case because there is no change in behavior. OK for trunk? I forgot to add: Regression-tested. Thomas

Re: [PATCH] rs6000: Remove integer abs/nabs/min/max patterns

2012-09-18 Thread David Edelsohn
On Tue, Sep 18, 2012 at 1:01 PM, Segher Boessenkool wrote: > Without these patterns, the exact same code is generated. This > is a leftover from when we still had a "doz" instruction. > > Tested on powerpc64-linux --enable-languages=c,c++,fortran; > no regressions. Also tested all these patterns

Re: [testsuite] vect effective targets should use arm_neon_ok

2012-09-18 Thread Janis Johnson
On 09/18/2012 12:54 PM, Janis Johnson wrote: > In most cases a test that requires ARM NEON should use effective target > arm_neon, which means that flags run for all tests include NEON support. > The result is cached the first time it is checked for a multilib. > > Vectorization tests, when run fo

Re: [patch] Fix PR rtl-optimization/54290

2012-09-18 Thread Eric Botcazou
> In any case, the change in the condition you noticed was introduced by a > recent patch by Bernd: > http://gcc.gnu.org/ml/gcc-patches/2012-08/msg00171.html > > It seems that we ought to use a similar test to what Bernd introduced in > gen_reload, that is, use the "replaced_subreg" routine on rld

Fix PR middle-end/54617

2012-09-18 Thread Eric Botcazou
In the recent patch for PR rtl-opt/44194 I left a case potentially ICEing because I managed to have it covered neither on x86-64 nor on SPARC 64-bit. This PR brings the missing coverage and it's from the gcc.c-torture/compile testsuite on PowerPC/Darwin 64-bit. Bootstrapped/regtested on x86-64/L

Re: [PATCH] Avoid bitfield stores to clobber adjacent variables (PR middle-end/48124)

2012-09-18 Thread saimf
Which gcc version does this patch apply to? Thanks! Bugzilla from ja...@redhat.com wrote: > > Hi! > > This patch changes get_best_mode, so that it doesn't suggest using larger > modes if it means it could clobber variables located after the containing > one. > > On the attached testcase on x

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

2012-09-18 Thread Paolo Carlini
Hi again, On 09/18/2012 08:33 PM, Paolo Carlini wrote: But I'm not surprised, frankly, I think the conflict is expected, *IF* (please check) TR1 says that those three overloads, for float, double an long double, must be declared in std::tr1 (likewise for all the other math functions) Now, give

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

2012-09-18 Thread Paolo Carlini
On 09/19/2012 01:12 AM, Paolo Carlini wrote: Hi again, On 09/18/2012 08:33 PM, Paolo Carlini wrote: But I'm not surprised, frankly, I think the conflict is expected, *IF* (please check) TR1 says that those three overloads, for float, double an long double, must be declared in std::tr1 (likewis

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

2012-09-18 Thread Paolo Carlini
On 09/19/2012 01:24 AM, Paolo Carlini wrote: On 09/19/2012 01:12 AM, Paolo Carlini wrote: Hi again, On 09/18/2012 08:33 PM, Paolo Carlini wrote: But I'm not surprised, frankly, I think the conflict is expected, *IF* (please check) TR1 says that those three overloads, for float, double an long

Re: [PATCH] Rs6000 infrastructure cleanup (switches), revised patch

2012-09-18 Thread David Edelsohn
On Mon, Sep 17, 2012 at 3:51 PM, Michael Meissner wrote: > This patch has support for all of the additonal cleanups I mentioned in the > first patch that I hadn't gotten to. At this point, I am not planning any > more > enhancements to the patch, and I would like to check it in. > > On my 64-bit

Re: [committed] Fix g++.dg/debug/dwarf2/nested-3.C to handle hppa assembler comment

2012-09-18 Thread Hans-Peter Nilsson
> From: John David Anglin > Date: Sun, 16 Sep 2012 19:13:19 +0200 > This adjusts the regexp to work on hppa. Tested on hppa2.0w-hp-hpux11.11 > and hppa64-hp-hpux11.11. Committed to trunk. > 2012-09-16 John David Anglin > > PR debug/54460 > * g++.dg/debug/dwarf2/nested-3.C: Add

RFA: Process '*' in '@'-output-template alternatives

2012-09-18 Thread Joern Rennecke
I am about to submit the ARCompact target port; this port needs a few patches to target-independent code. There is a move pattern with 20 alternatives; a few of them need a simple function call to decide which output pattern to use. With the '@'-syntax for multi-alternative templates, each alter

[testsuite] gcc.dg/vect/pr48765: fix option conflict

2012-09-18 Thread Janis Johnson
Test gcc.dg/vect/pr48765, run only for powerpc*-*-*, unconditionally adds -mcpu=power6 to the command line. It also adds -m64, although tests should rely on different multilib flags for that and skip the test if not lp64 if the test can only be run with -m64. For all of the Power multilib flags t

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

2012-09-18 Thread Jason Merrill
The compiler changes are OK. Jason

RE: Ping^2: [PATCH]Remove duplicate check on BRANCH_COST in fold-const.c

2012-09-18 Thread Bin Cheng
> -Original Message- > From: Richard Guenther [mailto:richard.guent...@gmail.com] > Sent: Tuesday, September 18, 2012 5:50 PM > To: Bin Cheng > Cc: Bin.Cheng; gcc-patches@gcc.gnu.org; Richard Earnshaw > Subject: Re: Ping^2: [PATCH]Remove duplicate check on BRANCH_COST in fold- > const.c >

Re: Backtrace library [1/3]

2012-09-18 Thread Gerald Pfeifer
On Mon, 17 Sep 2012, Ian Lance Taylor wrote: > While this should be fixed now, note that it should only arise if > i386-unknown-freebsd10.0 does not use ELF. If this is an ELF based > system, then we need to discover why it was not recognized as such. > What is the output of > > awk -f SRCDIR/lib

Re: Backtrace library [1/3]

2012-09-18 Thread Gerald Pfeifer
It turns out "make bootstrap" work now, but "make install" is still broken on i386-unknown-freebsd10.0: libtool: compile: gcc -DHAVE_CONFIG_H -I. -I/scratch2/tmp/gerald/gcc-HEAD/libbacktrace -W -Wall -Wwrite-strings -Wstrict-prototypes -Wmissing-prototypes -Wold-style-definition -Wmissing-form

libbacktrace patch committed: Add more -I options

2012-09-18 Thread Ian Lance Taylor
This patch to the libbacktrace Makefile.am adds some more -I options, so that it can find unwind.h under more conditions. This splits the -I options in AM_CPPFLAGS, which is where they belong anyhow. Bootstrapped and ran libbacktrace tests on x86_64-unknown-linux-gnu. Committed to mainline. Ian

Go patch committed: Fix multiple types with same name in function

2012-09-18 Thread Ian Lance Taylor
The Go frontend had a bug when there were multiple types with the same name in a single function. That could happen if the types appeared in different blocks. The compiler would use the same name for the type descriptor for each type, leading to a multiple definition error at assembly time. This

<    1   2