> 2011-09-04 David S. Miller
>
> * config.host: Add driver-sparc.o and sparc/x-sparc on
> native sparc*-*-linux* builds.
> * config/sparc/driver-sparc.c: Correct Linux strings.
> * gcc/config/sparc/linux.h: Add DRIVER_SELF_SPECS.
> * gcc/config/sparc/linux64.h: Like
> For now I schedule Niagara3 like Niagara2, and that's pretty accurate.
> Niagara4 will be a different beast scheduling wise, so I've left it
> out of the chips niagara2.md matches.
>
> 2011-09-04 David S. Miller
>
> * gcc/config/sparc/sparc-opts.h (PROCESSOR_NIAGARA3,
> PROCESSOR_N
> Hi,
>
> the patch below improves the comparisons of BINFOs in IPA-CP. The
> problem is that we can read different BINFOs for the same type (or a
> base type component) from the LTO summaries because BINFOs coming from
> different compilation units are not unified. Because we now have the
> BIN
On Sat, 3 Sep 2011, Eric Botcazou wrote:
> > Well, even when sign-extended there is a constant you can't negate
> > without overflow. I would start digging for a testcase with
> > such case - but as said, testcases involving TYPE_IS_SIZETYPE are
> > very hard to generate for me.
>
> We run thous
Hi,
this is patch I comitted fixing ipa-inline-analysis. Stil I guess we should
fill in PR for missed optimization and solve the forwprop issue?
Honza
Index: ChangeLog
===
*** ChangeLog (revision 178523)
--- ChangeLog (working c
On Mon, Sep 5, 2011 at 9:41 AM, Jan Hubicka wrote:
> Hi,
> this is patch I comitted fixing ipa-inline-analysis. Stil I guess we should
> fill in PR for missed optimization and solve the forwprop issue?
I'm looking at it, it seems to be cfgcleanups bug
Richard.
> Honza
>
> Index: ChangeLog
> ==
Richard Henderson writes:
> On 09/01/2011 12:13 AM, Richard Sandiford wrote:
>> Also, for the frame_pointer_required case, it looks like there's a
>> window between the restoration of the frame pointer and the deallocation
>> of the stack in which the CFA is still defined in terms of the frame
>>
Hi Eric,
During testing the approved-for-commit middle-end patch for bug 43864 on ARM, I
ran into a gcc.dg/torture/pr46068.c ICE.
An asm jump is not recognized as an unconditional jump, so its followed by a
fall-through block rather than a barrier.
...
(jump_insn 10 9 19 4 (asm_operands/v ("") ("
On Mon, Sep 05, 2011 at 10:29:18AM +0200, Tom de Vries wrote:
> Hi Eric,
>
> During testing the approved-for-commit middle-end patch for bug 43864 on ARM,
> I
> ran into a gcc.dg/torture/pr46068.c ICE.
>
> An asm jump is not recognized as an unconditional jump, so its followed by a
> fall-throug
This reverts parts of
2011-04-11 Richard Guenther
* stor-layout.c (layout_type): Compute all array index size
operations
in the original type.
(initialize_sizetypes): Add comment.
(set_sizetype): Do not set TREE_TYPE of a TREE_VEC.
and retains the use of size
Jan Hubicka wrote:
> this is patch I comitted fixing ipa-inline-analysis.
Thanks for fixing this!
Richard Guenther wrote:
> Can you open a bugreport?
Sorry, I didn't see this until now. Given that the fix is
already checked in (and the problem doesn't exist on any
release branch), I guess we
This makes merge-blocks fold all statements that have single-arg PHI
arguments propagated into them. fold_stmt_inplace does not fold
calls, thus the following makes us use fold_stmt instead of
fold_stmt_inplace.
Bootstrap and regtest ongoing on x86_64-unknown-linux-gnu.
Richard.
2011-09-05 Ri
Hi,
The checkin at r174965 in trunk fixed vect-27.c failure for ARM EABI target.
Detailed information please refer to
http://gcc.gnu.org/ml/gcc-patches/2011-06/msg00927.html.
Performed regression test on arm QEMU and no regression.
OK to ARM/embedded-4_6-branch?
BR,
Terry
2011-09-05 Terry Gu
On Mon, Sep 05, 2011 at 10:54:47AM +0200, Richard Guenther wrote:
> > 2011-09-02 Jakub Jelinek
> >
> > * common.opt: Add -ftree-strlen option.
>
> Maybe sth more generic? -foptimize-string-ops? Eventually guard
> the existing string op foldings with that flag as well.
I don't think ot
On Mon, Sep 5, 2011 at 1:45 AM, Thomas Klein wrote:
> +static int
> +stack_check_work_registers (rtx *workreg)
> +{
> + int reg, i, k, n, nregs;
> +
> + if (crtl->args.info.pcs_variant <= ARM_PCS_AAPCS_LOCAL)
> +{
> + nregs = crtl->args.info.aapcs_next_ncrn;
> +}
> + else
> +n
PR50289 is a minor issue with -mcall-prologues and global register variables
resp. fixed registers: Such registers shall be omitted in function
prologue/epilogue.
For test program
void bar (long long, long long, void*);
register char x asm ("7");
void foo (char a)
{
asm volatile ("":::"6");
On Mon, Sep 5, 2011 at 11:41 AM, Jakub Jelinek wrote:
> On Mon, Sep 05, 2011 at 10:54:47AM +0200, Richard Guenther wrote:
>> > 2011-09-02 Jakub Jelinek
>> >
>> > * common.opt: Add -ftree-strlen option.
>>
>> Maybe sth more generic? -foptimize-string-ops? Eventually guard
>> the existin
At 2011-08-18 10:21:01,"Ramana Radhakrishnan"
wrote:
> On 14 July 2011 08:45, Xinyu Qi wrote:
> >> Hi,
> >>
> >> It is the fourth part of iWMMXt maintenance.
> >>
>
> Can this be broken down further. ? I'll have to do this again but
> there are some initial comments below for some discussion.
This is the minimal patch that allows Ada to bootstrap (including
all the other frontend changes to make them regression-free). It
doesn't need any Ada changes for this.
I will send out Ada changes separately, piecewise, indicating which
regressions they will fix - I'd appreciate help there thou
This avoids setting the TREE_OVERFLOW bit on DECL_FIELD_OFFSET for
the shifted bounds field.
If DECL_FIELD_OFFSET of array-field is always a constant we could
as well use
DECL_FIELD_OFFSET (bounds_field)
= size_int (-int_byte_position (array_field));
also I notice that DECL_OFFSET_ALIGN i
This test case contains expression 1 << 18 which leads to FAIL for targets with
sizeof(int) < 4.
The mask seems not to be relevant for the test and can be set to, e.g. 14.
Ok?
testsuite/
* gcc.dg/ipa/ipcp-3.c (mark_cell): Use mask 1 << 14 instead of 1 << 18.
Index: gcc.dg/ipa/ipcp-3.c
Hi,
On Sat, 3 Sep 2011, Richard Guenther wrote:
> > your patch and apply mine entirely, as eliminating alloca (0) early
> > looks a interesting simplification. What do you think?
>
> I'm not sure we want to create a the replacement decl with DECL_SIZE
> zero though, so I suppose instead of ma
On 09/05/2011 10:53 AM, Jakub Jelinek wrote:
> On Mon, Sep 05, 2011 at 10:29:18AM +0200, Tom de Vries wrote:
>> Hi Eric,
>>
>> During testing the approved-for-commit middle-end patch for bug 43864 on
>> ARM, I
>> ran into a gcc.dg/torture/pr46068.c ICE.
>>
>> An asm jump is not recognized as an un
Hi,
On Mon, Sep 05, 2011 at 02:08:18PM +0200, Georg-Johann Lay wrote:
> This test case contains expression 1 << 18 which leads to FAIL for targets
> with
> sizeof(int) < 4.
>
> The mask seems not to be relevant for the test and can be set to, e.g. 14.
>
I have created the testcase by modifying
On Mon, Sep 05, 2011 at 02:31:32PM +0200, Tom de Vries wrote:
> --- gcc/recog.c (revision 178145)
> +++ gcc/recog.c (working copy)
> @@ -118,6 +118,46 @@ init_recog (void)
> }
>
>
> +/* Return true if labels in asm operands BODY are LABEL_REFs. */
> +
> +static bool
> +asm_labels_ok (rtx bod
The Alfa section of ALI files mentioned objects of task type or protected type
in effects, but it should not, because we do not consider effects on such
objects. The present patch fixes this.
Tested on x86_64-pc-linux-gnu, committed on trunk
2011-09-05 Johannes Kanig
* lib-xref-alfa.a
This patch fixes a bug where the compiler would give an incorrect error message
when an intrinsic operator is called with operands of private type, but with a
result of a non-private type.
The following test should compile quietly.
package Test_Intrinsics is
type T is private;
X : constant T;
In Ada2012, the expression that proides the body of an expression function must
be parenthesized. This patch improves the error recovery for what we expect to
be the common error of omitting those parentheses.
Compiling t.ads must yield:
t.ads:21:06: expression function must be enclosed in par
Constants cannot be modified, and so should never appear in the ALFA section
of ALI files as effects. This patch enforces this property.
Tested on x86_64-pc-linux-gnu, committed on trunk
2011-09-05 Johannes Kanig
* lib-xref-alfa.adb (Is_Alfa_Reference): Filter constants from effect
The test which filters constants objects so that they do not appear in the
ALFA sections of ALI files did not cover all the cases. This patch corrects
the problem.
Tested on x86_64-pc-linux-gnu, committed on trunk
2011-09-05 Johannes Kanig
* lib-xref-alfa.adb (Is_Alfa_Reference): Impr
When the SPARK restriction mode was set, check that a declaration of
unconstrained type is allowed only for constants of type string.
compiling above string_type.ad(s|b) with SPARK mode
gcc -c -gnat05 -gnatec=../spark.adc string_type.adb
the following output must yield:
string_type.adb:1:19: vi
In early versions of Ada, an illegal iterator of the form "for X in S" where
S is an array, is most likely an attempt to iterate over the range of S, and
this is the appropriate error message to emit. In Ada 2012, there is a new
iterator specification form "for E of S", and the error message must
Martin Jambor schrieb:
> Hi,
>
> On Mon, Sep 05, 2011 at 02:08:18PM +0200, Georg-Johann Lay wrote:
>> This test case contains expression 1 << 18 which leads to FAIL for targets
>> with
>> sizeof(int) < 4.
>>
>> The mask seems not to be relevant for the test and can be set to, e.g. 14.
>>
>
> I h
This patch sets properly the scope of an anonymous access type created for
an access to class-wide type declaration in a child unit. Previously the scope
was set to the parent unit, leading to anomalous behavior in the back end.
The following must compile quietly in Ada 2005 mode:
---
package Par
Committed as obvious.
http://gcc.gnu.org/viewcvs?view=revision&revision=178545
Martin Jambor schrieb:
> Hi,
>
> On Mon, Sep 05, 2011 at 02:08:18PM +0200, Georg-Johann Lay wrote:
>> This test case contains expression 1 << 18 which leads to FAIL for targets
>> with
>> sizeof(int) < 4.
>>
>> The
GNAT.Command_Line automatically computes a help message showing all supported
switches. This patch allows to replace this automatic message by a user given
one, in case the automatic one does not fit.
Tested on x86_64-pc-linux-gnu, committed on trunk
2011-09-05 Johannes Kanig
* g-coml
A view conversion to an interface type is expanded into a dereference of a
temporary that involves the class-wide interface type. Nevertheless such an
expression is not dynamically tagged, and is not a controlling argument in
a call.
Compiling main.adb must be rejected with:
main.adb:8:10: ca
This patch protects the compiler against cascaded errors and infinite loops
when analyzing a subunit that has serious syntax errors and an incomplete
context.
Compiling configurations.adb must yield:
configurations.adb:1:01: spec of this package does not allow a body
configurations-kernel-s
This patch adds task synchronization code to the mechanism which sets TSS
primitive Finalize_Address at run time. The following test should compile
and execute quietly.
-
-- Sources --
-
-- main.adb:
with Ada.Finalization; use Ada.Finalization;
with Ada.Text_IO; use
This patch properly documents the rule that rtsfind entities may not
be overloaded, and adds a check that this rule is met. This found
one violation of the rule in the run time which has been fixed. No
test is required, since this clean up has no external effect.
Tested on x86_64-pc-linux-gnu, com
From: Eric Botcazou
Date: Mon, 5 Sep 2011 09:20:35 +0200
> Great, thanks (superfluous gcc/ in the ChangeLog). No improvements to the
> SPARC port have been documented in http://gcc.gnu.org/gcc-4.7/changes.html
> yet, so I'm going to write something down.
Thanks Eric, and thanks also for findi
On 09/01/11 16:16, Richard Sandiford wrote:
> Bernd Schmidt writes:
>> On 08/26/11 14:57, Richard Sandiford wrote:
>>> Wouldn't a reverse post-order (inverted_post_order_compute) allow even
>>> more pre-opening (as well as being less code)?
>>
>> I can't really tell from the comments what that fun
This patch changes the rtl used by rs6000 for a TOC reference address.
The old rtl for -mcmodel=small is:
(plus (reg tocreg) (const (unspec [(symbol_ref sym)] UNSPEC_TOCREL)))
while the old rtl for -mcmodel=medium/large is:
(set (reg rn)
(const (plus (reg r2) (high (const (unspec [(symbol_r
> This avoids setting the TREE_OVERFLOW bit on DECL_FIELD_OFFSET for
> the shifted bounds field.
>
> If DECL_FIELD_OFFSET of array-field is always a constant we could
> as well use
>
> DECL_FIELD_OFFSET (bounds_field)
> = size_int (-int_byte_position (array_field));
>
> also I notice that DEC
On 09/03/2011 02:49 PM, Tobias Burnus wrote:
This patch implements the parsing/diagnostic for "DO[,] CONCURRENT
for-all-header", e.g.
do concurrent (i = 1:5)
A(i) = B(i)
end do
(Side remark: do concurrent also supports a logical mask expression as
FORALL does.)
I have attached an u
Commited the following backport of PR50289 from mainline to 4.6:
http://gcc.gnu.org/viewcvs?view=revision&revision=178529
PR target/50289
Backport from mainline r178528
* config/avr/avr.c (sequent_regs_live): Don't recognize sequences
that contain global register v
From: David Miller
Date: Mon, 05 Sep 2011 11:24:10 -0400 (EDT)
> From: Eric Botcazou
> Date: Mon, 5 Sep 2011 09:20:35 +0200
>
>> Great, thanks (superfluous gcc/ in the ChangeLog). No improvements to the
>> SPARC port have been documented in http://gcc.gnu.org/gcc-4.7/changes.html
>> yet, so
On 01/09/11 17:21, Andrew Stubbs wrote:
I wasn't sure how to find the mode of shift operand in the predicate
though, so I've assumed they're always the same size. How would one find
the proper mode in a predicate?
OK, no reply, so I'm just going to assume we're dealing with 32-bit
registers.
On 09/05/11 09:45, Ye Joey wrote:
+ /* check if we can use one of the argument registers r0..r3 as long as they
+ * not holding data*/
+ for (reg = 0; reg<= LAST_ARG_REGNUM&& i< 2; reg++)
...
+ n = (reg + 1) % 4;
Avoid immediate register number.
use ARG_REGISTER (1) to replace "r
On Mon, Sep 05, 2011 at 11:50:52AM +0200, Richard Guenther wrote:
> Yeah, I suppose update_call_from_tree could use a piecewise variant ...
> ok, let's defer this as a cleanup for whoever feels like updating some
> more code.
Attached are two patches, the first one contains two small changes,
one
There are a couple of typos in ada/gcc-infterface/Makefile.in that
result in many fails of x86_64-darwin10 bootstrap for the m32 lib
variant (thus breaking bootstrap with default switches)..
the attached removes a duplicated section of text and corrects the
target variants.
OK for trunk?
ld needs to be passed the correct arch flag when building shared libs.
This means that the multi-libs built for 4.6.x and trunk contain junk
(essentially all the input objects are rejected as having the wrong
arch).
The attached corrects this.
OK for trunk and 4.6 (since this is a wrong-co
Well, this was two needles in a haystack ...
... AFAICT from googling, powerpc-darwin9 has never bootstrapped ADA
(I see questions but no resolution).
Perhaps Adacore has a version ... but I was unable to find any
starting point - so this was somewhat tough to debug.
Anyway, there are
Darwin8 does not have _SC_NPROCESSORS_ONLN defined.
OK for trunk & 4.6?
Iain
Index: gcc/ada/adaint.c
===
--- gcc/ada/adaint.c(revision 178554)
+++ gcc/ada/adaint.c(working copy)
@@ -2460,7 +2460,10 @@ __gnat_number_of_cpus (
Hi!
On PowerPC64 PR50191 testcase when linked together with other code
results in linker crash. The problem is that a reference
to a constant pool stays in the .debug_loc section. During var-tracking
adjust_insn doesn't successfully avoid_constant_pool_reference it
because instead of a REG it us
> Jan Hubicka wrote:
>
> > this is patch I comitted fixing ipa-inline-analysis.
>
> Thanks for fixing this!
>
> Richard Guenther wrote:
>
> > Can you open a bugreport?
>
> Sorry, I didn't see this until now. Given that the fix is
> already checked in (and the problem doesn't exist on any
> re
> the attached removes a duplicated section of text and corrects the target
> variants.
>
> OK for trunk?
>
> ada:
>
> * gcc-interface/Makefile.in (X86_TARGET_PAIRS): Remove duplicate
> definition.
> (x86_64 darwin arch): Adjust LIBGNAT_TARGET_PAIRS for x86 and x86_64
> variant
> ld needs to be passed the correct arch flag when building shared libs.
>
> This means that the multi-libs built for 4.6.x and trunk contain junk
> (essentially all the input objects are rejected as having the wrong arch).
>
> The attached corrects this.
>
> OK for trunk and 4.6 (since this is
> Darwin8 does not have _SC_NPROCESSORS_ONLN defined.
Is Darwin8 still active/supported?
> OK for trunk & 4.6?
Not as submitted, you haven't provided a suitable changelog for review.
Also, when you repost your patch, can you please send a single version of
the patch (either inline or as an atta
On Sun, Sep 4, 2011 at 9:29 PM, Jason Merrill wrote:
> At the Bloomington C++ meeting we discussed some issues with the constexpr
> specification that the clang team encountered while trying to implement it.
> Among the issues was a problem that also came up recently for us as BZ
> 50248: if the
Hi Arno,
On 5 Sep 2011, at 20:04, Arnaud Charlet wrote:
Darwin8 does not have _SC_NPROCESSORS_ONLN defined.
Is Darwin8 still active/supported?
it works - and I test from time to time.
.. feedback from fink in the form of bug reports suggests that it is
still being used in the wild too.
As Bernd reported last week, mipsisa32-elf builds are broken in trunk.
The problem was a type error in the EABI part of mips_gimplify_va_arg_expr.
Tested on mipsisa32-elf and applied. There are still some lto
builtins failures that I need to look at, but I ran out of time.
Richard
gcc/
PR 49606 is about an ICE when using the combination of -mabi=o64
-mlong64 -mabicalls. In the PR I claimed that -mabi=o64 -mlong64
is an unsupported combination, and that this bug was therefore a
missing diagnostic bug. However, it seems that in more naive days
I'd explicitly OKed the non-abicalls
On Mon, Sep 05, 2011 at 08:26:49PM +0100, Iain Sandoe wrote:
> Hi Arno,
>
> On 5 Sep 2011, at 20:04, Arnaud Charlet wrote:
>
>>> Darwin8 does not have _SC_NPROCESSORS_ONLN defined.
>>
>> Is Darwin8 still active/supported?
>
> it works - and I test from time to time.
> .. feedback from fink in the f
If you configure a biarch Linux/Sparc compiler defaulting to
32-bit, but give --with-cpu= for a v9 cpu it erroneously
turns on 64-bit in TARGET_DEFAULT.
The right thing to do is what the Solaris/Sparc target does,
which is to key things off of a cpp macro (TARGET_64BIT_DEFAULT)
which is defined b
On Sun, 4 Sep 2011, David Miller wrote:
> * gcc/doc/invoke.texi: Document that Linux also supports
> -mcpu=native and -mtune=native on sparc.
:
> -Native Solaris toolchains also support the value @samp{native},
> +Native Solaris and Linux toolchains also support the value @samp{native},
From: Gerald Pfeifer
Date: Tue, 6 Sep 2011 00:57:08 +0200 (CEST)
> Should we use GNU/Linux in those three cases per the standing request
> from RMS? I can make the change if so.
I'm completely ambivalent, feel free to make the change.
On Sep 5, 2011, at 11:50 AM, Iain Sandoe
wrote:
> 1. rs6000.c (WORLD_SAVE_P) decides on the use of the out-of-line prologue
> saves on the basis of cfun->calls_setjmp && flag_exceptions.
> of course, flag_exceptions is NOT set by Ada for SjLj - which is what's
> used for the compiler build.
PING...
> -Original Message-
> From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-
> ow...@gcc.gnu.org] On Behalf Of Jiangning Liu
> Sent: Friday, August 26, 2011 5:56 PM
> To: gcc-patches@gcc.gnu.org
> Subject: [PATCH, testsuite, ARM] change XFAIL to pass for ARM on a case
> testing
On Fri, Sep 2, 2011 at 8:24 AM, Martin Jambor wrote:
> Hi,
>
> when I submitted the new IPA-CP a few months ago Honza requested that
> I store the jump_functions in a VECtor rather than an array in which
> they are now. The patch below does exactly that.
>
> The last remaining such request is to
On Sep 5, 2011, at 14:50, Iain Sandoe wrote:
> ... AFAICT from googling, powerpc-darwin9 has never bootstrapped ADA (I see
> questions but no resolution).
> Perhaps Adacore has a version ... but I was unable to find any starting point
> - so this was somewhat tough to debug.
Nope, if we had sup
On 09/05/2011 01:36 PM, Richard Sandiford wrote:
> Richard Henderson writes:
>> On 09/01/2011 12:13 AM, Richard Sandiford wrote:
>>> Also, for the frame_pointer_required case, it looks like there's a
>>> window between the restoration of the frame pointer and the deallocation
>>> of the stack in w
> -Original Message-
> From: Terry Guo
> Sent: Monday, September 05, 2011 17:18
> To: Joey Ye; gcc-patches@gcc.gnu.org
> Subject: [arm-embedded] Backport r174965 from trunk to ARM/embedded-
> 4_6-branch
>
> Hi,
>
> The checkin at r174965 in trunk fixed vect-27.c failure for ARM EABI
> tar
Hello Everyone,
This patch is for the Cilk Plus branch. It will add the include
directory path it the testsuite script (cilk_plus.exp) in both gcc.dg/cilk-plus
and g++.dg/cilk-plus directories.
Thanking You,
Yours sincerely,
Balaji V. Iyer.
diff --git a/gcc/testsuite/g++.dg/cilk-plus/c
Hi!
On Sun, 21 Aug 2011 02:14:10 +0200, Matthias Klose wrote:
Non-text part: multipart/mixed
> On 08/20/2011 09:51 PM, Matthias Klose wrote:
> > Multiarch [1] is the term being used to refer to the capability of a system
> > to
> > install and run applications of multiple different binary target
75 matches
Mail list logo
