Hi,
The GCC 5 branch is frozen now in preparation for a GCC 5.4 release candidate.
All changes require release manager approval until after the GCC 5.4 release.
Thanks,
Richard.
On 05/26/2016 10:14 PM, Alan Modra wrote:
On Thu, May 26, 2016 at 10:12:14AM -0400, Vladimir Makarov wrote:
On 05/26/2016 07:02 AM, Alan Modra wrote:
This fixes lack of bb_loop_depth info in some of the early parts of
ira, which has been the case for quite some time. All active branches
return
On Thu, May 26, 2016 at 09:50:18PM -0400, Michael Meissner wrote:
> It might be argued that this is a reload bug (since it runs on LRA), but
> sometimes it is simpler to place a simpler work around in the machine
> dependent
> code. If the maintainers decide that it should be fixed in reload inst
On Thu, May 26, 2016 at 10:12:14AM -0400, Vladimir Makarov wrote:
> On 05/26/2016 07:02 AM, Alan Modra wrote:
> >This fixes lack of bb_loop_depth info in some of the early parts of
> >ira, which has been the case for quite some time. All active branches
> >return 0 from bb_loop_depth() in update_e
It might be argued that this is a reload bug (since it runs on LRA), but
sometimes it is simpler to place a simpler work around in the machine dependent
code. If the maintainers decide that it should be fixed in reload instead of
via this patch, that is fine.
PR 71294 involves vectorization where
On 5/26/2016 21:55, Arnaud Charlet wrote:
>> Text mode translation should not be done for Cygwin, especially since it
>> does not
>> support unicode setmode calls. This also fixes ada builds for Cygwin.
>>
>> OK for trunk?
>
> OK, thanks.
>
Can someone please commit this? I don't have SVN write
On Thu, May 26, 2016 at 03:59:43PM -0500, Segher Boessenkool wrote:
> On Thu, May 26, 2016 at 01:04:59PM -0400, Michael Meissner wrote:
> > * config/rs6000/rs6000.md (SFDF2): New iterator to allow doing
> > conditional moves there the comparison type is different from move
> > type.
>
On Thu, 26 May 2016, Jan Hubicka wrote:
> > > +ffp-int-builtin-inexact
> > > +Common Report Var(flag_fp_int_builtin_inexact) Optimization
> > > +Allow built-in functions ceil, floor, round, trunc to raise \"inexact\"
> > > exceptions.
>
> When adding new codegen option which affects the correctn
On Thu, May 26, 2016 at 01:04:59PM -0400, Michael Meissner wrote:
> * config/rs6000/rs6000.md (SFDF2): New iterator to allow doing
> conditional moves there the comparison type is different from move
> type.
s/there/where/ ?
>
Don't forget to delete this line ;-)
> +/*
On Thu, May 26, 2016 at 07:39:01PM +0200, Uros Bizjak wrote:
> On Thu, May 26, 2016 at 7:05 PM, Jakub Jelinek wrote:
> > Hi!
> >
> > This patch adds an avx512dq alternative (EVEX vpinsrd requires that) and
> > enables EVEX vmovd and vpunpckldq.
> >
> > Bootstrapped/regtested on x86_64-linux and i6
Hi!
While porting doacross-1.c testcase to Fortran, I've discovered
that I've used there schedule(static, 0), which is invalid (I've meant
schedule(static) instead). This patch adds warning for this for all FEs.
Bootstrapped/regtested on x86_64-linux and i686-linux, committed to trunk.
2016-05-
Martin complained that -Wswitch-unreachable wouldn't warn on try-blocks,
either compiler-generated or user-written. This patch, which looks into
GIMPLE_TRY's body, seems to DTRT for both.
Bootstrapped/regtested on x86_64-linux, ok for trunk?
2016-05-26 Marek Polacek
* gimplify.c (gim
> > +ffp-int-builtin-inexact
> > +Common Report Var(flag_fp_int_builtin_inexact) Optimization
> > +Allow built-in functions ceil, floor, round, trunc to raise \"inexact\"
> > exceptions.
When adding new codegen option which affects the correctness, it is also
necessary to update can_inline_edge_p
On 26/05/16 19:49 +0200, Marc Glisse wrote:
On Thu, 26 May 2016, Jonathan Wakely wrote:
On 25/05/16 14:54 +0100, Jonathan Wakely wrote:
On 23/05/16 20:39 +0200, Marc Glisse wrote:
Ping
(re-attaching, I just added a one-line comment before the tag
class as asked by Ville)
This is OK for tr
On Thu, 26 May 2016, Jonathan Wakely wrote:
On 25/05/16 14:54 +0100, Jonathan Wakely wrote:
On 23/05/16 20:39 +0200, Marc Glisse wrote:
Ping
(re-attaching, I just added a one-line comment before the tag class as
asked by Ville)
This is OK for trunk - thanks.
On second thoughts - does thi
On Thu, May 26, 2016 at 7:05 PM, Jakub Jelinek wrote:
> Hi!
>
> This patch adds an avx512dq alternative (EVEX vpinsrd requires that) and
> enables EVEX vmovd and vpunpckldq.
>
> Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk?
>
> 2016-05-26 Jakub Jelinek
>
> * confi
Hello!
Some of these strings went out of control. Use if_then_else or cond
RTXes to make things readable and maintainable again.
No functional changes.
2016-05-26 Uros Bizjak
* config/i386/i386.md (*movqi_internal) : Use
if_then_else or cond RTXes to calculate attribute value.
*
On Thu, 26 May 2016, Marek Polacek wrote:
> The C++ FE has been changed, as a part of c++/50800, in such a way that it no
> longer considers types differentiating only in TYPE_REF_CAN_ALIAS_ALL
> incompatible. But the C FE still rejects the following testcase, so this
> patch
> makes the C FE fo
Hello,
On Thu, 26 May 2016, Nathan Sidwell wrote:
> Applied the attached to markup some more tests that PTX either crashes on or
> doesn't apply strict IEEE semantics to. In one case it's about debug info
> that we don't emit.
> Index: gcc.dg/torture/c99-contract-1.c
> =
On 05/19/2016 01:24 PM, Bin.Cheng wrote:
> On Thu, May 19, 2016 at 11:23 AM, Martin Liška wrote:
>> On 05/16/2016 03:55 PM, Martin Liška wrote:
>>> On 05/16/2016 12:13 PM, Bin.Cheng wrote:
Hi Martin,
Could you please rebase this patch and the profiling one against
latest trunk? The
Hello,
On Thu, 26 May 2016, Nathan Sidwell wrote:
> This patch removes the malloc/realloc/free wrappers from libgcc. I've
> implemented them completely in C and put them in the ptx newlib port --
> where one expects such functions.
It appears that the new Newlib code doesn't free 'p' on 'rea
The previous gcov behavior was to always output errors on the stderr channel.
This is fine for most uses, but some programs will require stderr to be
untouched by libgcov for certain tests. This change allows configuring
the gcov output via an environment variable which will be used to open
the app
On Thu, 26 May 2016, Kyrill Tkachov wrote:
> the early RTL optimisers. This better expresses the semantics of the
> operation as a whole, since the RBIT operation is represented as an
> UNSPEC anyway and so will not see the benefits of combine,
This doesn't affect your patch, but I think it wo
Hi!
Both vpunpcklqdq and vmovhps are available with XMM EVEX args in AVX512VL.
Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk?
2016-05-26 Jakub Jelinek
* config/i386/sse.md (*vec_concatv4si): Use v=v,v instead of
x=x,x and v=v,m instead of x=x,m.
Hi!
This patch adds an avx512dq alternative (EVEX vpinsrd requires that) and
enables EVEX vmovd and vpunpckldq.
Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk?
2016-05-26 Jakub Jelinek
* config/i386/sse.md (*vec_concatv2si_sse4_1): Add avx512dq v=Yv,rm
al
On Mon, May 09, 2016 at 09:31:43AM -0500, Segher Boessenkool wrote:
> On Thu, May 05, 2016 at 03:18:39PM -0400, Michael Meissner wrote:
> > At the present time, the code does not support comparisons involving >= and
> > <=
> > unless the -ffast-math option is used. I hope eventually to support
>
Hi Jeff,
>>> I may be missing something, but isn't it the transition to an FP
>>> relative address rather than a SP relative address that's the problem
>>> here?
>>
>> Yes, I believe so.
>>
>>> Where does that happen?
I think that it happens in dwarf2out.c:based_loc_descr() which
detects the use
On 25/05/16 14:54 +0100, Jonathan Wakely wrote:
On 23/05/16 20:39 +0200, Marc Glisse wrote:
Ping
(re-attaching, I just added a one-line comment before the tag class
as asked by Ville)
This is OK for trunk - thanks.
On second thoughts - does this change the passing conventions for
std::tupl
On Thu, May 26, 2016 at 1:46 AM, Joseph Myers wrote:
> In ISO C99/C11, the ceil, floor, round and trunc functions may or may
> not raise the "inexact" exception for noninteger arguments. Under TS
> 18661-1:2014, the C bindings for IEEE 754-2008, these functions are
> prohibited from raising "inex
On May 26, 2016 4:39:02 PM GMT+02:00, Richard Sandiford
wrote:
>tree-ssa-loop-ivopts.c:loop_body_includes_call was treating internal
>calls such as IFN_SQRT as clobbering all caller-saved registers, which
>I don't think is appropriate for any current internal function.
>
>Tested on aarch64-linux-
On 26/05/16 01:07 +0300, Ville Voutilainen wrote:
On 25 May 2016 at 16:55, Jonathan Wakely wrote:
On 24/05/16 19:49 +0300, Ville Voutilainen wrote:
On 24 May 2016 at 19:35, Ville Voutilainen
wrote:
Slight tweak. The avoidance of _NotSameTuple wasn't quite correct for
the templates that
tak
On 26/05/16 14:02 +0200, Christophe Lyon wrote:
I've seen you've backported the main patch to the gcc-6 branch, you
forgot to add the follow-up "Add dg-require-filesystem-ts directive to
test".
And likewise for the gcc-5 branch.
Both fixed now, sorry about that, again.
Andi Kleen writes:
Ping!
> Here's an updated version of the patchkit to enable autofdo bootstrap
> and testing. It also fixes some autofdo issues. The last patch is more a
> workaround
> (to make autofdo bootstrap not ICE), but may need a better fix.
>
> The main motivation is to get better tes
Applied the attached to markup some more tests that PTX either crashes on or
doesn't apply strict IEEE semantics to. In one case it's about debug info that
we don't emit.
nathan
2016-05-26 Nathan Sidwell
* gcc.dg/20060410.c: Xfail on ptx.
* gcc.dg/torture/c99-contract-1.c: Skip on ptx.
When using config-list.mk to build all configurations, it's useful
to filter out the configurations that are known to be broken.
This patch does so, adding a KNOWN_BROKEN variable.
contrib/ChangeLog:
* config-list.mk (LIST): Rename to...
(FULL_LIST): ...this.
(KNOWN_BROKEN
On 05/26/16 10:36, Nathan Sidwell wrote:
Ib. PTX appears to accept
'.extern .weak ...', but that has the same semantics as '.extern ...', which
IMHO is a bug. '.extern .weak' doesn't mean anything special. Working on a
GCC patch to stop us emitting it.
Pah, I'd misremembered what we emitte
r233165 marked three deprecated rtems targets as obsolete.
r233887 marked mep-elf as obsolete.
Update config-list.mk to add OPT-enable-obsolete to these 4
targets.
contrib/ChangeLog:
* config-list.mk (FULL_LIST): Add OPT-enable-obsolete to
avr-rtems, h8300-rtems, m32r-rtems, mep-e
config-list.mk currently requires the pwd to be in a sibling directory
of the source tree. However, building using config-list.mk can consume
over 400GB of disk space in this build directory (e.g. my machine
successfully built cc1 for 206 configurations last night, consuming
442GB of space). I've
tree-ssa-loop-ivopts.c:loop_body_includes_call was treating internal
calls such as IFN_SQRT as clobbering all caller-saved registers, which
I don't think is appropriate for any current internal function.
Tested on aarch64-linux-gnu and x86_64-linux-gnu. OK to install?
Thanks,
Richard
gcc/
On 05/25/16 11:49, Alexander Monakov wrote:
On Wed, 25 May 2016, Nathan Sidwell wrote:
With today's trunk and newlib, if I run
unresolved symbol __exitval_ptr
Is should work now, just pushed a patch to newlib. PTX appears to accept
'.extern .weak ...', but that has the same semantics as
This patch is effectively reverting a change from 1994. The reason
I think it's a hack is that store_bit_field_1 is creating a subreg
reference to one word of a field even though it has already proven that
the field spills into the following word. We then rely on the special
SUBREG handling in st
This patch removes the malloc/realloc/free wrappers from libgcc. I've
implemented them completely in C and put them in the ptx newlib port -- where
one expects such functions.
applied to trunk.
nathan
2016-05-26 Nathan Sidwell
* config/nvptx/free.asm: Delete.
* config/nvptx/malloc.as
The C++ FE has been changed, as a part of c++/50800, in such a way that it no
longer considers types differentiating only in TYPE_REF_CAN_ALIAS_ALL
incompatible. But the C FE still rejects the following testcase, so this patch
makes the C FE follow suit. After all, the may_alias attribute is not
On 05/26/2016 07:02 AM, Alan Modra wrote:
This fixes lack of bb_loop_depth info in some of the early parts of
ira, which has been the case for quite some time. All active branches
return 0 from bb_loop_depth() in update_equiv_regs, but whether that
actually causes mis-optimization anywhere but t
On Fri, May 06, 2016 at 04:00:28PM +0100, Jiong Wang wrote:
> This patch fixes PR63596.
>
> There is no need to push/pop all arguments registers. We only need to
> push and pop those registers used. These use info is calculated by a
> dedicated vaarg optimization tree pass "tree-stdarg", the backe
It seems like in cases of malformed input we should return the input
again rather than produce garbage like "K". Maybe catch this sort of situation in
d_lookup_template_parameter?
Jason
On Mon, May 2, 2016 at 11:21 AM, Marcel Böhme wrote:
> Hi,
>
> This fixes several stack overflows due to inf
> Text mode translation should not be done for Cygwin, especially since it
> does not
> support unicode setmode calls. This also fixes ada builds for Cygwin.
>
> OK for trunk?
OK, thanks.
> gcc/ada/ChangeLog:
> * sysdep.c (__gnat_set_binary_mode, __gnat_set_text_mode,
> __gnat_set_mo
On Thu, May 26, 2016 at 3:06 AM, Marek Polacek wrote:
> On Wed, May 25, 2016 at 03:21:00PM -0600, Martin Sebor wrote:
>> I see. Thanks for clarifying that. No warning on a declaration
>> alone makes sense in the case above but it has the unfortunate
>> effect of suppressing the warning when the
[Sorry for the large recipient list, I wasn't sure who of C++ and x86
maintainers should approve this]
Hi,
29_atomics/atomic/65913.cc test in libstdc++ is a runtime test that only rely
on atomic and gnu++11 support. Therefore I propose to require atomic-builtins
instead of an x86 (32 or 64 bit
On 18 May 2016 at 12:55, Richard Biener wrote:
>
> The following patch moves LIM before PRE to allow it to cleanup CSE
> (and copyprop) opportunities LIM exposes. It also moves the DCE done
> in loop before the loop pipeline as otherwise it is no longer executed
> uncoditionally at this point (si
Text mode translation should not be done for Cygwin, especially since it does
not
support unicode setmode calls. This also fixes ada builds for Cygwin.
OK for trunk?
gcc/ada/ChangeLog:
* sysdep.c (__gnat_set_binary_mode, __gnat_set_text_mode,
__gnat_set_mode): Disable text mode t
On 26 May 2016 at 13:58, Christophe Lyon wrote:
> On 5 May 2016 at 12:12, Jonathan Wakely wrote:
>> On 04/05/16 17:19 +0100, Andre Vieira (lists) wrote:
>>>
>>> On 20/04/16 18:40, Jonathan Wakely wrote:
On 19/04/16 19:07 +0100, Jonathan Wakely wrote:
>
> This was reported as a b
On 5 May 2016 at 12:12, Jonathan Wakely wrote:
> On 04/05/16 17:19 +0100, Andre Vieira (lists) wrote:
>>
>> On 20/04/16 18:40, Jonathan Wakely wrote:
>>>
>>> On 19/04/16 19:07 +0100, Jonathan Wakely wrote:
This was reported as a bug in the Filesystem library, but it's
actually a pro
James Greenhalgh wrote:
> I really don't like [1][2][3] this technique of attempting to work around
> register allocator issues using the disparaging mechanisms.
I don't see the issue as it is a standard mechanism to describe higher cost
to the register allocator. On the other had the use of '*' i
On Fri, May 06, 2016 at 04:00:13PM +0100, Jiong Wang wrote:
> This patch initialize va_list_gpr_counter_field and
> va_list_fpr_counter_field properly for AArch64 backend that tree-stdarg
> pass will be enabled.
>
> The "required register" analysis is largely target independent, but the
> user mig
This fixes lack of bb_loop_depth info in some of the early parts of
ira, which has been the case for quite some time. All active branches
return 0 from bb_loop_depth() in update_equiv_regs, but whether that
actually causes mis-optimization anywhere but trunk is yet to be
determined.
I played a li
On Fri, Apr 22, 2016 at 03:35:42PM +, Wilco Dijkstra wrote:
> SIMD operations like combine prefer to have their operands in FP registers,
> so increase the cost of integer registers slightly to avoid unnecessary
> int<->FP moves. This improves register allocation of scalar SIMD operations.
I r
On Fri, May 20, 2016 at 11:04:32AM +0100, Kyrill Tkachov wrote:
> Hi all,
>
> The recent -frename-registers change exposed a deficiency in the way we fuse
> AESE/AESMC instruction pairs in aarch64.
>
> Basically we want to enforce:
> AESE Vn, _
> AESMC Vn, Vn
>
> to enable the fusion, bu
Hi all,
In this PR we want to optimise:
int foo (int i)
{
return (i == 0) ? N : __builtin_clz (i);
}
on targets where CLZ is defined at zero to the constant 'N'.
This is determined at the RTL level through the CLZ_DEFINED_VALUE_AT_ZERO macro.
The obvious place to implement this would be in com
Hi all,
In a similar rationale to patch 1/3 this patch changes the AArch64 backend to
keep the CTZ expression
as a single RTX until after reload when it is split into an RBIT and a CLZ
instruction.
This enables CTZ-specific optimisations in the pre-reload RTL optimisers.
Bootstrapped and teste
Hi all,
On arm we don't have a dedicated instruction that corresponds to a CTZ rtx but
we synthesise it
with an RBIT instruction followed by a CLZ. This is currently done at expand
time.
However, I'd like to push that step until after reload and keep the CTZ rtx as
a single whole in
the early
On Wed, Apr 27, 2016 at 02:13:21PM -0700, Andrew Pinski wrote:
> Hi,
> AARCH64 ILP32 is like x32 where UNITS_PER_WORD > sizeof(void*) so we
> need to define REG_VALUE_IN_UNWIND_CONTEXT for ILP32. This fixes
> unwinding through the signal handler. This is independent of the ABI
> which Linux ker
Hi Jakub,
On 26 May 2016 at 18:18, Jakub Jelinek wrote:
> On Thu, May 26, 2016 at 02:17:56PM +1000, Kugan Vivekanandarajah wrote:
>> --- a/gcc/tree-ssa-reassoc.c
>> +++ b/gcc/tree-ssa-reassoc.c
>> @@ -3767,8 +3767,10 @@ swap_ops_for_binary_stmt (vec ops,
>>operand_entry temp = *oe3;
>>
Hi!
I came up with a simple C testcase for a PR, which apparently Martin
fixed yesterday in PR71239.
Thus, I've just committed following testcase as obvious and will close the
PR.
2016-05-26 Jakub Jelinek
PR tree-optimization/71280
* gcc.dg/pr71280.c: New test.
--- gcc/tests
Hi Ramana,
On 19/05/16 14:36, Ramana Radhakrishnan wrote:
On 11/05/16 15:32, Kyrill Tkachov wrote:
Hi all,
In this PR a NEON builtin is introduced during SLP vectorisation even when NEON
is not available
because arm_builtin_vectorized_function is missing an appropriate check in the
BSWAP han
On Mon, May 16, 2016 at 11:38:04AM +0100, Wilco Dijkstra wrote:
> GCC expands switch statements in a very simplistic way and tries to use a
> table
> expansion even when it is a bad idea for performance or codesize.
> GCC typically emits extremely sparse tables that contain mostly default
> entri
On Wed, May 18, 2016 at 02:13:53PM +0100, Jiong Wang wrote:
> Thanks for reporting this.
>
> Yes, reproduced. I should force those res* local variable into
> memory so they can be in the same order as the expected result
> which is kept in memory.
>
> The following patch fix this.
>
> vmul_elem_
On 26/05/16 09:24, Thomas Preudhomme wrote:
On Wednesday 25 May 2016 11:38:44 Mike Stump wrote:
On May 25, 2016, at 10:20 AM, Thomas Preudhomme
wrote:
2016-05-24 Thomas Preud'homme
* gcc.dg/plugin/plugin.exp: skip tail call tests for Thumb-1.
Is this ok for trunk?
Ok. Normally
On 18/05/16 14:13, Jiong Wang wrote:
On 18/05/16 09:17, Christophe Lyon wrote:
gcc/
2016-05-17 Jiong Wang
* config/aarch64/aarch64-simd.md
(*aarch64_mul3_elt_to_128df): Extend
to all supported modes. Rename to...
(*aarch64_mul3_elt_from_dup): ...this.
gcc/
On 23 May 19:21, Jakub Jelinek wrote:
> Hi!
>
> These insns are available in AVX512VL, so we can just use v instead of x.
>
> Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk?
OK.
--
Thanks, K
On 23 May 19:17, Jakub Jelinek wrote:
> Hi!
>
> This pattern is used to improve __builtin_shuffle in some cases;
> VPALIGNR is AVX512BW & AVX512VL.
>
> Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk?
OK.
--
Thanks, K
Hi Jakub,
On 23 May 19:15, Jakub Jelinek wrote:
> Hi!
>
> The vbroadcastss and vpermilps insns are already in AVX512F & AVX512VL,
> so can be used with v instead of x, the splitter case where we for AVX
> emit vpermilps plus vpermf128 is more problematic, because the latter
> insn isn't available
On Wednesday 25 May 2016 11:38:44 Mike Stump wrote:
> On May 25, 2016, at 10:20 AM, Thomas Preudhomme
wrote:
> > 2016-05-24 Thomas Preud'homme
> >
> >* gcc.dg/plugin/plugin.exp: skip tail call tests for Thumb-1.
> >
> > Is this ok for trunk?
>
> Ok. Normally I'd just punt to the ar
On Thu, May 26, 2016 at 02:17:56PM +1000, Kugan Vivekanandarajah wrote:
> --- a/gcc/tree-ssa-reassoc.c
> +++ b/gcc/tree-ssa-reassoc.c
> @@ -3767,8 +3767,10 @@ swap_ops_for_binary_stmt (vec ops,
>operand_entry temp = *oe3;
>oe3->op = oe1->op;
>oe3->rank = oe1->rank;
> +
On Wed, May 25, 2016 at 03:21:00PM -0600, Martin Sebor wrote:
> I see. Thanks for clarifying that. No warning on a declaration
> alone makes sense in the case above but it has the unfortunate
> effect of suppressing the warning when the declaration is followed
> by a statement, such as in:
>
>
Hi,
This patch is pending a careful review.
Best regards,
- Marcel
> On 2 May 2016, at 11:21 PM, Marcel Böhme wrote:
>
> Hi,
>
> This fixes several stack overflows due to infinite recursion in d_print_comp
> (https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70909).
>
> The method d_print_comp i
Hi: Pending review.
Best - Marcel
> On 3 May 2016, at 10:40 PM, Marcel Böhme wrote:
>
> Hi,
>
> This fixes four access violations
> (https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70926).
>
> Two of these first read the value of a length variable len from the mangled
> string, then strncpy l
77 matches
Mail list logo
