Re: [Patch, Fortran] FINAL (prep patches 4/5): Support noncontiguous arrays in the finalization wrapper function

2013-01-05 Thread Paul Richard Thomas
Dear Tobias, I think that the patch would be much less opaque if repeated operation to produce code were turned into functions, as I did for the defined assignment patch; eg resolve.c(build_assignment, add_code_to_chain)? What you have done is OK but it is HEAVY, as is much of the content of class

Re: [wwwdocs] SH 4.8 changes - document thread pointer built-ins

2013-01-05 Thread Oleg Endo
Hi, On Wed, 2013-01-02 at 19:13 -1000, Gerald Pfeifer wrote: > Hi Oleg, > > On Wed, 17 Oct 2012, Oleg Endo wrote: > >> +Added support for the built-in functions > >> +__builtin_thread_pointer and > >> +__builtin_set_thread_pointer. This assumes that > >> +GBR is used to hold the

Re: RFA: Fix ICE on PARALLEL returns when expand builtins

2013-01-05 Thread Richard Biener
On Fri, Jan 4, 2013 at 7:57 PM, Eric Botcazou wrote: >> Hmm, how would anyone else get at the "padding" info dealing with the >> parallel? This looks broken if we need access to the type :/ > > Yes, you need to access the type for return values, but that's not really new. Ick :/ >> Eric, any co

Re: [Patch, libfortran] Improve performance of byte swapped IO

2013-01-05 Thread Richard Biener
On Fri, Jan 4, 2013 at 11:35 PM, Andreas Schwab wrote: > Janne Blomqvist writes: > >> diff --git a/libgfortran/io/file_pos.c b/libgfortran/io/file_pos.c >> index c8ecc3a..bf2250a 100644 >> --- a/libgfortran/io/file_pos.c >> +++ b/libgfortran/io/file_pos.c >> @@ -140,15 +140,21 @@ unformatted_back

Re: Control dependence vs. builtin_unreachable

2013-01-05 Thread Steven Bosscher
On Thu, Jan 3, 2013 at 9:51 PM, Jeff Law wrote: > On 01/03/2013 12:01 PM, Steven Bosscher wrote: >> >> Hello, >> >> Consider the following test case: >> >> void bar (void); >> int foo (int b, int c, int d) >> { >>int r = 0; >>if (b) >> res = b * 2 + 4; >>if (c) >> { >>

[Patch, fortran] PR fortran/42769 typebound call resolved incorrectly.

2013-01-05 Thread Mikael Morin
Hello, here is a fix for PR42769, where a type bound call was resolved statically into a normal procedure call, but using the local procedure (with the same name) instead of the original (and correct) one. It turned out to not be OOP specific, and be a pure module loading problem in the way we

Re: [Patch, libfortran] Improve performance of byte swapped IO

2013-01-05 Thread Janne Blomqvist
On Sat, Jan 5, 2013 at 5:35 PM, Richard Biener wrote: > On Fri, Jan 4, 2013 at 11:35 PM, Andreas Schwab wrote: >> Janne Blomqvist writes: >> >>> diff --git a/libgfortran/io/file_pos.c b/libgfortran/io/file_pos.c >>> index c8ecc3a..bf2250a 100644 >>> --- a/libgfortran/io/file_pos.c >>> +++ b/libg

Re: Control dependence vs. builtin_unreachable

2013-01-05 Thread Steven Bosscher
On Sat, Jan 5, 2013 at 9:10 PM, Steven Bosscher wrote: > Bootstrapped&tested on powerpc64-unknown-linux-gnu. And to be clear, bootstrapped with this patch on top: Index: system.h === --- system.h(revision 194924) +++ system.h(

Re: [PING^4] PR 54805: __gthread_tsd* in vxlib-tls.c

2013-01-05 Thread rbmj
On 06-Dec-12 10:14, rbmj wrote: On 26-Nov-12 13:27, rbmj wrote: On 11/13/2012 10:22 PM, rbmj wrote: On 11/5/2012 12:57 PM, rbmj wrote: This removes warnings about implicit declarations and fixes one of the function calls in vxlib-tls.c for vxworks targets. I got the old prototypes from http:/

Re: [cxx-conversion] Change uses of htab_t in gcc/config to hash_table.

2013-01-05 Thread Xinliang David Li
I noticed that the traverse and traverse_noresize method takes Argument as the first template parameter. It should be moved to be the second after the callback. In most of the cases, the type of the Argument can be deduced from the callsite, so that the user only need to specify the callback: ht->