2013/6/16 Ondřej Bílka nel...@seznam.cz:
On Sat, Jun 15, 2013 at 05:13:31PM +0800, Chung-Ju Wu wrote:
2013/6/14 Joseph S. Myers jos...@codesourcery.com:
On Thu, 13 Jun 2013, Richard Biener wrote:
Btw, rather than these kind of patches I'd appreciate if someone would
look
at a simple
On Mon, Jun 17, 2013 at 07:59:15PM -0500, Aldy Hernandez wrote:
As discussed on IRC. Attached are these changes you requested, plus
changing OMP_CLAUSE__SIMDUID__UID to OMP_CLAUSE__SIMDUID__DECL.
I will tackle the dot named builtins in the next iteration.
Thanks.
---
On Mon, Jun 17, 2013 at 6:54 PM, Joseph S. Myers
jos...@codesourcery.com wrote:
On Mon, 17 Jun 2013, Jakub Jelinek wrote:
The following patch shows a performance regression caused by the C++ changes
to evaluate side-effects in x += foo (); first and only then do the
addition. Previously if x
Hi all,
This patch adjusts a number of patterns in arm.md for the IT block rules
in -mrestrict-it. The changes are mostly straightforward, disabling the
cond_exec version for -mrestrict-it in some cases and adding
alternatives that can produce a 16-bit encoding in others.
Bootstrapped on
On 13/6/18 上午3:05, Sandra Loosemore wrote:
Ping? I think these are the most recent versions of the unreviewed
patches in the series:
http://gcc.gnu.org/ml/gcc-patches/2013-06/msg00287.html
http://gcc.gnu.org/ml/gcc-patches/2013-05/msg00760.html
Hi,
During expand, function vcondmodemode inverses some CMP, e.g.
a LE b - b GE a
But if b is CONST0_RTX, b GE a will be an illegal insn.
(insn 933 932 934 113 (set (reg:V4SI 1027)
(unspec:V4SI [
(const_vector:V4SI [
(const_int 0 [0])
Hi,
the patch replaces next_real_insn with next_active_insn when checking
for JUMP TABLE insns.
This fixes ESA mode bootstrap on s390 which broke with r197266.
Comitted to mainline
Bye,
-Andreas-
2013-06-18 Andreas Krebbel andreas.kreb...@de.ibm.com
PR target/57609
*
On Tue, Jun 18, 2013 at 10:57 AM, Andreas Krebbel
kreb...@linux.vnet.ibm.com wrote:
Hi,
the patch replaces next_real_insn with next_active_insn when checking
for JUMP TABLE insns.
This fixes ESA mode bootstrap on s390 which broke with r197266.
Comitted to mainline
Please revert this and
2013/6/18 Steven Bosscher stevenb@gmail.com:
On Tue, Jun 18, 2013 at 10:57 AM, Andreas Krebbel
kreb...@linux.vnet.ibm.com wrote:
Hi,
the patch replaces next_real_insn with next_active_insn when checking
for JUMP TABLE insns.
This fixes ESA mode bootstrap on s390 which broke with
On Tue, Jun 18, 2013 at 11:29 AM, Chung-Ju Wu wrote:
Do you mean that your comment 6 in PR56809 is not a good solution
since you are going to remove JUMP_TABLE_DAT from active_insn_p ??
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56809
No, that one I'll fix, it's fall-out from the
On 06/18/13 09:50, Zhenqiang Chen wrote:
Hi,
During expand, function vcondmodemode inverses some CMP, e.g.
a LE b - b GE a
But if b is CONST0_RTX, b GE a will be an illegal insn.
(insn 933 932 934 113 (set (reg:V4SI 1027)
(unspec:V4SI [
(const_vector:V4SI [
On Mon, Jun 17, 2013 at 6:11 PM, Jakub Jelinek ja...@redhat.com wrote:
This instruction has the predicates/constraints wrong, the r/m argument is
the middle-one, the value from which it should be extracted, rather than
the packed start/length argument. This got broken with PR50766, where the
On Tue, Jun 18, 2013 at 11:47:29AM +0200, Uros Bizjak wrote:
On Mon, Jun 17, 2013 at 6:11 PM, Jakub Jelinek ja...@redhat.com wrote:
This instruction has the predicates/constraints wrong, the r/m argument is
the middle-one, the value from which it should be extracted, rather than
the
This fixes PR57334, properly merging the chain of symtab nodes
sharing the same assembler name.
LTO bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk.
Richard.
2013-06-18 Richard Biener rguent...@suse.de
PR lto/57334
* lto-symtab.c
Thanks,
Julian
ChangeLog
gcc/
* config/arm/arm.c (neon_vector_mem_operand): Add strict argument.
Permit virtual register pre-reload if !strict.
(coproc_secondary_reload_class): Adjust for neon_vector_mem_operand
change.
* config/arm/arm-protos.h
On Mon, 17 Jun 2013, Andi Kleen wrote:
Current trunk cannot build LTO kernels now, with your patch,
as reported earlier.
Please fix ASAP. I'm surprised that you checked in a patchkit
with such serious reported problems.
-Andi
backup/lsrc/git/linux-lto-2.6/lib/decompress_unlzo.c:
Hello Chung-Ju,
On 06/18/2013 05:12 AM, Chung-Ju Wu wrote:
2013/6/18 David Edelsohn dje@gmail.com:
gcc/testsuite/ChangeLog
2013-06-17 Sebastian Huber sebastian.hu...@embedded-brains.de
PR target/55033
* gcc.target/powerpc/pr55033.c: Fix options.
Okay.
Thanks, David
P.S. Please
Hi,
On 06/08/2013 10:57 AM, Paolo Carlini wrote:
Hi,
the bug reminds us to update the documentation about the value of
__cplusplus. I tentatively prepared the below, is it clear enough?
We could probably apply something to the branch too, without the
-std=c++1y bits, thus end simply like
Ping?
On Mon, Jun 10, 2013 at 2:13 PM, Igor Zamyatin izamya...@gmail.com wrote:
Hi!
This patch mentions support of Silvermont architecture in the
gcc-4.9/changes.html page.
OK to install?
Thanks,
Igor
Index: htdocs/gcc-4.9/changes.html
On Tue, Jun 18, 2013 at 3:52 AM, Oleg Endo oleg.e...@t-online.de wrote:
On Mon, 2013-06-17 at 10:41 +0200, Eric Botcazou wrote:
My mistake. It's because arm_legitimize_address cannot re-factor [r105 +
r165*4 + (-2064)] into rx = r105 + (-2064); [rx + r165*4]. Do you
suggest that this kind
On Tue, Jun 18, 2013 at 11:51 AM, Jakub Jelinek ja...@redhat.com wrote:
This instruction has the predicates/constraints wrong, the r/m argument is
the middle-one, the value from which it should be extracted, rather than
the packed start/length argument. This got broken with PR50766, where
On Mon, 17 Jun 2013, Andi Kleen wrote:
Richard Biener rguent...@suse.de writes:
Andi Kleen a...@firstfloor.org wrote:
Current trunk cannot build LTO kernels now, with your patch,
as reported earlier.
Please fix ASAP. I'm surprised that you checked in a patchkit
with such serious
Ping.
Th patch is OK, thanks!
I see you added gcov.exp file support, do you have a testcases?
Honza
When I fixed PR57630, I failed to adjust the expected message in
gcc.dg/c90-fordecl-1.c test (sorry for that); so we regressed. Fixed
thusly, will commit as obvious.
Tested with make check-gcc RUNTESTFLAGS=dg.exp=c90-fordecl-1.c
2013-06-18 Marek Polacek pola...@redhat.com
*
On 18/06/13 11:35, Steven Bosscher wrote:
On Tue, Jun 18, 2013 at 11:29 AM, Chung-Ju Wu wrote:
Do you mean that your comment 6 in PR56809 is not a good solution
since you are going to remove JUMP_TABLE_DAT from active_insn_p ??
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56809
No, that one
Hello!
Attached patch initializes return variable in get_fpu_except_flags.
Additionally, it uses __asm__ and __volatile__ consistently, as
recommended for header files and unifies a bunch of formatting issues
throughout the file.
2012-06-18 Uros Bizjak ubiz...@gmail.com
*
It is not necessary to maintain the pointer-map from cache entry to
cache index when reading trees. A quick estimate using the latest
WPA stats from firefox estimates its size to at least 1.5GB - apart
from the cost to maintain it. So the following patch makes it
possible to not maintain that
On Tue, Jun 18, 2013 at 10:59:56AM +0200, Steven Bosscher wrote:
On Tue, Jun 18, 2013 at 10:57 AM, Andreas Krebbel
kreb...@linux.vnet.ibm.com wrote:
Hi,
the patch replaces next_real_insn with next_active_insn when checking
for JUMP TABLE insns.
This fixes ESA mode bootstrap on s390
Hi,
This is the first of a series of patches to implement a single, unified and
fine grained instruction classification attribute.
The first few patches will propose a refactoring of the instruction
classifications we currently have in place.
This first patch moves the multiplication and
On 18/06/13 13:33, Sofiane Naci wrote:
Hi,
This is the first of a series of patches to implement a single, unified and
fine grained instruction classification attribute.
The first few patches will propose a refactoring of the instruction
classifications we currently have in place.
This first
On Sun, 16 Jun 2013, Joern Rennecke wrote:
Quoting Eric Botcazou ebotca...@adacore.com:
Could you also check that your patch also fixes PR opt/57569 and, if so, add
the reference to the ChangeLog as well as the testcase?
Attached is what I'm currently testing. bootstrap on
On Tue, Jun 18, 2013 at 12:12 AM, Alan Modra amo...@gmail.com wrote:
I'd like to apply the following set of patches supporting powerpc64le
to the 4.8 branch. David has stated that he's happy with the idea,
even though technically these are not regressions. OK?
On Tue, 2013-06-18 at 18:09 +0800, Bin.Cheng wrote:
On Tue, Jun 18, 2013 at 3:52 AM, Oleg Endo oleg.e...@t-online.de wrote:
My observation is, that legitimizing addressing modes in the backend by
looking at one isolated address works, but doesn't give good results.
In the SH backend there
This fixes the mv?.C ICEs in the testsuite (and transforms a subset
of them to execute fails for me).
Running target unix/
FAIL: g++.dg/ext/mv12.C -std=gnu++98 execution test
FAIL: g++.dg/ext/mv12.C -std=gnu++11 execution test
FAIL: g++.dg/ext/mv2.C -std=gnu++98 execution test
FAIL:
make oldconfig
make CC=gcc LD=ld-from-linux-binutils AR=gcc-ar -j ..
Ok, it doesn't use LTO for me, not even with adding CFLAGS=-O2 -flto
here.
Can you send me a build log with V=1 ?
There are some checks for the environment at the beginning, maybe they fail.
-Andi
--
On 06/17/2013 08:21 PM, Paolo Carlini wrote:
I see... There is a little difficulty in that 56794 involves a non-type
variadic parameter and in that case type_dependent_expression_p returns
false. If I use value_dependent_expression_p things work, but I'm not
sure it's 100% correct.
I don't
This makes us use a pointer-map for the hashtable in lto_tree_ref_encoder
which avoids gazillion of malloc/free calls.
LTO bootstrapped on x86_64-unknown-linux-gnu, testing in progress.
Richard.
2013-06-18 Richard Biener rguent...@suse.de
* Makefile.in (LTO_STREAMER_H): Add
Hi,
On 06/18/2013 04:15 PM, Jason Merrill wrote:
On 06/17/2013 08:21 PM, Paolo Carlini wrote:
I see... There is a little difficulty in that 56794 involves a non-type
variadic parameter and in that case type_dependent_expression_p returns
false. If I use value_dependent_expression_p things
Hi,
This patch updates the documentation for type attribute. It complements
the changes proposed in the previous patch
OK for trunk?
-
Thanks
Sofiane
arm-update-insn-class-doc.diff
Description: Binary data
On Mon, Jun 17, 2013 at 06:01:10PM +0200, Jakub Jelinek wrote:
On Mon, Jun 17, 2013 at 03:52:54PM +, Joseph S. Myers wrote:
On Mon, 17 Jun 2013, Jakub Jelinek wrote:
+; What the sanitizer should instrument
+Variable
+unsigned int flag_sanitize
Can't you just add
Hi,
The following patch removed pool_range/neg_pool_range attributes from
several instructions as a cleanup, which I believe to have been
incorrect:
http://gcc.gnu.org/ml/gcc-patches/2010-07/msg01036.html
On a Mentor-local branch, this caused problems with instructions like:
(insn 77 53 87
On 18/06/13 16:42, Julian Brown wrote:
Hi,
The following patch removed pool_range/neg_pool_range attributes from
several instructions as a cleanup, which I believe to have been
incorrect:
http://gcc.gnu.org/ml/gcc-patches/2010-07/msg01036.html
On a Mentor-local branch, this caused problems
That suggests
Index: gcc/expr.c
===
--- gcc/expr.c (revision 200164)
+++ gcc/expr.c (working copy)
@@ -9353,7 +9353,7 @@ expand_expr_real_1 (tree exp, rtx target
/* Variables inherited from containing functions
Ping.
On 06/06/2013 01:23 PM, Meador Inge wrote:
On 06/06/2013 08:11 AM, Richard Earnshaw wrote:
I understand (and agree with) this bit...
+(define_peephole2
+ [(set (reg:CC CC_REGNUM)
+(compare:CC (match_operand:SI 0 register_operand )
+(match_operand:SI 1
Richard Biener rguent...@suse.de writes:
It is not necessary to maintain the pointer-map from cache entry to
cache index when reading trees. A quick estimate using the latest
WPA stats from firefox estimates its size to at least 1.5GB - apart
from the cost to maintain it. So the following
Hi,
I checked in this patch to fix a typo in comments in config/i386/i386.c.
H.J.
---
Index: ChangeLog
===
--- ChangeLog (revision 200173)
+++ ChangeLog (working copy)
@@ -1,3 +1,8 @@
+2013-06-18 H.J. Lu hongjiu...@intel.com
On Tue, Jun 18, 2013 at 3:28 AM, Jan Hubicka hubi...@ucw.cz wrote:
Ping.
Th patch is OK, thanks!
I see you added gcov.exp file support, do you have a testcases?
Yes, I added support for verifying intermediate format in gcov.exp. I
also added a minimal testcase for intermediate format in
On Tue, Jun 18, 2013 at 2:17 PM, Andreas Krebbel wrote:
If you don't want me to use next_active_insn I probably have to do
something like this instead:
No. If the label is followed by jump table data, then NEXT_INSN(label)
will be the JUMP_TABLE_DATA rtx. See tablejump_p.
So the following
Just confirmed with the small build. It does. Running the large build
now.
Large build worked too.
Please check in.
On Wed, May 22, 2013 at 4:52 PM, Michael Meissner
meiss...@linux.vnet.ibm.com wrote:
2013-05-22 Michael Meissner meiss...@linux.vnet.ibm.com
* config/rs6000/predicates.md (fusion_gpr_addis): New predicates
to support power8 load fusion.
(fusion_gpr_mem_load):
Hi,
Forwarding this patch to gcc-patches...
Cheers!
Cindy
From: rtems-devel-boun...@rtems.org [rtems-devel-boun...@rtems.org] on behalf
of Sebastian Huber [sebastian.hu...@embedded-brains.de]
Sent: Tuesday, June 18, 2013 4:58 AM
To: rtems-de...@rtems.org
Please move simduid after force_vect, so that it is better packed.
Fixed.
I also rewrote the builtins to use the internal function doo-dah as
previously suggested.
Let me know if this is fine with y'all and you (Jakub) can keep this
patch and apply it on top of your pending patchset for
Quoting Michael Matz m...@suse.de:
So, first the potentially clobbering write, then the read. And indeed in
check_dependence d-exp is the write and x the read that is potentially
clobbered.
Oops, you are right. I got confused because what is X in cse.c:invalidate
ends up as d-exp in
On 06/05/13 16:18:52, Jan Hubicka wrote:
Hi,
this patch deals with C++ keyed methods and explicit instantiations.
Currently C++ calls mark_used that ultimately sets force_output
on the functions. This is equivalent to attribute ((used))
on the function/variable and it is bit too strong.
On Tue, Jun 18, 2013 at 04:42:51PM +0200, Marek Polacek wrote:
Ok, should be done now (together with other nit-fixes).
Regtested/bootstrapped on x86_64-linux, ok for trunk?
Looks good to me, the only thing I'm worried about are how this
interferes with the
%{fsanitize=address:...}
and
On Tue, Jun 18, 2013 at 10:19 PM, Gary Funck wrote:
It seems that GUPC may be calling assemble_addr_to_section() too early and
that
some other method of locating the UPC shared data related initialization into
the UPC upc_init_array section needs to be implemented. Either that,
or something
On Tue, Jun 18, 2013 at 08:04:15PM +0200, Andi Kleen wrote:
Just confirmed with the small build. It does. Running the large build
now.
Large build worked too.
Also it seems to be drastically faster. I haven't done a proper
measurement run, but the initial run was 58% faster than 4.8,
On 06/18/2013 10:46 AM, Paolo Carlini wrote:
Ah, fantastic. I really hoped we could say something like that but
seemed too easy ;) I'm finishing testing the below then.
OK.
Jason
On Tue, 18 Jun 2013, Jakub Jelinek wrote:
On Tue, Jun 18, 2013 at 04:42:51PM +0200, Marek Polacek wrote:
Ok, should be done now (together with other nit-fixes).
Regtested/bootstrapped on x86_64-linux, ok for trunk?
Looks good to me, the only thing I'm worried about are how this
On Mon, Jun 17, 2013 at 10:49 AM, Sriraman Tallam tmsri...@google.com wrote:
On Fri, Jun 14, 2013 at 11:08 AM, Sriraman Tallam tmsri...@google.com wrote:
On Fri, Jun 14, 2013 at 1:43 AM, Richard Biener
richard.guent...@gmail.com wrote:
On Fri, Jun 14, 2013 at 4:52 AM, Sriraman Tallam
Instead of fixing the bug three times I refactored the try_lock_xxx
functions into a mixin template and used that in the various timed
mutexes.
PR libstdc++/57641
* include/std/mutex (timed_mutex, recursive_timed_mutex): Move common
functionality to new __timed_mutex_impl
On 06/18/13 22:27:51, Steven Bosscher wrote:
The advice would have to be that the front end should not write out
anything to the assembler file.
Why not just emit the function as GIMPLE (even if your stmt_list is
empty) and let your main() call it?
The initialization function is currently
Thanks for fixing everything. This looks much better.
I don't have much, just small typos. It's up to Jason and Richard to
take it from here.
Aldy
+ else
+ {
+ val = convert_like_with_context (conv, arg, fn, i-is_method,
+
Hi,
if there are no objections, tomorrow I will apply this patchlet from
Manuel which I consider obvious. Tested x86_64-linux.
Thanks,
Paolo.
/
/cp
2013-06-19 Manuel Lopez-Ibanez m...@gcc.gnu.org
PR c++/57638
* pt.c (unify, [TEMPLATE_PARM_INDEX]): Pass
The notable changes since the last version:
First, it should properly handle signed single bit types, though I
haven't tested it with real code.
Second, the transformation is only applied when the result is used in a
conditional. Thus it's much less likely to pessimize targets with
Ping.
On Wed, Jun 12, 2013 at 2:44 PM, Wei Mi w...@google.com wrote:
Hi,
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57518
pr57518 happened because update_equiv_regs in IRA marked a reg
equivalent with a mem, lowered its mem_cost in scan_one_insn, set
NO_REGS to its rclass, but didn't
66 matches
Mail list logo