Re: [PATCH] ARMv6-M MI thunk fix

2013-06-20 Thread Cesar Philippidis
Ping. Cesar On 6/7/13 9:50 AM, Cesar Philippidis wrote: > On 6/6/13 9:00 AM, Richard Earnshaw wrote: >> The pipeline offset is 4 for Thumb2 as well. So at the very least you >> need to explain why your change doesn't apply then as well. > > Yes some context is los

[PATCH] if-to-switch pass

2013-06-21 Thread Cesar Philippidis
fold_binary () inside refine_range_plus (). Also, TODO_ggc_collect has been removed in the gimple_opt_pass struct. I bootstrapped and regression tested on x86_64-unknown-linux-gnu and arm-none-linux-gnueabi. OK for trunk? Cesar 2013-06-21 Tom de Vries Cesar Philippidis

[patch] fix ppc spe bootstrap error in dwf_regno

2014-04-30 Thread Cesar Philippidis
er for this application. Is this patch ok for trunk and 4.9? If so, please commit since I don't have an svn account. Cesar 2014-04-30 Cesar Philippidis gcc/ * dwarf2cfi.c (dwf_regno): Don't assert reg is a pseudo register. diff --git a/gcc/dwarf2cfi.c b/

Re: [patch ping] libstdc++ testsuite cxxflags

2014-05-20 Thread Cesar Philippidis
On 05/20/2014 02:11 AM, Jonathan Wakely wrote: > On 19/05/14 14:57 -0600, Sandra Loosemore wrote: >> On 05/17/2014 04:07 AM, Jonathan Wakely wrote: >>> On 17 May 2014 10:50, Jonathan Wakely wrote: On 17 May 2014 01:16, Sandra Loosemore wrote: > It appears that this patch from last fall nev

Re: [GOMP4, OpenACC] Fixed-form Fortran code failing to parse

2014-07-07 Thread Cesar Philippidis
couple of typos in scanner which caused gfc_next_char and friends to fail when handling fixed-form comments. There was also something being handled strangely in the parser, when I went ahead and fixed. I think there may be a few more bugs lurking in the fixed-form comment handling. Thomas, is t

Re: [GOMP4, OpenACC] Fixed-form Fortran code failing to parse

2014-07-08 Thread Cesar Philippidis
On 07/08/2014 02:21 AM, Tobias Burnus wrote: > Cesar Philippidis wrote: >> Thomas, is this OK for gomp-4_0-branch? > ... > > >> * gcc/fortran/scanner.c (gfc_next_char_literal): Fix the scan for >> *$acc. > > This changes looks good to me. >

Re: [patch,gomp-4_0-branch] openacc parallel reduction part 1

2014-07-08 Thread Cesar Philippidis
On 07/07/2014 02:55 AM, Thomas Schwinge wrote: > On Sun, 6 Jul 2014 16:10:56 -0700, Cesar Philippidis > wrote: >> This patch is the first step to enabling parallel reductions in openacc. > > Thanks! > >> As mentioned earlier, this patch isn't complete yet. For

Re: [patch,gomp-4_0-branch] openacc parallel reduction part 1

2014-07-08 Thread Cesar Philippidis
On 07/08/2014 07:28 AM, Cesar Philippidis wrote: > Thanks for catching those problems! I've committed this updated version > of the patch. I forgot to remove the support for the collapse clause in from the loop construct in the c frontend. I did so upstream, but not internally. I&#x

[patch,gomp-4_0-branch] misc reduction clause bug fixes

2014-07-10 Thread Cesar Philippidis
Hi Thomas, This patch addresses two bugs openacc reduction clause bugs. The first bug occurred because I didn't anticipate a GIMPLE_BIND stmt to be passed to process_reduction_data. Turns out, this could happen with the collapse clause. That's because the variables which were declared inside the i

[patch,gomp-4_0-branch] openacc collapse clause

2014-07-10 Thread Cesar Philippidis
Hi Thomas, These patch enables the collapse clause with a value greater than one. We had partial support for the collapse clause which supported a value of 1, because the fortran frontend always associated a collapse clause with each openacc for loop. However in terms of actual collapse support, a

Re: [patch,gomp-4_0-branch] misc reduction clause bug fixes

2014-07-14 Thread Cesar Philippidis
On 07/11/2014 02:15 AM, Thomas Schwinge wrote: > Hi Cesar! > > On Thu, 10 Jul 2014 11:43:11 -0700, Cesar Philippidis > wrote: >> This patch addresses two bugs openacc reduction clause bugs. > > Thanks! OK; one question/suggestion, though: > >> --- a/gcc

Re: [patch,gomp-4_0-branch] openacc collapse clause

2014-07-14 Thread Cesar Philippidis
On 07/11/2014 02:27 AM, Thomas Schwinge wrote: > Hi Cesar! > > On Thu, 10 Jul 2014 11:47:42 -0700, Cesar Philippidis > wrote: >> These patch enables the collapse clause with a value greater than one. > > Thanks! > >> Is this patch OK for gomp-4_0-branch? >

[patch,gomp-4_0-branch] acc nested function support

2014-07-18 Thread Cesar Philippidis
tions. Does this look OK to commit to gomp-4_0-branch? Thanks, Cesar 2014-07-17 Cesar Philippidis gcc/ * gcc/gimple.h (gimple_statement_oacc_kernels, gimple_statment_oacc_parallel): Derive from gimple_statement_omp_taskreg instestead of gimple_statement_omp_parallel_layout. (is_a_h

Re: FWD: Re: OpenACC subarray specifications in the GCC Fortran front end

2014-07-23 Thread Cesar Philippidis
xfail *-*-* } } >> + >> + !$omp target map(tt%j(1:)) ! { dg-bogus "Syntax error in OpenMP variable >> list" "" { xfail *-*-* } } >> + !$omp end target ! { dg-bogus "Unexpected !\\\$OMP END TARGET statement" >> "" { xfail *-*-

Re: FWD: Re: OpenACC subarray specifications in the GCC Fortran front end

2014-07-24 Thread Cesar Philippidis
the only conflicting one?) I moved all of the data clause matching back to gfc_match_omp_clauses, and I guarded the copyin clause with the openacc flag. It looks like the private clause may also require a special memory mapping, so I left the openacc flag in place. >>

[patch,gomp-4_0-branch] reductions with reference vars

2014-07-28 Thread Cesar Philippidis
Hi Thomas, After testing the reduction clause inside fortran subroutines, I noticed that I wasn't handling reference variables properly. This patch fixes that. Is this OK for gomp-4_0-branch? Thanks, Cesar 2014-07-24 Cesar Philippidis gcc/ * omp-low.c (get_base_type): New fun

Re: FWD: Re: OpenACC subarray specifications in the GCC Fortran front end

2014-07-28 Thread Cesar Philippidis
On 07/25/2014 09:01 AM, Thomas Schwinge wrote: > On Thu, 24 Jul 2014 15:44:13 -0700, Cesar Philippidis > wrote: >> On 07/24/2014 06:11 AM, Thomas Schwinge wrote: >>> I'd suggest to continue to handle all the data clauses [...] >> >> I moved a

Re: [patch,gomp-4_0-branch] reductions with reference vars

2014-07-28 Thread Cesar Philippidis
On 07/28/2014 09:27 AM, Thomas Schwinge wrote: > On Mon, 28 Jul 2014 07:26:30 -0700, Cesar Philippidis > wrote: >> After testing the reduction clause inside fortran subroutines, I noticed >> that I wasn't handling reference variables properly. This patch fixes >>

Re: [patch,gomp-4_0-branch] openacc parallel reduction part 1

2014-07-28 Thread Cesar Philippidis
On 07/28/2014 10:02 AM, Thomas Schwinge wrote: > Hi Cesar! > > On Sun, 6 Jul 2014 16:10:56 -0700, Cesar Philippidis > wrote: >> This patch is the first step to enabling parallel reductions in openacc. > > I think I have found one issue in this code -- but please verify

Re: FWD: Re: OpenACC subarray specifications in the GCC Fortran front end

2014-08-01 Thread Cesar Philippidis
On 08/01/2014 12:48 AM, Jakub Jelinek wrote: > On Wed, Jul 23, 2014 at 05:42:32PM -0700, Cesar Philippidis wrote: >>>> Jakub, before your Fortran OpenMP 4 target changes, Ilmir had written the >>>> test case gcc/testsuite/gfortran.dg/gomp/map-1.f90 (based on

Re: [PATCH] [gomp4] Initial support of OpenACC loop directive in C front-end.

2014-08-05 Thread Cesar Philippidis
On 07/29/2014 02:07 AM, Thomas Schwinge wrote: > On Thu, 20 Mar 2014 15:42:48 +0100, I wrote: >> On Tue, 18 Mar 2014 14:50:44 +0100, I wrote: >>> On Tue, 18 Mar 2014 16:37:24 +0400, Ilmir Usmanov >>> wrote: This patch introduces support of OpenACC loop directive (and combined directiv

[patch] make -flto -save-temps less verbose

2014-03-12 Thread Cesar Philippidis
27;ve also included a simple test case which would fail without the change. Is this OK for stage-4? If so, please check it in since I don't have an SVN account. Thanks, Cesar 2014-03-12 Cesar Philippidis gcc/ * lto-wrapper.c (maybe_unlink_file): Suppress diagnostic

Re: [patch] make -flto -save-temps less verbose

2014-03-13 Thread Cesar Philippidis
On 3/13/14, 2:52 AM, Richard Biener wrote: > On Thu, Mar 13, 2014 at 10:31 AM, Richard Biener > wrote: >> On Thu, Mar 13, 2014 at 1:10 AM, Cesar Philippidis >> wrote: >>> I noticed that the lto-wrapper is a little noisy without the -v option >>> when -sav

<    2   3   4   5   6   7