ok.
thanks,
David
On Fri, Feb 8, 2013 at 6:57 PM, Harshit Chopra wrote:
> 2013-02-08 Harshit Chopra
>
> Porting revisions r183548, r183888, r186118, r192231, r193381.
>
> diff --git a/gcc/c-family/c-common.c b/gcc/c-family/c-common.c
> index ab416ff..04b973f 100644
> --- a/gcc/c-famil
2013-02-08 Harshit Chopra
Porting revisions r183548, r183888, r186118, r192231, r193381.
diff --git a/gcc/c-family/c-common.c b/gcc/c-family/c-common.c
index ab416ff..04b973f 100644
--- a/gcc/c-family/c-common.c
+++ b/gcc/c-family/c-common.c
@@ -396,6 +396,13 @@ static tree ignore_attri
Since testsuite/lib/go.exp now uses [timeout_value], it needs to load
timeout.exp. This shows up when testing libgo using DejaGNU.
Bootstrapped and ran Go tests on x86_64-unknown-linux-gnu. Committed to
mainline.
Ian
2013-02-08 Ian Lance Taylor
* lib/go.exp: Load timeout.exp.
Ind
On Fri, Feb 8, 2013 at 5:05 PM, Peter Bergner wrote:
> On Wed, 2013-02-06 at 17:19 -0500, David Edelsohn wrote:
>> Because Peter Bergner most recently worked on embedded PPC targets, I
>> had asked him to double-check the patch before approving it.
>> Hopefully he will give me some feedback soon.
On Wed, 2013-02-06 at 17:19 -0500, David Edelsohn wrote:
> Because Peter Bergner most recently worked on embedded PPC targets, I
> had asked him to double-check the patch before approving it.
> Hopefully he will give me some feedback soon.
I've had a look at the patch and it looks good to me.
I al
The following patch fixes
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56246
LRA was not able to find enough regs for insn reloads. One reason of
that was
it did not try to reuse the same reload reg for the same value. Another
reason was it
tried to assign hard reg to a pseudo in any case, as
Hello world,
the attached patch fixes the PR by (re-)adding a search path
for the path used by intrinsic modules.
Regression-tested. OK for trunk?
Thomas
2013-02-08 Thomas Koenig
PR fortran/56224
* options.c (gfc_handle_option): Also call
gfc_add_include_p
Based on feedback from Jakub, I updated the testcase for this PR.
Updated patch attached.
Installed onto the trunk.
Jeff
commit 48b3095f4ef7da1a06ad2fa96db3967762ce4f0b
Author: Jeff Law
Date: Fri Feb 8 09:57:43 2013 -0700
PR debug/53948
* emit-rtl.c (reg_is_parm_p): N
On Fri, Feb 08, 2013 at 08:56:42PM +0100, Steven Bosscher wrote:
> On Fri, Feb 8, 2013 at 12:25 PM, Richard Biener wrote:
> >> Steven - can you reproduce your failures (and on which target?)
> >
> > Ping.
>
> I had ASAN failures on ppc64 but I cannot reproduce them anymore.
ASAN on ppc64 is stil
On Fri, Feb 8, 2013 at 12:25 PM, Richard Biener wrote:
>> Steven - can you reproduce your failures (and on which target?)
>
> Ping.
I had ASAN failures on ppc64 but I cannot reproduce them anymore.
Ciao!
Steven
On Fri, Feb 8, 2013 at 9:42 AM, Uros Bizjak wrote:
>
> Please note, that looking at generated libcalls.go, there are many
> other possible signed/unsigned issues of the same type. Since this all
> happens on x86_64 (where go int is apparently 64bit), it looks that it
> is very important to disting
The attached patch eliminates, PR55693, the failure of the libitm.c++/eh-1.C
execution test on
darwin10 and later. With the introduction of c++ weak-symbol coalescing in Mac
OS X 10.6, dyld no
longer looks up weak symbols that reside in object files rather than shared
libraries. This prevents
On 02/08/2013 10:55 AM, Jakub Jelinek wrote:
On Fri, Feb 08, 2013 at 09:54:19AM -0700, Jeff Law wrote:
PR debug/53948
* emit-rtl.c (reg_is_parm_p): New function.
* regs.h (reg_is_parm_p): New prototype.
* ira-conflicts.c (ira_build_conflicts): Allow parameters in
Hello!
As pointed out by Richi in PR56253 [1], we should enable standard
vector-mode FMA patterns also for !TARGET_SSE_MATH, otherwise various
transformations (convert_mult_to_fma) can't convert to a vectorized
FMA.
2013-02-08 Uros Bizjak
* config/i386/sse.md (FMAMODEM): New mode iter
On 02/08/2013 10:55 AM, Jakub Jelinek wrote:
On Fri, Feb 08, 2013 at 09:54:19AM -0700, Jeff Law wrote:
PR debug/53948
* emit-rtl.c (reg_is_parm_p): New function.
* regs.h (reg_is_parm_p): New prototype.
* ira-conflicts.c (ira_build_conflicts): Allow parameters in
On Fri, Feb 08, 2013 at 06:55:08PM +0100, Jakub Jelinek wrote:
> But the testcase is problematic.
>
> > --- /dev/null
> > +++ b/gcc/testsuite/gcc.dg/debug/dwarf2/pr53948.c
> > @@ -0,0 +1,10 @@
> > +/* Test that we emit a .line directive for the line
> > + with local variable initializations. */
On Fri, Feb 08, 2013 at 09:54:19AM -0700, Jeff Law wrote:
> PR debug/53948
> * emit-rtl.c (reg_is_parm_p): New function.
> * regs.h (reg_is_parm_p): New prototype.
> * ira-conflicts.c (ira_build_conflicts): Allow parameters in
> callee-clobbered registers.
This looks
Weddington, Eric wrote:
>> -Original Message- From: Georg-Johann Lay Sent: Friday, February
>> 08, 2013 9:00 AM To: Weddington, Eric Cc: gcc-patches@gcc.gnu.org; Denis
>> Chertykov; Joerg Wunsch Subject: Re: [Patch,avr] Implement PR56254
>>
>> Weddington, Eric wrote:
>>
>>> What does this
On Fri, Feb 8, 2013 at 6:10 PM, Uros Bizjak wrote:
> On Fri, Feb 8, 2013 at 5:53 PM, Ian Lance Taylor wrote:
>> On Fri, Feb 8, 2013 at 8:48 AM, Uros Bizjak wrote:
>>>
>>> utimensat returns zero on CentOS. Following is gdb session:
>>
>> Thanks--can you do the same session but with "step" instead
> -Original Message-
> From: Georg-Johann Lay
> Sent: Friday, February 08, 2013 9:00 AM
> To: Weddington, Eric
> Cc: gcc-patches@gcc.gnu.org; Denis Chertykov; Joerg Wunsch
> Subject: Re: [Patch,avr] Implement PR56254
>
> Weddington, Eric wrote:
>
> >
> > What does this do? How does it w
On Fri, Feb 8, 2013 at 5:53 PM, Ian Lance Taylor wrote:
> On Fri, Feb 8, 2013 at 8:48 AM, Uros Bizjak wrote:
>>
>> utimensat returns zero on CentOS. Following is gdb session:
>
> Thanks--can you do the same session but with "step" instead of "next"?
I think I got the problem.
breakpoint at utim
The basic problem here is parameters are marked with REG_USERVAR_P and
IRA creates conflicts for the parameters with the callee clobbered
registers.
This results in the copy from the parameter registers into callee
clobbered registers being removed. The move instructions happen to be
assoc
On Fri, Feb 8, 2013 at 8:48 AM, Uros Bizjak wrote:
>
> utimensat returns zero on CentOS. Following is gdb session:
Thanks--can you do the same session but with "step" instead of "next"?
Ian
On Fri, Feb 8, 2013 at 5:21 PM, Ian Lance Taylor wrote:
> On Fri, Feb 8, 2013 at 7:23 AM, Uros Bizjak wrote:
>> On Fri, Feb 8, 2013 at 4:02 PM, Ian Lance Taylor wrote:
>>
>> I did hit one new error that seems related:
>>
>> --- FAIL: TestChtimes (0.00 seconds)
>> os_test.go:681:
On Fri, Feb 8, 2013 at 7:23 AM, Uros Bizjak wrote:
> On Fri, Feb 8, 2013 at 4:02 PM, Ian Lance Taylor wrote:
>
> I did hit one new error that seems related:
>
> --- FAIL: TestChtimes (0.00 seconds)
> os_test.go:681: AccessTime didn't go backwards;
> was={63495872497 0
On 01/24/2013 04:39 PM, David Holsgrove wrote:
Hi Michael,
Please find attached updated patch for MicroBlaze Little Endian
support to reflect removal of microblaze*-*-* target.
Changelog updated as follows.
thanks,
David
gcc/Changelog
2013-01-25 Edgar E. Iglesias
* gcc/config.gcc (microb
Weddington, Eric wrote:
> From: Georg-Johann Lay
>> This adds variable delays to __builtin_avr_delay_cycles.
>>
>> Is this okay?
>
> What does this do? How does it work?
>
> Could you explain the statement in the documentation "if ticks is not a
> compile-time constant, the delay might be overe
Le 08/02/2013 14:06, Paul Richard Thomas a écrit :
> What's happened to the testcase? There are still lots of commented
> out lines that I presume are fixed.
>
Well, I was playing with the testcase just before I sent the patch for
review 😖.
Attached is what I have committed - testcase fixed, as w
On Fri, Feb 8, 2013 at 4:02 PM, Ian Lance Taylor wrote:
I did hit one new error that seems related:
--- FAIL: TestChtimes (0.00 seconds)
os_test.go:681: AccessTime didn't go backwards;
was={63495872497 0 47130825733376}, after={63495872497 0
47130825733376}
>
On 13-02-07 7:27 PM, Jakub Jelinek wrote:
Hi!
As the following testcase shows, get_reload_reg could sometimes return
a reused reg in a wrong mode. If curr_insn_input_reloads[i].input
and original have VOIDmode, they could compare equal (same integer
constant), yet mode might be different from
G
On Fri, Feb 8, 2013 at 6:42 AM, Uros Bizjak wrote:
> On Fri, Feb 8, 2013 at 3:07 AM, Ian Lance Taylor wrote:
>> On Thu, Feb 7, 2013 at 2:43 PM, Uros Bizjak wrote:
>>>
>>> I did hit one new error that seems related:
>>>
>>> --- FAIL: TestChtimes (0.00 seconds)
>>> os_test.go:681: AccessTi
On Fri, Feb 8, 2013 at 6:28 AM, Uros Bizjak wrote:
>
> Attached patch fixes several go testcase failures on CentOS 5.9 with
> ancient (GNU ld version 2.17.50.0.6-20.el5_8.3 20061020) binutils.
> These do not have CFI personality directive, so -fsplit-stack should
> not be used there.
>
> The compi
The -mcpu=XXX options enable processor features, including Altivec and
VSX. Some OS releases do not have support for that feature. GCC
included code to clear those features for Altivec, but it had not been
updated when VSX was added. This was fixed on mainline as part of the
general masks cleanu
On Fri, 8 Feb 2013, Jakub Jelinek wrote:
> Hi!
>
> As the testcase shows, treating (-something_unsigned) / c as
> -(something_unsigned / c) is wrong for unsigned types, seems to be
> fine for signed types even for -fwrapv and for multiplication
> instead of / or % looks fine to me too.
>
> Boots
On Fri, Feb 8, 2013 at 3:07 AM, Ian Lance Taylor wrote:
> On Thu, Feb 7, 2013 at 2:43 PM, Uros Bizjak wrote:
>>
>> I did hit one new error that seems related:
>>
>> --- FAIL: TestChtimes (0.00 seconds)
>> os_test.go:681: AccessTime didn't go backwards;
>> was={63495872497 0 47130825733376
Hello!
Attached patch fixes several go testcase failures on CentOS 5.9 with
ancient (GNU ld version 2.17.50.0.6-20.el5_8.3 20061020) binutils.
These do not have CFI personality directive, so -fsplit-stack should
not be used there.
The compiler detects this situation through
TARGET_SUPPORTS_SPLIT_
Hi!
As the testcase shows, treating (-something_unsigned) / c as
-(something_unsigned / c) is wrong for unsigned types, seems to be
fine for signed types even for -fwrapv and for multiplication
instead of / or % looks fine to me too.
Bootstrapped/regtested on x86_64-linux and i686-linux, ok for t
On 02/08/2013 08:18 AM, Paolo Carlini wrote:
On 02/08/2013 05:09 AM, Ed Smith-Rowland wrote:
Here is a reworked patch... The asyptotic stuff is removed for now.
Builds and tests cleanon x86_64-unknown-linux.
Please add a:
bool test __attribute__((unused)) = true;
at the beginning of test
> -Original Message-
> From: Georg-Johann Lay
> Sent: Friday, February 08, 2013 5:53 AM
> To: gcc-patches@gcc.gnu.org
> Cc: Denis Chertykov; Weddington, Eric
> Subject: [Patch,avr] Implement PR56254
>
> This adds variable delays to __builtin_avr_delay_cycles.
>
> Is this okay?
>
What
Mikael,
The patch itself is good for trunk - it's rather clever, in fact :-)
What's happened to the testcase? There are still lots of commented
out lines that I presume are fixed.
Cheers
Paul
On 7 February 2013 23:28, Mikael Morin wrote:
> Hello,
>
> this is the last remaining patch to fix a
This adds variable delays to __builtin_avr_delay_cycles.
Is this okay?
Johann
gcc/
PR target/56254
* config/avr/avr.c (avr_expand_builtin) :
Expand to delay_cycles for non-const delays.
* config/avr/avr.md (delay_cycles): New expander.
(*delay_cycles.libgc
This fixes broken include information printed on diagnostics from the
LTO frontend. We fail to properly leave the file we're still in
after processing one data_in. Instead of figuring out where exactly
we miss a clear_line_info call the following simply caches the
current file/line/column at the
On Fri, Feb 08, 2013 at 12:20:18PM +0100, Richard Biener wrote:
> Ok. Let's watch for fallout...
Yeah, we'll see. Commited.
Marek
Hello!
The patch replaces open-coded INTEGER_CLASS_P macro with its
definition from i386.h
2013-02-08 Uros Bizjak
* config/i386/i386.c (ix86_spill_class): Use INTEGER_CLASS_P macro.
Bootstrapped and regression tested on x86_64-pc-linux-gnu {,-m32},
with additional configure flags "--
This fixes LTO bootstrap which, after the IRA changes to properly
use loop_optimizer_init () and friends, now detects that we may
in fact have invalidated dominator information (and formerly
used this invalid dominator information to build up loops
via flow_loops_find - oops).
LTO bootstrap in st
On Fri, 8 Feb 2013, Marek Polacek wrote:
> On Fri, Feb 08, 2013 at 12:38:48PM +0100, Toon Moene wrote:
> > Maybe here:
> >
> > http://gcc.gnu.org/ml/gcc-testresults/2013-02/msg00835.html
>
> Well this one is definitely not caused by the new verifier bits above.
I've seen this locally when testi
On Fri, Feb 08, 2013 at 12:38:48PM +0100, Toon Moene wrote:
> Maybe here:
>
> http://gcc.gnu.org/ml/gcc-testresults/2013-02/msg00835.html
Well this one is definitely not caused by the new verifier bits above.
Marek
On 02/08/2013 12:20 PM, Richard Biener wrote:
On Thu, 7 Feb 2013, Marek Polacek wrote:
On Thu, Feb 07, 2013 at 02:56:48PM +0100, Richard Biener wrote:
+ /* Check the headers. */
+ FOR_EACH_BB (bb)
+{
+ /* Skip BBs in the root tree. */
+ if (bb->loop_father == current_loops->
On Fri, 1 Feb 2013, Richard Biener wrote:
> On Fri, 1 Feb 2013, Jakub Jelinek wrote:
>
> > On Fri, Feb 01, 2013 at 10:00:00AM +0100, Richard Biener wrote:
> > >
> > > This reduces compile-time of the testcase in PR56113 (with n = 4)
> > > from 575s to 353s. It does so by reducing the quadra
On Thu, 7 Feb 2013, Marek Polacek wrote:
> On Thu, Feb 07, 2013 at 02:56:48PM +0100, Richard Biener wrote:
> > > + /* Check the headers. */
> > > + FOR_EACH_BB (bb)
> > > +{
> > > + /* Skip BBs in the root tree. */
> > > + if (bb->loop_father == current_loops->tree_root)
> > > +
On Thu, Feb 7, 2013 at 5:07 PM, Georg-Johann Lay wrote:
> This patch solves a problem with VIEW_CONVERT_EXPR folding for fixed_cst and
> that use fixed-value.c:fixed_from_double_int.
>
> The patch sign/zero extends the double_int input according to the requested
> fixed-point mode.
>
> The patch b
On Thu, Feb 07, 2013 at 09:01:15PM -0500, David Miller wrote:
> From: Jakub Jelinek
> Date: Fri, 8 Feb 2013 01:27:02 +0100
>
> > +void
> > +fn (void)
> > +{
> > + if (b)
> > +{
> > + int *p, *q;
> > + char g;
> > +
> > + if (f++)
> > + for (;; e++);
> > +lbl:
> > +
52 matches
Mail list logo
