Hi Jason,
I recently did run into this again. Could you please have a look?
https://gcc.gnu.org/ml/gcc-patches/2015-06/msg00904.html
Thanks!
-Andreas-
On 04/19/2016 10:48 PM, Mikhail Maltsev wrote:
On 04/18/2016 12:14 PM, Richard Biener wrote:
Enlarging tree_function_decl is bad.
Probably using 3 bits for malloc_flag, operator_new_flag and free_flag is
redundant. I packed the state into 2 bits.
Passes should get at the info via flags_from_
On 04/22/2016 03:57 PM, Jason Merrill wrote:
This looks good, but can we move the code into c-common rather than
duplicate it?
That would be this patch. Also passes testing on x86_64-linux.
Bernd
* doc/invoke.texi (Warning Options): Add -Wmemset-elt-size.
(-Wmemset-elt-size): New item.
c-fa
On 03/02/2016 10:53 PM, Vladimir Makarov wrote:
The patch is ok for me. But I'd wait for GCC7. People are sensitive to
their code performance degradation. Even in most cases the patch
improves performance, in some case it can worsen code and people might
fill new PRs after such change.
I'v
On Tue, 2016-04-19 at 10:29 +0200, Arnaud Charlet wrote:
> >
> > The updated attachment was included in message
> > https://gcc.gnu.org/ml/gcc-patches/2016-03/msg01659.html
> You should just put a FSF copyright on s-osinte-gnu.adb
>
> OK with this change.
Is replacing
+-- Copyright
On 04/20/2016 08:22 AM, tbsaunde+...@tbsaunde.org wrote:
- rtx_insn_list *merged_insns = 0;
+ auto_vec, 10> merged_insns;
I see Jeff has already acked this, but some of the expressions here are
getting unwieldy. can we maybe shorten some of this using typedefs?
Bernd
On 04/20/2016 04:57 PM, H.J. Lu wrote:
On Wed, Apr 20, 2016 at 7:54 AM, Jakub Jelinek wrote:
https://gcc.gnu.org/ml/gcc-patches/2016-04/msg01080.html
This is wrong, see my other comment on the libgomp patch.
See my reply to your reply on the libgomp patch.
Since Jakub has said it is wrong
> Is replacing
>
> +-- Copyright (C) 1991-1994, Florida State
> University--
> +-- Copyright (C) 1995-2014,
> AdaCore --
> +-- Copyright (C) 2015-2016, Free Software Foundation,
> Inc. --
>
> with only
>
> +--
On Mon, Apr 25, 2016 at 11:46 AM, Bernd Schmidt wrote:
> On 04/20/2016 04:57 PM, H.J. Lu wrote:
>>
>> On Wed, Apr 20, 2016 at 7:54 AM, Jakub Jelinek wrote:
https://gcc.gnu.org/ml/gcc-patches/2016-04/msg01080.html
>>>
>>>
>>> This is wrong, see my other comment on the libgomp patch.
>>>
On 04/16/2016 01:12 AM, David Wohlferd wrote:
There were basically 3 changes I was trying for in that doc patch. Are
any of them worth keeping? Or are we done?
1) "Do not clobber flags if they are being used as outputs."
2) Output flags sample (with #if removed).
3) "On the x86 platform, flags
On Mon, Apr 25, 2016 at 11:50 AM, Uros Bizjak wrote:
> On Mon, Apr 25, 2016 at 11:46 AM, Bernd Schmidt wrote:
>> On 04/20/2016 04:57 PM, H.J. Lu wrote:
>>>
>>> On Wed, Apr 20, 2016 at 7:54 AM, Jakub Jelinek wrote:
>
> https://gcc.gnu.org/ml/gcc-patches/2016-04/msg01080.html
>>>
On Fri, Apr 22, 2016 at 9:40 PM, Martin Jambor wrote:
> Hi,
>
> On Fri, Apr 22, 2016 at 09:24:31PM +0200, Richard Biener wrote:
>> On April 22, 2016 7:04:31 PM GMT+02:00, Martin Jambor
>> wrote:
>> >Hi,
>> >
>> >this patch adds verification that __builtin_unreachable and
>> >__builtin_trap are n
On Wed, Apr 20, 2016 at 07:56:16AM -0700, H.J. Lu wrote:
> On Wed, Apr 20, 2016 at 7:53 AM, Jakub Jelinek wrote:
> > On Wed, Apr 20, 2016 at 07:43:27AM -0700, H.J. Lu wrote:
> >> From 12c6ddcf67593ed7137764ca74043f1a9c2d8fda Mon Sep 17 00:00:00 2001
> >> From: "H.J. Lu"
> >> Date: Wed, 30 Mar 201
On Mon, Apr 25, 2016 at 11:02 AM, Bernd Schmidt wrote:
> On 04/19/2016 10:48 PM, Mikhail Maltsev wrote:
>>
>> On 04/18/2016 12:14 PM, Richard Biener wrote:
>>>
>>>
>>> Enlarging tree_function_decl is bad.
>>
>> Probably using 3 bits for malloc_flag, operator_new_flag and free_flag is
>> redundant.
On Fri, Apr 22, 2016 at 11:37 PM, Mikhail Maltsev wrote:
> On 04/20/2016 05:12 PM, Richard Biener wrote:
>> You have
>>
>> +static tree
>> +handle_free_attribute (tree *node, tree name, tree /*args*/, int /*flags*/,
>> + bool *no_add_attrs)
>> +{
>> + tree decl = *node;
>> +
On 04/18/2016 02:26 PM, Dhole wrote:
A few months ago I submited a patch to allow the embedded timestamps by
C/C++ macros to be set externally [2], which was already an improvement
over [1]. I was told to wait until the GCC 7 stage 1 started to send
this patch again.
+/* Read SOURCE_DATE_EPOC
Ping!
Regards
Senthil
Senthil Kumar Selvaraj writes:
> Bernd Schmidt writes:
>
>> On 04/07/2016 01:52 PM, Senthil Kumar Selvaraj wrote:
>>>The below patch fixes PR 60040 by not halting with a hard error on
>>>a spill failure, if reload knows that it has to run again anyway.
>>
>> Some a
On Mon, 2016-04-25 at 11:50 +0200, Arnaud Charlet wrote:
> >
> > Is replacing
> >
> > +-- Copyright (C) 1991-1994, Florida State
> > University--
> > +-- Copyright (C) 1995-2014,
> > AdaCore --
> > +-- Copyright (C) 2015-201
> Attaching the modified ada-hurd.diff. Maybe it is ready for inclusion in
> upstream now?
Patch is OK, go ahead and commit it, thanks.
> 2016-03-31 Svante Signell
>
> * gcc-interface/Makefile.in: Add support for x86 GNU/Hurd.
> * s-osinte-gnu.ads: New specification file.
>
The following patch fixes PR70780 uncovered by a mistake I made when
updating the iteration scheme in PRE antic compute.
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied.
Richard.
2016-04-25 Richard Biener
PR tree-optimization/70780
* tree-ssa-pre.c (compute_anti
> Did you manage to do this yet? I'm flushing my stage1 queue of
> "simple cleanups" right now.
No, I'm going to have a look this week.
--
Eric Botcazou
Hi Guys,
I am applying the following patch to update the prototypes in the
MSP430 specific part of libgcc. It adds missing prototypes for
exported ABI functions, and it changes the prototypes for the
arithmetic shift functions so that they explicitly take a signed
char parameter.
Cheer
On 04/21/2016 02:16 PM, Kirill Yukhin wrote:
Hello,
This looks like a typo to me.
GCC 6 Stage 4 (starts 2016-01-20)GCC 5.3 release (2015-12-04)
|
+-- GCC 5 branch created +
| \
v v
On 04/22/2016 09:45 PM, Sandra Loosemore wrote:
On 04/22/2016 10:42 AM, paul_kon...@dell.com wrote:
Would you expect people to conform to the abridged version or the
full standard? If the full standard, then publishing an abridged
version is not a good idea, it will just cause confusion. Let
On 6 April 2016 at 14:54, Richard Biener wrote:
> On Wed, 6 Apr 2016, Richard Biener wrote:
>
>> On Wed, 6 Apr 2016, Prathamesh Kulkarni wrote:
>>
>> > On 6 April 2016 at 13:44, Richard Biener wrote:
>> > > On Wed, 6 Apr 2016, Prathamesh Kulkarni wrote:
>> > >
>> > >> On 5 April 2016 at 18:28, Ri
Hi Guys,
I am applying this patch, developed by DJ, to improve the code
generated for the MSP430 when performing a shift by a single bit.
Normally a helper function is used to perform N-bit shifts, but
for one bit we can save time, and not use up any more space, by
performing the shift i
On 04/20/2016 08:22 AM, tbsaunde+...@tbsaunde.org wrote:
From: Trevor Saunders
+ unsigned int len = cond_list.length ();
+ for (unsigned int i = len - 1; i < len; i--)
This is a really icky way to write a loop, the i < len condition makes
it look like a forw
On 04/21/2016 01:24 AM, Trevor Saunders wrote:
On Wed, Apr 20, 2016 at 06:03:01AM -0700, Andi Kleen wrote:
A vector can have very different performance than a list, depending how
it is used. Do your patches cause any measure performance difference for
the compiler?
I haven't measured, but I
On 04/20/2016 08:22 AM, tbsaunde+...@tbsaunde.org wrote:
- remove_node_from_insn_list (insn, &nonlocal_goto_handler_labels);
+
+ unsigned int len = vec_safe_length (nonlocal_goto_handler_labels);
+ for (unsigned int i = 0; i < len; i++)
+ if ((*nonlocal_goto_handler_labels)[i
Tested on Linux/x86-64. OK for trunk?
BTW, I have a followup patch to use SSE for TImode bitwise operation.
H.J.
128-bit SSE load and store instructions can be used for load and store
of 128-bit integers if they are the only operations on 128-bit integers.
To convert load and store of 128-b
On 04/20/2016 08:22 AM, tbsaunde+...@tbsaunde.org wrote:
-static rtx_expr_list *
+static vec
extract_mentioned_regs (rtx x)
{
- rtx_expr_list *mentioned_regs = NULL;
+ vec mentioned_regs = vNULL;
subrtx_var_iterator::array_type array;
FOR_EACH_SUBRTX_VAR (iter, array, x, NONCONST)
Hi,
the patch below moves an assert from expand_expr_real_1 to gimple
verification. It triggers when we do a sloppy job outlining stuff
from one function to another (or perhaps inlining too) and leave in
the IL of a function a local declaration that belongs to a different
function.
Like I wrote
On Mon, Apr 25, 2016 at 02:56:07PM +0200, Bernd Schmidt wrote:
> On 04/20/2016 08:22 AM, tbsaunde+...@tbsaunde.org wrote:
> >
> >-static rtx_expr_list *
> >+static vec
> > extract_mentioned_regs (rtx x)
> > {
> >- rtx_expr_list *mentioned_regs = NULL;
> >+ vec mentioned_regs = vNULL;
> >sub
On Mon, Apr 25, 2016 at 11:35:16AM +0200, Bernd Schmidt wrote:
> On 04/20/2016 08:22 AM, tbsaunde+...@tbsaunde.org wrote:
> >- rtx_insn_list *merged_insns = 0;
> >+ auto_vec, 10> merged_insns;
>
> I see Jeff has already acked this, but some of the expressions here are
> getting unwieldy. can we
On Mon, Apr 25, 2016 at 02:43:27PM +0200, Bernd Schmidt wrote:
> On 04/20/2016 08:22 AM, tbsaunde+...@tbsaunde.org wrote:
> >- remove_node_from_insn_list (insn, &nonlocal_goto_handler_labels);
> >+
> >+ unsigned int len = vec_safe_length (nonlocal_goto_handler_labels);
> >+ for (unsi
On Mon, Apr 25, 2016 at 02:28:51PM +0200, Bernd Schmidt wrote:
> On 04/20/2016 08:22 AM, tbsaunde+...@tbsaunde.org wrote:
> >From: Trevor Saunders
>
> >+ unsigned int len = cond_list.length ();
> >+ for (unsigned int i = len - 1; i < len; i--)
>
> This is a really icky
On 04/20/2016 08:22 AM, tbsaunde+...@tbsaunde.org wrote:
-/* Remove INSN from queue. */
+/* Remove INSN at idx from queue. */
+static void
+queue_remove (unsigned int q, unsigned int idx)
+{
+ QUEUE_INDEX (insn_queue[q][idx]) = QUEUE_NOWHERE;
+ insn_queue[q].ordered_remove (idx);
+ q_size--;
Hi Segher,
Here's the fix for the obvious pasto separated out. CCing Richi and
Jakub as I'd appreciate release manager approval to include this in
gcc-6-branch. This fixes some cases where built-in functions are
connected to the wrong expanders because of copy-paste issues. These
tend not to be
On Mon, Apr 25, 2016 at 03:55:15PM +0200, Bernd Schmidt wrote:
> On 04/20/2016 08:22 AM, tbsaunde+...@tbsaunde.org wrote:
> >-/* Remove INSN from queue. */
> >+/* Remove INSN at idx from queue. */
> >+static void
> >+queue_remove (unsigned int q, unsigned int idx)
> >+{
> >+ QUEUE_INDEX (insn_qu
Hi
This patch adds a plugin event when evaluating a call expression in constexpr.
The goal of this patch is to allow the plugins to analyze and or
modify the evaluation of constant expressions.
Changelog 2016-4-25 Andres Tiraboschi
*gcc/plugin.c (PLUGIN_EVAL_CALL_CONSTEXPR): New event.
On 21 Apr 18:29, Kirill Yukhin wrote:
> Hello,
> On 21 Apr 14:50, Kirill Yukhin wrote:
> > Hello,
> > Patch in the bottom fixes mentioned PR by separating
> > AVX and AVX-512BW constraints.
> >
> > gcc/
> > * gcc/config/i386/sse.md (define_insn "3"):
> > Extract AVX-512BW constraint from A
On Mon, Apr 25, 2016 at 2:51 PM, H.J. Lu wrote:
> Tested on Linux/x86-64. OK for trunk?
> + /* FIXME: Since the CSE pass may change dominance info, which isn't
> + expected by the fwprop pass, call free_dominance_info to
> + invalidate dominance info. Otherwise, the fwprop pass may cra
On 04/25/2016 03:30 PM, Trevor Saunders wrote:
On Mon, Apr 25, 2016 at 02:28:51PM +0200, Bernd Schmidt wrote:
On 04/20/2016 08:22 AM, tbsaunde+...@tbsaunde.org wrote:
From: Trevor Saunders
+ unsigned int len = cond_list.length ();
+ for (unsigned int i = len
On 04/20/2016 08:22 AM, tbsaunde+...@tbsaunde.org wrote:
From: Trevor Saunders
It used the gc vector type, but isn't marked as a gc route, and appears
to be manually managed, so it should be safe to use the normal heap
vector.
gcc/ChangeLog:
2016-04-19 Trevor Saunders
* ira-emit.c
On 22/04/16 12:20, Kyrill Tkachov wrote:
On 22/04/16 11:34, Marc Glisse wrote:
On Fri, 22 Apr 2016, Kyrill Tkachov wrote:
On 22/04/16 10:43, Kyrill Tkachov wrote:
On 22/04/16 10:42, Marc Glisse wrote:
On Fri, 22 Apr 2016, Kyrill Tkachov wrote:
2016-04-21 Marc Glisse
gcc/
* match
On 04/25/2016 04:21 PM, Bernd Schmidt wrote:
On 04/25/2016 03:30 PM, Trevor Saunders wrote:
On Mon, Apr 25, 2016 at 02:28:51PM +0200, Bernd Schmidt wrote:
On 04/20/2016 08:22 AM, tbsaunde+...@tbsaunde.org wrote:
From: Trevor Saunders
+ unsigned int len = cond_list.length ();
+
On Mon, Apr 25, 2016 at 09:09:03AM -0500, Bill Schmidt wrote:
> Here's the fix for the obvious pasto separated out. CCing Richi and
> Jakub as I'd appreciate release manager approval to include this in
> gcc-6-branch. This fixes some cases where built-in functions are
> connected to the wrong exp
On Mon, Apr 25, 2016 at 7:18 AM, Uros Bizjak wrote:
> On Mon, Apr 25, 2016 at 2:51 PM, H.J. Lu wrote:
>> Tested on Linux/x86-64. OK for trunk?
>
>> + /* FIXME: Since the CSE pass may change dominance info, which isn't
>> + expected by the fwprop pass, call free_dominance_info to
>> + in
On 04/22/16 10:04, Alexander Monakov wrote:
echo 'int v __attribute__((section("foo")));' |
x86_64-pc-linux-gnu-accel-nvptx-none-gcc -xc - -o /dev/null
:1:5: error: section attributes are not supported for this target
Presumably it's missing a necessary hook? Couldn't such a hook check the
Hello.
Please consider application of the following patch, it fixes
a coding style issue and a memory leak.
Thanks,
Martin
>From 6afc975de0b6de76aa51b8c2ef741cd72c76dc75 Mon Sep 17 00:00:00 2001
From: marxin
Date: Mon, 25 Apr 2016 13:50:41 +0200
Subject: [PATCH 1/4] Fix coding style and a memory
On Mon, Apr 25, 2016 at 3:49 PM, Martin Liška wrote:
> Hello.
>
> Please consider application of the following patch, it fixes
> a coding style issue and a memory leak.
Hi Martin,
Will do, thanks very much for the help.
Thanks,
bin
>
> Thanks,
> Martin
Hello.
As I've been playing with branch predictions and contrib/analyze_brprob script,
I've decided to replace the old script with a Python implementation.
Improvements:
+ fixed horizontal formatting
+ remove ugly utilization of bc that is used for arithmetics
+ script is a bit faster (tramp3d du
On 04/15/2016 02:52 PM, Senthil Kumar Selvaraj wrote:
For both testcases in the PR, reload fails to take into account that
FP-SP elimination can no longer be performed, and tries to find reload
regs for an rtx generated when FP-SP elimination was valid.
1. reload initializes elim table with FP-
On Mon, Apr 25, 2016 at 05:06:34PM +0300, Kirill Yukhin wrote:
> On 21 Apr 18:29, Kirill Yukhin wrote:
> > Hello,
> > On 21 Apr 14:50, Kirill Yukhin wrote:
> > > Hello,
> > > Patch in the bottom fixes mentioned PR by separating
> > > AVX and AVX-512BW constraints.
> > >
> > > gcc/
> > > * gcc/co
On Mon, Apr 25, 2016 at 4:47 PM, H.J. Lu wrote:
> On Mon, Apr 25, 2016 at 7:18 AM, Uros Bizjak wrote:
>> On Mon, Apr 25, 2016 at 2:51 PM, H.J. Lu wrote:
>>> Tested on Linux/x86-64. OK for trunk?
>>
>>> + /* FIXME: Since the CSE pass may change dominance info, which isn't
>>> + expected by
On Mon, 2016-04-25 at 17:06 +0200, Jakub Jelinek wrote:
> On Mon, Apr 25, 2016 at 09:09:03AM -0500, Bill Schmidt wrote:
> > Hi Segher,
> >
> > Here's the fix for the obvious pasto separated out. CCing Richi and
> > Jakub as I'd appreciate release manager approval to include this in
> > gcc-6-bran
On Mon, Apr 25, 2016 at 09:09:03AM -0500, Bill Schmidt wrote:
> Hi Segher,
>
> Here's the fix for the obvious pasto separated out. CCing Richi and
> Jakub as I'd appreciate release manager approval to include this in
> gcc-6-branch. This fixes some cases where built-in functions are
> connected
On Mon, Apr 25, 2016 at 8:10 AM, Uros Bizjak wrote:
> On Mon, Apr 25, 2016 at 4:47 PM, H.J. Lu wrote:
>> On Mon, Apr 25, 2016 at 7:18 AM, Uros Bizjak wrote:
>>> On Mon, Apr 25, 2016 at 2:51 PM, H.J. Lu wrote:
Tested on Linux/x86-64. OK for trunk?
>>>
+ /* FIXME: Since the CSE pass m
On Mon, Apr 25, 2016 at 8:27 AM, H.J. Lu wrote:
> On Mon, Apr 25, 2016 at 8:10 AM, Uros Bizjak wrote:
>> On Mon, Apr 25, 2016 at 4:47 PM, H.J. Lu wrote:
>>> On Mon, Apr 25, 2016 at 7:18 AM, Uros Bizjak wrote:
On Mon, Apr 25, 2016 at 2:51 PM, H.J. Lu wrote:
> Tested on Linux/x86-64. O
Thanks for the info with regard to contributing,
On Fri, Apr 22, 2016 at 09:40:11AM +0100, James Greenhalgh wrote:
> This patch will need a ChangeLog entry [1], please draft one that I can
> use when I apply the patch.
* gcc/config/aarch64/arm_neon.h: Remove spurious attribute __unused__ from
p
On Fri, Apr 22, 2016 at 03:28:27PM -0400, Jason Merrill wrote:
> On Fri, Apr 22, 2016 at 2:12 PM, Marek Polacek wrote:
> > +cp_stabilize_reference (tree ref)
> > +{
> > + if (TREE_CODE (ref) == PREINCREMENT_EXPR
> > + || TREE_CODE (ref) == PREDECREMENT_EXPR)
>
> I think we want to do this f
Hello.
To make LTO wrappers (gcc-nm, gcc-ar, gcc-ranlib) more smart, I would like to
prevent execution
of the same binary by these wrapper. For LTO testing I symlink ar (nm, ranlib)
to these wrappers instead
of hacking a build system to respect NM (AR, RANLIB) environment variables. The
only pr
2016-04-25 18:27 GMT+03:00 H.J. Lu :
>
> Ilya, can you take a look?
>
> Thanks.
>
> --
> H.J.
Hi,
Algorithmic part of the patch looks OK to me except the following piece of code.
+/* Check REF's chain to add new insns into a queue
+ and find registers requiring conversion. */
Comment is wron
On Fri, Apr 22, 2016 at 11:47 AM, Richard Biener
wrote:
> On Fri, Apr 22, 2016 at 12:33 PM, Bin.Cheng wrote:
>> On Fri, Apr 22, 2016 at 11:25 AM, Richard Biener
>> wrote:
>>> On Fri, Apr 22, 2016 at 12:07 PM, Bin Cheng wrote:
Hi,
Tree if-conv has below code checking on virtual PHI nod
On Fri, Apr 22, 2016 at 11:26:11AM +0200, Thomas Schwinge wrote:
> Index: htdocs/gcc-6/changes.html
> ===
> RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-6/changes.html,v
> retrieving revision 1.75
> diff -u -p -r1.75 changes.html
LGTM.
> --
On Mon, Apr 25, 2016 at 05:39:45PM +0200, Wladimir J. van der Laan wrote:
>
> Thanks for the info with regard to contributing,
>
> On Fri, Apr 22, 2016 at 09:40:11AM +0100, James Greenhalgh wrote:
> > This patch will need a ChangeLog entry [1], please draft one that I can
> > use when I apply the
Hi,
This is a simple patch for tree ifcvt. It renames variable any_mask_load_store
to any_pred_load_store, as well as makes the variable visible in file scope.
First rationale is name of that variable is confusing with masked load store.
In fact, it also covers cases in which data race store
On Sat, 2016-04-23 at 20:21 +0200, Bernhard Reutner-Fischer wrote:
> On March 7, 2016 3:57:16 PM GMT+01:00, David Malcolm <
> dmalc...@redhat.com> wrote:
> > On Sat, 2016-03-05 at 23:46 +0100, Bernhard Reutner-Fischer wrote:
> > [...]
> >
> > > diff --git a/gcc/fortran/misc.c b/gcc/fortran/misc.c
The C++17 maybe_unused attribute is mostly equivalent to the GNU unused
attribute, except that it can also be applied to enumerators.
I was surprised to see that there currently isn't a table of C++
standard attributes; all the standard attributes we already support are
handled by translating
There doesn't seem to be any need to step through the is-a inline
functions. OK for trunk?
commit 1b74375b17e37ab7c5f96944148ff5a6bff3f8bc
Author: Jason Merrill
Date: Wed Apr 20 10:21:02 2016 -0400
* gdbinit.in: Skip is-a.h.
diff --git a/gcc/gdbinit.in b/gcc/gdbinit.in
index af7d51a..d2
On 04/25/2016 11:28 AM, Jason Merrill wrote:
There doesn't seem to be any need to step through the is-a inline
functions. OK for trunk?
Yes, please :-)
jeff
On 04/25/2016 11:35 AM, Marek Polacek wrote:
On Fri, Apr 22, 2016 at 03:28:27PM -0400, Jason Merrill wrote:
On Fri, Apr 22, 2016 at 2:12 PM, Marek Polacek wrote:
+cp_stabilize_reference (tree ref)
+{
+ if (TREE_CODE (ref) == PREINCREMENT_EXPR
+ || TREE_CODE (ref) == PREDECREMENT_EXPR)
On Sun, Apr 17, 2016 at 2:01 PM, Patrick Palka wrote:
> When an in-class unscoped enumeration is defined out-of-line its
> enumerators currently don't inherit the access of the enumeration. This
> patch makes the access of the enumerations defined out-of-line match the
> access of the enumerator.
On Mon, 25 Apr 2016, Nathan Sidwell wrote:
> On 04/22/16 10:04, Alexander Monakov wrote:
> > echo 'int v __attribute__((section("foo")));' |
> >x86_64-pc-linux-gnu-accel-nvptx-none-gcc -xc - -o /dev/null
> > :1:5: error: section attributes are not supported for this target
>
> Presumably it's
On 04/25/2016 05:07 AM, Bernd Schmidt wrote:
+ if (TREE_CODE (arg2) == CONST_DECL)
+ arg2 = DECL_INITIAL (arg2);
+ int literal_mask = ((!!integer_zerop (arg1) << 1)
+ | (!!integer_zerop (arg2) << 2));
Are you delibera
On 04/18/2016 01:12 PM, Jim Wilson wrote:
On 04/11/2016 01:41 PM, Jim Wilson wrote:
Here is a patch to correct the -fabi-version docs on the GCC 5 branch.
https://gcc.gnu.org/ml/gcc-patches/2016-04/msg00480.html
ping^2
Jim
On 04/25/2016 08:44 PM, Jim Wilson wrote:
On 04/18/2016 01:12 PM, Jim Wilson wrote:
On 04/11/2016 01:41 PM, Jim Wilson wrote:
Here is a patch to correct the -fabi-version docs on the GCC 5 branch.
https://gcc.gnu.org/ml/gcc-patches/2016-04/msg00480.html
ping^2
Cc'ing Jason as the most li
On 04/22/16 11:15, Wilco Dijkstra wrote:
This patch fixes that by setting the default aarch64_case_values_threshold to
16 when the per-CPU tuning is not set. On SPEC2006 this improves the switch
heavy benchmarks GCC and perlbench both in performance (1-2%) as well as size
(0.5-1% smaller).
I a
On 25.04.2016 16:57, Martin Liška wrote:
Hello.
As I've been playing with branch predictions and contrib/analyze_brprob script,
I've decided to replace the old script with a Python implementation.
Improvements:
+ fixed horizontal formatting
+ remove ugly utilization of bc that is used for arith
On 04/21/16 03:15, Kyrill Tkachov wrote:
Ok to commit?
LGTM
--
Evandro Menezes
On 04/22/16 10:35, Wilco Dijkstra wrote:
OK for trunk?
LGTM
--
Evandro Menezes
2016-04-25 Uros Bizjak
* config/i386/i386.md (*movoi_internal_avx): Set mode attribute to XI
for SSE constm1 operands and TARGET_AVX512VL.
(*movti_internal): Ditto.
(*mov_or): Use constm1_operand predicate.
* config/i386/sse.md (*mov_internal): Set mode attribute to XI
f
On 03/10/16 10:37, James Greenhalgh wrote:
On Thu, Mar 10, 2016 at 10:32:15AM -0600, Evandro Menezes wrote:
I agree to postpone until GCC 7.
[AArch64] Replace insn to zero up SIMD registers
gcc/
* config/aarch64/aarch64.md
(*movhf_aarch64): Add "movi %0,
Evandro Menezes wrote:
> I assume that you mean that such improvements are true for
> -mcpu=generic, yes? On which target, A53 or A57 or other?
It's true for any CPU setting. The SPEC results are for Cortex-A57
however I wrote a microbenchmark that shows improvements on
all targets I have access
On 04/25/2016 10:08 AM, Andres Tiraboschi wrote:
*gcc/cp/constexpr.c (constexpr_fundef): Moved to gcc/cp/cp-tree.h.
*gcc/cp/constexpr.c (constexpr_call): Ditto.
*gcc/cp/constexpr.c (constexpr_ctx): Ditto.
Let's create a constexpr.h rather than expose constexpr internals to all
o
Hello,
a simple transform to replace a more complicated one in fold-const.c.
This patch breaks the testcase gcc.dg/gomp/loop-1.c. Indeed, the C
front-end folds too eagerly
newrhs = c_fully_fold (newrhs, false, NULL);
in build_modify_expr, and by the time the OMP code checks that the
inc
Hello!
(const_int 0) and (const_int 1) are never const_wide_int.
2016-04-25 Uros Bizjak
* config/i386/predicates.md (const0_operand): Do not match
const_wide_int code.
(const1_operand): Ditto.
Bootstrapped and regression tested on x86_64-linux-gnu {,-m32}.
Committed to mainline
OK.
Jason
On 04/25/16 14:21, Wilco Dijkstra wrote:
Evandro Menezes wrote:
I assume that you mean that such improvements are true for
-mcpu=generic, yes? On which target, A53 or A57 or other?
It's true for any CPU setting. The SPEC results are for Cortex-A57
however I wrote a microbenchmark that shows im
Martin Liška writes:
> #endif
> + /* Do not search original location in the same folder. */
> + char *exe_folder = lrealpath (av[0]);
> + exe_folder[strlen (exe_folder) - strlen (lbasename (exe_folder))] =
> '\0';
> + char *location = concat (exe_folder, PERSONALITY, NULL);
Hi!
As mentioned in the PR, some checking code, in particular the one
in C++ FE's build_non_dependent_expr, may affect code generation, as it can
instantiate templates that aren't instantiated otherwise, which affects
the various counters like cfun->funcdef_no, DECL_UID etc.
I'd like to commit th
On Thu, Apr 21, 2016 at 1:15 AM, Kyrill Tkachov
wrote:
> Jim, you added support for the qdf24xx identifier to -mcpu and -mtune.
> Could you please suggest an appropriate entry to describe it?
> I think the same format as the Cortex-A35 entry in this patch would be
> appropriate.
This is tricky, a
Looks good to me.
Jason
On Mon, Apr 25, 2016 at 3:30 PM, Jakub Jelinek wrote:
> Hi!
>
> As mentioned in the PR, some checking code, in particular the one
> in C++ FE's build_non_dependent_expr, may affect code generation, as it can
> instantiate templates that aren't instantiated otherwise, whic
Uros Bizjak writes:
> On Fri, Apr 22, 2016 at 7:10 PM, Uros Bizjak wrote:
>> On Fri, Apr 22, 2016 at 4:19 PM, H.J. Lu wrote:
>>> On Fri, Apr 22, 2016 at 5:11 AM, Uros Bizjak wrote:
On Thu, Apr 21, 2016 at 10:58 PM, H.J. Lu wrote:
> Here is the updated patch with my standard_sse_c
On Mon, Apr 25, 2016 at 9:45 PM, Richard Sandiford
wrote:
>>> Can you please investigate, what is wrong with all_ones_operand so it
>>> doesn't accept all (-1) operands?
>>
>> Does following work:
>>
>> ;; Return true if operand is a (vector) constant with all bits set.
>> (define_predicate "all_
Hmm, this seems to assume that operator delete itself doesn't do
anything with the object being deleted. This is true of the default
implementation, but I don't see anything in the standard that
prohibits a user-supplied replacement or class-specific deallocation
function from accessing the memory
Evandro Menezes wrote:
> I agree with your assessment, but I'm more curious to understand how
> this change affects code built with the default -mcpu=generic when run
> on both A53 and A57, the typical configuration of big.LITTLE machines.
I wouldn't expect the result to be any different as the -m
On 04/25/16 14:58, Wilco Dijkstra wrote:
Evandro Menezes wrote:
I agree with your assessment, but I'm more curious to understand how
this change affects code built with the default -mcpu=generic when run
on both A53 and A57, the typical configuration of big.LITTLE machines.
I wouldn't expect th
Bernd Schmidt writes:
> (Apologies if you get this twice, the mailing list didn't like the html
> attachment in the first attempt).
>
> We frequently get malformatted patches, and it's been brought to my
> attention that some people don't even make the effort to read the GNU
> coding standards
On 11 April 2016 at 20:09, Prasad Ghangal wrote:
>
> Hi!
>
> This is proposed patch for
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=48116 (-Wreturn-type does
> not work as advertised)
>
> Currently gcc doesn't give any warning with -Wreturn-type or -Wall
> option for test cases like :
>
> void
1 - 100 of 106 matches
Mail list logo
