Hi,
this is a regression present on the mainline and 6 branch: the compiler now
generates wrong code for the attached testcase at -O because of an internal
conflict about boolean types. The sequence is as follows. In .mergephi3:
boolean _22;
p__enum res;
:
if (_22 != 0)
goto ;
On Mon, Oct 17, 2016 at 5:26 AM, Richard Biener wrote:
>
> $subject, applied as obvious.
I think you should do the same for the vectorizer too. I noticed that
when testing the patch for loop splitting.
Thanks,
Andrew
>
> Richard.
>
> 2016-10-17 Richard Biener
>
> * bb-reorder.c (reo
On October 17, 2016 6:09:02 PM GMT+02:00, Thomas Schwinge
wrote:
>Hi!
>
>On Mon, 17 Oct 2016 15:17:13 +0200, Richard Biener
> wrote:
>> On Mon, Oct 17, 2016 at 3:12 PM, Thomas Schwinge
>> wrote:
>> > On Mon, 17 Oct 2016 14:26:42 +0200 (CEST), Richard Biener
> wrote:
>> >> --- gcc/bb-reorder.c (
On 18 October 2016 at 02:46, Jeff Law wrote:
> On 10/15/2016 11:59 PM, Prathamesh Kulkarni wrote:
>>
>> This patch is mostly the same as previous one, except it drops
>> targeting __udivmoddi4() because it gave undefined reference link
>> error for calling __udivmoddi4() on aarch64-linux-gnu. It a
On 2016.10.18 at 05:13 +0200, Markus Trippelsdorf wrote:
> On 2016.10.17 at 17:23 -0500, Bill Schmidt wrote:
> > Hi,
> >
> > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77916 identifies a situation
> > where SLSR will ICE when exposed to a cast from integer to pointer. This
> > is because we try
On 2016.10.17 at 17:23 -0500, Bill Schmidt wrote:
> Hi,
>
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77916 identifies a situation
> where SLSR will ICE when exposed to a cast from integer to pointer. This
> is because we try to convert a PLUS_EXPR with an addend of -1 * S into a
> MINUS_EXPR
On Mon, Oct 17, 2016 at 06:02:52PM -0700, Jerry DeLisle wrote:
> Hi all,
>
> The attached patch enables the size= specifier in a READ statement to work
> with
> child DTIO procedures. This is accomplished by moving the size_used variable
> from the dtp structure to the gfc_unit structure so tha
2016-05-24 Max Filippov
gcc/
* config/xtensa/xtensa.opt (mpreferred-stack-boundary=): New
option.
* config/xtensa/xtensa.h (STACK_BOUNDARY): Redefine as 64 for
windowed ABI/32 for call0 ABI.
(PREFERRED_STACK_BOUNDARY): New definition.
* config/xten
Hi all,
The attached patch enables the size= specifier in a READ statement to work with
child DTIO procedures. This is accomplished by moving the size_used variable
from the dtp structure to the gfc_unit structure so that the accumulation of
bytes during READ is carried across the procedures v
Define LIB2ADDEH_XTENSA_UNWIND_DW2_FDE to unwind-dw2-fde.c in
xtensa/t-elf and to unwind-dw2-fde-dip.c in xtensa/t-linux and use
LIB2ADDEH_XTENSA_UNWIND_DW2_FDE in LIB2ADDEH definition.
2016-10-17 Max Filippov
libgcc/
* config/xtensa/t-elf (LIB2ADDEH_XTENSA_UNWIND_DW2_FDE): New
Hi Segher,
> On 17 Oct 2016, at 09:13, Segher Boessenkool
> wrote:
>
> We cannot use {SAVE,REST}_MULTIPLE and separate shrink-wrapping together,
> not without checking when actually emitting the prologue/epilogue that the
> registers to save/restore are actually still one contiguous block up to
Whoops. Discovered I had inadvertently left in a comment that made no
sense when fixing PR77916. This patch removes it. Committed as obvious.
Thanks,
Bill
2016-10-17 Bill Schmidt
* gimple-ssa-strength-reduction.c (record_increment): Remove
garbage comment.
Index: gcc/gim
> On ia64 I get this regression:
>
> FAIL: gcc.dg/Wtrampolines.c (test for warnings, line 31)
>
> Since ia64 never uses trampolines this is probably ok and the test
> should be adjusted.
This actually revealed a problem: the same regression should have appeared on
PowerPC64/Linux (ELFv1 ABI) b
Hi,
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77916 identifies a situation
where SLSR will ICE when exposed to a cast from integer to pointer. This
is because we try to convert a PLUS_EXPR with an addend of -1 * S into a
MINUS_EXPR with a subtrahend of S, but the base operand is unexpectedly
o
On 10/17/2016 01:11 PM, Jeff Law wrote:
On 10/02/2016 02:10 PM, Martin Sebor wrote:
The attached patch fixes a number of outstanding test failures
and ILP32-related bugs in the gimple-ssa-sprintf pattch pointed
out in bug 77676 and 77735). The patch also fixes c_strlen to
correctly handle wid
Hi!
This patch starts using DW_FORM_implicit_const (new in DWARF5), though only
for cases where the constants are the same among all DIEs with the same
abbreviation (and only for abbrevs used only in the main compilation unit,
because right now we don't give all DIEs abbreviations first before
out
On Oct 17, 2016, at 2:38 PM, Ximin Luo wrote:
>
> Mike Stump:
>> On Oct 17, 2016, at 11:00 AM, Ximin Luo wrote:
>>> Therefore, it is better to emit it in all circumstances, in case the reader
>>> needs to know what the working
>>> directory was at compile-time.
>>
>> I can't help but wonder if
Hi!
DWARF5 has a new 128-bit constant class form, this patch uses it
e.g. for __int128 values.
Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk?
2016-10-17 Jakub Jelinek
* dwarf2out.c (size_of_die, value_format, output_die): Use
DW_FORM_data16 for 128-bit d
Mike Stump:
> On Oct 17, 2016, at 11:00 AM, Ximin Luo wrote:
>> Therefore, it is better to emit it in all circumstances, in case the reader
>> needs to know what the working
>> directory was at compile-time.
>
> I can't help but wonder if this would break ccache some?
>
Could you explain this
Hi!
This is on top of the
http://gcc.gnu.org/ml/gcc-patches/2016-10/msg01202.html
patch. This is yet another GNU extension that got accepted into DWARF5,
though in this case various attributes got renamed (other than just dropping
GNU_ middle parts), or extra attributes have been added instead of
On Oct 17, 2016, at 11:00 AM, Ximin Luo wrote:
> Therefore, it is better to emit it in all circumstances, in case the reader
> needs to know what the working
> directory was at compile-time.
I can't help but wonder if this would break ccache some?
On 10/15/2016 11:59 PM, Prathamesh Kulkarni wrote:
This patch is mostly the same as previous one, except it drops
targeting __udivmoddi4() because it gave undefined reference link
error for calling __udivmoddi4() on aarch64-linux-gnu. It appears
aarch64 has hardware insn for DImode div, so __udiv
David Malcolm writes:
>> I assume you didn't turn the functions above into member functions
>> because they aren't primitives. OTOH it might seem inconsistent
>> to have things like traverse_enum_types as member functions but
>> traverse_md_constants not.
>
> In the following I've converted trave
On Wed, Sep 21, 2016 at 1:53 AM, Richard Biener
wrote:
> On Wed, Sep 21, 2016 at 10:45 AM, Andrew Pinski wrote:
>> On Wed, Sep 21, 2016 at 4:32 PM, Richard Biener
>> wrote:
>>> On Wed, Sep 21, 2016 at 9:13 AM, Andrew Pinski wrote:
Hi,
While looking through bug reports, I noticed som
From: Trevor Saunders
gcc/ChangeLog:
2016-10-17 Trevor Saunders
* ccmp.c (expand_ccmp_expr_1): Adjust.
(expand_ccmp_expr): Likewise.
(expand_ccmp_next): Likewise.
* config/aarch64/aarch64.c (aarch64_gen_ccmp_next): Likewise.
(aarch64_gen_ccmp_first): L
From: Trevor Saunders
gcc/ChangeLog:
2016-10-17 Trevor Saunders
* cfgrtl.c (delete_insn_chain): Change argument type to rtx_insn *
and adjust for that.
* cfgrtl.h (delete_insn_chain): Adjust prototype.
---
gcc/cfgrtl.c | 8 +++-
gcc/cfgrtl.h | 2 +-
2 files chang
From: Trevor Saunders
gcc/ChangeLog:
2016-10-17 Trevor Saunders
* config/alpha/alpha.c (alpha_legitimize_address_1): Change
variable types from rtx to rtx_insn *.
(alpha_emit_xfloating_libcall): Likewise.
* config/arc/arc.c (arc_emit_call_tls_get_addr): Likewi
From: Trevor Saunders
While chainging LABEL_REF_LABEL it might as well become an inline
function, so that its clearer what types are involved. Unfortunately
because it is still possible to use XEXP and related macros on a
LABEL_REF rtx you can still set the field to be a non insn rtx. The
other
From: Trevor Saunders
gcc/ChangeLog:
2016-10-17 Trevor Saunders
* emit-rtl.c (prev_nonnote_insn_bb): Change argument type to
rtx_insn *.
* rtl.h (prev_nonnote_insn_bb): Adjust prototype.
---
gcc/emit-rtl.c | 3 +--
gcc/rtl.h | 2 +-
2 files changed, 2 insertions
From: Trevor Saunders
gcc/ChangeLog:
2016-10-17 Trevor Saunders
* cfgcleanup.c (merge_blocks_move_successor_nojumps): Adjust.
(outgoing_edges_match): Likewise.
(try_crossjump_to_edge): Likewise.
* cfgrtl.c (try_redirect_by_replacing_jump): Likewise.
(r
From: Trevor Saunders
gcc/ChangeLog:
2016-10-17 Trevor Saunders
* config/rl78/rl78.c (gen-and_emit_move): Change argument type
to rtx_insn *.
(transcode_memory_rtx): Likewise.
(move_to_acc): Likewise.
(move_from_acc): Likewise.
(move_acc_to_reg
From: Trevor Saunders
Hi,
Rather late (travel and then busy day job interfiered), but this includes the
promised conversion of LABEL_REF_LABEL and tablejump_p to use rtx_insn *.
Followed by a bit more random conversion of things to use rtx_insn *.
patches individually bootstrapped + regtested
> Here is a patch implementing what I think has been discussed in this
thread.
>
> OK?
Looks good to me.
Uli, do you agree with that change for S/390 or would you rather see us
fixing the float_t type definition in Glibc?
-Andreas-
On 10/17/2016 01:12 PM, Adhemerval Zanella wrote:
This test adds the expect va_end after va_start on split-stack tests.
gcc/ChangeLog:
* gcc/testsuite/gcc.dg/split-3.c (down): Call va_end after va_start.
* gcc/testsuite/gcc.dg/split-6.c (down): Likewise.
OK. Would probably fal
On 10/08/2016 04:50 PM, Trevor Saunders wrote:
On Fri, Oct 07, 2016 at 03:18:07PM +, Bernd Edlinger wrote:
Hi!
This extends -Wint-in-bool-context to uses of enum values in boolean
context, and fixes one place where accidentally an enum value was
passed to a bool parameter.
I excluded enum
On 10/12/2016 01:08 AM, Richard Biener wrote:
On Fri, 7 Oct 2016, Richard Biener wrote:
This turns the switch (which also requires propagating into ASSERT_EXPRs,
otherwise those will end up with released SSA names eventually).
A [3/2] would be to let ASSERT_EXPRs be removed by the propagator
On 10/17/16 20:05, Joseph Myers wrote:
> On Sun, 16 Oct 2016, Bernd Edlinger wrote:
>
>> Second, the declaration in the glibc header files simply look wrong,
>> because the type of argv, and envp is "char *const *" while the
>> builtin function wants "const char**", thus only the array of char*
>>
From: Adhemerval Zanella
Changes from previous version:
* Add missing new function comments;
* Using gen_rtx_fmt_ee instead of gen_rtx_IF_THEN_ELSE to create
conditional jumps;
* Some typos and withspace issues.
--
This patch adds the split-stack support on aarch64 (PR #67877). As for
o
This test adds the expect va_end after va_start on split-stack tests.
gcc/ChangeLog:
* gcc/testsuite/gcc.dg/split-3.c (down): Call va_end after va_start.
* gcc/testsuite/gcc.dg/split-6.c (down): Likewise.
---
gcc/ChangeLog | 5 +
gcc/testsuite/gcc.dg/split-3.
On 10/16/2016 04:53 AM, Marek Polacek wrote:
Found when looking at something else. find_array_ref_with_const_idx_r would
uselessly keep on looking for other array refs with constant indices, even
though finding one is enough. So return when something is found, instead of
just ignoring the subtr
On 10/02/2016 02:10 PM, Martin Sebor wrote:
The attached patch fixes a number of outstanding test failures
and ILP32-related bugs in the gimple-ssa-sprintf pattch pointed
out in bug 77676 and 77735). The patch also fixes c_strlen to
correctly handle wide strings (previously it accepted them but
Hi David,
On Mon, Oct 17, 2016 at 09:54:35AM -0400, David Edelsohn wrote:
> This patch shifts earlier the point during compilation where the
> rs6000 backend adds decoration to symbols: to encode_section_info.
> - /* Currently C++ toc references to vtables can be emitted before it
> - is dec
> On Oct 17, 2016, at 12:26 PM, Bin.Cheng wrote:
>
> On Sat, Oct 15, 2016 at 3:07 AM, kugan
> wrote:
>> Hi Bin,
>>
>> On 15/10/16 00:15, Bin Cheng wrote:
>>>
>>> +/* Test for likely overcommitment of vector hardware resources. If a
>>> + loop iteration is relatively large, and too large a
On Mon, Oct 17, 2016 at 2:55 AM, Richard Earnshaw (lists)
wrote:
> On 16/10/16 22:39, Andrew Pinski wrote:
>> Hi,
>> These testcases use long for 64bit integer which means they will
>> fail with -mabi=ilp32 on aarch64. This reduces the number of failures
>> down for ILP32.
>>
>> OK? Bootstrappe
On Sun, 16 Oct 2016, Bernd Edlinger wrote:
> Second, the declaration in the glibc header files simply look wrong,
> because the type of argv, and envp is "char *const *" while the
> builtin function wants "const char**", thus only the array of char*
> itself is const, not the actual char stings th
On 10/03/2016 09:45 AM, Eric Botcazou wrote:
Ping: https://gcc.gnu.org/ml/gcc-patches/2016-09/msg01781.html
2016-09-26 Eric Botcazou
* expmed.c (expand_shift_1): Add MAY_FAIL parameter and do not assert
that the result is non-zero if it is true.
(maybe_expand_shift):
(Please keep me on CC, I am not subscribed)
In working on another patch for DW_AT_comp_dir behaviour, I tried to write a
test for that other patch. This test did not work as expected - DW_AT_comp_dir
was not even generated in the resulting output! But this had nothing to do with
my patch so I dug
Dear All,
Committed as 'obvious' on trunk as revision 241274.
I will commit to 5- and 6-branches at the end of the week.
Cheers
Paul
2016-10-17 Paul Thomas
PR fortran/61420
PR fortran/78013
* resolve.c (resolve_variable): Obtain the typespec for a
variable expression, when
On 2016.10.17 at 17:30 +, Bernd Edlinger wrote:
> On 10/17/16 19:11, Markus Trippelsdorf wrote:
> >>> I'm seeing this warning a lot in valid low level C code for unsigned
> >>> integers. And I must say it look bogus in this context. Some examples:
> >
> > (All these examples are from qemu trunk
Hi!
I'd like to ping the
[C++ PATCH] Fix -Wimplicit-fallthrough in templates (PR c++/77886)
patch:
On Sat, Oct 08, 2016 at 08:13:50AM +0200, Jakub Jelinek wrote:
> 2016-10-08 Jakub Jelinek
>
> PR c++/77886
> * pt.c (tsubst_expr) Copy over
> FALLTHROUGH_LABEL_P flag to the n
On 10/17/16 19:11, Markus Trippelsdorf wrote:
>>> I'm seeing this warning a lot in valid low level C code for unsigned
>>> integers. And I must say it look bogus in this context. Some examples:
>
> (All these examples are from qemu trunk.)
>
>>> return ((a.high & 0x7fff) == 0x7fff) && (a.low<<1);
On Sat, Oct 15, 2016 at 3:07 AM, kugan
wrote:
> Hi Bin,
>
> On 15/10/16 00:15, Bin Cheng wrote:
>>
>> +/* Test for likely overcommitment of vector hardware resources. If a
>> + loop iteration is relatively large, and too large a percentage of
>> + instructions in the loop are vectorized, the
On 2016.10.17 at 16:51 +, Bernd Edlinger wrote:
> On 10/17/16 17:23, Markus Trippelsdorf wrote:
> > On 2016.09.29 at 18:52 +, Bernd Edlinger wrote:
> >> On 09/29/16 20:03, Jason Merrill wrote:
> >>> On Wed, Sep 28, 2016 at 12:09 PM, Bernd Edlinger
> >>> wrote:
> On 09/28/16 16:41, Jas
On Mon, 17 Oct 2016, Bernd Schmidt wrote:
> On 10/14/2016 06:39 PM, Alexander Monakov wrote:
> > I'm resending the patch series with backend prerequisites for OpenMP
> > offloading to the NVIDIA PTX ISA. The patches are rebased on trunk.
>
> What's the status of the branch? Is it expected to work
On Sat, Oct 15, 2016 at 01:07:12PM +1100, kugan wrote:
> Hi Bin,
>
> On 15/10/16 00:15, Bin Cheng wrote:
> >+/* Test for likely overcommitment of vector hardware resources. If a
> >+ loop iteration is relatively large, and too large a percentage of
> >+ instructions in the loop are vectorized
Richard Biener writes:
> On Mon, Oct 17, 2016 at 12:21 PM, Senthil Kumar Selvaraj
> wrote:
>> Hi,
>>
>> The fix for PR 52085 went into trunk when trunk was 6.0. I ran into the
>> same issue on a gcc 5.x and found that the fix didn't get backported.
>>
>> Bootstrapped and reg tested below p
This patch to libgo copies the rdebug code from the Go 1.7 runtime to libgo.
While we're at it, this updates the runtime/debug package, and starts
running its testsuite by default. I'm not sure why runtime/debug was
not previously updated to 1.7. Doing that led me to fix some minor
aspects of ru
On 10/17/16 17:23, Markus Trippelsdorf wrote:
> On 2016.09.29 at 18:52 +, Bernd Edlinger wrote:
>> On 09/29/16 20:03, Jason Merrill wrote:
>>> On Wed, Sep 28, 2016 at 12:09 PM, Bernd Edlinger
>>> wrote:
On 09/28/16 16:41, Jason Merrill wrote:
> On Tue, Sep 27, 2016 at 11:10 AM, Bernd
On 30/09/16 14:34, Bernd Edlinger wrote:
On 09/30/16 12:14, Bernd Edlinger wrote:
Eric Botcazou wrote:
A comment before the SETs and a testcase would be nice. IIRC
we do have stack size testcases via using -fstack-usage.
Or -Wstack-usage, which might be more appropriate here.
Yes. good ide
I'm still looking for a review of the patch below:
https://gcc.gnu.org/ml/gcc-patches/2016-10/msg00043.html
The patch should clean up the remaining test suite failures on
ILP32 targets and also fixes up some remaining issues in the
gimple-ssa-sprintf pass that stand in the way of re-enabling
On Thu, Oct 13, 2016 at 06:14:16PM +0100, Kyrill Tkachov wrote:
> Hi all,
>
> This patch moves the aarch64-specific FMA steering pass registration into the
> new framework
> that Jakub introduced. With this patch the RTL dump for the steering pass is
> now numbered properly
> so that it appears
On Mon, 2016-10-17 at 12:37 +0100, Richard Sandiford wrote:
> David Malcolm writes:
> > @@ -2388,8 +2389,10 @@ gen_mnemonic_setattr (htab_t mnemonic_htab,
> > rtx insn)
> > static int
> > mnemonic_htab_callback (void **slot, void *info ATTRIBUTE_UNUSED)
> > {
> > - obstack_grow (&string_obstac
On 10/17/2016 08:17 AM, Maxim Kuvyrkov wrote:
>> The patch here, https://gcc.gnu.org/ml/gcc-patches/2014-10/msg01872.html,
>> attempted to scale down the register limit used by -fsched-pressure for the
>> case where the block in question executes as frequently as the entry block
>> to just the c
Hi all,
For the attached testcase the code ends up trying to extract bits outside the
range of the normal register
widths. The aarch64 patterns for ubfz and tbnz end up accepting such operands
and emitting invalid assembly
such as 'ubfx x18,x2,192,32'
The solution is to add proper predicates a
Hi!
On Mon, 17 Oct 2016 15:17:13 +0200, Richard Biener
wrote:
> On Mon, Oct 17, 2016 at 3:12 PM, Thomas Schwinge
> wrote:
> > On Mon, 17 Oct 2016 14:26:42 +0200 (CEST), Richard Biener
> > wrote:
> >> --- gcc/bb-reorder.c (revision 241228)
> >> +++ gcc/bb-reorder.c (working copy)
> >> @@ -23
On 09/09/16 15:32, Andre Vieira (lists) wrote:
> On 27/05/16 15:51, Ulrich Weigand wrote:
>> Andre Vieira (lists) wrote:
>>> On 07/04/16 10:30, Andre Vieira (lists) wrote:
On 17/03/16 16:33, Andre Vieira (lists) wrote:
> On 23/10/15 12:31, Bernd Schmidt wrote:
>> On 10/12/2015 11:58 AM
Hi!
Ping.
On Thu, 29 Sep 2016 15:18:00 +0200, Thomas Schwinge
wrote:
> On Mon, 19 Sep 2016 13:25:01 +0200, Richard Biener
> wrote:
> > On Mon, Sep 19, 2016 at 1:19 PM, Thomas Schwinge
> > wrote:
> > > On Mon, 19 Sep 2016 10:18:35 +0200, Richard Biener
> > > wrote:
> > >> On Fri, Sep 16, 20
Hi!
On Fri, 30 Sep 2016 09:47:56 +0200, Richard Biener
wrote:
> On Thu, Sep 29, 2016 at 4:48 PM, Thomas Schwinge
> wrote:
> > On Mon, 19 Sep 2016 13:25:01 +0200, Richard Biener
> > wrote:
> >> On Mon, Sep 19, 2016 at 1:19 PM, Thomas Schwinge
> >> wrote:
> >> > On Mon, 19 Sep 2016 10:18:35 +0
On 10/14/2016 06:39 PM, Alexander Monakov wrote:
I'm resending the patch series with backend prerequisites for OpenMP
offloading to the NVIDIA PTX ISA. The patches are rebased on trunk.
What's the status of the branch? Is it expected to work? I'm trying to
compile the OpenMP version of these
On 10/14/2016 04:59 PM, Segher Boessenkool wrote:
On Fri, Oct 14, 2016 at 02:37:42PM -0500, Bill Seurer wrote:
[PATCH, rs6000] pr65479 Add option to fix failing asan test cases.
This patch adds the -fasynchronous-unwind-tables option to several of the asan
test cases. The option causes a full
On 2016.09.29 at 18:52 +, Bernd Edlinger wrote:
> On 09/29/16 20:03, Jason Merrill wrote:
> > On Wed, Sep 28, 2016 at 12:09 PM, Bernd Edlinger
> > wrote:
> >> On 09/28/16 16:41, Jason Merrill wrote:
> >>> On Tue, Sep 27, 2016 at 11:10 AM, Bernd Edlinger
> >>> wrote:
> On 09/27/16 16:42,
On 10/14/2016 11:34 AM, Jakub Jelinek wrote:
Hi!
Now that DWARF5 public review draft has been released, I went through
the document looking for double dagger marked constants and added them to
dwarf2.{def,h}.
Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk?
It's just adding
The __sample algorithms should work with the iterator's
difference_type, not the user-supplied _Distance type.
PR libstdc++/77994
* include/bits/stl_algo.h (sample): Convert size argument to iterator
difference type.
* include/experimental/algorithm (experimental::
On 10/14/2016 10:29 PM, Andrew Pinski wrote:
>>> >> This patch bumps the iteration count by 1 for loops with the exit at the
>>> >> end so that it represents the number of times the loop body is executed,
>>> >> and therefore removes the need to always execute that first peeled copy.
>>> >> With
This patch shifts earlier the point during compilation where the
rs6000 backend adds decoration to symbols: to encode_section_info.
AIX uses the XCOFF file format, which extends COFF in a few ways,
including some increased granularity of sections through a mechanism
called Storage Mapping Class --
Ping
On 10/06/16 18:18, Bernd Edlinger wrote:
> Hi,
>
> I'd like to ping for this patch:
> https://gcc.gnu.org/ml/gcc-patches/2016-09/msg02362.html
>
> Thanks,
> Bernd.
>
>
> On 09/30/16 15:33, Bernd Edlinger wrote:
>> On 09/30/16 12:14, Bernd Edlinger wrote:
>>> Eric Botcazou wrote:
> A c
On 10/14/16 12:17, David Malcolm wrote:
How about splitting out the bail-out code into a separate function:
diagnostic_handle_max_errors
Having discovered how fortran buffers errors, I think this approach is
necessary. I think I'm going to be in assignment-limbo for a little while though
This refactors propagation vs. substitution and handles condition
simplification properly as well as passing a known taken edge down
to the DOM walker (avoiding useless work and properly handling PHIs).
If we do all the work it's stupid to not fold away dead code...
Bootstrap and regtest pending
Hi!
On Mon, 17 Oct 2016 14:08:44 +0200, Richard Biener
wrote:
> On Mon, Oct 17, 2016 at 1:47 PM, Thomas Schwinge
> wrote:
> > On Mon, 17 Oct 2016 13:22:17 +0200, Richard Biener
> > wrote:
> >> On Mon, Oct 17, 2016 at 11:38 AM, Thomas Schwinge
> >> wrote:
> >> > On Fri, 14 Oct 2016 13:06:59 +
We are incorrectly requiring unique_ptr deleters to be copyable here:
explicit
unique_ptr(pointer __p) noexcept
: _M_t(__p, deleter_type())
{ }
We could just do:
explicit
unique_ptr(pointer __p) noexcept
: _M_t()
{ std::get<0>(_M_t) = __p; }
But having t
> On Oct 7, 2016, at 6:08 PM, Pat Haugen wrote:
>
> The patch here, https://gcc.gnu.org/ml/gcc-patches/2014-10/msg01872.html,
> attempted to scale down the register limit used by -fsched-pressure for the
> case where the block in question executes as frequently as the entry block to
> just the
On Mon, Oct 17, 2016 at 3:12 PM, Thomas Schwinge
wrote:
> Hi!
>
> On Mon, 17 Oct 2016 14:26:42 +0200 (CEST), Richard Biener
> wrote:
>> --- gcc/bb-reorder.c (revision 241228)
>> +++ gcc/bb-reorder.c (working copy)
>> @@ -2355,7 +2355,10 @@ reorder_basic_blocks_simple (void)
>> To start w
Hi!
On Mon, 17 Oct 2016 14:26:42 +0200 (CEST), Richard Biener
wrote:
> --- gcc/bb-reorder.c (revision 241228)
> +++ gcc/bb-reorder.c (working copy)
> @@ -2355,7 +2355,10 @@ reorder_basic_blocks_simple (void)
> To start with, everything points to itself, nothing is assigned yet. */
>
>
> On Oct 17, 2016, at 3:27 AM, Richard Biener
> wrote:
>
> On Fri, Oct 14, 2016 at 3:51 PM, Bill Schmidt
> wrote:
>> Hi Richard,
>>
>>> On Oct 14, 2016, at 4:19 AM, Richard Biener
>>> wrote:
>>>
>>> On Thu, Oct 13, 2016 at 5:38 PM, Bill Schmidt
>>> wrote:
The previous patch for
ping
From: Wilco Dijkstra
Sent: 12 September 2016 15:50
To: Richard Earnshaw; GCC Patches
Cc: nd
Subject: Re: [PATCH v2][AArch64] Fix symbol offset limit
Wilco wrote:
> The original example is from GCC itself, the fixed_regs array is small but
> due to
> optimization we can end up wit
Bootstrapped on x86_64-unknown-linux-gnu, testing in progress.
Richard.
2016-10-17 Richard Biener
PR tree-optimization/77988
* tree-vrp.c (remove_range_assertions): Use replace_uses_by.
* gcc.dg/torture/pr77988.c: New testcase.
Index: gcc/testsuite/gcc.dg/torture/pr
ping
From: Wilco Dijkstra
Sent: 02 September 2016 12:31
To: Ramana Radhakrishnan; GCC Patches
Cc: nd
Subject: Re: [PATCH][AArch64 - v3] Simplify eh_return implementation
Ramana Radhakrishnan wrote:
> Can you please file a PR for this and add some testcases ? This sounds like
> a serious
ping
From: Wilco Dijkstra
Sent: 08 September 2016 14:35
To: GCC Patches
Cc: nd
Subject: [PATCH][AArch64] Align FP callee-saves
If the number of integer callee-saves is odd, the FP callee-saves use 8-byte
aligned
LDP/STP. Since 16-byte alignment may be faster on some CPUs, align the FP
cal
ping
From: Wilco Dijkstra
Sent: 10 August 2016 17:20
To: Richard Earnshaw; GCC Patches
Cc: nd
Subject: Re: [PATCH][AArch64] Improve stack adjustment
Richard Earnshaw wrote:
> I see you've added a default argument for your new parameter. I think
> doing that is fine, but I have two comments
This fixes matching of toplevel (cond (lt @1 @2) ...) as reported by
Bin to me privately.
Bootstrapped on x86_64-unknown-linux-gnu, testing in progress.
Richard.
2016-10-17 Richard Biener
* genmatch.c (dt_operand::gen_gimple_expr): Use get_name to
get at the operand to look
On Sun, Oct 16, 2016 at 08:37:30AM -0400, David Edelsohn wrote:
> Maybe something like the following:
>
> * config/rs6000/rs6000.c (rs6000_savres_strategy) [AIX,ELFv2]: Inline
> FPR save and restore if shrink-wrapping and not optimizing for size.
>
> Index: rs6000.c
>
$subject, applied as obvious.
Richard.
2016-10-17 Richard Biener
* bb-reorder.c (reorder_basic_blocks_simple): Clear BB_VISITED
before using it.
Index: gcc/bb-reorder.c
===
--- gcc/bb-reorder.c(revision 2412
This typedef uses the type before it's complete, which Clang rejects.
It's not necessary anyay, we can just use decltype directly where
needed. The _Invoker type isn't user-facing and doesn't need a
result_type.
PR libstdc++/77998
* include/std/future (__future_base::_Deferred_sta
On Mon, Oct 17, 2016 at 1:47 PM, Thomas Schwinge
wrote:
> Hi!
>
> On Mon, 17 Oct 2016 13:22:17 +0200, Richard Biener
> wrote:
>> On Mon, Oct 17, 2016 at 11:38 AM, Thomas Schwinge
>> wrote:
>> > On Fri, 14 Oct 2016 13:06:59 +0200, Richard Biener
>> > wrote:
>> >> On Fri, Oct 14, 2016 at 1:00 P
This is a very simple change, to allow conversions that are supposed
to be valid according to DR 2118. The fix was slightly complicated by
the tests being a bit of a mess. We had tests for this requirement,
but they were in files which mixed positive tests and negative tests.
I've split the tests
In C++17 the emplace_front and emplace_back members return a
reference. There isn't a very neat way to implement this, so it's just
lots of conditional compilation.
This isn't an ABI break, because these member functions are templates
and so the return type is part of the mangled name. We can't a
On Fri, Oct 14, 2016 at 2:07 PM, Yuri Rumyantsev wrote:
> Richard,
>
> Here is updated patch with the changes proposed by you.
>
> Bootstrapping and regression testing did not show any new failures.
> Is it OK for trunk?
Ok with dropping the free_dominance_info_for_region variant without
struct f
Hi!
On Mon, 17 Oct 2016 13:22:17 +0200, Richard Biener
wrote:
> On Mon, Oct 17, 2016 at 11:38 AM, Thomas Schwinge
> wrote:
> > On Fri, 14 Oct 2016 13:06:59 +0200, Richard Biener
> > wrote:
> >> On Fri, Oct 14, 2016 at 1:00 PM, Nathan Sidwell wrote:
> >> > On 10/14/16 05:28, Richard Biener wr
On 10/13/2016 11:43 AM, Richard Biener wrote:
> On Wed, Oct 12, 2016 at 3:52 PM, Martin Liška wrote:
>> On 10/04/2016 11:45 AM, Richard Biener wrote:
>>> On Thu, Sep 15, 2016 at 12:00 PM, Martin Liška wrote:
On 09/07/2016 02:09 PM, Richard Biener wrote:
> On Wed, Sep 7, 2016 at 1:37 PM,
David Malcolm writes:
> @@ -2388,8 +2389,10 @@ gen_mnemonic_setattr (htab_t mnemonic_htab, rtx insn)
> static int
> mnemonic_htab_callback (void **slot, void *info ATTRIBUTE_UNUSED)
> {
> - obstack_grow (&string_obstack, (char*)*slot, strlen ((char*)*slot));
> - obstack_1grow (&string_obstack
1 - 100 of 125 matches
Mail list logo