Go patch committed: Detect invalid import statements

2012-09-16 Thread Ian Lance Taylor
This patch to the Go compiler checks for invalid import filenames, including checks for cases that are technically valid but likely to be wrong. This corresponds to the tests done by the gc Go compiler. Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline. Will com

Fix small paste-o in rice_distribution.

2012-09-16 Thread Ed Smith-Rowland
I initialized an adapted urng that was not needed or used in the rice_distribution extension. 2012-09-16 Edward Smith-Rowland <3dw...@verizon.net> * include/ext/random.tcc (__gnu_cxx::rice_distribution<> ::__generate_impl): Remove bogus _Adaptor usage. Index: include/ext/rand

Re: Add Rice distribution to libstdc++

2012-09-16 Thread Ed Smith-Rowland
On 09/16/2012 06:29 PM, Paolo Carlini wrote: On 09/16/2012 11:18 PM, Paolo Carlini wrote: On 09/16/2012 10:02 PM, Marc Glisse wrote: On Sun, 16 Sep 2012, Ed Smith-Rowland wrote: This adds the Rice distribution to the C++11 facility and a Gnu extension. Hello, libstdc++ not in copy? The c

Re: Add Rice distribution to libstdc++

2012-09-16 Thread Paolo Carlini
On 09/16/2012 10:02 PM, Marc Glisse wrote: On Sun, 16 Sep 2012, Ed Smith-Rowland wrote: This adds the Rice distribution to the C++11 facility and a Gnu extension. Hello, libstdc++ not in copy? The change to beta_distribution is not mentioned in the ChangeLog. You don't appear to have a te

Re: Add Rice distribution to libstdc++

2012-09-16 Thread Paolo Carlini
Hi >This adds the Rice distribution to the C++11 facility and a >Gnu extension. Thanks. I realize now that the features we have been adding to include/ext/random should be tested in testsuite/ext/random. If nobody beats me one of these days I will take care of the move. Paolo

Re: GCC 4.7.2 Status Report (2012-09-14), branch frozen

2012-09-16 Thread Jack Howarth
On Fri, Sep 14, 2012 at 05:05:05PM +0200, Jakub Jelinek wrote: > The GCC 4.7 branch is now frozen for creating a first release candidate > of the GCC 4.7.2 release. > > All changes need explicit release manager approval until the final > release of GCC 4.7.2 which should happen roughly one week af

Re: [Patch, Fortran, OOP] PR 54594: Type-bound ASSIGNMENTs (elemental + array version) rejected as ambiguous

2012-09-16 Thread Janus Weil
>> here is an OOP patch for type-bound generics. The problem was that >> specifics with polymorphic arguments of different ranks were wrongly >> rejected as ambiguous, because 'gfc_compare_types' did not properly >> handle CLASS arrays. The patch fixes this is regtests cleanly on >> x86_64-unknown-

Re: [Patch, Fortran] PR 54387: [F03] Wrongly accepts non-proc result variable on the RHS of a proc-pointer assignment

2012-09-16 Thread Janus Weil
Hi, >> here is a small patch which fixes an accepts-invalid problem with >> proc-pointer assignments. Regtested on x86_64-unknown-linux-gnu. Ok >> for trunk? > > > OK. Thanks for the patch. > > (Though, I am wondering whether the error message could be improved. We know > that the RHS is not the p

Re: Add Rice distribution to libstdc++

2012-09-16 Thread Marc Glisse
On Sun, 16 Sep 2012, Ed Smith-Rowland wrote: This adds the Rice distribution to the C++11 facility and a Gnu extension. Hello, libstdc++ not in copy? The change to beta_distribution is not mentioned in the ChangeLog. You don't appear to have a test like those added for PR 54376, and like

Re: [committed] Fix gnat.dg/lto15.adb on hppa

2012-09-16 Thread John David Anglin
On 16-Sep-12, at 1:27 PM, Eric Botcazou wrote: Thanks for fixing this. The testcase is also on the 4.7 branch. I'm aware of that. Will fix branch after the 4.7.2 release. -- John David Anglin dave.ang...@bell.net

Add Rice distribution to libstdc++

2012-09-16 Thread Ed Smith-Rowland
This adds the Rice distribution to the C++11 facility and a Gnu extension. 2012-09-14 Edward Smith-Rowland <3dw...@verizon.net> * include/ext/random: Add __gnu_cxx::rice_distribution<> class. * include/ext/random.tcc: Add out-of-line functions for __gnu_cxx::rice_dist

Re: [Patch, fortran] PR46897 - [OOP] type-bound defined ASSIGNMENT(=) not used for derived type component in intrinsic assign

2012-09-16 Thread Tobias Burnus
Am 10.09.2012 20:58, schrieb Paul Richard Thomas: Bootstrapped and regtested on FC9/x86_64 - OK for trunk? The following test case doesn't work; it should print "Overloaded" - and does so with crayftn. But with your patch, it doesn't. Tobias module a_mod type :: a contains procedu

Re: [committed] Fix gnat.dg/lto15.adb on hppa

2012-09-16 Thread Eric Botcazou
> Tested on hppa2.0w-hp-hpux11.11 and committed to trunk. Thanks for fixing this. The testcase is also on the 4.7 branch. -- Eric Botcazou

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

2012-09-16 Thread John David Anglin
This adjusts the regexp to work on hppa. Tested on hppa2.0w-hp-hpux11.11 and hppa64-hp-hpux11.11. Committed to trunk. Dave -- J. David Anglin dave.ang...@nrc-cnrc.gc.ca National Research Council of Canada (613) 990-0752 (FAX: 952-6602) 2012-09-16

[committed] Fix gnat.dg/lto15.adb on hppa

2012-09-16 Thread John David Anglin
Tested on hppa2.0w-hp-hpux11.11 and committed to trunk. Dave -- J. David Anglin dave.ang...@nrc-cnrc.gc.ca National Research Council of Canada (613) 990-0752 (FAX: 952-6602) 2012-09-16 John David Anglin PR testsuite/54007 * gnat.d

Re: [Patch, Fortran, OOP] PR 54594: Type-bound ASSIGNMENTs (elemental + array version) rejected as ambiguous

2012-09-16 Thread Tobias Burnus
Janus Weil wrote: here is an OOP patch for type-bound generics. The problem was that specifics with polymorphic arguments of different ranks were wrongly rejected as ambiguous, because 'gfc_compare_types' did not properly handle CLASS arrays. The patch fixes this is regtests cleanly on x86_64-unk

[committed] Adjust scan assembler check in gfortran.dg/bind_c_array_params_2.f90 for hppa*-*-hpux*

2012-09-16 Thread John David Anglin
This fixes the assembler scan on hppa*-*-hpux*. Tested on hppa2.0w-hp-hpux11 and hppa64-hp-hpux11.11. Committed to trunk. Dave -- J. David Anglin dave.ang...@nrc-cnrc.gc.ca National Research Council of Canada (613) 990-0752 (FAX: 952-6602) 2012-09-

[Patch, Fortran, OOP] PR 54594: Type-bound ASSIGNMENTs (elemental + array version) rejected as ambiguous

2012-09-16 Thread Janus Weil
Hi all, here is an OOP patch for type-bound generics. The problem was that specifics with polymorphic arguments of different ranks were wrongly rejected as ambiguous, because 'gfc_compare_types' did not properly handle CLASS arrays. The patch fixes this is regtests cleanly on x86_64-unknown-linux-

[PATCH] Fix missing EH updates in execute_cse_sincos (PR tree-optimization/54563)

2012-09-16 Thread Jakub Jelinek
Hi! As discussed in the PR, this patch fixes the ICE by calling gimple_purge_dead_eh_edges if the last stmt has been replaced by the pass. Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? Still, I think it would be nice if e.g. powf or cosl were throw() in C++, at least if the

Re: Finish up PR rtl-optimization/44194

2012-09-16 Thread Eric Botcazou
> I hit one problem building libgfortran for mips64-linux-gnu. > The calls.c change was: > > Index: calls.c > === > --- calls.c (revision 191198) > +++ calls.c (working copy) > @@ -3272,16 +3272,8 @@ expand_call (tree exp, rtx tar

FW: Wish we had a chance to meet...

2012-09-16 Thread Jacob Adopley
Please track this idiot down and stop it! This idiot is giving a bad rap to your organization! Thanks. jacob k. adopley From: java-ow...@gcc.gnu.org [java-ow...@gcc.gnu.org] on behalf of Reynalda Bonilla [gio6778sei...@yahoo.com] Sent: Saturday, Septembe

[Patch, Fortran] (3/n) Fix bit/logical bugs found by Coverity

2012-09-16 Thread Tobias Burnus
Committed as obvious (Rev. 191358) after building and successfully regtesting on x86-64-linux. I collected some futher bugs in http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54599. There several additional ones, but many are bogus. I think the most common "defect" is that one dereferences a poin

Re: [Patch, Fortran] PR 54387: [F03] Wrongly accepts non-proc result variable on the RHS of a proc-pointer assignment

2012-09-16 Thread Tobias Burnus
Am 15.09.2012 17:22, schrieb Janus Weil: here is a small patch which fixes an accepts-invalid problem with proc-pointer assignments. Regtested on x86_64-unknown-linux-gnu. Ok for trunk? OK. Thanks for the patch. (Though, I am wondering whether the error message could be improved. We know that

Re: Finish up PR rtl-optimization/44194

2012-09-16 Thread Richard Sandiford
Eric Botcazou writes: > This is the PR about the useless spilling to memory of structures that are > returned in registers. It was essentially addressed last year by Easwaran > with > an enhancement of the RTL DSE pass, but Easwaran also noted that we still > spill > to memory in the simples

Re: [Patch, Fortran] Fix some issues found by Coverity's static-code analysis scan

2012-09-16 Thread Tobias Burnus
Mikael Morin wrote: On 15/09/2012 19:36, Tobias Burnus wrote: As a pre-script: Will you look at Paul's revised assignment patch - or should I do it? I looked at it twice or so, and couldn't get the double temporary thing. I'll get back to it. Thanks! Thanks for the analysis. Will you create

Re: [Patch, Fortran] Fix some issues found by Coverity's static-code analysis scan

2012-09-16 Thread Mikael Morin
On 15/09/2012 19:36, Tobias Burnus wrote: > Hi Mikael, > > thanks for your comments. > > As a pre-script: Will you look at Paul's revised assignment patch - or > should I do it? I looked at it twice or so, and couldn't get the double temporary thing. I'll get back to it. > If you want to know th

Ping^2: [PATCH 3/6] Thread pointer built-in functions, arm

2012-09-16 Thread Richard Sandiford
Second ping for the ARM part of Chung-Lin's __builtin_thread_pointer patch: http://gcc.gnu.org/ml/gcc-patches/2012-08/msg01914.html I think this is the only part that hasn't been approved. Thanks, Richard

Re: [PATCH] Changes in mode switching

2012-09-16 Thread Richard Sandiford
Vladimir Yakovlev writes: > I reproduced the failure and found reason of it. I understood haw it > resolve and now I need small changes only - additional argument of > EMIT_MODE_SET. Is it good fo trunk? I'm not sure I understand why you need to know the instruction. The x86 code was: + if

Re: [patch, mips] Patch for new mips triplet - mips-mti-elf

2012-09-16 Thread Richard Sandiford
"Steve Ellcey " writes: > +#undef SYSROOT_SUFFIX_SPEC > +#define SYSROOT_SUFFIX_SPEC \ > + > "%{mips32:/mips32}%{mips64:/mips64}%{mips64r2:/mips64r2}%{mips16:/mips16}%{msoft-float:/sof}%{mel|EL:/el}%{mabi=64:/64}%{mabi=n32:/n32}" This shouldn't be needed for *-elf targets. > +/* This is iden