Hi Matthew,
Please find attached the patch to fix the calling convention issue,
where argument and result passing convention differed for MSA and
non-MSA variants.
The implementation of TARGET_RETURN_IN_MEMORY is altered to block V4SF to be
returned in registers.
Ok for trunk?
- Thanks and r
On Tue, 7 Feb 2017, Jakub Jelinek wrote:
> Hi!
>
> While looking at the RTL problem when combiner optimizes away
> (x & 0xfffe) % 0x into just (x & 0xfffe), I've looked at VRP
> which optimizes that case well (-O2 only, the PR is -O1), but discovered
> that it isn't generic enough, we actuall
> I believe we're in. Thanks for the help!
Congratulations. There are a few post-integration items to be done:
"If the back end is added to the official GCC source repository, the
following are also necessary:
* An entry for the target architecture in `readings.html' on the GCC
web sit
> @Eric: thanks for the offer of sparc testing; please can you let me know
> if there are any issues?
Sure, but can you post a consolidated patch as an attachment? (btw it's SPARC
instead of sparc, like MIPS instead of mips I presume).
--
Eric Botcazou
Hi Gerald,
> Index: readings.html
> ===
> RCS file: /cvs/gcc/wwwdocs/htdocs/readings.html,v
> retrieving revision 1.257
> diff -u -r1.257 readings.html
> --- readings.html 6 Feb 2017 17:47:51 - 1.257
> +++ readings.html
Hi Steve,
I see Mikael has okayed the patch, but I did not have any time to comment
prior. I wanted to note that:
- the choice was deliberate, as the standard allows us to choose which real
kind REAL128 corresponds to when there are several matching choices. The idea
behind the current choice
Tested on x86_64-unknown-linux-gnu, applied.
Richard.
2017-02-08 Richard Biener
PR tree-optimization/71824
PR tree-optimization/79409
* gcc.dg/graphite/pr71824-3.c: New testcase.
Index: gcc/testsuite/gcc.dg/graphite/pr71824-3.c
===
It was spelled Altivec in two user-facing messages in rs6000.opt; let's
be consistent.
Committing to trunk. Will also backport as needed.
Segher
2017-02-08 Segher Boessenkool
PR translation/79397
* config/rs6000/rs6000.opt (maltivec=le, maltivec=be): Fix spelling
o
Eric Botcazou writes:
> > @Eric: thanks for the offer of sparc testing; please can you let me
> > know if there are any issues?
>
> Sure, but can you post a consolidated patch as an attachment? (btw it's
> SPARC instead of sparc, like MIPS instead of mips I presume).
I've attached the consolidat
On 08/02/17 08:17 +0100, Gerald Pfeifer wrote:
Index: src/c++11/snprintf_lite.cc
===
--- src/c++11/snprintf_lite.cc (revision 245267)
+++ src/c++11/snprintf_lite.cc (working copy)
@@ -52,7 +52,7 @@
const size_t __len = __bufend
On Wed, Feb 08, 2017 at 10:42:58AM +, Jonathan Wakely wrote:
> On 08/02/17 08:17 +0100, Gerald Pfeifer wrote:
> > Index: src/c++11/snprintf_lite.cc
> > ===
> > --- src/c++11/snprintf_lite.cc (revision 245267)
> > +++ src/c++11
On 08/02/17 11:50 +0100, Jakub Jelinek wrote:
On Wed, Feb 08, 2017 at 10:42:58AM +, Jonathan Wakely wrote:
On 08/02/17 08:17 +0100, Gerald Pfeifer wrote:
> Index: src/c++11/snprintf_lite.cc
> ===
> --- src/c++11/snprintf_lite.cc
On Wed, Feb 08, 2017 at 10:58:20AM +, Jonathan Wakely wrote:
> On 08/02/17 11:50 +0100, Jakub Jelinek wrote:
> > On Wed, Feb 08, 2017 at 10:42:58AM +, Jonathan Wakely wrote:
> > > On 08/02/17 08:17 +0100, Gerald Pfeifer wrote:
> > > > Index: src/c++11/snprintf_lite.cc
> > > > ==
On Tue, Feb 07, 2017 at 04:17:48PM -0500, Jason Merrill wrote:
> On Tue, Feb 7, 2017 at 9:13 AM, Jonathan Wakely wrote:
> > On 07/02/17 15:04 +0100, Marek Polacek wrote:
> >>
> >> Thanks much for the review. Looks ok now?
>
> I'd suggest adding something to say that the reason these are now
> be
On 7 February 2017 at 20:06, Richard Biener wrote:
> On Tue, 7 Feb 2017, Prathamesh Kulkarni wrote:
>
>> Hi Richard,
>> The attached patch tries to handle ABS_EXPR in gimple-fe.
>> I am not sure if __ABS_EXPR should be parsed as id (like __MEM)
>> or parse it as keyword (like __GIMPLE). Currently
From 205aa8e97bab553e5e6fe45896325e97962de704 Mon Sep 17 00:00:00 2001
From: Rolf Eike Beer
Date: Wed, 8 Feb 2017 11:42:52 +0100
Subject: [PATCH] asan: fix missing include of signal.h
This breaks when building gcc 5.4.0 with
target_platform=powerpc-unknown-linux-gnu with glibc 2.25:
../../../..
On 08/02/17 12:02 +0100, Jakub Jelinek wrote:
On Wed, Feb 08, 2017 at 10:58:20AM +, Jonathan Wakely wrote:
On 08/02/17 11:50 +0100, Jakub Jelinek wrote:
> On Wed, Feb 08, 2017 at 10:42:58AM +, Jonathan Wakely wrote:
> > On 08/02/17 08:17 +0100, Gerald Pfeifer wrote:
> > > Index: src/c++1
On Mon, Jan 23, 2017 at 05:25:45PM +0100, Bernd Schmidt wrote:
> There's still a a few details that need addressing, and some questions I
> have. Also Ccing Jakub to have another pair of eyes on the name of the
> section - I don't know if we want some sort of .gnu.something name.
So Jakub, Richard
Hi Palmer,
On Tue, 7 Feb 2017, Palmer Dabbelt wrote:
> These were suggested as part of our review process, but I never got an
> OK on them. Am I meant to just commit these?
I am pretty sure I acked htdocs/readings.html (see the mail below).
backends.html I am not sufficiently knowledgable, but
On Wed, Feb 08, 2017 at 12:18:05PM +0100, Torsten Duwe wrote:
> On Mon, Jan 23, 2017 at 05:25:45PM +0100, Bernd Schmidt wrote:
> > There's still a a few details that need addressing, and some questions I
> > have. Also Ccing Jakub to have another pair of eyes on the name of the
> > section - I don'
Hi,
The attached patch replaces couple of calls to c_parser_gimple_unary_expression
by c_parser_gimple_postfix_expression, to reject invalid cases like:
~t0_1 = ~a;
OK to commit after bootstrap+test ?
Thanks,
Prathamesh
diff --git a/gcc/c/gimple-parser.c b/gcc/c/gimple-parser.c
index 942597a..0080
On 8 February 2017 at 17:18, Prathamesh Kulkarni
wrote:
> Hi,
> The attached patch replaces couple of calls to
> c_parser_gimple_unary_expression
> by c_parser_gimple_postfix_expression, to reject invalid cases like:
> ~t0_1 = ~a;
> OK to commit after bootstrap+test ?
Oops no, this would reject v
On Wed, 8 Feb 2017, Prathamesh Kulkarni wrote:
> On 7 February 2017 at 20:06, Richard Biener wrote:
> > On Tue, 7 Feb 2017, Prathamesh Kulkarni wrote:
> >
> >> Hi Richard,
> >> The attached patch tries to handle ABS_EXPR in gimple-fe.
> >> I am not sure if __ABS_EXPR should be parsed as id (like
On 2017.02.08 at 12:05 +0100, Marek Polacek wrote:
> On Tue, Feb 07, 2017 at 04:17:48PM -0500, Jason Merrill wrote:
> > On Tue, Feb 7, 2017 at 9:13 AM, Jonathan Wakely wrote:
> > > On 07/02/17 15:04 +0100, Marek Polacek wrote:
> > >>
> > >> Thanks much for the review. Looks ok now?
> >
> > I'd s
This fixes a number of links that Gerald pointed out are either
redirecting or broken.
* doc/xml/manual/policy_data_structures.xml: Fix spelling of author's
name.
* doc/xml/manual/policy_data_structures_biblio.xml: Likewise. Remove
broken links to texts that are no
On 8 February 2017 at 17:24, Richard Biener wrote:
> On Wed, 8 Feb 2017, Prathamesh Kulkarni wrote:
>
>> On 7 February 2017 at 20:06, Richard Biener wrote:
>> > On Tue, 7 Feb 2017, Prathamesh Kulkarni wrote:
>> >
>> >> Hi Richard,
>> >> The attached patch tries to handle ABS_EXPR in gimple-fe.
>>
On Wed, 8 Feb 2017, Prathamesh Kulkarni wrote:
> On 8 February 2017 at 17:24, Richard Biener wrote:
> > On Wed, 8 Feb 2017, Prathamesh Kulkarni wrote:
> >
> >> On 7 February 2017 at 20:06, Richard Biener wrote:
> >> > On Tue, 7 Feb 2017, Prathamesh Kulkarni wrote:
> >> >
> >> >> Hi Richard,
> >>
Hi,
the patch below is my attempt at properly deprecating switch
-fipa-cp-alignment. I have not found any clear example to follow but
hopefully I have not missed anything. If so, please let me know.
So far I have only tested this with a normal build and "make info" but
I will give it a proper b
On Tue, Feb 7, 2017 at 7:31 PM, Jeff Law wrote:
>
> This patch addresses issues Richi raised from V1. Specifically it moves
> EXACT_DIV_EXPR handling into extract_range_from_binary_expr_1 and less
> aggressively uses ~[0,0] when intersecting ranges -- only doing so when
> vr1's range is > 65536 e
On Wed, Feb 08, 2017 at 01:41:24PM +0100, Martin Jambor wrote:
> 2017-02-08 Martin Jambor
>
> * common.opt (-finstrument-functions-exclude-file-list): Remove Var
> and Optimization, Document as deprecated and superseded by
> -fipa-bit-cp.
> * doc/invoke.texi (Option Summ
On Tue, Feb 7, 2017 at 7:34 PM, Jeff Law wrote:
> On 02/07/2017 01:39 AM, Richard Biener wrote:
>>
>> On Mon, Feb 6, 2017 at 10:57 PM, Jeff Law wrote:
>>>
>>> On 02/06/2017 08:33 AM, Richard Biener wrote:
>>>
ah, indeed vr0type is VR_ANTI_RANGE and yes we have the case of a
range with a
On Wed, Feb 08, 2017 at 12:54:44PM +0100, Markus Trippelsdorf wrote:
> On 2017.02.08 at 12:05 +0100, Marek Polacek wrote:
> > On Tue, Feb 07, 2017 at 04:17:48PM -0500, Jason Merrill wrote:
> > > On Tue, Feb 7, 2017 at 9:13 AM, Jonathan Wakely
> > > wrote:
> > > > On 07/02/17 15:04 +0100, Marek Po
On Wed, Feb 8, 2017 at 1:49 PM, Jakub Jelinek wrote:
> On Wed, Feb 08, 2017 at 01:41:24PM +0100, Martin Jambor wrote:
>> 2017-02-08 Martin Jambor
>>
>> * common.opt (-finstrument-functions-exclude-file-list): Remove Var
>> and Optimization, Document as deprecated and superseded by
>
On 2017.02.08 at 13:56 +0100, Marek Polacek wrote:
> On Wed, Feb 08, 2017 at 12:54:44PM +0100, Markus Trippelsdorf wrote:
> > On 2017.02.08 at 12:05 +0100, Marek Polacek wrote:
> > > On Tue, Feb 07, 2017 at 04:17:48PM -0500, Jason Merrill wrote:
> > > > On Tue, Feb 7, 2017 at 9:13 AM, Jonathan Wake
On 07/02/17 23:29 +0200, Ville Voutilainen wrote:
On 7 February 2017 at 22:00, Ville Voutilainen
wrote:
Currently, clang rejects all attempts to visit a libstdc++ variant:
https://godbolt.org/g/kSmBTg
While gcc doesn't reject such code, that seems like an access checking
bug. This patch makes
On 08/02/17 13:56 +, Jonathan Wakely wrote:
Definitely a clang bug, see
http://melpon.org/wandbox/permlink/PkbNSpvjPrLh8rov
If we remove the nested namespace we see that the friend declaration
is declaring a second, unrelated function, and so we get an ambiguity:
http://melpon.org/wandbox/pe
This patch fixes 79393, but I'm not 100% sure it's right.
[15.4]/8 says:
'The exception specification for an implicitly-declared destructor, or a
destructor without a noexcept-specifier, is potentially-throwing if and
only if any of the destructors for any of its potentially constructed
suboje
On Wed, Feb 8, 2017 at 10:08 PM, Nathan Sidwell wrote:
>
> 'potentially constructed subobject' appears to be a term without definition.
[special]/5:
For a class, its non-static data members, its non-virtual direct base
classes, and, if the class is not abstract, its virtual base classes
are call
The following fixes walking the SESE region when determining if it is
valid.
Bootstrap and regtest running on x86_64-unknown-linux-gnu.
Will commit as obvious once that passed.
Thanks,
Richard.
2017-02-08 Richard Biener
PR tree-optimization/69823
* graphite-scop-detection.
This patch teaches lower_oacc_reductions how to handle reduction
variables that were transferred to the accelerator via
GOMP_MAP_FIRSTPRIVATE_POINTER, e.g. fortran dummy variables. One side
effect of this change is that the ref_to_res variable gets remapped to
the (void *) field decl that gets tran
> PR target/78660
> * lra-constraints.c (curr_insn_transform): Handle
> WORD_REGISTER_OPERATIONS requirements when reloading SUBREGs.
> ---
> gcc/lra-constraints.c | 9 -
> 1 file changed, 8 insertions(+), 1 deletion(-)
>
> diff --git a/gcc/lra-constraints.c b/gcc/lra-co
> This patch is a minimal change to prevent (subreg(mem)) from being
> simplified to use the outer mode for WORD_REGISTER_OPERATIONS. There
> is high probability of refining and/or re-implementing this for GCC 8
> but such a change would be too invasive. This change at least ensures
> correctness
On 02/08/2017 09:33 AM, Tim Song wrote:
On Wed, Feb 8, 2017 at 10:08 PM, Nathan Sidwell wrote:
'potentially constructed subobject' appears to be a term without definition.
[special]/5:
For a class, its non-static data members, its non-virtual direct base
classes, and, if the class is not ab
Dear All,
The attached rework of the patch functions in the same way as
yesterday's but is based in resolve.c rather than trans-decl.c. It
looks to me to be by far cleaner.
Bootstraps and regtests on FC23/x86_64 - OK for trunk?
Cheers
Paul
2017-02-08 Paul Thomas
PR fortran/79344
*
On Wed, Feb 08, 2017 at 10:25:22AM +0100, FX wrote:
> Hi Steve,
>
> I see Mikael has okayed the patch, but I did not have any
> time to comment prior. I wanted to note that:
>
> - the choice was deliberate, as the standard allows us to
> choose which real kind REAL128 corresponds to when there a
On Wed, Feb 8, 2017 at 8:34 AM, Richard Biener wrote:
>
> The following fixes walking the SESE region when determining if it is
> valid.
>
> Bootstrap and regtest running on x86_64-unknown-linux-gnu.
>
> Will commit as obvious once that passed.
The change looks good.
Thanks,
Sebastian
>
> Thank
On Wed, Feb 08, 2017 at 02:14:14PM +0100, Markus Trippelsdorf wrote:
> I'm not suggesting to drop both. But:
>
> »As a consequence, the following examples are invalid and G++ will no
> longer compile them, because, in the following examples, G++ used to...«
>
> The second occurrence of "the follo
On Wed, Feb 8, 2017 at 10:09 AM, Nathan Sidwell wrote:
> On 02/08/2017 09:33 AM, Tim Song wrote:
>> On Wed, Feb 8, 2017 at 10:08 PM, Nathan Sidwell wrote:
>>> 'potentially constructed subobject' appears to be a term without
>>> definition.
>>
>> [special]/5:
>>
>> For a class, its non-static dat
On Wed, Feb 8, 2017 at 12:45 PM, Marek Polacek wrote:
> On Wed, Feb 08, 2017 at 02:14:14PM +0100, Markus Trippelsdorf wrote:
>> I'm not suggesting to drop both. But:
>>
>> »As a consequence, the following examples are invalid and G++ will no
>> longer compile them, because, in the following exampl
The following patch fixes the pr by adding code for integer comparisons to
invert the condition and swap operands to avoid generation of a compound test.
Bootstrap/regtest on powerpc64-unknown-linux-gnu with no new regressions. Ok
for trunk?
-Pat
2017-02-08 Pat Haugen
PR target/78
Sorry, I must have missed that one. They're committed.
Thanks!
On Wed, 08 Feb 2017 03:33:19 PST (-0800), ger...@pfeifer.com wrote:
> Hi Palmer,
>
> On Tue, 7 Feb 2017, Palmer Dabbelt wrote:
>> These were suggested as part of our review process, but I never got an
>> OK on them. Am I meant to ju
2017-02-08 Palmer Dabbelt
* config-list.mk (LIST): Add riscv32-unknown-elf and
riscv64-unknown-elf.
---
contrib/ChangeLog | 5 +
contrib/config-list.mk | 1 +
2 files changed, 6 insertions(+)
diff --git a/contrib/ChangeLog b/contrib/ChangeLog
index f89ff38..23c6d7a 10
2017-02-08 Palmer Dabbelt
* MAINTAINERS (CPU Port Maintainers): Add Kito Cheng, Palmer Dabbelt,
and Andrew Waterman as the RISC-V maintainers.
---
ChangeLog | 5 +
MAINTAINERS | 3 +++
2 files changed, 8 insertions(+)
diff --git a/ChangeLog b/ChangeLog
index d2c498a..262
On Wed, 08 Feb 2017 00:43:09 PST (-0800), ebotca...@adacore.com wrote:
>> I believe we're in. Thanks for the help!
>
> Congratulations. There are a few post-integration items to be done:
>
> "If the back end is added to the official GCC source repository, the
> following are also necessary:
>
>
On 02/08/2017 12:53 PM, Jason Merrill wrote:
Perhaps the defect is in the standard. Will you email core about it?
Correct, done.
nathan
--
Nathan Sidwell
Le 08/02/2017 à 10:25, FX a écrit :
Hi Steve,
I see Mikael has okayed the patch, but I did not have any time to comment
prior. I wanted to note that:
- the choice was deliberate, as the standard allows us to choose which real
kind REAL128 corresponds to when there are several matching choice
On Wed, Feb 08, 2017 at 12:45:22PM -0600, Pat Haugen wrote:
> The following patch fixes the pr by adding code for integer comparisons to
> invert the condition and swap operands to avoid generation of a compound
> test. Bootstrap/regtest on powerpc64-unknown-linux-gnu with no new
> regressions.
Hello world,
the attached patch fixes a 5/6/7 regression where we accept
code that is invalid for F95. Regression-tested.
OK for trunk?
(For some reason attaching the file doesn't work, and I don't want to
reboot my computer or log out just now because of some long-running
calculation. Therefo
On Wed, 8 Feb 2017, Jakub Jelinek wrote:
> > fipa-cp-alignment
> > -Common Report Var(flag_ipa_cp_alignment) Optimization
> > -Perform alignment discovery and propagation to make Interprocedural
> > constant propagation stronger.
> > +Common Report
> > +This switch is deprecated. Use -fipa-bit-
On Wed, 8 Feb 2017, Gerald Pfeifer wrote:
> So, both are okay.
Except http://riscv.org actually redirects to https://riscv.org . :-}
Fixed thusly.
Gerald
Index: index.html
===
RCS file: /cvs/gcc/wwwdocs/htdocs/index.html,v
retrievi
On Wed, 8 Feb 2017, Jonathan Wakely wrote:
>> + "(Please submit full bug report at http://gcc.gnu.org/bugs/):\n";
> Shouldn't that be https ? :-)
You're right! Fixed by the patch at the end.
On Wed, 8 Feb 2017, Jakub Jelinek wrote:
> Note ACX_BUGURL in gcc/configure.ac needs adjustments
On Wed, 08 Feb 2017 13:43:55 PST (-0800), ger...@pfeifer.com wrote:
> On Wed, 8 Feb 2017, Gerald Pfeifer wrote:
>> So, both are okay.
>
> Except http://riscv.org actually redirects to https://riscv.org . :-}
>
> Fixed thusly.
>
> Gerald
>
> Index: index.html
> ==
On Tue, Feb 7, 2017 at 4:30 PM, Jakub Jelinek wrote:
> P0017R1 added in [dcl.init.aggr]/1 that classes with inherited constructors
> are not aggregate. CLASSTYPE_NON_AGGREGATE is set slightly before
> add_implicitly_declared_members is called and so we don't know about the
> inherited ctors yet.
On Wed, Feb 08, 2017 at 04:51:54PM -0500, Jason Merrill wrote:
> On Tue, Feb 7, 2017 at 4:30 PM, Jakub Jelinek wrote:
> > P0017R1 added in [dcl.init.aggr]/1 that classes with inherited constructors
> > are not aggregate. CLASSTYPE_NON_AGGREGATE is set slightly before
> > add_implicitly_declared_m
On 7 February 2017 19:40:17 CET, Daniel Santos wrote:
>+{ "-mmoutline-msabi-xlogues", MASK_OUTLINE_MSABI_XLOGUES }
> };
s/mm/m/
>+thus reducing function size at the cost of and few extra instructions.
s/and/a/
On Wed, Feb 8, 2017 at 5:24 PM, Jakub Jelinek wrote:
> On Wed, Feb 08, 2017 at 04:51:54PM -0500, Jason Merrill wrote:
>> On Tue, Feb 7, 2017 at 4:30 PM, Jakub Jelinek wrote:
>> > P0017R1 added in [dcl.init.aggr]/1 that classes with inherited constructors
>> > are not aggregate. CLASSTYPE_NON_AGG
The libgomp.oacc-c-c++-common/loop-dim-default.c test uses asms and needs to be
skipped on hppa.
We don't have the complex.h header on hppa*-*-hpux*. The
libgomp.oacc-c-c++-common/reduction-cplx-dbl.c
and libgomp.oacc-c-c++-common/reduction-cplx-flt.c tests compile fine on hppa
without it.
As
* Claudiu Zissulescu [2017-01-30 10:35:08
+]:
> Hi,
>
> >
> > Andrew Burgess (2):
> > ARC: Make arc_selected_cpu global
> > ARC: Better creation of __NPS400__ define
> >
> > gcc/ChangeLog | 31
> > gcc/config/arc/arc-arch.h | 50 ++
On 7 February 2017 at 13:40, Richard Biener wrote:
> On Mon, Feb 6, 2017 at 7:00 AM, Prasad Ghangal
> wrote:
>> On 4 January 2017 at 16:02, Richard Biener
>> wrote:
>>> On Wed, Dec 28, 2016 at 7:27 PM, Prasad Ghangal
>>> wrote:
Hi,
The attached patch tries fix ICE due to undeclared
69 matches
Mail list logo
