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
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
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 +
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
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
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
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,
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
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
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
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
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
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
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
> -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
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
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
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
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
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
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
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)
>
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
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
24 matches
Mail list logo