Hi,
On 17/11/2016 18:14, Jason Merrill wrote:
How about changing cp_parser_non_integral_constant_expression to issue
a pedwarn instead of error for NIC_FLOAT?
Indeed. And a relevant observation is that we don't pass NIC_FLOAT from
anywhere else in the parser, please correct me if I'm wrong. The
OK.
On Thu, Nov 17, 2016 at 3:25 PM, Paolo Carlini wrote:
> Hi,
>
> On 17/11/2016 18:14, Jason Merrill wrote:
>>
>> How about changing cp_parser_non_integral_constant_expression to issue
>> a pedwarn instead of error for NIC_FLOAT?
>
> Indeed. And a relevant observation is that we don't pass NIC_
On 17/11/16 09:10, Kyrill Tkachov wrote:
On 16/11/16 18:09, Thomas Preudhomme wrote:
Hi,
I've rebased the patch to make arm_feature_set agree with type of FL_* macros
on top of trunk rather than on top of the optional -mthumb patch. That
involved doing the changes to gcc/config/arm/arm-proto
Ping?
Best regards,
Thomas
On 08/11/16 13:35, Thomas Preudhomme wrote:
Ping,
Best regards,
Thomas
On 02/11/16 10:04, Thomas Preudhomme wrote:
Ping?
Best regards,
Thomas
On 28/10/16 10:49, Thomas Preudhomme wrote:
On 22/09/16 16:47, Richard Earnshaw (lists) wrote:
On 22/09/16 15:51, Th
Ping?
Best regards,
Thomas
On 08/11/16 13:36, Thomas Preudhomme wrote:
Ping?
Best regards,
Thomas
On 02/11/16 10:05, Thomas Preudhomme wrote:
Ping?
Best regards,
Thomas
On 27/10/16 15:26, Thomas Preudhomme wrote:
Hi Kyrill,
On 27/10/16 10:45, Kyrill Tkachov wrote:
Hi Thomas,
On 24/1
Ping?
Best regards,
Thomas
On 08/11/16 13:36, Thomas Preudhomme wrote:
Ping?
Best regards,
Thomas
On 02/11/16 10:05, Thomas Preudhomme wrote:
Ping?
Best regards,
Thomas
On 24/10/16 09:07, Thomas Preudhomme wrote:
Ping?
Best regards,
Thomas
On 13/10/16 16:35, Thomas Preudhomme wrote:
On Wed, Nov 16, 2016 at 11:52:23AM -0700, Jeff Law wrote:
> >2016-10-30 Segher Boessenkool
> >
> > PR rtl-optimization/71785
> > * bb-reorder.c (duplicate_computed_gotos_find_candidates): New
> > function, factored out from pass_duplicate_computed_gotos::execute.
> > (duplicate_c
For code like the testcase in PR71785 GCC factors all the indirect branches
to a single dispatcher that then everything jumps to. This is because
having many indirect branches with each many jump targets does not scale
in large parts of the compiler. Very late in the pass pipeline (right
before p
On 11/05/2016 04:04 PM, Martin Sebor wrote:
The patch below documents the rule of thumb for what changes are
appropriate for bug-fix release branches that we discussed in
the gcc-patches thread Re: relax rule for flexible array members
in 6.x (78039 - fails to compile glibc tests)
https://gcc.g
On 11/06/2016 09:32 AM, Waldemar Brodkorb wrote:
Hi,
Some platforms, such as Blackfin, have a special prefix for assembly
symbols as opposed to C symbols. For this reason, a function named
"foo()" in C will in fact be visible as a symbol called "_foo" in the
ELF binary.
The current linker versi
On 11/06/2016 09:39 AM, Waldemar Brodkorb wrote:
Hi,
This commit makes the Blackfin platform use the generic linker version
information, rather than a completely duplicated file, specific for the
Blackfin architecture.
This is made possible using the newly introduced skip_underscore
variable of
GCC maintainers:
The Power ABI document lists a number of built-ins that it is supposed
to support. There are still some missing. This patch adds the built-in
support for the following built-ins:
vector bool char vec_cmpeq vector bool char vector bool char
vector bool int vec_c
On 11/08/2016 05:49 PM, Iain Sandoe wrote:
On 8 Nov 2016, at 13:39, Mike Stump wrote:
On Nov 8, 2016, at 1:05 PM, Iain Sandoe
wrote:
Simple for the simple case is already part of my patch, but
capability for the expert and non-simple case is also present,
I'm trying to ask a specific que
On 11/17/2016 11:24 AM, Jakub Jelinek wrote:
On Thu, Nov 17, 2016 at 11:14:26AM -0700, Martin Sebor wrote:
--- a/gcc/fortran/interface.c
+++ b/gcc/fortran/interface.c
@@ -2821,7 +2821,8 @@ compare_actual_formal (gfc_actual_arglist **ap,
gfc_formal_arglist *formal,
for (f = formal; f; f = f->
The first patch does everything but the container adaptor parts (which
are wrong in the p/r) and the tuple part (which is icky). The second
part does the tuple parts, and needs some serious RFC. I know
it's ugly as sin, but it works. I don't think we should try to teach
our tuple to give the right
The C++17 copy elision code needed to handle CONSTRUCTORs.
Tested x86_64-pc-linux-gnu, applying to trunk.
commit 8b82502696188d92fc560cc55e0a564c507e6144
Author: Jason Merrill
Date: Thu Nov 17 15:37:56 2016 -0500
PR c++/78369 - {} as default argument
* call.c (buil
finish_parenthesized_expr uses TREE_NO_WARNING to mark parenthesized
expressions, so I think it's reasonable to use that to recognize them
here.
Tested x86_64-pc-linux-gnu, applying to trunk.
commit 043b2830c4e296951a8209ade25b8d01394c7386
Author: Jason Merrill
Date: Thu Nov 17 15:25:00 2016 -0
An inherited ctor makes the class non-aggregate. We weren't catching
this before because we add the inheriting ctors after we set
CLASSTYPE_NON_AGGREGATE, but it's easy enough to set it earlier, here.
Tested x86_64-pc-linux-gnu, applying to trunk.
commit d462f9acb6cdb05dcc49bee32f0741633de33aed
A
Hi Carl,
On Thu, Nov 17, 2016 at 01:19:57PM -0800, Carl E. Love wrote:
> 2016-11-15 Carl Love
Two spaces between your name and email address.
> * gcc.target/powerpc/builtins-3.c : New file to test the new
No space before colon.
> built-ins for vecotr compare equal and vector
> -Original Message-
> From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-
> ow...@gcc.gnu.org] On Behalf Of Matthew Fortune
> Sent: Thursday, November 17, 2016 8:47 AM
> To: Toma Tabacu ; Andrew Bennett
> ; Moore, Catherine
> ; 'gcc-patches@gcc.gnu.org' patc...@gcc.gnu.org>
> Subje
On Thu, Nov 17, 2016 at 01:56:03PM -0700, Jeff Law wrote:
> On 11/17/2016 11:24 AM, Jakub Jelinek wrote:
> >On Thu, Nov 17, 2016 at 11:14:26AM -0700, Martin Sebor wrote:
> >>--- a/gcc/fortran/interface.c
> >>+++ b/gcc/fortran/interface.c
> >>@@ -2821,7 +2821,8 @@ compare_actual_formal (gfc_actual_a
On Thu, Nov 17, 2016 at 4:26 AM, Andrew Senkevich
wrote:
> 16 Ноя 2016 г. 23:45 пользователь "Uros Bizjak" написал:
>>
>> On Tue, Nov 15, 2016 at 10:50 PM, Andrew Senkevich
>> wrote:
>> > Hi,
>> >
>> > this is follow-up with tests for new __target__ attributes and
>> > __builtin_cpu_supports upd
On Thu, Nov 17, 2016 at 4:20 AM, Andrew Senkevich
wrote:
> 16 Ноя 2016 г. 19:21 пользователь "Bernd Schmidt"
> написал:
>
>
>>
>> On 11/15/2016 05:31 PM, Andrew Senkevich wrote:
>>>
>>> 2016-11-15 17:56 GMT+03:00 Jeff Law :
On 11/15/2016 05:55 AM, Andrew Senkevich wrote:
>
>
>>>
The issue here was that we were still introducing extra temporaries of
an empty class in a CALL_FROM_THUNK_P call to the inherited
constructor.
Tested x86_64-pc-linux-gnu, applying to trunk.
commit 312678eec4a6dac8fe05e2ecfde60a717543ae76
Author: Jason Merrill
Date: Thu Nov 17 17:18:26 2016 -05
* Claudiu Zissulescu [2016-11-17 13:23:44
+]:
>
> > > Note on tests: It will be nice to add a test where the added
> > > peephole kicks in. If you consider to add this test to the current
> > > patch, please resubmit it.
> >
> > There were cmem-bit-{1,2,3,4}.c added in that patch. All of
On 11/17/2016 03:03 PM, Jakub Jelinek wrote:
On Thu, Nov 17, 2016 at 01:56:03PM -0700, Jeff Law wrote:
On 11/17/2016 11:24 AM, Jakub Jelinek wrote:
On Thu, Nov 17, 2016 at 11:14:26AM -0700, Martin Sebor wrote:
--- a/gcc/fortran/interface.c
+++ b/gcc/fortran/interface.c
@@ -2821,7 +2821,8 @@ co
> gcc/testsuite/ChangeLog:
>
> 2016-11-15 Toma Tabacu
>
> * gcc.target/mips/branch-cost-1.c (dg-options): Use
> (HAS_MOVN) instead
> of isa>=4, in order to downgrade to R5.
>
> diff --git a/gcc/testsuite/gcc.target/mips/branch-cost-1.c
> b/gcc/testsuite/gcc.target/mips/branch-cost
Like so many ports, the m68k port forgot to emit a barrier prior to
deallocation of the stack in its epilogue.
As a result, a memory reference could schedule after the stack
deallocation. That memory reference might refer to an object in the
just allocated stack. If we get an interrupt bet
AFAICT the m68k was the last target needing twiddling to avoid
unexpected failures on vrp47.c (from reading comments in the bz).
The m68k (like others already mentioned) breaks the conditionals down
rather than combining them into a logical with single compare/test.
Fixed in the obvious way
This patch to libgo rewrites the code that converts from a Go type to
a libffi type in Go. As we move toward the Go 1.7 garbage collector,
it's essential that all allocation of values that can contain Go
pointers be done using the correct type descriptor. That is simplest
if we do all such alloca
Attached is an update to the patch that avoids duplicating the
-Walloca-larger-than warnings. This version also avoids warning
for calls with zero allocation size to functions declared with
the returns_nonnull attribute (like libiberty's xmalloc). Since
such functions cannot return null there's
On 11/17/2016 03:52 PM, Jeff Law wrote:
On 11/17/2016 03:03 PM, Jakub Jelinek wrote:
On Thu, Nov 17, 2016 at 01:56:03PM -0700, Jeff Law wrote:
On 11/17/2016 11:24 AM, Jakub Jelinek wrote:
On Thu, Nov 17, 2016 at 11:14:26AM -0700, Martin Sebor wrote:
--- a/gcc/fortran/interface.c
+++ b/gcc/for
Hi,
I was relying on ipa_get_callee_param_type to get type of parameter and
then convert arguments to this type while computing jump functions.
However, in cases like shown in PR78365, ipa_get_callee_param_type,
instead of giving up, would return the wrong type. I think the current
uses of ip
On Thu, Nov 17, 2016 at 09:10:51AM +0100, Jakub Jelinek wrote:
> On Wed, Nov 16, 2016 at 04:29:05PM -0800, Marek Polacek wrote:
> > Sorry. So consider the following:
> >
> > class S
> > {
> > virtual void foo () = 0;
> > };
> >
> > struct T {
> > T &operator << (const char *s);
> > };
> >
>
On 11/17/2016 05:32 PM, Martin Sebor wrote:
On 11/17/2016 03:52 PM, Jeff Law wrote:
On 11/17/2016 03:03 PM, Jakub Jelinek wrote:
On Thu, Nov 17, 2016 at 01:56:03PM -0700, Jeff Law wrote:
On 11/17/2016 11:24 AM, Jakub Jelinek wrote:
On Thu, Nov 17, 2016 at 11:14:26AM -0700, Martin Sebor wrote:
This patch to libgo updates the configure script and a few binary
files. The configure script was changed without going through the
external repo in a way that somehow dropped the
--with-system-libunwind argument and made a few other changes. This
patch restores these changes. A few binary files
This patch to gcc/go/gofrontend and libgo removes the remaining
extraneous differences between the GCC repo and gofrontend repo. A
few files were somehow not deleted from the GCC repo. The aclocal.m4
file was changed recently, and gcc/go/gofrontend/lex.cc was changed a
while back. This patch sim
Hi,
Please find attached the patch that fixes PR78382.
The "SYMBOL_SMALL_TLSGD" was not handled for ILP32.
Hence it generates error when compiled for ILP32.
The attached patch adds the support and handles it properly as expected
for ILP32.
Please review the patch and let me know if its okay?
On 11/16/2016 09:49 AM, Martin Sebor wrote:
I'm looking for an approval of the attached patch.
I've adjusted the documentation based on Sandra's input (i.e.,
documented the negative of the option rather than the positive;
thank you for the review, btw.)
On 11/08/2016 08:13 PM, Martin Sebor wrot
Hi Jakub,
Jakub Jelinek wrote,
> On Wed, Nov 16, 2016 at 07:31:59AM +0100, Waldemar Brodkorb wrote:
> > > On Wed, Nov 09, 2016 at 04:08:39PM +0100, Bernd Schmidt wrote:
> > > > On 11/05/2016 06:14 PM, Waldemar Brodkorb wrote:
> > > > >Hi,
> > > > >
> > > > >the following patch fixes PR68468.
> > >
James,
I incorporated all your suggestions, and successfully bootstrapped and re-ran
the testsuite.
Okay for trunk?
2016-11-18 Michael Collison
* config/aarch64/aarch64-protos.h
(aarch64_and_split_imm1, aarch64_and_split_imm2)
(aarch64_and_bitmask_imm): New prototype
On Thu, 17 Nov 2016, Jeff Law wrote:
> On 11/17/2016 01:20 AM, Richard Biener wrote:
> > On Wed, 16 Nov 2016, Jeff Law wrote:
> >
> > > On 10/28/2016 05:51 AM, Richard Biener wrote:
> > > >
> > > > These are the middle-end changes and additions to the testsuite.
> > > >
> > > > They are pretty
On Thu, Nov 17, 2016 at 05:32:31PM -0700, Martin Sebor wrote:
> >The point is warning on an alloca (0) may not be as clear cut as it
> >might seem. It's probably a reasonable thing to do on the host, but on
> >a target, which might be embedded and explicitly overriding the builtin
> >alloca, warni
101 - 143 of 143 matches
Mail list logo
