On Tue, 6 Mar 2018, Jakub Jelinek wrote:
> On Tue, Mar 06, 2018 at 02:28:40AM -0300, Alexandre Oliva wrote:
> > On Mar 5, 2018, Jakub Jelinek wrote:
> >
> > > * tree.c (stabilize_reference_1): Return save_expr (e) for
> > > STATEMENT_LIST even if it doesn't have side-effects.
> >
> > I'm c
On Sat, 3 Mar 2018, Martin Sebor wrote:
> On 03/02/2018 01:05 AM, Richard Biener wrote:
> > On Thu, 1 Mar 2018, Martin Sebor wrote:
> >
> > > While testing my recent changes to the handling of attributes
> > > on C++ templates I noticed that the -finline-limit=N option
> > > is not recognized in
Hi Steve,
For the mangled code in the new testcase, gfortran issues
a somewhat obtuse error message. The problem is the matcher
for a complex entity runs prior to the matcher for an implied
do-loop. Errors are queued in that order so an error messagei
for a mangled complex constant is emitted.
On Tue, Mar 06, 2018 at 03:13:11AM -0300, Alexandre Oliva wrote:
> On Mar 2, 2018, Alexandre Oliva wrote:
>
> > Mark Wielaard is implementing support for LVU and IEPM in elfutils, and
> > he was surprised by the encoding of DW_AT_GNU_entry_view; so was I!
> > When GCC computes and outputs views
On Tue, Mar 06, 2018 at 02:28:40AM -0300, Alexandre Oliva wrote:
> On Mar 5, 2018, Jakub Jelinek wrote:
>
> > * tree.c (stabilize_reference_1): Return save_expr (e) for
> > STATEMENT_LIST even if it doesn't have side-effects.
>
> I'm concerned about codegen differences in case the state
On March 5, 2018 9:40:28 PM GMT+01:00, Jakub Jelinek wrote:
>Hi!
>
>This patch clears DECL_BUILT_IN on simd clones, similarly how
>cgraphclones.c
>does:
> /* When signature changes, we need to clear builtin info. */
> if (DECL_BUILT_IN (new_decl)
> && args_to_skip
> && !bitmap_empty_p
On Mar 2, 2018, Jason Merrill wrote:
>> + gcc_assert (TREE_CODE (type) == REFERENCE_TYPE);
>> + init = fold (convert (type, integer_zero_node));
> Maybe build_zero_cst?
> OK either way.
Here's what I'm installing:
[PR c++/84593] ice on braced init with uninit ref field
If an initi
On Mar 2, 2018, Alexandre Oliva wrote:
> Mark Wielaard is implementing support for LVU and IEPM in elfutils, and
> he was surprised by the encoding of DW_AT_GNU_entry_view; so was I!
> When GCC computes and outputs views internally (broken without internal
> view resets), it outputs entry_view a
On Mar 2, 2018, Jason Merrill wrote:
> Let's add a comment along the lines of
> /* Let lvalue_kind know this was a glvalue. */
> OK with that change.
Thanks, here's what I'm about to check in.
[C++] [PR84231] overload on cond_expr in template
A non-type-dependent COND_EXPR within a templat
On Mar 2, 2018, Jason Merrill wrote:
> On Fri, Mar 2, 2018 at 2:57 AM, Alexandre Oliva wrote:
>> + gcc_assert (TREE_CODE (type) == REFERENCE_TYPE);
>> + init = fold (convert (type, integer_zero_node));
> Maybe build_zero_cst?
Sure.
I wonder, is there any reason to not change any o
On Mar 5, 2018, Jakub Jelinek wrote:
> * tree.c (stabilize_reference_1): Return save_expr (e) for
> STATEMENT_LIST even if it doesn't have side-effects.
I'm concerned about codegen differences in case the statement list is
created because of debug stmts not present in a nondebug com
On 03/05/2018 12:30 PM, Michael Matz wrote:
> Hi,
>
> On Mon, 5 Mar 2018, Jeff Law wrote:
>
The single successor test was strictly my paranoia WRT abnormal/EH
edges.
I don't immediately see why the CFG would be incorrect if the
successor of the setjmp block has multiple
Hi!
On the following testcase on powerpc64-linux -m32 we end up calling
combine_simplify_rtx on
(plus:SI (ltu:SI (plus:SI (if_then_else:SI (eq (reg:CC 147)
(const_int 0 [0]))
(subreg:SI (reg:DI 126) 4)
(reg:SI 146))
(subreg:SI (reg:DI
On 02/23/2018 07:32 PM, Jason Merrill wrote:
On Sun, Feb 18, 2018 at 11:39 PM, Jason Merrill wrote:
On Fri, Feb 16, 2018 at 4:33 PM, Martin Sebor wrote:
On 02/16/2018 07:04 AM, Jason Merrill wrote:
On Thu, Feb 15, 2018 at 6:36 PM, Martin Sebor wrote:
A failed template deduction in templa
Hi!
This patch clears DECL_BUILT_IN on simd clones, similarly how cgraphclones.c
does:
/* When signature changes, we need to clear builtin info. */
if (DECL_BUILT_IN (new_decl)
&& args_to_skip
&& !bitmap_empty_p (args_to_skip))
{
DECL_BUILT_IN_CLASS (new_decl) = NOT_BUIL
Looking for fake NSDMI 'this' in scope_chain only works for non-local
classes. But if we can't find it, we can make our own fake.
Tested x86_64-pc-linux-gnu, applying to trunk.
commit 8fd062c984befc3419f8a0a61224d0f4ecf02495
Author: Jason Merrill
Date: Mon Mar 5 16:45:36 2018 -0500
OK.
On Mon, Mar 5, 2018 at 3:31 PM, Jakub Jelinek wrote:
> Hi!
>
> This doesn't actually fix this PR (Marek is working on that), but
> just something I've noticed while analyzing the PR.
> We have the hashes saved in the structure (to speed up hash table
> expansion), so it is a waste not to test
* Claudiu Zissulescu [2018-02-20 15:38:32 +0200]:
> From: Claudiu Zissulescu
>
> Cleanup unsed functions and macros.
>
> OK to apply?
> Claudiu
Looks good. Nice cleanup.
Thanks,
Andrew
>
> gcc/
> 2018-01-26 Claudiu Zissulescu
>
> * config/arc/arc.c (arc_finalize_pic): Remove fu
Hi!
The following testcase ICEs, because ARRAY_REF on lhs of op=
has a statement list with no side-effects as index (containing
DEBUG_BEGIN_STMT and integer_zero_node), stabilize_reference
does nothing to it (as it has no side-effects), then the array ref
is unshared in unshare_body (mostly_copy_t
Attribute nonstring is currently only allowed on arrays and
pointers to plain char, but -Wstringop-truncation triggers
even for strncpy calls whose arguments are arrays of signed
or unsigned char (with or without a cast to char*).
To help deal with -Wstringop-truncation in the Linux kernel
it was
Hi!
If we discover some bad inline-asm during reg-stack processing and we
error on those, we replace that inline-asm with a (use (const_int 0))
and therefore the various assumptions of reg-stack pass may not hold.
Seems we already have a couple of spots which are more permissive if
any_malformed_a
Hi!
This doesn't actually fix this PR (Marek is working on that), but
just something I've noticed while analyzing the PR.
We have the hashes saved in the structure (to speed up hash table
expansion), so it is a waste not to test those also in the equal hook,
by giving up cheaply in cases of hash t
Hi,
On Mon, 5 Mar 2018, Jeff Law wrote:
> >> The single successor test was strictly my paranoia WRT abnormal/EH
> >> edges.
> >>
> >> I don't immediately see why the CFG would be incorrect if the
> >> successor of the setjmp block has multiple preds.
> >
> > Actually, without further condition
On 03/02/2018 08:12 AM, Andrew Sadek wrote:
Hello Michael,
I tried running the whole GCC test suite on the current head (without my
patch) along with 'microblaze-qemu' but I have the following problems:
1) The test is hanging at 'gcc.c-torture/string-large-1.c' , the gcc is
making a 100% CPU
For POWER4..POWER8 we align loops of 5..8 instructions to 32 bytes
(instead of to 16 bytes) because that executes faster. This is no
longer the case on POWER9, so we can just as well only align to 16
bytes.
Bootstrapped and tested on a p9 powerpc64le-linux. Committing to trunk.
Segher
2018-0
For the mangled code in the new testcase, gfortran issues
a somewhat obtuse error message. The problem is the matcher
for a complex entity runs prior to the matcher for an implied
do-loop. Errors are queued in that order so an error messagei
for a mangled complex constant is emitted.
Regression
On 03/05/2018 11:30 AM, Michael Matz wrote:
> Hi,
>
> On Wed, 28 Feb 2018, Jeff Law wrote:
>
>> The single successor test was strictly my paranoia WRT abnormal/EH edges.
>>
>> I don't immediately see why the CFG would be incorrect if the successor
>> of the setjmp block has multiple preds.
>
> A
On Fri, 2018-02-23 at 08:54 -0600, Joshua Watt wrote:
> On Mon, 2018-02-19 at 17:24 -0800, Cary Coutant wrote:
> > Please see this patch posted to the binutils list:
> >
> >https://sourceware.org/ml/binutils/2018-02/msg00260.html
> >
> > where Joshua proposes to add the ax_pthread.m4 script,
This libgo patch by Than McIntosh fixes a small typo in the
mksysinfo.sh script (incorrect input file for a grep command).
Bootstrapped and ran Go testsuite on x86_64-pc-linux-gnu. Committed
to mainline.
Ian
Index: gcc/go/gofrontend/MERGE
==
This fixes an ICE where we were not correctly marking a lookup as kept,
resulting in an assertion failure later on.
nathan
--
Nathan Sidwell
2018-03-05 Nathan Sidwell
PR c++/84702
* pt.c (process_template_arg): Mark lookup_keep on a default arg.
PR c++/84702
* g++.dg/lookup/pr84702.C: N
Hi!
I'd like to ping following patch:
http://gcc.gnu.org/ml/gcc-patches/2018-02/msg01461.html
PR84564 - fix ICE with -mforce-indirect-call
Thanks
Jakub
Hi,
On Wed, 28 Feb 2018, Jeff Law wrote:
> The single successor test was strictly my paranoia WRT abnormal/EH edges.
>
> I don't immediately see why the CFG would be incorrect if the successor
> of the setjmp block has multiple preds.
Actually, without further conditions I don't see how it woul
Hi All,
Taking the subreg of a vector mode on big-endian may result in an infinite
recursion and eventually a segfault once we run out of stack space.
As an example, taking a subreg of V4HF to SImode we end up in the following
loop on big-endian:
#861 0x008462e9 in operand_subword_force
On Mon, Mar 05, 2018 at 09:58:50AM -0600, Bill Schmidt wrote:
>
> > On Mar 5, 2018, at 9:04 AM, Segher Boessenkool
> > wrote:
> >
> > Hi!
> >
> > On Sun, Mar 04, 2018 at 01:52:20PM -0600, Bill Schmidt wrote:
> >> We realized recently that the use of "rs6000" in a builtin name doesn't
> >> agr
Dne 2018-03-05 17:13, Jakub Jelinek napsal:
Hi!
I'd like to ping following patch:
http://gcc.gnu.org/ml/gcc-patches/2018-02/msg01461.html
PR84564 - fix ICE with -mforce-indirect-call
OK,
thanks!
Honza
Thanks
Jakub
This fixes an ICE with a friend decl. Although the reported source is
invalid, we can turn it into valid, but useless, source. ISTM that we
should be generating a raw identifier_node here, but finding the
TEMPLATE_DECL at parse time. But that's a change for another day. This
restores the pr
On 03/05/2018 09:47 AM, Marek Polacek wrote:
Since Nathan's r253489 we seem to not use anon_identifier anymore; rather, the
DECL_NAME is simply NULL. This crashed in duplicate_decls on this invalid code
because UDLIT_OPER_P was blithely used on a possibly null tree. Other spots in
this function
> On Mar 5, 2018, at 9:04 AM, Segher Boessenkool
> wrote:
>
> Hi!
>
> On Sun, Mar 04, 2018 at 01:52:20PM -0600, Bill Schmidt wrote:
>> We realized recently that the use of "rs6000" in a builtin name doesn't agree
>> with our normal naming conventions. Thus this patch changes such a builtin
>>
Hello Jakub,
On Friday, March 2, 2018, Jakub Jelinek wrote:
> Hi!
>
> I'd like to ping 2 patches:
>
> http://gcc.gnu.org/ml/gcc-patches/2018-02/msg01340.html
> - PR target/84524 avx512* wrong-code bug
Patch is OK.
>
> http://gcc.gnu.org/ml/gcc-patches/2018-02/msg01337.html
> - fix c-c++-c
On Thu, Feb 8, 2018 at 1:41 AM, Kugan Vivekanandarajah
wrote:
> Hi Richard,
>
> On 1 February 2018 at 23:21, Richard Biener
> wrote:
>> On Thu, Feb 1, 2018 at 5:07 AM, Kugan Vivekanandarajah
>> wrote:
>>> Hi Richard,
>>>
>>> On 31 January 2018 at 21:39, Richard Biener
>>> wrote:
On Wed,
Hi!
On Sun, Mar 04, 2018 at 01:52:20PM -0600, Bill Schmidt wrote:
> We realized recently that the use of "rs6000" in a builtin name doesn't agree
> with our normal naming conventions. Thus this patch changes such a builtin
> to __builtin_powerpc_speculation_barrier instead.
>
> Bootstrapped and
The following fixes a missed optimization caused by code hoisting.
When PRE inserts expressions somewhere the SSA names created in that
process do not have any associated info like alignment.
Fixed for the very specific case of __builtin_assume_aligned.
Bootstrapped and tested on x86_64-unknown-
Hi Peter,
On Sat, Mar 03, 2018 at 10:55:28PM -0600, Peter Bergner wrote:
> In PR84264, we hit an assert in rs6000_emit_le_vsx_store causing an ICE
> in LRA. We get there, because LRA called the movv4si expander to generate
> a spill and the mov pattern calls rs6000_emit_le_vsx_move which in turn
Since Nathan's r253489 we seem to not use anon_identifier anymore; rather, the
DECL_NAME is simply NULL. This crashed in duplicate_decls on this invalid code
because UDLIT_OPER_P was blithely used on a possibly null tree. Other spots in
this function check this, too.
Bootstrapped/regtested on x8
On Fri, Mar 2, 2018 at 3:12 PM, Richard Sandiford
wrote:
> Richard Biener writes:
>> On Thu, Mar 1, 2018 at 12:38 PM, Richard Sandiford
>> wrote:
>>> Richard Biener writes:
On Wed, Feb 28, 2018 at 3:20 PM, Richard Sandiford
wrote:
> GCC 6 and 7 would vectorise:
>
> void
>
OK.
On Sun, Mar 4, 2018 at 8:55 PM, Paolo Carlini wrote:
> Hi,
>
> a rather simple ice on invalid (not sure why only P4 given that no
> meaningful diagnostic is emitted before ICEing). What happens is that the
> ill-formed capture naming b, an OVERLOAD, escapes the early check in
> cp_parser_lamb
On Fri, Mar 2, 2018 at 11:18 PM, Jeff Law wrote:
> On 02/28/2018 03:43 AM, Richard Biener wrote:
> [ More snipping ]
>
>>
>>> It's actually pretty easy to fix the CFG. We just need to recognize
>>> that a "returns twice" function returns not to the call, but to the
>>> point immediately after th
On 03/02/2018 01:43 PM, Jason Merrill wrote:
On Fri, Mar 2, 2018 at 11:07 AM, Nathan Sidwell wrote:
NEEDS_CONSTRUCTING && !HAS_CONSTEXPR_CTOR && !HAS_DEFAULT_CONSTRUCTOR seems
like it would be sufficient. and indeed that works in this case.
Do you mean !HAS_TRIVIAL_DFLT rather than !HAS_DEF
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied.
Richard.
2018-03-05 Richard Biener
PR tree-optimization/84650
* tree-ssa-loop-im.c (pass_lim::execute): Reset the SCEV cache
if executed in the loop pipeline.
* gcc.dg/graphite/pr84650.c: New testc
On Tue, Feb 27, 2018 at 11:39 AM, H.J. Lu wrote:
> For x86 targets, when -fno-plt is used, external functions are called
> via GOT slot, in 64-bit mode:
>
> [bnd] call/jmp *foo@GOTPCREL(%rip)
>
> and in 32-bit mode:
>
> [bnd] call/jmp *foo@GOT[(%reg)]
>
> With -mindirect-branch=, t
On Mon, Feb 26, 2018 at 12:48 PM, H.J. Lu wrote:
> Function return thunks shouldn't be aliased to indirect branch thunks
> since indirect branch thunks are placed in COMDAT section and a COMDAT
> section with indirect branch may not have function return thunk. This
> patch generates function retu
The following fixes a thinko in the assert I added. It's true that
the value-set of ANTIC_IN shouldn't grow during iteration but there's
an exception for when the successors we intersect are not all visited
(recursively). For example when the latch wasn't visited and thus
its ANTIC_IN is first c
The compiler builtin will use the hardware instruction cdsg if the
memory operand is properly aligned and will fall back to the
library call otherwise.
In case the compiler for one part is able to detect that the
location is aligned and fails to do so for another usage of the hw
instruction and the
53 matches
Mail list logo
