Hi,
This patch enhances mixed condition pattern detection to work with
non-constant integral then and else clauses. It checks that 'then' and
'else' are results of type conversion from the comparison type to their
current type, and generates the whole cond_epxr in comparison type
(ignoring the co
Thomas,
The test fails on powerpc-apple-darwin9 because double are not
vectorized. I think the following patch is required:
--- /opt/gcc/_gcc_clean/gcc/testsuite/gfortran.dg/vect/vect-8.f90
2012-02-05 23:32:31.0 +0100
+++ /opt/gcc/work/gcc/testsuite/gfortran.dg/vect/vect-8.f90
On Sat, Feb 4, 2012 at 3:48 AM, Bin Cheng wrote:
> Hi,
>
> Here is the patch fixing pr51867 by removing the redundant check on
> DECL_ASSEMBLER_NAME_SET_P.
> I also changed '-O0' to '-O1' in signbit-2.c and added a new test.
> The new test case won't bite if target cpu does not support hardware sq
On Fri, Feb 3, 2012 at 7:07 PM, Jakub Jelinek wrote:
> Hi!
>
> On some targets e.g. sms-7.c test fails, because fprintf is called
> with %s format and NULL argument, GLIBC prints for that e.g.
> SMS loop num: 1, file: (null), line: 0
> but it isn't portable. print-rtl.c guards the locator printin
Weddington, Eric wrote:
>
>> -Original Message-
>> From: Georg-Johann Lay [mailto:a...@gjlay.de]
>> Sent: Friday, February 03, 2012 12:47 PM
>> To: gcc-patches@gcc.gnu.org
>> Cc: Denis Chertykov; Weddington, Eric
>> Subject: [Patch,AVR]: Clean up hard-coded SFR addresses
>>
>> This patch r
Hi!
The attached testcase is miscompiled on arm*, by doing a sibcall when setup
of one argument overwrites incoming arguments used to setup parameters in
later insns.
The reason why
mem_overlaps_already_clobbered_arg_p/check_sibcall_argument_overlap
fails to detect is that the caller has non-zero
Hi!
On the following testcase we ICE in dwarf2out, because
apparently reg-stack changed a
(var_location:SI D#1 (subreg:SI (reg:SF 8) 0))
into
(var_location:SI D#1 (reg:SF 8))
(note the subreg is gone and mismatching mode). The problem seems
to be in get_true_reg function that subst_stack_regs_in_
This patch adds DImode shift support in NEON registers/instructions.
The patch causes delays any lowering until the split2 pass, after the
register allocator has chosen whether to do the shift in NEON (VFP)
registers, or in core-registers.
The core-registers case depends on the patch I previo
On Mon, Feb 6, 2012 at 2:01 PM, Jakub Jelinek wrote:
> Hi!
>
> The attached testcase is miscompiled on arm*, by doing a sibcall when setup
> of one argument overwrites incoming arguments used to setup parameters in
> later insns.
> The reason why
> mem_overlaps_already_clobbered_arg_p/check_sibcal
On Mon, Feb 06, 2012 at 01:13:58PM +, Andrew Stubbs wrote:
> Anyway, is this patch OK?
Are you asking approval for 4.8, or 4.7? For the latter this doesn't
seem to be a regression bugfix that would be suitable for stage4.
> 2012-02-06 Andrew Stubbs
>
> gcc/
> * config/arm/arm
This fixes PR50955 - IVOPTs can end up expressing a pointer value
using unrelated pointer bases. This confuses alias analysis.
The fix is to extend the existing stop-gap we have in place
to cover the case in question.
Bootstrapped and tested on x86_64-unknonw-linux-gnu.
I have also benchmarked
Hello!
This patch updates baseline symbols for alpha-linux-gnu.
2012-02-06 Uros Bizjak
* config/abi/post/alpha-linux-gnu/baseline_symbols.txt: Regenerated.
OK for mainline?
Uros.
Index: config/abi/post/alpha-linux-gnu/baseline_symbols.txt
On Mon 06 Feb 2012 13:18:34 GMT, Jakub Jelinek wrote:
On Mon, Feb 06, 2012 at 01:13:58PM +, Andrew Stubbs wrote:
Anyway, is this patch OK?
Are you asking approval for 4.8, or 4.7? For the latter this doesn't
seem to be a regression bugfix that would be suitable for stage4.
Yes, sorry, t
Looks good to me.
Thanks,
Richard.
Thanks folks. I have committed the patch, and will close the PR.
Aldy
This fixes PR 52155.
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk.
Richard.
2012-02-06 Richard Guenther
PR tree-optimization/52115
* tree-sra.c (access_has_replacements_p): New function.
(sra_modify_assign): Use it to decide whether a use is
On Fri, 3 Feb 2012, Richard Guenther wrote:
> On Fri, 3 Feb 2012, Richard Guenther wrote:
>
> > On Fri, 3 Feb 2012, Richard Guenther wrote:
> >
> > > On Thu, 2 Feb 2012, Aldy Hernandez wrote:
> > >
> > > > Linus Torvalds writes:
> > > >
> > > > > Seriously - is there any real argument *agains
On Tue, Nov 8, 2011 at 3:53 PM, Richard Henderson wrote:
>
> On 11/07/2011 07:28 PM, Quentin Neill wrote:
> > + Add FMA4 to bdver2.
> > + * config/i386/i386.c (ix86_option_override_internal): Add FMA4
> > to bdver2.
>
> Ok.
>
>
> r~
This patch was okay'd in stage 3 but was never commi
On Sat, 4 Feb 2012, Richard Sandiford wrote:
> > I don't think _mcount has ever worked for dynamic libraries, has it? --
> > please correct me if I am wrong, but I believe `gprof' relies on memory
> > segments to be contiguous which is certainly not the case for a dynamic
> > executable where
On Sat, Feb 4, 2012 at 12:46 AM, Alan Modra wrote:
> http://gcc.gnu.org/ml/gcc-patches/2007-01/msg01835.html changed the
> code I'm tweaking here to use DFmode subregs when loading a TFmode
> constant into regs for e500. This just extends that change to all
> rs6000 targets, the simplest fix I fo
On Feb 5, 2012, at 12:26 PM, Jonathan Wakely wrote:
> PRs libstdc++/51296 and libstdc++/51906 are both caused by problems
> with the Pthreads static initializer macros such as
> PTHREAD_MUTEX_INITIALIZER.
> On Mac OS X 10.7 the PTHREAD_RECURSIVE_MUTEX_INITIALIZER is buggy.
Thanks for all you work
On Feb 4, 2012, at 11:20 AM, Robert Millan wrote:
> El 1 de febrer de 2012 1:06, Mike Stump ha escrit:
>> On Jan 31, 2012, at 2:29 PM, Gerald Pfeifer wrote:
>>> On Sun, 29 Jan 2012, Robert Millan wrote:
Please consider this patch to stddef.h. GNU/kFreeBSD has the same
problem with __size
"Maciej W. Rozycki" writes:
>> > We have only about now got MIPS16 shared libraries to work -- are you
>> > sure removing code to save/restore $2/$3 in the dynamic linker is going to
>> > hit anyone?
>>
>> Pretty sure. There are two separate points here. Support for MIPS16
>> shared librarie
[forking for different topics]
"Maciej W. Rozycki" writes:
> On Sat, 4 Feb 2012, Richard Sandiford wrote:
>> > I don't think _mcount has ever worked for dynamic libraries, has it? --
>> > please correct me if I am wrong, but I believe `gprof' relies on memory
>> > segments to be contiguous whi
Thanks for the patch.
"Moore, Catherine" writes:
>> -Original Message-
>> From: Chung-Lin Tang [mailto:clt...@codesourcery.com]
>> Sent: Monday, January 30, 2012 4:36 AM
>> To: gcc-patches@gcc.gnu.org; rdsandif...@googlemail.com
>> Cc: Moore, Catherine
>> Subject: Re: [committed] PR 51931
On Mon, Feb 6, 2012 at 9:32 AM, Quentin Neill
wrote:
> On Tue, Nov 8, 2011 at 3:53 PM, Richard Henderson wrote:
>>
>> On 11/07/2011 07:28 PM, Quentin Neill wrote:
>> > + Add FMA4 to bdver2.
>> > + * config/i386/i386.c (ix86_option_override_internal): Add FMA4
>> > to bdver2.
>>
>> Ok.
Attached patch applied
2012-02-06 François Dumont
* include/debug/safe_iterator.h
(_Safe_iterator::_M_before_dereferenceable): Avoid the expensive
creation of a _Safe_iterator instance to do the check.
François
On 02/05/2012 06:30 PM, Paolo Carlini wrote:
On 02/05/20
Hi!
combine_simplify_rtx and its helpers (called from subst) apparently modify
the given RTL in-place, there are many SUBST () calls all around those
functions. On the attached testcase in particular on arm when
newpat = subst (newpat, i1dest, i1src, 0, 0, 0); is called, force_to_mode
modifies th
Dear All,
The attached is obvious fix to this regression and I will commit
tomorrow evening if there is no objection.
Cheers
Paul
2012-02-06 Paul Thomas
* resolve.c (resolve_fl_derived0): Typebound functions support
assumed character length results.
2012-02-06 Paul Thomas
On 02/05/2012 07:27 AM, Uros Bizjak wrote:
> Hello!
>
> Attached patch enables post-reload compare optimization pass for x86 targets.
Hmm. Well, the only thing that's going to work for x86 is the double-compare
elimination portion.
If we want to use this pass for x86, then for 4.8 we should als
In this testcase, we ended up trying to call an uninstantiated template,
and get confused as a result. This patch changes the compiler to not
consider a template to be a valid candidate for a default conversion.
Tested x86_64-pc-linux-gnu, applied to trunk
commit 06d4f1b83e5c393fd22421bbd12135
Hi Dominique,
The test fails on powerpc-apple-darwin9 because double are not
vectorized. I think the following patch is required:
Thanks for noticing this!
I applied this patch as obvious after regression-testing.
Thomas
2012-02-06 Thomas König
PR fortran/32373
*
On Mon, Feb 6, 2012 at 9:59 PM, Richard Henderson wrote:
> On 02/05/2012 07:27 AM, Uros Bizjak wrote:
>> Hello!
>>
>> Attached patch enables post-reload compare optimization pass for x86 targets.
>
> Hmm. Well, the only thing that's going to work for x86 is the double-compare
> elimination portio
On Feb 6, 2012, at 12:59 PM, Richard Henderson wrote:
> And, really, we ought to come up with some trick to eliminate some of the
> redundancy in patterns in the md file too.
:-) That'd be awesome...
And fix a linker error (which I can't reproduce) on TLS targets, which
can use an explicit instantiation because it doesn't refer to an
unnamed closure type.
PR libstdc++/52128
* src/c++11/future.cc: Add explicit instantiation.
Tested x86_64-linux, powerpc-linux, i686-linux, x86_6
When passing a CLASS to a TYPE, the "_data" component wasn't added for
scalar variables. (Polymorphic arrays are/were handled correctly.)
This patch adds the _data, fixing this wrong-code issue.
Build and regtested on x86-64-linux.
OK for the trunk?
Tobias
2012-02-06 Tobias Burnus
PR fort
> -Original Message-
> From: Georg-Johann Lay [mailto:a...@gjlay.de]
> Sent: Monday, February 06, 2012 4:45 AM
> To: Weddington, Eric
> Cc: gcc-patches@gcc.gnu.org; Denis Chertykov
> Subject: Re: [Patch,AVR]: Clean up hard-coded SFR addresses
>
>
> Here is an updated patch without the S
Hi Jakub
Instead of disabling the sibcall, it could also be a valid tail call
optimization by moving the str after ldmia, and change the used
register(It should be handled by RA automatically), as following
...
add r4, r1, r4, lsl #2
ldmia r2, {r1, r2}
Dear Tobias,
On Mon, Feb 6, 2012 at 11:27 PM, Tobias Burnus wrote:
> When passing a CLASS to a TYPE, the "_data" component wasn't added for
> scalar variables. (Polymorphic arrays are/were handled correctly.)
>
> This patch adds the _data, fixing this wrong-code issue.
>
> Build and regtested on
38 matches
Mail list logo
