Re: [PATCH] Add fields to struct gomp_thread for debugging purposes

2017-11-04 Thread Kevin Buettner
On Tue, 31 Oct 2017 08:03:22 +0100 Jakub Jelinek wrote: > On Mon, Oct 30, 2017 at 04:06:15PM -0700, Kevin Buettner wrote: > > This patch adds a new member named "pthread_id" to the gomp_thread > > struct. It is initialized in team.c. > > That part is reasonable, though it is unclear how the d

Re: [PATCH] RISC-V: Emit "i" suffix for instructions with immediate operands

2017-11-04 Thread Palmer Dabbelt
Committed, after adding a second space before the email and before the closing comment. On Fri, 03 Nov 2017 09:14:25 PDT (-0700), Palmer Dabbelt wrote: > From: Michael Clark > > This changes makes GCC asm output use instruction names that are > consistent with the RISC-V ISA manual. The assemble

Re: [PATCH] RISC-V: If -m[no-]strict-align is not passed, assume its value from -mtune

2017-11-04 Thread Palmer Dabbelt
On Fri, 03 Nov 2017 09:14:10 PDT (-0700), Palmer Dabbelt wrote: > From: Andrew Waterman > > 2017-11-03 Andrew Waterman > > * config/riscv/riscv.c (riscv_option_override): Conditionally set > TARGET_STRICT_ALIGN based upon -mtune argument. > --- > gcc/config/riscv/riscv.c | 10 +

Re: [PATCH] RISC-V: Set SLOW_BYTE_ACCESS=1

2017-11-04 Thread Palmer Dabbelt
Committed. On Fri, 03 Nov 2017 08:34:38 PDT (-0700), Palmer Dabbelt wrote: > From: Andrew Waterman > > When implementing the RISC-V port, I took the name of this macro at > face value. It appears we were mistaken in what this means, here's a > quote from the SPARC port that better describes what

Re: [C++ Patch] PR 80955 (Macros expanded in definition of user-defined literals)

2017-11-04 Thread Mukesh Kapoor
On 11/3/2017 7:31 AM, Paolo Carlini wrote: Hi, On 02/11/2017 15:42, Jason Merrill wrote: This is a good suggestion. I have attached the revised patch. Thanks. OK, thanks! Thanks Jason. I was about to volunteer committing the patch but then noticed that the testcase includes quite a lot, e

Re: [PATCH] Initialize variable in order to survive PGO bootstrap.

2017-11-04 Thread Richard Biener
On November 4, 2017 7:53:12 PM GMT+01:00, Richard Sandiford wrote: >Richard Biener writes: >> On Fri, Nov 3, 2017 at 9:52 AM, Martin Liška wrote: >>> Hi. >>> >>> This is oneliner that fixes PGO bootstrap. I've discussed that with >>> Richi and the core is correct. >>> However we probably don't

Re: [PATCH v3 1/14] D: The front-end (DMD) language implementation and license.

2017-11-04 Thread Walter Bright
On 10/24/2017 4:58 PM, Jeff Law wrote: On 10/03/2017 03:36 PM, Joseph Myers wrote: On Tue, 3 Oct 2017, Jeff Law wrote: /* Copyright (c) 2010-2014 by Digital Mars * All Rights Reserved, written by Walter Bright * http://www.digitalmars.com * Distributed under the Boost Software License,

Re: [PATCH] Initialize variable in order to survive PGO bootstrap.

2017-11-04 Thread Richard Sandiford
Richard Biener writes: > On Fri, Nov 3, 2017 at 9:52 AM, Martin Liška wrote: >> Hi. >> >> This is oneliner that fixes PGO bootstrap. I've discussed that with >> Richi and the core is correct. >> However we probably don't have an attribute that will ignore the warning? > > I think > > wide_int r

Re: [Patch, fortran] PR81447 - [7/8] gfortran fails to recognize the exact dynamic type of a polymorphic entity that was allocated in a external procedure

2017-11-04 Thread Thomas Koenig
Hi Andre, Shouldn't that better be if ((gfc_option.allow_std & GFC_STD_F2003) > 0 I think that if ((gfc_option.allow_std & GFC_STD_F2003) would be better - the allow_std field is signed, and things could get hariy if we ever have close to 32 standards we would like to support. Hm, come

Re: [Patch, fortran] PR81447 - [7/8] gfortran fails to recognize the exact dynamic type of a polymorphic entity that was allocated in a external procedure

2017-11-04 Thread Andre Vehreschild
Hi Paul, On Thu, 2 Nov 2017 20:11:29 + Paul Richard Thomas wrote: > *** resolve_fl_derived (gfc_symbol *sym) > *** 14075,14080 > --- 14078,14097 > if (!resolve_typebound_procedures (sym)) > return false; > > + /* Generate module vtables subject to their acc

Re: [PATCH][AArch64] Improve aarch64_legitimate_constant_p

2017-11-04 Thread Andreas Schwab
FAIL: gfortran.dg/class_array_1.f03 -O3 -fomit-frame-pointer -funroll-loops -fpeel-loops -ftracer -finline-functions (test for excess errors) Excess errors: /opt/gcc/gcc-20171104/gcc/testsuite/gfortran.dg/class_array_1.f03:31:0: Error: could not split insn (insn 527 1444 562 (set (reg:TI 0 x0

Re: [Patch, fortran] PR78641 - [6/7/8 Regression] [OOP] ICE on polymorphic allocatable function in array constructor

2017-11-04 Thread Steve Kargl
On Sat, Nov 04, 2017 at 12:02:58PM +, Paul Richard Thomas wrote: > Dear All, > > This patch allows the assignment of class array constructors to > derived type arrays. It is straightforward enough that the ChangeLogs > and the comment are sufficient explanation. > > Bootstraps and regtests on

Re: [Committed, Fortran, Patch, v1] Three small patches for character arrays

2017-11-04 Thread Andre Vehreschild
Hi Paul, thanks for the review. Committed as r254407 to trunk and r254408 to gcc-7. Regards, Andre On Sat, 4 Nov 2017 12:06:15 + Paul Richard Thomas wrote: > Hi Andre, > > These patches look fine to me. Thanks for taking account of the array > descriptor change between trunk and 7

[test case, fortran, committed] Test case for PR 70330

2017-11-04 Thread Thomas Koenig
Hi, I have just committed the test case from PR 70330 to trunk and closed the (error-recovery) PR. Regards Thomas https://gcc.gnu.org/viewcvs?rev=254406&root=gcc&view=rev

RE: [PATCH 08/22] Add Intel CET support for EH in libgcc.

2017-11-04 Thread Tsimbalist, Igor V
> -Original Message- > From: Jeff Law [mailto:l...@redhat.com] > Sent: Tuesday, October 31, 2017 5:49 AM > To: Tsimbalist, Igor V ; gcc- > patc...@gcc.gnu.org > Cc: i...@airs.com > Subject: Re: [PATCH 08/22] Add Intel CET support for EH in libgcc. > > On 10/12/2017 01:56 PM, Tsimbalist, Ig

Re: [Ping, Fortran, Patch, v1] Three small patches for character arrays

2017-11-04 Thread Paul Richard Thomas
Hi Andre, These patches look fine to me. Thanks for taking account of the array descriptor change between trunk and 7-branch :-) OK for trunk and 7-branch. Thanks Paul On 4 November 2017 at 10:27, Andre Vehreschild wrote: > Ping! > > On Wed, 1 Nov 2017 12:27:11 +0100 > Andre Vehreschild wro

[Patch, fortran] PR78641 - [6/7/8 Regression] [OOP] ICE on polymorphic allocatable function in array constructor

2017-11-04 Thread Paul Richard Thomas
Dear All, This patch allows the assignment of class array constructors to derived type arrays. It is straightforward enough that the ChangeLogs and the comment are sufficient explanation. Bootstraps and regtests on FC23/x86_64 - OK for all three branches? Paul 2017-11-04 Paul Thomas PR

Re: Drop frequencies from basic blocks

2017-11-04 Thread Andreas Schwab
This breaks gfortran.dg/bounds_check_15.f90 with -O3 -funroll-loops -fbounds-check on ia64: Error: qsort comparator non-negative on sorted output: 1 during RTL pass: mach ../gcc/testsuite/gfortran.dg/bounds_check_15.f90:32:0: internal compiler error: qsort checking failed 0x401cc1ff qsort

Re: [Ping, Fortran, Patch, v1] Three small patches for character arrays

2017-11-04 Thread Andre Vehreschild
Ping! On Wed, 1 Nov 2017 12:27:11 +0100 Andre Vehreschild wrote: > Hi all, > > during development on OpenCoarrays I found these three issues in gfortran: > > - caf_send: Did not treat char arrays as arrays when calling caf_send. > - gfc_trans_assignment_1: Conversion of character kind creates

Re: libbacktrace patch committed: Support compressed debug sections

2017-11-04 Thread Gerald Pfeifer
On Fri, 6 Oct 2017, Ian Lance Taylor wrote: > Thanks for the report. I committed this patch, which I hope will fix > the problem. > * ztest.c (test_large): Pass unsigned long *, not size_t *, to > zlib uncompress function. Thank you, yes it did. (Sorry, I thought I had responded back then, but

More fold_negate in match.pd

2017-11-04 Thread Marc Glisse
Hello, this copies some more transformations from fold_negate_expr to match.pd. Mostly, I wanted to add (negate (pointer_diff @0 @1)), and couldn't find the corresponding transformation with minus_expr... We can see about generalizing a bit with conversions later, I wanted to have something at

Re: PR82816: Widening multiplies of bitfields

2017-11-04 Thread Richard Sandiford
Richard Sandiford writes: > In this PR we tried to create a widening multiply of two 3-bit numbers, > but that isn't a widening multiply at the optab/rtl level, since both > the input and output still have the same mode. > > We could trap this either in is_widening_mult_p or (as the patch does) >

Re: [Patch, fortran] PR81735 - [6/7/8 Regression] double free or corruption (fasttop) error (SIGABRT) with character(:) and custom return type with allocatable

2017-11-04 Thread Paul Richard Thomas
Corrected in trunk in revision 254404. Cheers Paul On 3 November 2017 at 19:22, Paul Richard Thomas wrote: > This was already fixed by the patch for PR82375 on trunk, albeit less > well than the patch applied here. I will correct trunk tomorrow. > > Committed as 'obvious' on 6-branch(r254393) a

Re: [PATCH 2/2] [i386] PR82002 Part 2: Correct non-immediate offset/invalid INSN

2017-11-04 Thread Uros Bizjak
On Fri, Nov 3, 2017 at 10:22 PM, Daniel Santos wrote: > On 11/03/2017 02:09 AM, Uros Bizjak wrote: >> On Thu, Nov 2, 2017 at 11:43 PM, Daniel Santos >> wrote: >> >int_registers_saved = (frame.nregs == 0); >sse_registers_saved = (frame.nsseregs == 0); > + save_stub_call_neede