> From: Ramana Radhakrishnan [mailto:ramana@googlemail.com]
> Sent: Tuesday, February 17, 2015 4:08 PM
> To: Thomas Preud'homme
> Cc: gcc-patches; Richard Earnshaw; Ramana Radhakrishnan; Marcus
> Shawcroft; Richard Biener; Jakub Jelinek
> Subject: Re: [PATCH, ARM] Backport fix for PR59593 (mini
On 03/02/15 01:38, Richard Biener wrote:
On Mon, Mar 2, 2015 at 6:34 AM, Aldy Hernandez wrote:
As I mention in the PR...
What's happening here is that the ipa_polymorphic_call_context constructor
is calling walk_ssa_copies on a PHI node that has no arguments. This
happens because finalize_jum
Hi,
fix for comdat sections I am going to send incrementally triggers bug in
tree-sra that is trying to access a non-existing call statement in thunk.
While theoretically tree-sra can handle thunks - simply turn them into
wrappers when doing the transformation, I just hardened a check to disable
i
On Tue, Mar 03, 2015 at 06:20:05AM -0800, H.J. Lu wrote:
> For x86-64, protected data symbols are impossible.
Impossible? This is not even true currently since -fPIC emits code
that looks like it would fully support protected visibiliy variables
in shared libraries.
If you meant to say it is imp
On Wed, Mar 4, 2015 at 10:44 AM, Terry Guo wrote:
> On Mon, Mar 2, 2015 at 9:08 PM, Maxim Kuvyrkov
> wrote:
>>> On Mar 2, 2015, at 4:44 AM, Terry Guo wrote:
>>>
>>> Hi there,
>>>
>>> If target mode isn't specified via either gcc configuration option
>>> --with-mode or command line, this patch in
03.03.2015 10:25, Jan Hubicka writes:
>>>
>>> The hash itself is quite simple (borrowing incremental hash for constants
>>> adding
>>> very simple match for other stuff + logic to skip things that may match
>>> even if
>>> they are syntactticaly different). The hash can be strenghtened
>>> sign
On Mon, Mar 2, 2015 at 9:08 PM, Maxim Kuvyrkov
wrote:
>> On Mar 2, 2015, at 4:44 AM, Terry Guo wrote:
>>
>> Hi there,
>>
>> If target mode isn't specified via either gcc configuration option
>> --with-mode or command line, this patch intends to improve gcc driver to
>> automatically add option -m
Done for backport to 4.8.
Best regards,
Thomas
> -Original Message-
> From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-
> ow...@gcc.gnu.org] On Behalf Of Thomas Preud'homme
> Sent: Tuesday, March 03, 2015 5:35 PM
> To: Ramana Radhakrishnan
> Cc: gcc-patches; Richard Biener; Jakub
The attached patch resolves PR 65286 by adding --disable-multilib
when configuring for a ppc64le target.
Martin
diff --git a/ChangeLog b/ChangeLog
index bd43281..46ec13d 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+2015-03-03 Martin Sebor
+
+ PR target/65286
+ * configure.ac (powerpc
The attached patches are to fix PR target/65249 which is a 4.9/5
regression. It avoids yet another R0_REGS spill failure with
pre-allocating R0 reg in symGOT_load when the loading symbol is
__stack_chk_guard. Although the problem went away with -mlra
and LRA would be the real fix for this issue,
More edge cases regarding far addresses. Committed.
* config/rl78/rl78-real.md (*addqi_real): Allow SADDR types for
inc/dec.
(*addhi3_real): Likewise.
* config/rl78/rl78-virt.md (*inc3_virt): Additional
pattern to match incrementing memory.
* confi
This is the backport of the fixes for PR 65138/target (not using --with-cpu
during configuration means that several lto tests fail on little endian 64-bit
PowerPC systems). I found in doing the patches, I also needed to apply the
changes on January 19th to properly set the TARGET_DEFAULT on 64-bit
ok for google branch.
Dehao
On Tue, Mar 3, 2015 at 12:26 PM, Cary Coutant wrote:
>>> @@ -21817,22 +21823,39 @@ out_subprog_directive (subprog_entry *su
>>> {
>>>tree decl = subprog->decl;
>>>tree decl_name = DECL_NAME (decl);
>>> - const char *name;
>>> + tree origin;
>>
>> Explicitly
>> @@ -21817,22 +21823,39 @@ out_subprog_directive (subprog_entry *su
>> {
>>tree decl = subprog->decl;
>>tree decl_name = DECL_NAME (decl);
>> - const char *name;
>> + tree origin;
>
> Explicitly initialize origin to NULL_TREE;
Done.
>> + /* For inlined subroutines, use the linkage n
> On Mon, Mar 2, 2015 at 6:34 AM, Aldy Hernandez wrote:
> > As I mention in the PR...
> >
> > What's happening here is that the ipa_polymorphic_call_context constructor
> > is calling walk_ssa_copies on a PHI node that has no arguments. This
> > happens because finalize_jump_threads eventually re
Hello, gentle maintainer.
This is a message from the Translation Project robot.
A revised PO file for textual domain 'gcc' has been submitted
by the German team of translators. The file is available at:
http://translationproject.org/latest/gcc/de.po
(This file, 'gcc-5.1-b20150208.de.po', h
> > Hi,
> >
> > I built with r221117. I see errors while building following targets:
> > aarch64_be-none-linux-gnu, aarch64_be-none-linux-gnu,
> > arm-none-linux-gnueabihf, arm-none-linux-gnueabi.
>
> Indeed, it's broken again since r221099.
Accidentally I reverted the var->alias check (probably
> The problem appears to be that GCC will create a multiply-plus-add
> instruction to access the table regardless of whether the backend
> supports such an instruction. I could not work out where in the
> middle end this was occurring, so instead I created the patch below
> which contai
On Tue, Mar 3, 2015 at 11:25 AM, Cary Coutant wrote:
> This patch is for the google/gcc-4_9 branch.
>
> With this patch, we output an empty name for non-inlined functions,
> and change the -ftwo-level-all-subprogs option to override this
> behavior to output linkage names instead. The consumer can
> OK to apply ?
Ok.
> Andrew Pinski wrote:
> On Tue, Mar 3, 2015 at 10:06 AM, Wilco Dijkstra wrote:
> > This patch makes aarch64_min_divisions_for_recip_mul configurable for float
> > and double. This
> allows
> > CPUs with really fast or multiple dividers to return 3 (or even 4) if that
> > happens to be
> faster
On Tue, Mar 3, 2015 at 10:06 AM, Wilco Dijkstra wrote:
> This patch makes aarch64_min_divisions_for_recip_mul configurable for float
> and double. This allows
> CPUs with really fast or multiple dividers to return 3 (or even 4) if that
> happens to be faster
> overall. No code generation change
On Wed, Feb 25, 2015 at 7:43 AM, Martin Jambor wrote:
> Hi Eric and Richard,
>
> On Tue, Jan 06, 2015 at 06:07:12PM +0100, Eric Botcazou wrote:
>> Martin,
>>
>> > I suppose that could be done by something like the following, which I
>> > have tested only very mildly so far, in particular I have no
On 03/02/15 10:37 +, Iain Sandoe wrote:
the following additional tweaks provide further work-arounds.
... checked on darwin12 and darwin14.
Also checked on x86_64-linux, committed to trunk.
In constexpr handling, we can request an rval for a return-value both by
an invisible reference and by virtue of named return value optimization.
Fixed by removing the assert.
Approved by Jason off-line.
Committed to mainline.
commit 1a8fdc708eeab70adc237193febf5f8db1eab995
Author: Aldy Hernan
This patch makes aarch64_min_divisions_for_recip_mul configurable for float and
double. This allows
CPUs with really fast or multiple dividers to return 3 (or even 4) if that
happens to be faster
overall. No code generation change - bootstrap & regression OK.
ChangeLog:
2015-03-03 Wilco Dijkst
> -Original Message-
> From: Kyrylo Tkachov
> Sent: 27 February 2015 14:30
> To: Kyrylo Tkachov; GCC Patches
> Cc: Ramana Radhakrishnan; Richard Earnshaw
> Subject: RE: [PATCH][ARM] PR target/64600 Fix another ICE with -
> mtune=xscale: properly sign-extend mask during constant splitting
The fix for PR c/60226
https://gcc.gnu.org/viewcvs/gcc?view=revision&revision=212346
changed the type of the 'divisor' parameter of the round_up_loc function from
'int' to 'unsigned int'. Now the function does the usual masking dance with
powers of two, in particular:
t = build_int_cst (TRE
On Tue, Mar 3, 2015 at 7:13 PM, augustine.sterl...@gmail.com
wrote:
> On Sat, Feb 28, 2015 at 10:14 AM, Max Filippov wrote:
>> call0 is an ABI that doesn't use register windows.
>
> This is OK for xtensa.
Applied to trunk. Thanks!
-- Max
> Hello.
>
> There's suggested patch for the issue, where regression test have been
> running on x86_64-linux-pc.
>
> Ready for trunk after tests will be finished?
> Thanks,
> Martin
> >From a186fc97c5f192b94cc571f611a50eb0f4f8a354 Mon Sep 17 00:00:00 2001
> From: mliska
> Date: Tue, 3 Mar 201
> gcc/testsuite/ChangeLog:
>
> 2015-03-03 Martin Liska
>
> * gcc.dg/ipa/pr65287.c: New test.
>
> gcc/ChangeLog:
>
> 2015-03-03 Martin Liska
>
> PR ipa/65287
> * ipa-icf.c (sem_variable::parse): Skip all alias variables.
OK
Sorry, I must have inadverty reverted this.
Ho
> Hello.
>
> There's suggested patch for the issue, where regression test have been
> running on x86_64-linux-pc.
>
> Ready for trunk after tests will be finished?
> Thanks,
> Martin
> >From a186fc97c5f192b94cc571f611a50eb0f4f8a354 Mon Sep 17 00:00:00 2001
> From: mliska
> Date: Tue, 3 Mar 201
On Tue, Mar 3, 2015 at 6:52 PM, H.J. Lu wrote:
> On Tue, Mar 3, 2015 at 7:05 AM, Max Filippov wrote:
>> On Tue, Mar 3, 2015 at 5:51 PM, Richard Henderson wrote:
>>> On 03/02/2015 10:42 PM, Max Filippov wrote:
gcc/
* builtins.c (expand_builtin_return_addr): Add
RETURN_A
It's been reported that Paul's address is no longer active and Paul
hasn't been active in some time. Obviously if he returns to GCC
development, we'd welcome him with open arms. But in the mean time,
listing him as a maintainer doesn't make much sense.
Separately, I've had a conversation w
On Tue, Mar 3, 2015 at 3:14 PM, Uros Bizjak wrote:
> On Mon, Mar 2, 2015 at 7:23 PM, Richard Henderson wrote:
>
>>> Unfortunately, while "normal" bootstrap works OK with alpha-linux-gnu
>>> and alphaev68-linux-gnu, the bootstrap still fails when configured
>>> --with-build-config=bootstrap-lto:
>
Hi all,
I have just committed this simple and obvious patch to fix a typo for
"^" multiple alternative constraints in md.texi.
Doc builds Okay.
SVN revision link:
https://gcc.gnu.org/viewcvs/gcc?view=revision&revision=221152
ChangeLog:
2015-03-03 Renlin Li
* gcc/doc/md.texi (@item ^)
Expand abs into a compare and conditional negate. This is the most obvious
expansion, enables
merging of the comparison into ALU instructions and is faster on all
implementations. Bootstrapped &
regression tested.
int f(int x) { return abs (x + 1); }
Before:
add w0, w0, 1
sx
On Sat, Feb 28, 2015 at 10:14 AM, Max Filippov wrote:
> call0 is an ABI that doesn't use register windows.
>
> 2015-02-28 Max Filippov
>
> gcc/
> * config/xtensa/constraints.md ("a" constraint): Include stack
> pointer in case of call0 ABI.
> ("q" constraint): Make empty
On 19/02/15 17:26, Richard Henderson wrote:
On 02/19/2015 09:08 AM, Alex Velenko wrote:
Your suggestion seem to fix gcc.target/arm/long-calls-1.c, but has to be
thoroughly tested.
Before you do complete testing, please also delete the TREE_STATIC test.
That bit should never be relevant to func
On Tue, Mar 3, 2015 at 7:05 AM, Max Filippov wrote:
> On Tue, Mar 3, 2015 at 5:51 PM, Richard Henderson wrote:
>> On 03/02/2015 10:42 PM, Max Filippov wrote:
>>> gcc/
>>> * builtins.c (expand_builtin_return_addr): Add
>>> RETURN_ADDR_IN_PREVIOUS_FRAME to 'if' condition, remove
>>>
Hi DJ,
Compiling this test case with the RL78 toolchain results in an ICE:
unsigned char g_X;
unsigned char g_Y;
const __far unsigned char TabA[2][2] = { { 3, 0 }, { 4, 5 } };
void main (void) { g_X = 1; g_Y = TabA[g_X][g_X]; }
The problem appears to be that GCC will create a
Hi DJ,
Compiling a program with one of the real RL78 registers marked as
call-used (eg via -fcall-used-ax), can trigger an internal compiler
error. Although modifying the ABI in this way may not be a good idea,
it is still wrong that the compiler generates an ICE.
The patch below fixes
2015-03-03 Joern Rennecke
* MAINTAINERS (CPU Port Maintainers): Update my email address.
2015-03-03 Joern Rennecke
* MAINTAINERS (CPU Port Maintainers): Update my email address.
Index: MAINTAINERS
===
--- MAINTAINERS
On 3 March 2015 at 13:44, Alex Velenko wrote:
> On 02/03/15 22:04, Christophe Lyon wrote:
>>
>> On 2 March 2015 at 21:21, Jan Hubicka wrote:
On 01/03/15 16:47, Christophe Lyon wrote:
>
> On 27 February 2015 at 21:49, Jan Hubicka wrote:
>>>
>>>
>>> ../sysde
On Tue, Mar 3, 2015 at 5:51 PM, Richard Henderson wrote:
> On 03/02/2015 10:42 PM, Max Filippov wrote:
>> gcc/
>> * builtins.c (expand_builtin_return_addr): Add
>> RETURN_ADDR_IN_PREVIOUS_FRAME to 'if' condition, remove
>> surrounding #ifdef.
>> * config/sparc/sparc.h (RETU
On 03/02/2015 10:42 PM, Max Filippov wrote:
> gcc/
> * builtins.c (expand_builtin_return_addr): Add
> RETURN_ADDR_IN_PREVIOUS_FRAME to 'if' condition, remove
> surrounding #ifdef.
> * config/sparc/sparc.h (RETURN_ADDR_IN_PREVIOUS_FRAME): Change
> definition to 1.
>
On Tue, Mar 3, 2015 at 12:12 AM, Alan Modra wrote:
> On Mon, Mar 02, 2015 at 05:36:24AM -0800, H.J. Lu wrote:
>> On Sun, Mar 1, 2015 at 11:48 PM, Uros Bizjak wrote:
>> > On Sat, Feb 28, 2015 at 5:42 PM, H.J. Lu wrote:
>> >> Ue copy relocation in PIE improves performance. But copy relocation
>>
On Mon, Mar 2, 2015 at 7:23 PM, Richard Henderson wrote:
>> Unfortunately, while "normal" bootstrap works OK with alpha-linux-gnu
>> and alphaev68-linux-gnu, the bootstrap still fails when configured
>> --with-build-config=bootstrap-lto:
>>
>> libbackend.a(tree-data-ref.o): In function `non_affin
As discussed last year, we've converted the GNAT main documentation
(gnat_rm.texi and gnat_ugn.texi) to reST/sphinx, so the master doc
can now be found under gcc/ada/doc.
there is an empty directory left in SVN:
gcc/ada/doc/share/_static
ok to remove?
No, this (empty currently) directory
On Mon, Mar 02, 2015 at 08:40:17PM +0100, Georg-Johann Lay wrote:
> The new specs file generation introduces several problems. This patch
>
> - Fix build warnings
>
> - Clean up unused code and the old, now dead specs definitions.
>
> - Removes unused files and adjust build scripts / rules.
>
On 02/03/15 22:04, Christophe Lyon wrote:
On 2 March 2015 at 21:21, Jan Hubicka wrote:
On 01/03/15 16:47, Christophe Lyon wrote:
On 27 February 2015 at 21:49, Jan Hubicka wrote:
../sysdeps/gnu/siglist.c:72:1: internal compiler error: in address_matters_p,
at symtab.c:1908
versioned_sym
As outlined in the PR, a relatively recent change to DOM unconditionally
modifies the available expression hash table after a partial hit, but a
failure by the alias walker to find an available expression.
This interacts badly with jump threading which modifies gimple
statements in-place, trie
On Tue, Mar 3, 2015 at 11:40 AM, Martin Liška wrote:
> Hello.
>
> There's suggested patch for the issue, where regression test have been
> running on x86_64-linux-pc.
>
> Ready for trunk after tests will be finished?
Ok.
Thanks,
Richard.
> Thanks,
> Martin
On 02/20/2015 10:17 AM, Arnaud Charlet wrote:
> As discussed last year, we've converted the GNAT main documentation
> (gnat_rm.texi and gnat_ugn.texi) to reST/sphinx, so the master doc
> can now be found under gcc/ada/doc.
there is an empty directory left in SVN:
gcc/ada/doc/share/_static
ok t
Hello.
There's suggested patch for the issue, where regression test have been running
on x86_64-linux-pc.
Ready for trunk after tests will be finished?
Thanks,
Martin
>From a186fc97c5f192b94cc571f611a50eb0f4f8a354 Mon Sep 17 00:00:00 2001
From: mliska
Date: Tue, 3 Mar 2015 11:33:30 +0100
Subje
Hello.
There's suggested patch for the issue that survives regression tests on
x86_64-linux-pc.
Ready for trunk?
Thanks,
Martin
>From d4e9d2906a90151d1239e7b32b00f7cb46b045fa Mon Sep 17 00:00:00 2001
From: mliska
Date: Tue, 3 Mar 2015 10:36:59 +0100
Subject: [PATCH] Fix PR ipa/65287.
gcc/test
On 03/03/2015 10:50 AM, Marek Polacek wrote:
On Tue, Mar 03, 2015 at 10:31:17AM +0100, Martin Liška wrote:
@@ -2952,9 +2959,12 @@ sem_item_optimizer::merge_classes (unsigned int
prev_class_count)
alias->dump_to_file (dump_file);
}
- source->merge (alias)
On Tue, Mar 03, 2015 at 10:31:17AM +0100, Martin Liška wrote:
> @@ -2952,9 +2959,12 @@ sem_item_optimizer::merge_classes (unsigned int
> prev_class_count)
> alias->dump_to_file (dump_file);
> }
>
> - source->merge (alias);
> + if (source->merge (alias))
On Tue, Mar 3, 2015 at 9:56 AM, Eric Botcazou wrote:
> Hi,
>
> the test fails at -O2 because of the assertion recently added to gen_reg_rtx
> that crtl->emit.regno_pointer_align_length is non-zero. Now the function
> expand_vec_perm_interleave_2 can be invoked at the GIMPLE level in "testing"
> m
And I forgot to update the date in the rush so I committed the fix as
obvious:
diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index ed8ca31..809f5cf 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,4 +1,4 @@
-2015-01-23 Thomas Preud'homme
+2015-03-03 Thomas Preud'homme
Backport fro
Steve Ellcey writes:
> On Mon, 2015-03-02 at 15:54 -0800, Matthew Fortune wrote:
> > Thanks for looking through and catching this.
> >
> > I had conflicting thoughts on whether the new condition should
> > reference both !ISA_HAS_FP_CONDMOVE || ISA_HAS_SEL but if we take it
> > that FP_CONDMOVE is
Just committed to 4.9 branch, 4.8 to follow once regression testsuite for
4.8 backport finishes running (backport was done quite some time ago now).
Best regards,
Thomas
> -Original Message-
> From: Ramana Radhakrishnan [mailto:ramana@googlemail.com]
> Sent: Tuesday, February 17, 201
On 03/02/2015 06:20 PM, Jan Hubicka wrote:
Hello.
Following one line patch is fix for PR ipa/65087. No regression has been
seen on x86_64-linux-pc.
OK, it won't happend large programs, but can you arrange its execute method to
return TODO_remove_functions only if some merging actually happened
Hi,
the test fails at -O2 because of the assertion recently added to gen_reg_rtx
that crtl->emit.regno_pointer_align_length is non-zero. Now the function
expand_vec_perm_interleave_2 can be invoked at the GIMPLE level in "testing"
mode and crtl->emit is not initialized by that time.
Tested on
On Mon, 2 Mar 2015, Jan Hubicka wrote:
> Hi,
> in the testcase bellow we manage to produce cycle in inline plan because
> there is
> indirectly recrusive function that appears called once.
>
> I am having problems with the testcase. The second file outht to be compiled
> with
> -O2, but it is n
On 02/25/2015 05:40 PM, Maxim Ostapenko wrote:
On 02/16/2015 10:58 AM, Maxim Ostapenko wrote:
Hi,
when testing I noticed, that if compile with both -fsanitize=address and
-fstack-protector for 32-bit architectures and run with
ASAN_OPTIONS=detect_stack_use_after_return=1, libsanitizer fails wi
On Mon, Mar 02, 2015 at 05:36:24AM -0800, H.J. Lu wrote:
> On Sun, Mar 1, 2015 at 11:48 PM, Uros Bizjak wrote:
> > On Sat, Feb 28, 2015 at 5:42 PM, H.J. Lu wrote:
> >> Ue copy relocation in PIE improves performance. But copy relocation
> >> can't be used to access protected symbols defined in sh
67 matches
Mail list logo