> -Original Message-
> From: Paolo Carlini [mailto:paolo.carl...@oracle.com]
> Sent: Wednesday, March 28, 2012 9:15 AM
> To: Terry Guo
> Cc: gcc-patches@gcc.gnu.org; libstd...@gcc.gnu.org; r...@cebitec.uni-
> Bielefeld.DE; mikest...@comcast.net; Richard Earnshaw; 'Paolo Bonzini'
> Subject:
): Likewise.
* common.opt (fdiagnostics-show-caret): New option.
caret-diagnostics-20120406.diff
Description: Binary data
Richard Guenther writes:
>> They can affect shared memory in some ways like a call, but don't have many
>> of the other attributes of call. They are really more like an assignment or
>> other operation with arbitrary shared memory side effects. I do hope to be
>> able to teach the optimizers the
Matt Turner writes:
> 2012-04-04 Matt Turner
>
> gcc/
> * doc/install.texi: Correct typo "-mno-lsc" -> "-mno-llsc".
Thanks, applied to trunk, 4.7, 4.6 and 4.5.
Richard
Since in C++11 the things you can do with an lvalue are no longer a
superset of the things you can do with an rvalue (specifically, you
can't bind one to an rvalue reference) we can't just conservatively
assume that an expression is an lvalue in a template and then get a
better answer later. S
On Apr 6, 2012, at 12:50 AM, Terry Guo wrote:
> This patch was committed into trunk at March 28. And I just verified that it
> also works for gcc 4.7 branch and 4.6 branch. So can I back port it to 4.7
> and 4.6 branch?
Ok. As always please be on the lookout for any problems.
Hi,
With ptr_mode = SImode and Pmode == DImode, given
(note 21 8 17 2 (expr_list:REG_DEP_TRUE (concat:SI (reg:SI 5 di)
(subreg:SI (plus:DI (reg/f:DI 16 argp)
(const_int -20 [0xffec])) 0))
(nil)) NOTE_INSN_CALL_ARG_LOCATION)
when
(plus:DI (reg/f:DI 16 ar
On Apr 6, 2012, at 1:11 AM, Manuel López-Ibáñez wrote:
> A simple implementation of caret diagnostics.
>
> In the testsuite, pruning the caret output does not always work
> because of several known deficiencies of DejaGNU, thus in some places
> I disable the caret explicitly.
>
> Bootstrapped and
On 6 April 2012 15:42, Mike Stump wrote:
> On Apr 6, 2012, at 1:11 AM, Manuel López-Ibáñez wrote:
>> A simple implementation of caret diagnostics.
>>
>> In the testsuite, pruning the caret output does not always work
>> because of several known deficiencies of DejaGNU, thus in some places
>> I dis
Hi,
The attached patch removes unneeded forward declarations from
sh-protos.h.
Tested with 'make all-gcc'.
OK?
Cheers,
Oleg
ChangeLog:
* config/sh/sh-protos.h (fp_int_operand, symbol_ref_operand,
general_movsrc_operand, general_movdst_operand,
arith_reg_operand, fp_arit
Hi,
The attached patch removes the 'high_life_started' macro and replaces
its use with 'reload_in_progress'.
Tested with 'make all-gcc'.
OK?
Cheers,
Oleg
ChangeLog:
* config/sh/sh.h (high_life_started): Remove
* config/sh/predicates.md (general_movdst_operand): Use
'rel
The suggestion was made in the PR to use -w; found this by testing on such a
target, and didn't want to endlessly add systems to the skip list.
2012-04-06 Mike Stump
PR testsuite/50722
* gcc.dg/pr49994-3.c: Use -w to squelch non-portable warnings.
Index: gcc.dg/pr49994-3.c
=
Hi,
The 'prepare_move_operands' function currently always returns 0, which
makes checking its return value (e.g. in sh.md) useless. The attached
patch makes 'prepare_move_operands' return void and removes the return
value checks.
Tested with 'make all-gcc'.
OK?
Cheers,
Oleg
ChangeLog:
Hello Alan,
On Apr 6, 2012, at 02:32 , Alan Modra wrote:
[...]
> In the epilogue, the previous frame read must be prevented from moving after
> the stack adjust. If the adjacent write/read uses r1 as a base reg,
> then we don't need a stack tie at all.
Right
> Writes/reads further away won't be
Back on this! It turn out that this breaks the shared Ada runtime.
Indeed, exported variables on Ada packages in a DLLs are only accessible
when linking against DLL (thanks to runtime pseudo reloc).
With the patch applied it is not possible to build any Ada application
using the shared runtime. T
> 2012-04-03 Martin Jambor
>
> * expr.c (expand_expr_real_1): Pass type, not the expression, to
> set_mem_attributes for a memory temporary. Do not call the
> function for temporaries with a different alias set.
The last sentence is unprecise, this would rather be: "Do not ca
The following patch makes GCC bootstrap successful on mips64el with LRA
usage.
Committed as rev. 186198.
2012-04-06 Vladimir Makarov
* config/mips/mips.c: Include lra.h.
(mips_expand_fcc_reload): Add code for LRA.
* lra-int.h (lra_get_allocno_class, lra_create_new_re
> @@ -9870,7 +9871,14 @@ expand_expr_real_1 (tree exp, rtx target
> if (op0 == orig_op0)
> op0 = copy_rtx (op0);
>
> - set_mem_attributes (op0, exp, 0);
> + /* If op0 is a temporary because of forcing to memory, pass only the
> +type to set_mem_attributes so that the o
On Fri, Apr 6, 2012 at 6:30 AM, H.J. Lu wrote:
> Hi,
>
> With ptr_mode = SImode and Pmode == DImode, given
>
> (note 21 8 17 2 (expr_list:REG_DEP_TRUE (concat:SI (reg:SI 5 di)
> (subreg:SI (plus:DI (reg/f:DI 16 argp)
> (const_int -20 [0xffec])) 0))
> (nil)) NOT
On Thu, Apr 5, 2012 at 5:27 PM, Diego Novillo wrote:
> On 4/5/12 6:13 AM, Richard Guenther wrote:
>>
>>
>> Currently we release the memory for released SSA names (which we keep
>> around on a freelist, mainly to try to keep the SSA version namespace
>> dense) after early optimizations. This has t
Hello,
PR 52870 is another crash in vectorizer pattern detection that was uncovered
by Ira's patch to enable patterns for SLP as well.
In this case, the problem is that vect_recog_widen_mult_pattern detects a
statement as part of a pattern, but that statement is actually outside of
the basic bloc
Hello world,
after some time with a defective computer, I am back online.
Here is a fix for PR 52893 (which I just submitted, I wanted to
set a new record between bug posting and patch submissin :-), a
wrong-code regression for trunk and 4.7. Regression-tested.
OK for both?
Thomas
201
On Thu, Mar 15, 2012 at 11:54 AM, Mike Stump wrote:
> On Mar 15, 2012, at 11:09 AM, Pedro Alves wrote:
>> Still, kfail is standard DejaGnu, not a GDB invention. It'd be nice not to
>> need to fork the script for this.
>
> The change is fine for the gcc tree.
Committed (to the gcc tree, already i
Oleg Endo wrote:
> The attached patch removes the 'high_life_started' macro and replaces
> its use with 'reload_in_progress'.
>
> Tested with 'make all-gcc'.
> OK?
OK.
Regards,
kaz
Oleg Endo wrote:
> The attached patch removes unneeded forward declarations from
> sh-protos.h.
> Tested with 'make all-gcc'.
>
> OK?
OK.
Regards,
kaz
Oleg Endo wrote:
> The 'prepare_move_operands' function currently always returns 0, which
> makes checking its return value (e.g. in sh.md) useless. The attached
> patch makes 'prepare_move_operands' return void and removes the return
> value checks.
>
> Tested with 'make all-gcc'.
> OK?
OK.
R
On Fri, 2012-04-06 at 10:46 +0900, Kaz Kojima wrote:
> From: Oleg Endo
> >> Exposing three-letter macro MSW and LSW globally looks not
> >> a good idea to me.
> >
> > Would 'HIGH_WORD' and 'LOW_WORD' be OK as an alternative?
>
> Sounds better but still a bit too generic. SH_{HIGH,LOW}_WORD
> wo
On 04/06/2012 04:11 AM, Manuel López-Ibáñez wrote:
+++ gcc/testsuite/gcc.dg/torture/tls/tls.exp(working copy)
@@ -48,10 +48,10 @@ dg-init
torture-init
set-torture-options $TLS_TORTURE_OPTIONS {{}} $LTO_TORTURE_OPTIONS
# Main loop.
gcc-dg-runtest [lsort [glob -nocomplain $srcdir/$subdir/*
On 7 April 2012 00:04, Jason Merrill wrote:
> On 04/06/2012 04:11 AM, Manuel López-Ibáńez wrote:
>>
>> +++ gcc/testsuite/gcc.dg/torture/tls/tls.exp (working copy)
>> @@ -48,10 +48,10 @@ dg-init
>> torture-init
>> set-torture-options $TLS_TORTURE_OPTIONS {{}} $LTO_TORTURE_OPTIONS
>>
>> # Main
On 2012/04/06 00:35:12, asharif1 wrote:
On 2012/04/05 17:25:44, shenhan wrote:
> Hi Jing and Ahmad,
>
> This adds a new test base line for x86_64-cros-linux-gnu. Please
take a look.
>
> Thanks,
> Han
lgtm.
ok
http://codereview.appspot.com/5990044/
This fixes a warning Applied as obvious.
Index: ChangeLog
===
--- ChangeLog (revision 186201)
+++ ChangeLog (working copy)
@@ -1,3 +1,7 @@
+2012-04-06 Mike Stump
+
+ * gimple-fold.c (gimple_fold_stmt_to_constant_1):
On 04/06/2012 06:30 PM, Manuel López-Ibáñez wrote:
width if it's set; otherwise I would lean toward unlimited width. And I'm
not sure why we need a right margin at all.
The right margin is because:
>[snip]
Ah, I read "margin" and assumed it meant you were leaving blank space at
the right sid
On Wed, Apr 4, 2012 at 11:09 PM, Paolo Carlini wrote:
> Hi,
>
>> The attached patches fix http://gcc.gnu.org/PR52822, and have been
>> tested with `make check-c++` on linux-x86_64. The trunk patch applies
>> and tests cleanly on gcc-4_7-branch. The gcc-4_6-branch patch is
>> significantly simpler,
This is another installment in my series of cleanups to invoke.texi.
In this patch I have taken a break from nit-picking grammar and have
nit-picked some Texinfo markup issues instead.
Since this is supposed to be a content-free patch, I went ahead and
checked it in after inspecting the genera
34 matches
Mail list logo
