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.
> --- a/gcc/builtin-typ
On Mon, Jun 17, 2013 at 6:54 PM, Joseph S. Myers
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 was say int and
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 Cortex-A
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
> http://gcc.gnu.org/ml/gcc-patches/2013-0
Hi,
During expand, function vcond 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
PR target/57609
* config/s390/s390.c (s390_chunki
On Tue, Jun 18, 2013 at 10:57 AM, Andreas Krebbel
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 find another solutio
2013/6/18 Steven Bosscher :
> On Tue, Jun 18, 2013 at 10:57 AM, Andreas Krebbel
> 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
>
>
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 JUMP_TAB
On 06/18/13 09:50, Zhenqiang Chen wrote:
Hi,
During expand, function vcond 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 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
> patch hasn't
On Tue, Jun 18, 2013 at 11:47:29AM +0200, Uros Bizjak wrote:
> On Mon, Jun 17, 2013 at 6:11 PM, Jakub Jelinek 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/l
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
PR lto/57334
* lto-symtab.c (lto_symtab_merge_decls): Process nodes p
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 (neon_vector_m
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
Hello Chung-Ju,
On 06/18/2013 05:12 AM, Chung-Ju Wu wrote:
2013/6/18 David Edelsohn :
gcc/testsuite/ChangeLog
2013-06-17 Sebastian Huber
PR target/55033
* gcc.target/powerpc/pr55033.c: Fix options.
Okay.
Thanks, David
P.S. Please explicitly copy me on patches.
Hi, Sebastian,
Since Dav
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 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 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 of trans
On Tue, Jun 18, 2013 at 11:51 AM, Jakub Jelinek 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 Mon, 17 Jun 2013, Andi Kleen wrote:
> Richard Biener writes:
>
> > 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 probl
> 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
* gcc.dg/c90-fordecl-1.c: Adju
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
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
* config/fpu-387.h: Use __asm__ and
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 ma
On Tue, Jun 18, 2013 at 10:59:56AM +0200, Steven Bosscher wrote:
> On Tue, Jun 18, 2013 at 10:57 AM, Andreas Krebbel
> 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 wi
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 divisio
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 p
On Sun, 16 Jun 2013, Joern Rennecke wrote:
> Quoting Eric Botcazou :
>
> > 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 i686-pc-linux-gnu
> finishe
On Tue, Jun 18, 2013 at 12:12 AM, Alan Modra 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?
>
> http://gcc.gnu.org/ml/gcc-patches/2013-04/
On Tue, 2013-06-18 at 18:09 +0800, Bin.Cheng wrote:
> On Tue, Jun 18, 2013 at 3:52 AM, Oleg Endo 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 is this a pa
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: g++.dg/ext/mv
> > 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
--
a...@li
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 thin
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
* Makefile.in (LTO_STREAMER_H): Add pointer-set.h dependency.
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 work,
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
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 (set
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 wit
> 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 function
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_opera
Richard Biener 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 patch makes it
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
+
+ * config/i3
On Tue, Jun 18, 2013 at 3:28 AM, Jan Hubicka 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
testsuite/g++.dg/
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 sho
> 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
wrote:
> 2013-05-22 Michael Meissner
>
> * config/rs6000/predicates.md (fusion_gpr_addis): New predicates
> to support power8 load fusion.
> (fusion_gpr_mem_load): Likewise.
>
> * config/rs6000/rs6000-modes.def (
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 c
Quoting Michael Matz :
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 cse.c:check_depend
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
%{fsanit
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 som
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
> in
On Mon, Jun 17, 2013 at 10:49 AM, Sriraman Tallam wrote:
> On Fri, Jun 14, 2013 at 11:08 AM, Sriraman Tallam wrote:
>> On Fri, Jun 14, 2013 at 1:43 AM, Richard Biener
>> wrote:
>>> On Fri, Jun 14, 2013 at 4:52 AM, Sriraman Tallam
>>> wrote:
On Thu, Jun 13, 2013 at 12:40 PM, Jan Hubicka w
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 m
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 current
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,
+ c
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
PR c++/57638
* pt.c (unify, [TEMPLATE_PARM_INDEX]): Pass to unify_type_mis
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
and-not
Ping.
On Wed, Jun 12, 2013 at 2:44 PM, Wei Mi 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 consider the
Uros Bizjak wrote:
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.
OK. Thanks for having a second look and improving the
66 matches
Mail list logo
