[PATCH] Do array index calculations in gfc_array_index_type

2018-09-22 Thread Janne Blomqvist
It was recently noticed that for a few of the coarray intrinsics array index calculations were done in integer_type_node instead of gfc_array_index_type. This patch fixes this. Regtested on x86_64-pc-linux-gnu, Ok for trunk? gcc/fortran/ChangeLog: 2018-09-22 Janne Blomqvist *

Re: [PATCH 02/25] Propagate address spaces to builtins.

2018-09-22 Thread Andreas Schwab
On Sep 05 2018, wrote: > At present, pointers passed to builtin functions, including atomic operators, > are stripped of their address space properties. This doesn't seem to be > deliberate, it just omits to copy them. > > Not only that, but it forces pointer sizes to Pmode, which isn't

Re: [patch, fortran] Clobber some intent(out) variables on call

2018-09-22 Thread Thomas Koenig
Hi Jerry, s/tue/due/ a = 12345689  < missing 7 digit? Yep, you're right. Corrected, committed as r264506. Regards Thomas

Re: [PATCH] Cleanup strcpy/stpcpy no nul warning code

2018-09-22 Thread Martin Liška
Hi Jeff. I noticed that your commit r264328 introduced this: gcc/builtins.c: ... 579tree rhs1 = gimple_assign_rhs1 (stmt); 580tree_code code = gimple_assign_rhs_code (stmt); 581if (code == ADDR_EXPR 582&& TREE_CODE (TREE_OPERAND (rhs1, 0)) ==

Re: [patch, fortran] Clobber some intent(out) variables on call

2018-09-22 Thread Jerry DeLisle
Minor typo, see below. On 9/22/18 10:23 AM, Thomas Koenig wrote: Hello world, the attached patch lets the middle-end know that variables associated with intent(out) arguments become undefined, by issuing an assignment to a special value (a "clobber") before entering the procedure. Originally,

[patch, fortran] PR87318 gfortran.dg/dtio_1.f90 is invalid

2018-09-22 Thread Jerry DeLisle
Committed as OKed by Janus on bugzilla. Author: jvdelisle Date: Sat Sep 22 17:49:19 2018 New Revision: 264505 URL: https://gcc.gnu.org/viewcvs?rev=264505=gcc=rev Log: 2018-09-22 Jerry DeLisle PR fortran/87318 * gfortran.dg/dtio_1.f90: Update test to valid code. Modified:

[patch, fortran] Clobber some intent(out) variables on call

2018-09-22 Thread Thomas Koenig
Hello world, the attached patch lets the middle-end know that variables associated with intent(out) arguments become undefined, by issuing an assignment to a special value (a "clobber") before entering the procedure. Originally, I had also planned to do so on entry to the procedure, see

[wwwdocs] Disable empty sections in gcc-8/porting_to.html

2018-09-22 Thread Gerald Pfeifer
I noticed this when pointing someone to this document. Clearly it does not make much sense to have those empty sections which just carry a title but no content, so I commented those (both in case someone is adding material later, and also since this is going to be used as a template for the GCC 9

Re: [PATCH 08/14] Add D2 Testsuite files.

2018-09-22 Thread Iain Buclaw
On 22 September 2018 at 15:23, Florian Weimer wrote: > * Iain Buclaw: > >> This patch adds part of the D2 testsuite, which includes D source code >> files that are considered compilable; files that are considered >> uncompilable, but should not ICE; and files that should execute on >> targets

Re: [PATCH 08/14] Add D2 Testsuite files.

2018-09-22 Thread Florian Weimer
* Iain Buclaw: > This patch adds part of the D2 testsuite, which includes D source code > files that are considered compilable; files that are considered > uncompilable, but should not ICE; and files that should execute on > targets with crash or assertion failures. > >

[Patc, fortran] PR85603 - ICE with character array substring assignment

2018-09-22 Thread Paul Richard Thomas
Yet another 'obvious' deferred character fix. Committed to trunk as r264502. Will backport in about ten days time. Paul 2018-09-22 Paul Thomas PR fortran/85603 * trans-array.c (gfc_alloc_allocatable_for_assignment): Test the charlen backend_decl before using the VAR_P macro.

Re: [PATCH v4 01/10] Initial TI PRU GCC port

2018-09-22 Thread Dimitar Dimitrov
On Thursday, 9/13/2018 13:02:21 EEST Richard Sandiford wrote: > Dimitar Dimitrov writes: > > +(define_insn > > "sub_impl_ > _zext_op2>" + [(set (match_operand:EQD 0 "register_operand" "=r,r,r") > > + (minus:EQD > > +(zero_extend:EQD > > + (match_operand:EQS0 1 "reg_or_ubyte_operand"

Re: Fold more boolean expressions

2018-09-22 Thread MCC CS
Thanks a lot for the review, Richard! - I have updated the patch. - Had done a full "make check" with the second draft. - Did a "make check-gcc" with the final patch. - Both tests have the same tests failing, that always fail: g++.dg/pr80481.C g++.dg/pr83239.C Would be great if anyone willing