On Wed, Nov 23, 2011 at 02:24:34PM -0700, Tom Tromey wrote:
> > "Jakub" == Jakub Jelinek 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 agai
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 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 seein
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)
> >> +
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 sysc
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"
wrote:
>
> Hi Xinyu,
>
> This doesn't apply cleanly currently on trunk and the reject appears
> to come from iwmmxt.md and I've not yet investigated why.
>
At 2011-11-19 07:08:22,"Ramana Radhakrishnan"
wrote:
> On 20 October 2011 08:39, Xinyu Qi wrote:
> > Ping
> >
> > http://gcc.gnu.org/ml/gcc-patches/2011-07/msg01103.html
> >
> > * config/arm/arm.c (enum arm_builtins): Revise built-in fcode.
> > (builtin_description bdesc_2arg): Re
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 Paol
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, e
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 use
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, { r4-r1
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
th
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
> "__atomic
Ping 2.
r~
On 11/03/2011 04:24 PM, Richard Henderson wrote:
> Cc: Richard Earnshaw
> ---
> 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
> --- a/gcc/config/arm/ar
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 2
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
> "Jakub" == Jakub Jelinek 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 throw()
> "Jakub" == Jakub Jelinek writes:
Jakub> 2011-11-23 Jakub Jelinek
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
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:
template struct A {};
template void foo()
{
A a;
}
This is what happens. First we build anyway a
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 except
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, wi
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
PR target/51261
* config/i386/i386.c (de
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 Sherrill writes:
Does this patch apply OK for others?
Ran
On Sat, Nov 5, 2011 at 07:02, Iain Sandoe
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 (which should always be ca
Richard Henderson 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
>> /var/gcc/regression/t
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 compu
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 11/23/2011 07:35 AM, Rainer Orth wrote:
> Richard Henderson 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
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 patch
Richard Henderson 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:
$ > isainfo -v
64-
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 jus
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.
> * gcc.dg/atomic-s
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
an
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-
Eric Botcazou 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 rather than 0x00_
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.
>
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 sl
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 wrote:
> The failures are independent which will be looked into at some point.
> Teresa's patch is/will be tested
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_COMDA
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.
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
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"
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
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 Hernandez wrote:
This looks weird -- you're seting D_C_G after H_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;
> }
>
On Tue, Nov 22, 2011 at 8:06 AM, Aldy Hernandez 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 (working copy
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 discrimi
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
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 a
Richard Henderson 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:
http://arc.opensolaris.org/caselog/PSARC/2010/022/mail
Unfortun
> 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 bootstr
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 continues an
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.
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
* s-osinte-hpux.ads, s-taprop-vxworks.adb, s-taprop-tru64.adb,
s-osinte-vxworks.ads, s-osinte-aix.ads, s-osin
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 desi
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 e
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 w
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, be
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 interface
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 a
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 e
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
beha
On Mon, Nov 21, 2011 at 7:07 AM, Revital Eres 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.
>
> This part of th
64 matches
Mail list logo
