On Thu, Dec 13, 2012 at 11:44:12AM +0400, Konstantin Serebryany wrote:
We are discussing it from time to time.
Sometimes, if e.g. an error happens inside a qsort callback,
the fp-based unwinder fails to unwind through libc, while _Unwind would work.
I was opposed to this sometime ago because
On Thu, Dec 13, 2012 at 10:38:13AM +0400, Dmitry Vyukov wrote:
On Wed, Dec 12, 2012 at 11:50 PM, Jakub Jelinek ja...@redhat.com wrote:
Various TM tests ICE when built with -fgnu-tm -fsanitizer=address.
The problem is that asan.c pass adds calls to builtins that weren't there
before and TM
When PRE creates representatives for expressions during PHI translation it never
releases the SSA names again. The following patch arranges for SCCVN to release
them.
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk.
Richard.
p3
Description: Binary data
On Wed, Dec 12, 2012 at 6:43 PM, Steven Bosscher stevenb@gmail.com wrote:
Hello,
This patch fixes a regression in the size of bitmap_head that resulted
from the removal of all the #ifdef GATHER_STATISTICS tests.
Instead of a pointer to a descriptor, this patch gives each bitmap an
Hi!
As the PR54402 go testcase shows, sometimes we end up with VALUEs with huge
(many thousands) long locs lists because we add too many reverse ops for the
same value. Such huge locs lists of equivalences are hardly useful in
practice, reverse ops mainly help when there are just a few locs and
On Wed, Dec 12, 2012 at 7:32 PM, Uros Bizjak ubiz...@gmail.com wrote:
On Wed, Dec 12, 2012 at 3:45 PM, Richard Biener
richard.guent...@gmail.com wrote:
I assume that this is not right way for fixing such simple performance
anomaly since we need to do redundant work - combine load to
On Wed, Dec 12, 2012 at 5:23 PM, Kai Tietz ktiet...@googlemail.com wrote:
2012/12/12 Richard Biener richard.guent...@gmail.com:
On Wed, Dec 12, 2012 at 4:11 PM, Richard Henderson r...@redhat.com wrote:
On 12/12/2012 02:57 AM, Richard Biener wrote:
That looks wrong. Having both TYPE_PACKED and
On Wed, Dec 12, 2012 at 8:41 PM, Jakub Jelinek ja...@redhat.com wrote:
Hi!
On the attached testcase prg_ctr_mask is non-zero, presumably set
while there still were some functions in the TU, but later on none of them
are being emitted. This leads to n_functions in coverage_obj_finish being
On Thu, Dec 13, 2012 at 11:03:58AM +0100, Richard Biener wrote:
struct X
{
char c;
short s;
char c2;
short s2;
} __attribute__((packed,aligned(2)));
As struct-layout-1.exp tests show, this is something that was ABI-wise
changed already several times. That said, for
On Wed, Dec 12, 2012 at 8:58 PM, Jakub Jelinek ja...@redhat.com wrote:
Hi!
Before http://gcc.gnu.org/viewcvs?root=gccview=revrev=193595
aka the big vec.h changes, nonlocalized_list parameter to remap_decls
used to be **, but the changes changed it to *. That is a problem,
because then there
On 12/12/2012 06:04 PM, Andi Kleen wrote:
Richard Biener rguent...@suse.de writes:
Probably not suitable for trunk because I use popen/pclose/fileno
which I don't know whether they are available on all host platforms.
Just add a ifdef HAVE_popen or somesuch around it?
Better yet, use the
On Thu, Dec 13, 2012 at 10:51 AM, Richard Biener
richard.guent...@gmail.com wrote:
I assume that this is not right way for fixing such simple performance
anomaly since we need to do redundant work - combine load to
conditional and then split it back in peephole2? Does it look
reasonable? Why
On Thu, Dec 13, 2012 at 11:07 AM, Jakub Jelinek ja...@redhat.com wrote:
On Thu, Dec 13, 2012 at 11:03:58AM +0100, Richard Biener wrote:
struct X
{
char c;
short s;
char c2;
short s2;
} __attribute__((packed,aligned(2)));
As struct-layout-1.exp tests show, this is something that
On Thu, Dec 13, 2012 at 11:20 AM, Uros Bizjak ubiz...@gmail.com wrote:
On Thu, Dec 13, 2012 at 10:51 AM, Richard Biener
richard.guent...@gmail.com wrote:
I assume that this is not right way for fixing such simple performance
anomaly since we need to do redundant work - combine load to
On Thu, Dec 13, 2012 at 11:29:47AM +0100, Richard Biener wrote:
On Thu, Dec 13, 2012 at 11:07 AM, Jakub Jelinek ja...@redhat.com wrote:
On Thu, Dec 13, 2012 at 11:03:58AM +0100, Richard Biener wrote:
struct X
{
char c;
short s;
char c2;
short s2;
}
This fixes PR55660 where a different setting of -f[un-]signed-char
at compile vs. link-time lead to ICEs. This is because the
not preloading of char_type_node isn't working on i?86 as
char_type_node is reached via its va_list_type_node type
(which is char *, not unsigned char * or signed char *
*ping*
Thanks,
James
-Original Message-
From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-
ow...@gcc.gnu.org] On Behalf Of James Greenhalgh
Sent: 06 December 2012 15:10
To: gcc-patches@gcc.gnu.org
Cc: Marcus Shawcroft
Subject: [AArch64] Add support for
2012/12/13 Jakub Jelinek ja...@redhat.com:
On Thu, Dec 13, 2012 at 11:29:47AM +0100, Richard Biener wrote:
On Thu, Dec 13, 2012 at 11:07 AM, Jakub Jelinek ja...@redhat.com wrote:
On Thu, Dec 13, 2012 at 11:03:58AM +0100, Richard Biener wrote:
struct X
{
char c;
short s;
char
Hi John,
John Tytgatj...@bass-software.com
* config/arm/fp16.c (__gnu_f2h_internal): Fix inaccuracy.
Approved and applied.
Cheers
Nick
On 06/12/12 15:09, James Greenhalgh wrote:
Hi,
This patch wires up TARGET_VECTORIZE_AUTOVECTORIZE_VECTOR_SIZES on
AArch64 to allow for 64-bit or 128-bit vectorization if the
preferred vector mode is not acceptable.
The patch has been regression tested on aarch64-none-elf with
no regressions.
On Thu, Dec 13, 2012 at 1:02 PM, Kai Tietz ktiet...@googlemail.com wrote:
2012/12/13 Jakub Jelinek ja...@redhat.com:
On Thu, Dec 13, 2012 at 11:29:47AM +0100, Richard Biener wrote:
On Thu, Dec 13, 2012 at 11:07 AM, Jakub Jelinek ja...@redhat.com wrote:
On Thu, Dec 13, 2012 at 11:03:58AM
Hi!
I keep seeing a -Wsign-compare warning from GCC 4.7 here,
alloc_count is const int = 16, but length () method returns unsigned.
Perhaps GCC 4.8 folds it to 16 early enough for the -Wsign-compare warning
(might be related to the sizeof changes), but still I think it doesn't hurt
to fix this
On Thu, Dec 13, 2012 at 1:45 PM, Jakub Jelinek ja...@redhat.com wrote:
Hi!
I keep seeing a -Wsign-compare warning from GCC 4.7 here,
alloc_count is const int = 16, but length () method returns unsigned.
Perhaps GCC 4.8 folds it to 16 early enough for the -Wsign-compare warning
(might be
Hi Guys,
The patch proposed by Uros is useless since we don't have free scratch
register to do splitting of memory operand:
;; regs ever live 0[ax] 1[dx] 2[cx] 3[bx] 4[si] 5[di] 6[bp] 7[sp]
17[flags]
...
(insn 96 131 132 7 (set (reg/v/f:SI 6 bp [orig:70 trie_root ] [70])
Hi all,
This patch adds a copyright notice to the recently added unspecs.md file in
the arm backend (r193204). The file just gathered some unspecs definitions
into one place.
Ok for trunk?
Thanks,
Kyrill
gcc/ChangeLog
2012-12-13 Kyrylo Tkachov kyrylo.tkac...@arm.com
*
On Thu, Dec 13, 2012 at 3:23 PM, Yuri Rumyantsev ysrum...@gmail.com wrote:
The patch proposed by Uros is useless since we don't have free scratch
register to do splitting of memory operand:
;; regs ever live 0[ax] 1[dx] 2[cx] 3[bx] 4[si] 5[di] 6[bp] 7[sp]
17[flags]
...
(insn 96
OK.
Jason
On Thu, Dec 13, 2012 at 3:23 PM, Yuri Rumyantsev ysrum...@gmail.com wrote:
Hi Guys,
The patch proposed by Uros is useless since we don't have free scratch
register to do splitting of memory operand:
;; regs ever live 0[ax] 1[dx] 2[cx] 3[bx] 4[si] 5[di] 6[bp] 7[sp]
17[flags]
...
On Thu, Dec 13, 2012 at 3:27 PM, Uros Bizjak ubiz...@gmail.com wrote:
The patch proposed by Uros is useless since we don't have free scratch
register to do splitting of memory operand:
;; regs ever live 0[ax] 1[dx] 2[cx] 3[bx] 4[si] 5[di] 6[bp] 7[sp]
17[flags]
...
(insn 96 131
Uros,
We did not see any performance improvement on Atom in 32-bit mode at
routelookup from eembc_2_0 (eembc_1_1).
Best regards.
Yuri.
2012/12/13 Uros Bizjak ubiz...@gmail.com:
On Thu, Dec 13, 2012 at 3:27 PM, Uros Bizjak ubiz...@gmail.com wrote:
The patch proposed by Uros is useless since
On Fri, Dec 07, 2012 at 10:31:57AM -0500, Diego Novillo wrote:
On Thu, Dec 6, 2012 at 1:12 PM, Bernhard Reutner-Fischer
rep.dot@gmail.com wrote:
@@ -210,12 +211,12 @@ def IsInterestingResult(line):
if '|' in line:
(_, line) = line.split('|', 1)
line = line.strip()
- return
This fixes the weak alias regression FAIL: gcc.dg/attr-weakref-1.c.
I tracked it down to the fact that this has both DECL_EXTERNAL
and TREE_STATIC (and RTL) set but the trunk code differs from the
4.7 code in that it has an additional !DECL_EXTERNAL check
(for no apparent reason, that is, no
On Thu, Dec 13, 2012 at 02:22:52PM +0400, Konstantin Serebryany wrote:
I've added two flags, both on by default for now.
// Use fast (frame-pointer-based) unwinder on fatal errors (if available).
bool fast_unwind_on_fatal;
// Use fast (frame-pointer-based) unwinder on malloc/free (if
Hello!
Could you please take a look at the attached patch that implements
the target libc_has_function hook? I didn't change so far the default presence
of c99, but rather tried to preserve the current behaviour of
TARGET_C99_FUNCTIONS.
Thank you,
Alexander
2012/12/1 Joseph S. Myers
On Fri, Dec 7, 2012 at 5:15 AM, H.J. Lu hjl.to...@gmail.com wrote:
On Wed, Nov 28, 2012 at 8:18 PM, H.J. Lu hjl.to...@gmail.com wrote:
Binutils supports 2 linkers, ld.gold and ld.bfd. One of them is
configured as the default linker, ld, which is used by GCC. Sometimes,
we want to use the
On Thu, Dec 13, 2012 at 4:02 PM, Yuri Rumyantsev ysrum...@gmail.com wrote:
We did not see any performance improvement on Atom in 32-bit mode at
routelookup from eembc_2_0 (eembc_1_1).
I assume that for x86_64 the patch works as expected. Let's take a
bigger hammer for 32bit targets - the
Season's greetings to you! :)
I've made zero_extend versions of SI mode patterns that write
to W registers in order to make the implicit zero_extend that
they do explicit, so GCC can be smarter about when it actually
needs to plant a zero_extend (uxtw).
This patch significantly reduces the
Hi Ahmad and Luis, I'd like you to do a code review.
Temporarily adds below failures to baseline. Needs to be removed after
a thorough analysis/fix.
Patch below -
--- /dev/null 2012-11-30 16:08:50.372341021 -0800
+++ contrib/testsuite-management/i686-pc-linux-gnu 2012-12-13
10:30:39.496677271
Hi,
On 12/13/2012 07:42 PM, Steve Ellcey wrote:
I have been running the libstdc++ testsuite using the gnu simulator for
my mips-mti-elf target. Some of the tests fail due to the amount of
memory they need and I would like to 'shrink' them when run on a
simulator using the same technique I see
Try the following one. 1) -minline-all-stringops
-mstringop-strategy=rep_8byte -O2 vs 1) -mstringop_strategy=libcall
-O2.
David
#include string.h
#include stdio.h
#include stdlib.h
#ifndef LEN
#define LEN 16
#endif
void copy(char* s1, char* s2,int len) __attribute__((noinline));
On Wed, Dec 12, 2012 at 7:32 PM, Uros Bizjak ubiz...@gmail.com wrote:
On Wed, Dec 12, 2012 at 3:45 PM, Richard Biener
richard.guent...@gmail.com wrote:
I assume that this is not right way for fixing such simple performance
anomaly since we need to do redundant work - combine load to
Honza, I think the pass manager should call default_rtl_profile () before
each
RTL pass to avoid this, no?
Please note that we have plenty of existing peephole2s that use
optimize_insn_for_speed_p predicate. It is assumed to work ...
It is set by peep2 pass
static void
Here's me wishing that NOTE_INSN_CALL_ARG_LOCATION was a reg note
and not an insn note. Mainly because it's required to be adjacent
to the call insn, modulo barriers apparently.
This just started failing recently, though I didn't check to see
what sort of other change might have precipitated
On Wed, Dec 12, 2012 at 10:21 PM, Jakub Jelinek ja...@redhat.com wrote:
On Wed, Dec 12, 2012 at 10:09:14PM -0800, Xinliang David Li wrote:
On Wed, Dec 12, 2012 at 5:19 PM, Jan Hubicka hubi...@ucw.cz wrote:
libcall is not faster up to 8KB to rep sequence that is better for
regalloc/code
Hi,
On Thu, Dec 13, 2012 at 11:05:49AM -0800, Lawrence Crowl wrote:
On 12/12/12, Jakub Jelinek ja...@redhat.com wrote:
On Tue, Dec 11, 2012 at 02:44:41PM -0800, Lawrence Crowl wrote:
+/* Hash a tree in a uid_decl_map. */
+
+inline hashval_t
+uid_decl_hasher::hash (const value_type
On Thu, Dec 13, 2012 at 11:37:24AM -0800, Richard Henderson wrote:
Here's me wishing that NOTE_INSN_CALL_ARG_LOCATION was a reg note
and not an insn note. Mainly because it's required to be adjacent
to the call insn, modulo barriers apparently.
It can be changed (though I guess for 4.9?).
On Thu, 2012-12-13 at 20:27 +0100, Paolo Carlini wrote:
It's the same approach that we are following in many other places, right?
Yes, it is the same approach.
OK to checkin?
Yes, thanks.
Done.
Steve Ellcey
sell...@mips.com
On Wed, Dec 12, 2012 at 10:21 PM, Jakub Jelinek ja...@redhat.com wrote:
On Wed, Dec 12, 2012 at 10:09:14PM -0800, Xinliang David Li wrote:
On Wed, Dec 12, 2012 at 5:19 PM, Jan Hubicka hubi...@ucw.cz wrote:
libcall is not faster up to 8KB to rep sequence that is better for
On Thu, Dec 13, 2012 at 12:26 PM, Jan Hubicka hubi...@ucw.cz wrote:
On Wed, Dec 12, 2012 at 10:21 PM, Jakub Jelinek ja...@redhat.com wrote:
On Wed, Dec 12, 2012 at 10:09:14PM -0800, Xinliang David Li wrote:
On Wed, Dec 12, 2012 at 5:19 PM, Jan Hubicka hubi...@ucw.cz wrote:
libcall is not
On 12/13/2012 09:25 AM, Ian Bolton wrote:
+ add\\t%w0, %w2, %w, suxtSHORT:size
^^^ %w1
r~
Here we speak about memcpy/memset only. I never got around to modernize
strlen and friends, unfortunately...
memcmp and friends are different beats. They realy need some TLC...
memcpy and memset in glibc are also extremely fast.
The default strategy now is to inline only when the
The following patch fixes most GCC testsuite regressions on PPC64 for
LRA (now there are only 2 failed tests in comparison with reload. the
tests expect a specific assembler code and LRA generates a bit different
code).
The patch also adds a parameter can be used to form EBB in which
On 12/13/2012 09:25 AM, Ian Bolton wrote:
This patch significantly reduces the number of redundant
uxtw instructions seen in a variety of programs.
(There are further patterns that can be done, but I have them
in a separate patch that's still in development.)
What do you get if you enable
On Thu, Dec 13, 2012 at 12:40 PM, Jan Hubicka hubi...@ucw.cz wrote:
Here we speak about memcpy/memset only. I never got around to modernize
strlen and friends, unfortunately...
memcmp and friends are different beats. They realy need some TLC...
memcpy and memset in glibc are also
Hi
As part of a performance patch proposed in an other mailing thread
was a patch to improve management of hash functor with state. This part
is I think less sensible than the performance patch so I propose it
independently. I only would like to commit the modification on the
performance
me libc starts to be win only for rather large blocks (i.e. 8KB)
Which glibc are you using?
2.15 as it comes with opensuse 12.2
Honza
--
H.J.
This patch to the Go compiler fixes it to accept a trailing comma after
a varargs parameter, as required by the language spec.
Testing that uncovered a couple of other related bugs: the compiler was
not seeing interface types that appeared only in a function or method
declaration, as in
func (t
Hello,
Looks like a remnant from pre GIMPLE tuples days.
Will commit as obvious.
Ciao!
Steven
* cgraph.c (verify_cgraph_node): Don't allocate/free visited_nodes set.
Index: cgraph.c
===
--- cgraph.c(revision 194491)
The attached patch disables two asan tests introduced in r194458 which are
invalid
on darwin. The g++.dg/asan/interception-test-1.C test is invalid on darwin
since
mac function interposition is used instead of interception. The
c-c++-common/asan/swapcontext-test-1.c
test is invalid as
Hi,
As per discussion in http://gcc.gnu.org/ml/gcc/2012-12/msg00056.html,
the attached patch updates loop unroll passes to use new dump
infrastructure.
This patch filters relevant dump messages into the following
three categories
- optimized: an optimization was successfully applied
- missed:
-Original Message-
From: H.J. Lu [mailto:hjl.to...@gmail.com]
Sent: Tuesday, November 27, 2012 12:56
To: Joey Ye
Cc: gcc-patches@gcc.gnu.org
Subject: Re: [PATCH, libgcc] Make possible to disable JCR in crtstuff.c
OK to trunk?
2012-09-21 Joey Ye joey...@arm.com
A couple of comments:
1) please dump with source location if possible. What is the use of
the information if there is no line number
2) Please do not use the existing dump report -- Loop 1,2,3 etc means
nothing to user
3) The optimization report should be standardized with some template
(similar
Hello Everyone,
This patch is for the Cilk Plus branch mainly affecting both C and C++
compilers. Some of the pragma simd structure's indices were getting lost and
this patch will restore them. It will also insert the appropriate error
messages when assert (in pragma simd) is requested
On Thu, Dec 13, 2012 at 8:58 PM, Xinliang David Li davi...@google.com wrote:
A couple of comments:
1) please dump with source location if possible. What is the use of
the information if there is no line number
The google branches have the code to identify a source location of the
loop, and a
2012-12-13 Teresa Johnson tejohn...@google.com
PR gcov-profile/55674
* lto-cgraph.c (merge_profile_summaries): Set min correctly the
first time we merge into a histogram entry.
Index: lto-cgraph.c
===
---
-Original Message-
From: H.J. Lu [mailto:hjl.to...@gmail.com]
Sent: Friday, December 14, 2012 11:55
To: Joey Ye
Cc: gcc-patches@gcc.gnu.org; Joseph Prostko
Subject: Re: [PATCH, libgcc] Make possible to disable JCR in crtstuff.c
2012-12-12 Joey Ye joey...@arm.com
*
2012-12-13 Teresa Johnson tejohn...@google.com
PR gcov-profile/55674
* lto-cgraph.c (merge_profile_summaries): Set min correctly the
first time we merge into a histogram entry.
OK,
thanks!
Honza
On Thu, Dec 13, 2012 at 08:21:49PM -0500, Jack Howarth wrote:
2012-12-13 Jack Howarth howa...@bromo.med.uc.edu
PR 55679/sanitizer
PR sanitizer/55679 instead
* g++.dg/asan/interception-test-1.C: Skip on darwin.
This I can live with.
---
68 matches
Mail list logo