On Mon, Nov 21, 2011 at 7:07 AM, Revital Eres revital.e...@linaro.org wrote:
Hello,
This patch support the estimation of register pressure in SMS.
Although GCC is in stage 3 I would appreciate comments on it.
Thanks to Richard and Ayal for discussing the implementation and their
insights.
I provided dummy functions for the libitm weak refs in the first
version of this.
(a) this is unnecessary (except in the corner-case of testing lto)
because there is never a need to use the crts unless libitm is linked.
(b) it no longer works across all *x86*-darwin* because weak linkage
This patch reduces VTA memory consumption and even speeds it up
somewhat, by avoiding recording permanent equivalences in dataflow sets
(and propagating them all the way down the control flow graph), keeping
them in cselib tables only. This saves some micro-operations, some
duplicate attempts to
The source location of an expression may not be the best place to put a mess in
the case of a failed precondition/postcondition/invariant. For example, it gets
located on the last and keyword in a chain of boolean expressiond and'ed
together. It is best put the message on the first character of an
If a function returns a class-wide interface object then the compiler
generates code that leaves the interface object not well initialized.
This causes wrong dispatching calls at runtime. The following test
must compile and execute without run-time errors.
package Pkg is
type Iface is
There were several issues with the implementation of the map iterator.
(1) Ordered maps support reverse iteration, so the type returned by the
iterator factory function was changed from Forward_Iterator'Class to
Reversible_Iterator'Class.
(2) The concrete type Iterator was declared as limited,
On Mon, 21 Nov 2011, Jeff Law wrote:
Just mind the long line
Do we wrap earlier in the texi file (first line is 79chars I think)?
Or are you referring to an output file?
I fail to find the reference now, but usually try to wrap things
such that a patch (which adds + or - ) still fits with
This patch adds code to the semantic analyzer to properly check
that entry family bounds are in range. Previously this check was
done during code expansion, leading to messages posted at the
wrong point, omission of the check in -gnatc mode, and in the
case of task entry families a blow up in the
The concrete type Iterator is declared as limited, because there is no need for
assignment for iterator objects. Hashed maps also do not support partial
iteration, so the unnecessary node component is also removed from that type.
The Next operation vets the cursor parameter to ensure that it
Remove hard-coded clock ids in s-taprop*.adb; instead, generate them
in System.OS_Constants.
Tested on x86_64-pc-linux-gnu, committed on trunk
2011-11-23 Thomas Quinot qui...@adacore.com
* s-osinte-hpux.ads, s-taprop-vxworks.adb, s-taprop-tru64.adb,
s-osinte-vxworks.ads,
On 11/22/2011 08:53 PM, Jeff Law wrote:
* lang.c (java_init_options_struct): Disable optimizations
which assume a NULL pointer dereference will cause a fault.
This is OK.
Thanks,
Andrew.
Tru64 UNIX Ada bootstrap recently got broken:
s-taprop.adb:892:12: access to volatile object cannot yield
access-to-non-volatile type
make[6]: *** [s-taprop.o] Error 1
s-taprop-tru64.adb missed a patch already applied to s-taprop-{irix,
solaris}.adb. With that change, the bootstrap continues
On 11/23/2011 7:31 AM, Rainer Orth wrote:
Tru64 UNIX Ada bootstrap recently got broken:
s-taprop.adb:892:12: access to volatile object cannot yield
access-to-non-volatile type
make[6]: *** [s-taprop.o] Error 1
s-taprop-tru64.adb missed a patch already applied to s-taprop-{irix,
solaris}.adb.
Tru64 UNIX Ada bootstrap recently got broken:
s-taprop.adb:892:12: access to volatile object cannot yield
access-to-non-volatile type
make[6]: *** [s-taprop.o] Error 1
s-taprop-tru64.adb missed a patch already applied to s-taprop-{irix,
solaris}.adb. With that change, the bootstrap
Richard Henderson r...@redhat.com writes:
What support does the Solaris 11 assembler have for this feature?
The documentation only mentions the compiler...
I've asked exactly the same question when the spec was posted early last
year:
On 2011/11/2 11:47 PM, Bernd Schmidt wrote:
On 10/31/11 10:11, Eric Botcazou wrote:
I'm suggesting a new patch, as attached. Before reload_completed, we
directly return 0 upon nlabel == NULL, which should be identical with
old behavior, while asserting fail if after reload (where we assume the
There were several issues with the implementation of the set iterators.
The concrete type Iterator is declared as limited, because there is no need for
assignment for iterator objects.
For the ordered forms (which support specifying the start position when
iterating), the First and Last selector
This patch corrects the creation of equivalent class-wide types for records
with unknown discriminants. Equivalent class-wide types are used by the back
end to determine the size of an object. As a result it is now possible to
deallocate a class-wide object whose root base type has unknown
On Tue, Nov 22, 2011 at 8:06 AM, Aldy Hernandez al...@redhat.com wrote:
David, can you try the following and see if it fixes the problem on your
end?
Index: trans-mem.c
===
--- trans-mem.c (revision 181588)
+++ trans-mem.c
On Tue, Nov 22, 2011 at 01:53:54PM -0700, Jeff Law wrote:
+
+ /* Java catches catch NULL pointer exceptions, thus we can not necessarily
catches catch?
+ rely on a pointer having a non-NULL value after a dereference. */
+ opts-x_flag_delete_null_pointer_checks = 0;
}
Richard, do you have an opinion on either one of these approaches?
Both bootstrap and regtest on x86-64 Linux and David AIX :-). OK?
Which one?
On 11/22/11 07:58, David Edelsohn wrote:
On Tue, Nov 22, 2011 at 8:06 AM, Aldy Hernandezal...@redhat.com wrote:
This looks weird -- you're
The presence of a body freezes all entities previously declared
in the current list of declarations, but this does not apply
if the body does not come from source. A type invariant is
transformed into a subprogram body which is placed at the end of
the private part of the current package, but this
The syntax of Ada 2012 accepts aspect specifications on renaming declarations,
but no language-defined aspects exist for them, so that pre- and postconditions
on these declarations must be rejected.
Compiling db.ads in Ada 2012 mode must yield:
db.ads:9:07: incorrect placement of aspect PRE
This change makes the target names used by various tools consistent across the
whole tool chain. In particular, Standard'Target_Name and the default search
path for projects are changed to match the GCC target name.
Tested on x86_64-pc-linux-gnu, committed on trunk
2011-11-23 Thomas Quinot
This change ensures that virtual extending projects generated when processing
an EXTENDS ALL project have a closure that is consistent with that of the
extending project. This is required to build DSA applications with the
gnatdist partitioning tool when RCI units come from some imported project.
On Wed, Nov 23, 2011 at 07:47:46AM -0600, Aldy Hernandez wrote:
@@ -4198,7 +4198,7 @@ ipa_tm_create_version_alias (struct cgra
TREE_SYMBOL_REFERENCED (tm_name) = 1;
/* Perform the same remapping to the comdat group. */
- if (DECL_COMDAT (new_decl))
+ if (HAVE_COMDAT_GROUP
Tests clean on google 4-6 branch, so I will be committing this to
google/main and backporting to google/4-6 shortly.
Thanks,
Teresa
On Tue, Nov 22, 2011 at 1:26 PM, Xinliang David Li davi...@google.com wrote:
The failures are independent which will be looked into at some point.
Teresa's patch
Hi,
On Tue, 22 Nov 2011, Gary Funck wrote:
Thanks for the tip. Typically, we don't build with
--enable-checking=all, however ...
We build with --enable-checking=all when we're testing major changes,
moving GUPC to a new target architecture, etc.
In the past, it may have been slower,
Mikael Pettersson writes:
Richard Henderson writes:
The first patch adds support for the m68k-linux syscall. The second
patch adds native support for the m680[2346]0 CAS instruction, and
the m68000/Coldfire TAS instruction.
Both tested only via cross-compile.
Eric Botcazou ebotca...@adacore.com writes:
OK, I see how the two are different now, but I still don't get why the
current version is right. If we say words are 16 bits for simplicity,
and we're storing VALUE == 0x0001_2345 into a 3-byte BLKmode bitfield,
surely we want to store 0x01_2345
On 11/23/2011 03:47 PM, Richard Sandiford wrote:
Since Andreas was fixing C++ bugs, I assume the same situation occurs there.
What does it look like in a C++ context?
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50325#c4
value was (subreg:DI (reg:TI 70) 8)
and fieldmode == BLKmode
-Andreas-
As described in the PR, several atomic tests were failing on bi-arch
32-bit x86 for the 64-bit multilib. As Uros found, this is obvious and
corrected by the following patch, which also fixes a typo in
atomic-other-int128.c.
Tested with the appropriate runtest invocations on i386-pc-solaris2.10
On 11/23/2011 07:14 AM, Rainer Orth wrote:
PR testsuite/51258
* gcc.dg/atomic-compare-exchange-5.c: Add -mcx16 on i?86-*-*.
* gcc.dg/atomic-exchange-5.c: Likewise.
* gcc.dg/atomic-load-5.c: Likewise.
* gcc.dg/atomic-op-5.c: Likewise.
*
On 21/11/11 08:02, Eric Botcazou wrote:
This seems to be a little risky for stage 3 though.
Being in stage 3 shouldn't stop us trying to fix bugs in the compiler:
we're not in the final run-up to a release yet (that could still be five
months away if history is anything to go by). It should
Richard Henderson r...@redhat.com writes:
But why don't we just change the default 64-bit target to have this enabled?
I somehow doubt that there are many people that have those very first
machines that didn't have this feature...
I just saw that we still have them in large numbers:
$
On 11/23/11 04:16, Gerald Pfeifer wrote:
On Mon, 21 Nov 2011, Jeff Law wrote:
Just mind the long line
Do we wrap earlier in the texi file (first line is 79chars I think)?
Or are you referring to an output file?
I fail to find the reference now, but usually try to wrap things
such that a
On 11/23/2011 07:35 AM, Rainer Orth wrote:
Richard Henderson r...@redhat.com writes:
But why don't we just change the default 64-bit target to have this enabled?
I somehow doubt that there are many people that have those very first
machines that didn't have this feature...
I just saw
On 11/23/11 06:45, Jakub Jelinek wrote:
On Tue, Nov 22, 2011 at 01:53:54PM -0700, Jeff Law wrote:
+
+ /* Java catches catch NULL pointer exceptions, thus we can not necessarily
catches catch?
Fixed.
jeff
On 22/11/11 19:06, Richard Henderson wrote:
On 11/21/2011 05:38 PM, Jiangning Liu wrote:
But I still want to know why we don't want to support this? I don't see any
GCC documentation saying not allowing this usage.
Before reload, which_alternative doesn't make much sense, yet you compute it
Richard Henderson r...@redhat.com writes:
On 11/22/2011 04:15 AM, Rainer Orth wrote:
This patch broke bootstrap on Solaris 8 and 9/x86 with Sun as which
doesn't support .hidden: linking the stage2 lto-plugin fails like this:
ld: fatal: relocation error: R_386_GOTOFF: file
On Sat, Nov 5, 2011 at 07:02, Iain Sandoe
develo...@sandoe-acoustics.co.uk wrote:
On 28 Oct 2011, at 13:57, Richard Guenther wrote:
We fail to keep the cannot-inline flag up-to-date when turning
indirect to direct calls. The following patch arranges to do
this during statement folding
Thanks.
Works for me. I posted test results for powerpc-rtems4.11
at http://gcc.gnu.org/ml/gcc-testresults/2011-11/msg02314.html
From the rtems perspective, you can commit it.
--joel
On 11/21/2011 12:08 PM, Rainer Orth wrote:
Joel Sherrilljoel.sherr...@oarcorp.com writes:
Does this patch
Hi!
With -O0 decide_alg wouldn't initialize *dynamic_check and the callers
would use an uninitialized variable.
Fixed thusly, bootstrapped/regtested on x86_64-linux and i686-linux,
committed to trunk as obvious.
2011-11-23 Jakub Jelinek ja...@redhat.com
PR target/51261
*
Hi!
As discussed in the PR, libjava fails to build against latest
glibc, because prims.cc is compiled with -fnon-call-exceptions,
uses ctype.h and libgcj isn't linked against -lsupc++ or -lstdc++.
isspace in latest glibc is a throw() inline that calls a throw()
function pointer. Unfortunately,
Hi!
As mentioned in the PR, if gimple_purge_dead_eh_edges is ever called
after ehcleanup2 pass (or in its after todos as in the testcase below),
nothing removes the unreachable EH regions and we crash either in
cleanup_dead_labels_eh or, if we ignore those there, later on during
emitting of
Hi,
today I have been analyzing this issue - not terribly urgent, it's a
regression but the ICE is on *invalid*. It happens only in C++11 mode
and the test is the following, very simple:
templateint struct A {};
templatetypename void foo()
{
Aint(x) a;
}
This is what happens. First
Jakub == Jakub Jelinek ja...@redhat.com writes:
Jakub 2011-11-23 Jakub Jelinek ja...@redhat.com
Jakub PR bootstrap/50888
Jakub * prims.cc: Don't include ctype.h.
Jakub (c_isspace): Define.
Jakub (next_property_key, next_property_value): Use it instead
Jakub of isspace.
This is ok.
Tom
Jakub == Jakub Jelinek ja...@redhat.com writes:
Jakub As discussed in the PR, libjava fails to build against latest
Jakub glibc, because prims.cc is compiled with -fnon-call-exceptions,
Jakub uses ctype.h and libgcj isn't linked against -lsupc++ or -lstdc++.
Jakub isspace in latest glibc is a
On Mon, 21 Nov 2011, Matthew Gretton-Dann wrote:
The attached patch updates gcc-4.7/changes.html to document the addition
of support in the ARM backend for Cortex-A7 via the -mcpu=cortex-a7
command line option.
Thanks, Matthew. I just applied that patch on your behalf.
Gerald
I found and fixed another problem in the latest memcpy/memest changes
- with this fix all the failing tests mentioned in #51134 started
passing. Bootstraps are also ok.
Though I still see fails in 32-bit make check, so probably, it'd be
better to revert the changes till these fails are fixed.
On
Ping 2.
r~
On 11/03/2011 04:24 PM, Richard Henderson wrote:
Cc: Richard Earnshaw richard.earns...@arm.com
---
gcc/config/arm/arm.c |4
1 files changed, 4 insertions(+), 0 deletions(-)
diff --git a/gcc/config/arm/arm.c b/gcc/config/arm/arm.c
index 5f0d562..9963faa 100644
---
On 11/23/2011 06:46 AM, Mikael Pettersson wrote:
+FAIL: c-c++-common/gomp/atomic-10.c scan-tree-dump-times ompexp
__atomic_fetch_add 4
+FAIL: c-c++-common/gomp/atomic-3.c scan-tree-dump-times ompexp xyzzy, 4 1
+FAIL: c-c++-common/gomp/atomic-9.c scan-tree-dump-times ompexp
To get the ball rolling for other targets, and to let port maintainers see how
easy it really is, here's a first cut at a port to ARM.
Only cross-compiled as yet, and qemu-linux-user isn't good enough to emulate.
I'll do another build on the armv7 host once my current bootstrap and test for
On 11/23/2011 03:47 PM, Richard Henderson wrote:
+GTM_longjmp:
+ ldm r0, { r4-r11, sp, pc }
Bah. This should be
mov r2, r0 /* Move the jmpbuf out of the way. */
mov r0, r1 /* Move the return value into place. */
ldm r2, {
On Wed, 23 Nov 2011, Richard Henderson wrote:
+ __asm volatile (swi %1
+ : +r(sc_0)
+ : i(SYS_futex), r(sc_1), r(sc_2), r(sc_3)
+ : memory);
That looks wrong. Passing the syscall number to swi is the old-ABI
approach; the EABI uses syscall number
Hi again,
Thus, what to do? Definitely adding CAST_EXPR to the switch works, but
I'm wondering if we could do something else... but doesn't seem easy
to me given the above. For example, I suppose changing
potential_constant_expression to return true for error_mark_node would
be catastrophic,
Hi,
another case of spurious -Wzero-as-null-pointer-constant, rather
straightforward fix, IMHO. Well, with hindsight, I should have grepped
more carefully for integer_zero_node, a couple of weeks ago ;)
Anyway, tested x86_64-linux. Ok?
Thanks,
Paolo.
///
/cp
2011-11-23
At 2011-11-19 07:08:22,Ramana Radhakrishnan ramana.radhakrish...@linaro.org
wrote:
On 20 October 2011 08:39, Xinyu Qi x...@marvell.com wrote:
Ping
http://gcc.gnu.org/ml/gcc-patches/2011-07/msg01103.html
* config/arm/arm.c (enum arm_builtins): Revise built-in fcode.
Hi Ramana,
I solve the conflict, please try again. The new diff is attached.
Thanks,
Xinyu
At 2011-11-19 07:36:15,Ramana Radhakrishnan ramana.radhakrish...@linaro.org
wrote:
Hi Xinyu,
This doesn't apply cleanly currently on trunk and the reject appears
to come from iwmmxt.md and I've
On 11/23/2011 04:00 PM, Joseph S. Myers wrote:
On Wed, 23 Nov 2011, Richard Henderson wrote:
+ __asm volatile (swi %1
+ : +r(sc_0)
+ : i(SYS_futex), r(sc_1), r(sc_2), r(sc_3)
+ : memory);
That looks wrong. Passing the syscall number to swi is the
On Wed, 23 Nov 2011, Richard Henderson wrote:
On 11/23/2011 04:00 PM, Joseph S. Myers wrote:
On Wed, 23 Nov 2011, Richard Henderson wrote:
+ __asm volatile (swi %1
+: +r(sc_0)
+: i(SYS_futex), r(sc_1), r(sc_2), r(sc_3)
+: memory);
That looks
On Wed, Nov 23, 2011 at 12:05:56PM -0800, Joel Brobecker wrote:
We were looking at -Wshadow warnings that we thought were counter
productive, and found that you had written a patch that would help us.
The patch was okayed, but never applied. Is there a reason for it?
I must have missed seeing
On 11/23/2011 08:09 PM, Paolo Carlini wrote:
if (null_test)
{
- tree zero = cp_convert (TREE_TYPE (expr), integer_zero_node);
+ tree zero = cp_convert (TREE_TYPE (expr),
+ want_pointer ? nullptr_node : integer_zero_node);
This ?: is unnecessary; if
On Wed, Nov 23, 2011 at 02:24:34PM -0700, Tom Tromey wrote:
Jakub == Jakub Jelinek ja...@redhat.com writes:
Jakub As discussed in the PR, libjava fails to build against latest
Jakub glibc, because prims.cc is compiled with -fnon-call-exceptions,
Jakub uses ctype.h and libgcj isn't linked
64 matches
Mail list logo