On Mon, Jun 06, 2011 at 02:32:29AM -0300, Alexandre Oliva wrote:
Those that remove sets whose DESTs may still be receved by debug insns
ought to adjust debug insns, yeah. I think (hope) we have them all
covered. Do you know of any we missed?
delete_trivially_dead_insns is already covered,
On Jun 4, 2011, Steven Bosscher stevenb@gmail.com wrote:
I'm curious, though: What CFG changes or other transformations are
performed without this patch? It could be a sign of a missed
optimization before CPROP. Have you looked at that too?
IIRC the transformation that was possible but
On Jun 3, 2011, Eric Botcazou ebotca...@adacore.com wrote:
Does the same logic need to be replicated in all passes that do? On
the other hand, these passes call into DF when they remove insns, so
DF is a central place here.
I went over (again?) a number of passes that call delete_insn
It might be too late for DF to do anything sensible to preserve the
debug info rather than just throw it away, as your partial approval
suggests.
OK, let me think about this a little more.
Indeed, sorry, I misread it.
Mind installing these bits separately? My understanding is that they are
On Fri, 3 Jun 2011, Diego Novillo wrote:
As discussed in http://gcc.gnu.org/ml/gcc-patches/2011-06/msg00063.html,
this patch moves decl registration in symbol tables to the LTO front
end. It makes type and symbol registration happen at the same time in
uniquify_nodes.
Tested with LTO
OK, the only difference is that we don't have the node analyzed here, so
externally_visible, etc are not set. With the initial proposal the warning
was emitted only if the function could not be inlined. Now it will be
emitted for each DECL_COMDAT (decl) !DECL_DECLARED_INLINE_P (decl)) even
Ping. This patch
http://gcc.gnu.org/ml/gcc-patches/2011-05/msg01907.html is pending
review.
--
Joseph S. Myers
jos...@codesourcery.com
Ping. This patch
http://gcc.gnu.org/ml/gcc-patches/2011-05/msg02172.html is pending
review (as is the patch on which it depends).
--
Joseph S. Myers
jos...@codesourcery.com
Paolo Bonzini bonz...@gnu.org writes:
* Except for Darwin, the code uses TRAMPOLINE_SIZE. This only exists in
the backend headers. While it could be duplicated somewhere in the
libgcc configury, I'd rather propose that gcc define
__TRAMPOLINE_SIZE__ (in gcc/c-family/c-cppbuiltin.c
On 06/06/2011 11:17 AM, Rainer Orth wrote:
* Instead of __FreeBSD__, one could use HAVE_SYSCTLBYNAME instead, but
that would need a new libgcc config.h header. In addition, we might
have to check for kern.stackprot to make sure the code really works.
* Similarly, instead of testing
On Fri, Jun 3, 2011 at 3:55 PM, Jakub Jelinek ja...@redhat.com wrote:
Hi!
If memcpy is folded into an assignment, that assignment can be for C++
folded into nothing (if it is copying of 1 byte from or to empty C++ class).
gimple-fold.c was changed to handle that case in some spots, but not
On Fri, Jun 3, 2011 at 3:59 PM, Jakub Jelinek ja...@redhat.com wrote:
Hi!
This limits this testcase to i?86/x86_64 (moving to gcc.target/ would
be harder because it relies on all the weirdo vectorization options to be
passed), because apparently on strict alignment targets we don't handle
On Fri, Jun 3, 2011 at 4:33 PM, Alexandre Oliva aol...@redhat.com wrote:
According to http://gcc.gnu.org/ml/gcc-patches/2009-11/msg00999.html
on Nov 19, 2009, Richard Guenther richard.guent...@gmail.com wrote:
On Thu, Nov 19, 2009 at 4:05 AM, Alexandre Oliva aol...@redhat.com wrote:
On Nov
On Fri, Jun 3, 2011 at 4:17 PM, Alexandre Oliva aol...@redhat.com wrote:
A recent change introduced decl_uid in the “;; Function ” header in dump
files. This breaks -fcompare-debug (and bootstrap-debug-lean), because
decl uids aren't kept in sync between -g and non-g compilations.
This patch
Jakub Jelinek ja...@redhat.com writes:
On Fri, Jun 03, 2011 at 09:38:31PM +0200, Rainer Orth wrote:
Rainer Orth r...@cebitec.uni-bielefeld.de writes:
Jakub, any suggestion how to properly test for -fpie/-fPIE support?
Otherwise, I'll remove that part of the patch for now and just commit
the
On 06/06/2011 11:32 AM, Rainer Orth wrote:
On top of the IRIX 6 toplevel libgcc move, I noticed that IRIX doesn't
use crtfastmath.o yet. This patch corrects this, bootstrapped without
regressions on mips-sgi-irix6.5.
While doing this, I noticed that libgcc/config/mips/t-crtfm is identical
Richard Guenther richard.guent...@gmail.com writes:
On Fri, Jun 3, 2011 at 3:59 PM, Jakub Jelinek ja...@redhat.com wrote:
Hi!
This limits this testcase to i?86/x86_64 (moving to gcc.target/ would
be harder because it relies on all the weirdo vectorization options to be
passed), because
On 06/03/11 23:50, Ramana Radhakrishnan wrote:
How are things handled for Thumb1 in case someone builds a routine for
Thumb1 which uses tlsdesc ? I went and read the doc and didn't see any
difference between T1 and T2 in the specification . Would the linker
and everyone else do the right thing
If you're asking something else, can you rephrase the question?
Sorry if I wasn't too clear. My question really should have read why
do we have to special case Thumb2 ? The linker should be able to
veneer the t- a calls either through the veneering sequence (in case
of T1 without blx) or
On Mon, Jun 6, 2011 at 10:58 AM, Christian Bruel christian.br...@st.com wrote:
OK, the only difference is that we don't have the node analyzed here, so
externally_visible, etc are not set. With the initial proposal the
warning
was emitted only if the function could not be inlined. Now it
On 06/06/11 10:53, Ramana Radhakrishnan wrote:
If you're asking something else, can you rephrase the question?
Sorry if I wasn't too clear. My question really should have read why
do we have to special case Thumb2 ? The linker should be able to
veneer the t- a calls either through the
Taft amendment types are incomplete types declared in a package spec whose
completion is declared in the body. This means that other units don't have
access to their full view at all and treat them as incomplete types. This is
problematic when debugging information is requested because their
On Jun 6, 2011, Eric Botcazou ebotca...@adacore.com wrote:
Indeed, sorry, I misread it.
Mind installing these bits separately?
Nope. Testing it separately now.
My understanding is that they are
independent correctness fixes.
Yeah, I guess they are. They won't make debug info any
This is again the compiler trying to create a temporary, for an unconstrained
type hence whose size isn't fixed. Fixed by not creating the temporary as it
is superfluous here.
Tested on i586-suse-linux, applied on the mainline
2011-06-06 Eric Botcazou ebotca...@adacore.com
*
Self-explanatory. Tested on i586-suse-linux, applied on the mainline.
2011-06-06 Eric Botcazou ebotca...@adacore.com
* gcc-interface/utils2.c (gnat_stabilize_reference): Propagate
TREE_THIS_NOTRAP flag.
--
Eric Botcazou
Index: gcc-interface/utils2.c
On Wed, Jun 1, 2011 at 9:27 PM, William J. Schmidt
wschm...@linux.vnet.ibm.com wrote:
This patch cleans up the FIXME logic in gimple_expand_builtin_pow by
introducing gimple_val_nonnegative_real_p for the same purpose that
tree_expr_nonnegative_p served in the expand logic. This completes the
On Wed, Jun 1, 2011 at 11:23 PM, Xinliang David Li davi...@google.com wrote:
On Wed, Jun 1, 2011 at 2:12 PM, Basile Starynkevitch
bas...@starynkevitch.net wrote:
On Wed, 1 Jun 2011 13:26:24 -0700
Xinliang David Li davi...@google.com wrote:
Hi, this is a simple patch that support dump_before
On Thu, Jun 2, 2011 at 2:02 PM, Mikael Pettersson mi...@it.uu.se wrote:
GCC has attribute((returns_twice)) which is supposed to allow the safe
use of alternate implementations of setjmp-like functions. In particular,
a function that calls a setjmp-like function must itself not be inlined,
On Mon, Jun 06, 2011 at 11:30:19AM +0200, Richard Guenther wrote:
On Fri, Jun 3, 2011 at 3:55 PM, Jakub Jelinek ja...@redhat.com wrote:
--- gcc/tree-inline.c.jj 2011-06-02 10:15:20.0 +0200
+++ gcc/tree-inline.c 2011-06-03 09:29:15.0 +0200
@@ -4108,6 +4108,14 @@
On Wed, Jun 1, 2011 at 7:24 PM, Xinliang David Li davi...@google.com wrote:
The attached is the split #1 patch that enhances -fenable/disable.
Ok after testing?
I expect the testcases will be quite fragile, so while I appreciate
test coverage for new options I think we should go without those
Ping^3 for the C6X port. Now with extra patches:
Additional preliminary scheduler tweaks:
http://gcc.gnu.org/ml/gcc-patches/2011-05/msg02408.html
Allow alternatives in attr predicable:
http://gcc.gnu.org/ml/gcc-patches/2011-06/msg00094.html
regrename across basic block boundaries:
On Thu, Jun 2, 2011 at 9:12 AM, Xinliang David Li davi...@google.com wrote:
This is the version of the patch that walks through pass lists.
Ok with this one?
+/* Dump all optimization passes. */
+
+void
+dump_passes (void)
+{
+ struct cgraph_node *n, *node = NULL;
+ tree save_fndecl =
Tom Tromey tro...@redhat.com a écrit:
Dodji + context of 'P'. The problem is, if we are beeing
Typo, being.
Ok with that fixed.
Thanks. Fixed and committed to revision r174694.
--
Dodji
On Mon, 2011-06-06 at 13:00 +0200, Richard Guenther wrote:
On Wed, Jun 1, 2011 at 9:27 PM, William J. Schmidt
wschm...@linux.vnet.ibm.com wrote:
snip
+/* Return true iff VAL is a gimple expression that is known to be
+ non-negative. Restricted to floating-point inputs. When changing
This is follow-up patch that fixes rtx costs for CONST_INT in PLUS
pattern. Original discussion is here:
http://gcc.gnu.org/ml/gcc-patches/2011-05/msg01427.html
2011-06-06 Dmitry PLotnikov dplotni...@ispras.ru
gcc/
* config/arm/arm.c (arm_rtx_costs_1): Fixed costs for CONST_INT
in PLUS
On 06/06/2011 02:53 PM, Gerald Pfeifer wrote:
not a direct approval for any of the outstanding patches, but I am happy
to report that the steering committee is appointing you maintainer of the
C6X port.
Thanks!
Thanks for contributing this work, and happy hacking! And do think of
Hi Jing Yu,
2011-05-08 Jing Yu jin...@google.com
* configure.ac: Skip target-libiberty for
arm*-*-linux-androideabi.
* configure: Regenerated.
Approved - please apply.
Cheers
Nick
Richard Guenther richard.guent...@gmail.com writes:
Thanks. I would hope that we eventually can get rid of the
pattern recognizer ... at least for SSE there is also always
a scalar variant instruction for each vectorized one.
AFAIK, that isn't true for ARM and NEON. E.g. I don't know of a
On 06/06/2011 04:41 PM, Andrew Stubbs wrote:
On 06/06/11 13:15, Dmitry Plotnikov wrote:
+ (const_ok_for_op (INTVAL (x), outer)
+ || const_ok_for_op (~INTVAL (x), outer
The second call is redundant. const_ok_for_op should already do that.
Fixed patch is attached. Ok?
On Jun 6, 2011, Eric Botcazou ebotca...@adacore.com wrote:
It might be too late for DF to do anything sensible to preserve the
debug info rather than just throw it away, as your partial approval
suggests.
OK, let me think about this a little more.
Indeed, sorry, I misread it.
Mind
On 06/06/11 14:26, Dmitry Plotnikov wrote:
if (const_ok_for_arm (INTVAL (x))
- || const_ok_for_arm (~INTVAL (x)))
+ || const_ok_for_arm (~INTVAL (x))
+ || (TARGET_THUMB2 outer == PLUS
+ (const_ok_for_op (INTVAL (x), outer
Sorry, I should have
On May 30, 2011, Alexandre Oliva aol...@redhat.com wrote:
In an -O3 bootstrap, out.clauses are reported as uninitialized. GCC is
not smart enough to realize accesses to the uninitialized members will
never happen. It shouldn't be too expensive to initialize them all, so
this is what this
On May 30, 2011, Alexandre Oliva aol...@redhat.com wrote:
In an -O3 bootstrap, out.clauses are reported as uninitialized. GCC is
not smart enough to realize accesses to the uninitialized members will
never happen. It shouldn't be too expensive to initialize them all, so
this is what
On 06/05/11 21:16, Peter O'Gorman wrote:
Ok for trunk?
Peter
Index: ChangeLog
===
--- ChangeLog (revision 174678)
2011-??-?? Peter O'Gorman po...@thewrittenword.com
* inclhack.def (aix_net_if_arp): New fix.
On May 30, 2011, Alexandre Oliva aol...@redhat.com wrote:
In an -O3 bootstrap, out.clauses are reported as uninitialized. GCC is
not smart enough to realize accesses to the uninitialized members will
never happen. It shouldn't be too expensive to initialize them all, so
this is what
So, I'm getting this:
Running /home/dnovillo/pph/svn/src/gcc/testsuite/g++.dg/pph/pph.exp ...
XPASS: g++.dg/pph/c120060625-1.cc -I. (test for bogus messages, line )
XPASS: g++.dg/pph/c1eabi1.cc -I. (test for bogus messages, line )
XPASS: g++.dg/pph/x1autometh.cc -I. (test for bogus
On 06/06/2011 05:33 PM, Andrew Stubbs wrote:
On 06/06/11 14:26, Dmitry Plotnikov wrote:
if (const_ok_for_arm (INTVAL (x))
- || const_ok_for_arm (~INTVAL (x)))
+ || const_ok_for_arm (~INTVAL (x))
+ || (TARGET_THUMB2 outer == PLUS
+ (const_ok_for_op (INTVAL (x),
On Mon, Jun 6, 2011 at 2:12 PM, William J. Schmidt
wschm...@linux.vnet.ibm.com wrote:
On Mon, 2011-06-06 at 13:00 +0200, Richard Guenther wrote:
On Wed, Jun 1, 2011 at 9:27 PM, William J. Schmidt
wschm...@linux.vnet.ibm.com wrote:
snip
+/* Return true iff VAL is a gimple expression that is
On Mon, Jun 6, 2011 at 04:50, Richard Guenther rguent...@suse.de wrote:
I'd have it done in the loop that computes canonical types, at this
place we do not gain the advantage that the decl register functions
get completely fixed up trees.
Hm, yes, I had forgotten about the call to
On Mon, Jun 6, 2011 at 3:04 PM, Richard Sandiford
richard.sandif...@linaro.org wrote:
Richard Guenther richard.guent...@gmail.com writes:
Thanks. I would hope that we eventually can get rid of the
pattern recognizer ... at least for SSE there is also always
a scalar variant instruction for
On 06/06/11 15:26, Dmitry Plotnikov wrote:
On 06/06/2011 05:33 PM, Andrew Stubbs wrote:
On 06/06/11 14:26, Dmitry Plotnikov wrote:
if (const_ok_for_arm (INTVAL (x))
- || const_ok_for_arm (~INTVAL (x)))
+ || const_ok_for_arm (~INTVAL (x))
+ || (TARGET_THUMB2 outer == PLUS
+ (const_ok_for_op
Hi,
the attached patch fixed a failure with -mzarch -m31. The smul_ppmm
implementation in longlong.h uses the registers of the result in the
wrong order.
A corresponding patch for glibc will be posted soon.
Committed to 4.6 and mainline.
Bye,
-Andreas-
2011-06-06 Andreas Krebbel
On Sat, 4 Jun 2011, Diego Novillo wrote:
On Wed, Jun 1, 2011 at 15:19, Richard Guenther rguent...@suse.de wrote:
Yes, I see no benefit of using a global function to get access
to the address of a global variable.
There is the minor benefit of being able to control access to it, but
I
On Mon, Jun 6, 2011 at 12:16 AM, Peter O'Gorman
po...@thewrittenword.com wrote:
Hi,
We ran across an issue with qt-4.7 built with gcc-4.4 on AIX 5.2, 5.3,
6.1, and 7.1 where some static constructors were not being called. It
turned out to be a header file issue, see, for example,
On Mon, Jun 6, 2011 at 10:50, Richard Guenther rguent...@suse.de wrote:
Do you remember if it was only void_zero_node that causes problems?
We could just special-case it in
lto_input_integer_cst/lto_output_integer_cst. Or even fix the C family
frontends to not create or use this strange
On Sun, Jun 5, 2011 at 9:54 PM, H.J. Lu hongjiu...@intel.com wrote:
I'd like to start submitting a series of patches to enable x32:
https://sites.google.com/site/x32abi/
The GCC x32 branch is very stable. There are no unexpected failures in
C, C++, Fortran and Objective C testsuites. SPEC
On Mon, Jun 6, 2011 at 4:22 AM, Richard Guenther
richard.guent...@gmail.com wrote:
On Wed, Jun 1, 2011 at 7:24 PM, Xinliang David Li davi...@google.com wrote:
The attached is the split #1 patch that enhances -fenable/disable.
Ok after testing?
I expect the testcases will be quite fragile, so
On Mon, Jun 6, 2011 at 5:53 PM, Xinliang David Li davi...@google.com wrote:
On Mon, Jun 6, 2011 at 4:22 AM, Richard Guenther
richard.guent...@gmail.com wrote:
On Wed, Jun 1, 2011 at 7:24 PM, Xinliang David Li davi...@google.com wrote:
The attached is the split #1 patch that enhances
On Mon, Jun 6, 2011 at 4:38 AM, Richard Guenther
richard.guent...@gmail.com wrote:
On Thu, Jun 2, 2011 at 9:12 AM, Xinliang David Li davi...@google.com wrote:
This is the version of the patch that walks through pass lists.
Ok with this one?
+/* Dump all optimization passes. */
+
+void
On 06/03/11 18:41, Julian Brown wrote:
On Thu, 02 Jun 2011 16:35:01 +0100
Richard Earnshawrearn...@arm.com wrote:
I see Paul has already approved this, but I've just spotted one
potential problem that might cause latent bugs sometime in the future.
The code to register the libcalls is
On 6 June 2011 10:56, Nathan Sidwell nat...@codesourcery.com wrote:
On 06/06/11 10:53, Ramana Radhakrishnan wrote:
If you're asking something else, can you rephrase the question?
Sorry if I wasn't too clear. My question really should have read why
do we have to special case Thumb2 ? The
On Thu, May 26, 2011 at 07:02, Richard Guenther
richard.guent...@gmail.com wrote:
81764 dnovillo {
81764 dnovillo if (TREE_CODE (type) == BOOLEAN_TYPE)
81764 dnovillo {
81764 dnovillo arg0 = copy_node (arg0);
81764 dnovillo
On Mon, Jun 06, 2011 at 11:05:25AM -0400, David Edelsohn wrote:
David, do you have any idea if this is what it's supposed to be?
Ok for trunk?
The header certainly does not make sense as is and does not follow AIX
header file conventions. typedef is the only thing that makes sense,
Your patch doesn't really improve this but adds to the confusion.
+ /* Override dump TODOs. */
+ if (dump_file (pass-todo_flags_finish TODO_dump_func)
+ (dump_flags TDF_BEFORE))
+ {
+ pass-todo_flags_finish = ~TODO_dump_func;
+ pass-todo_flags_start |=
On 05/10/11 07:49, Nathan Sidwell wrote:
This patch implements TLS descriptor support in GCC. TLS descriptors
are described at
http://www.codesourcery.com/publications/RFC-TLSDESC-ARM.txt and
blessed by ARM, who have reserved the relocation numbers.
Binutils and GLIBC patches are already
Richard Guenther rguent...@suse.de writes:
2011-06-01 Richard Guenther rguent...@suse.de
* tree.c (build_common_tree_nodes): Also initialize size_type_node.
Call set_sizetype from here.
c-family/
* c-common.c (c_common_nodes_and_builtins): Do not set
Hi,
noticed while working on std::vector. Tested x86_64-linux, committed.
Paolo.
///
2011-06-06 Paolo Carlini paolo.carl...@oracle.com
* include/bits/move.h (move_if_noexcept): Use __and_ and __not_.
Index: include/bits/move.h
As described in the PR, gfortran.dg/graphite/vect-pr40979.f90 FAILs on
Solaris 8 and 9/x86, which defaults to -march=pentiumpro:
FAIL: gfortran.dg/graphite/vect-pr40979.f90 -O scan-tree-dump-times vect
vectorized 1 loops 1
The dump contains 'vectorized 0 loops' instead. The test passes with
On 05/20/11 11:13, Greta Yorsh wrote:
This patch replaces a hardcoded . of local label prefix with
LOCAL_LABEL_PREFIX macro in the ARM backend (generating a trampoline in
thumb mode).
The patch also contains a new test to make sure that the local label
.Ltrampoline_start: is generated
The trunk version has been approved and committed as r174710. Backport
it to google/main. The google/main version has the same logic but is
slightly different since trunk has a different code structure here. OK
for google/main?
2011-06-06 Jing Yu jin...@google.com
Backport trunk
On 6/2/11 2:42 PM, Langton, Asher langt...@llnl.gov wrote:
On 6/2/11 2:11 PM, Steve Kargl s...@troutmask.apl.washington.edu wrote:
On Thu, Jun 02, 2011 at 01:59:03PM -0700, Langton, Asher wrote:
This patch fixes a bug where the wrong code was generated for
assumed-size
Cray pointees. The fix is
OK.
thanks
Carrot
On Tue, Jun 7, 2011 at 1:09 AM, jin...@google.com wrote:
The trunk version has been approved and committed as r174710. Backport
it to google/main. The google/main version has the same logic but is
slightly different since trunk has a different code structure here. OK
for
Hi,
This patch fixes an issue in the ctypewchar_t implementation when the
newlib C library is used.
The generic version of ctypewchar_t::_M_convert_to_wmask() in
config/locale/generic/ctype_members.cc assumes that a character type
mask is either a bitmask with only 1 bit set or a bitwise-OR
This time with the patch attached.
Yufeng
-Original Message-
From: Yufeng Zhang [mailto:yufeng.zh...@arm.com]
Sent: 06 June 2011 18:31
To: gcc-patches@gcc.gnu.org
Subject: [PATCH, libstdc++-v3] Add newlib specific ctype_members.cc
Hi,
This patch fixes an issue in the
On 6 Jun 2011, at 18:21, Nicola Pero wrote:
This patch switches all the testcases in the ObjC/ObjC++ testsuite
to use the
Modern Objective-C runtime API when executing with the GNU runtime.
This
will allow me to complete removing the Traditional Objective-C
runtime API
from libobjc. :-)
On Mon, Jun 06, 2011 at 01:41:14PM +0200, Richard Guenther wrote:
I think NULL new_stmt would have the advantage that we wouldn't duplicate
the complex code looping through all kinds of clones.
Yeah, I'd prefer that variant. Honza?
Ok, after further discussions with Honza on IRC here is
Hi!
Trying to create vector zero_extend is wrong, while mem_loc_descriptor
will for now fail for vector modes, the code attempted to expand zero_extend
anyway.
Fixed thusly, committed to trunk as obvious after bootstrapping/regtesting
on x86_64-linux and i686-linux.
2011-06-06 Jakub Jelinek
Hi!
As usually, -g -dA output differs between targets that can't merge
debug strings and those that do. Instead of writing regexps to match both,
this patch does what most other testcases do. Additionally, it fixes the
testcase also for -gdwarf-4 defaulting compilers.
Regtested on x86_64-linux
Hi!
I forgot to decrement the counter, but given that nobody has hit it
until now it is possible that no frontend leaves RANGE_EXPR indexes in
CONSTRUCTORs around till late. varasm.c seems to handle them though,
so fixed thusly, committed to trunk as obvious.
2011-06-06 Jakub Jelinek
Hi!
When an indirect noreturn call has a likely call target, value-prof.c
ICEs on it, as there is no fallthru edge after the call (nor a join bb).
Fixed thusly, bootstrapped/regtested on x86_64-linux and i686-linux,
ok for trunk/4.6?
2011-06-06 Jakub Jelinek ja...@redhat.com
PR
It's been some time since I last posted about the address lowering issue from
PR46556 (http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46556). I've had a basic
prototype in place for some time, but I ran into some issues that initially
caused performance regressions; I've had to jump through several
On Jun 6, 2011, at 10:21 AM, Nicola Pero wrote:
This patch switches all the testcases in the ObjC/ObjC++ testsuite to use the
Modern Objective-C runtime API when executing with the GNU runtime.
OK to commit ?
Ok, from me with one possible exception, I'd like Iain or Jack to weigh in on
wether
On 6/6/11, Diego Novillo dnovi...@google.com wrote:
So, I'm getting this:
Running /home/dnovillo/pph/svn/src/gcc/testsuite/g++.dg/pph/pph.exp ...
XPASS: g++.dg/pph/c120060625-1.cc -I. (test for bogus messages, line )
XPASS: g++.dg/pph/c1eabi1.cc -I. (test for bogus messages, line )
On Mon, Jun 6, 2011 at 13:02, Diego Novillo dnovi...@google.com wrote:
On Mon, Jun 6, 2011 at 10:27, Diego Novillo dnovi...@google.com wrote:
On Mon, Jun 6, 2011 at 04:50, Richard Guenther rguent...@suse.de wrote:
I'd have it done in the loop that computes canonical types, at this
place we do
Hi Richard,
Here's the reworked patch addressing the noted concerns. Regtested for
powerpc-linux. OK for trunk?
Thanks,
Bill
2011-06-06 Bill Schmidt wschm...@linux.vnet.ibm.com
PR tree-optimization/46728
* tree-ssa-math-opts.c (gimple_expand_builtin_pow): Change FIXME
Please take a look at the revised one.
Thanks,
David
On Mon, Jun 6, 2011 at 4:22 AM, Richard Guenther
richard.guent...@gmail.com wrote:
On Wed, Jun 1, 2011 at 7:24 PM, Xinliang David Li davi...@google.com wrote:
The attached is the split #1 patch that enhances -fenable/disable.
Ok after
This patch fixes PR obj-c++/48275. It's a routine parser ingenuity.
OK to commit ?
Thanks
Index: testsuite/ChangeLog
===
--- testsuite/ChangeLog (revision 174657)
+++ testsuite/ChangeLog (working copy)
@@ -1,3 +1,9 @@
+2011-06-06
This is the patch with max id removed.
Thanks,
David
On Mon, Jun 6, 2011 at 9:00 AM, Xinliang David Li davi...@google.com wrote:
On Mon, Jun 6, 2011 at 4:38 AM, Richard Guenther
richard.guent...@gmail.com wrote:
On Thu, Jun 2, 2011 at 9:12 AM, Xinliang David Li davi...@google.com wrote:
I wrote:
Hello world,
the attached patch extends removing trailing TRIMs in assignments for
cases like a // trim(b). Regression-tested. OK for trunk?
Thomas
This time with the test case corrected (cleanup of the *.original file)
and a more meaningful Subject line.
OK?
Thomas
Nicola,
On x86_64-apple-darwin10 I have the following failures with -m32
FAIL: obj-c++.dg/threedotthree-abi-1.mm -fnext-runtime (test for excess errors)
FAIL: obj-c++.dg/torture/strings/const-str-10.mm -O0 -fnext-runtime (test for
excess errors)
FAIL:
potential_constant_expression_1 didn't know how to deal with a
pointer-to-member expression in a template.
Tested x86_64-pc-linux-gnu, applied to trunk.
commit a07ee89ff40f73dcb1fc11fb66931ee79bf8a2d9
Author: Jason Merrill ja...@redhat.com
Date: Mon Jun 6 12:05:39 2011 -0400
PR
OK.
Jason
On 6 Jun 2011, at 21:07, Nicola Pero wrote:
On x86_64-apple-darwin10 I have the following failures with -m32
darwin10 is Mac OS X 10.6, right ? I have access to that. So, how
do you test with -m32 ?
I thought the testsuite would do that (test both with -m32 and -m64
if they are
On 06/02/2011 03:25 PM, David Krauss wrote:
Optimally the re-opened context would be the preceding operator- function
itself, to create the illusion of nested calls. However, the result of
build_new_op may be a target_expr or a call_expr. I'm not sure of the best way to
recover the function
On 6 Jun 2011, at 21:23, IainS wrote:
It doesn't...
.. if you want to be pedantic the following should cover all bases
on a given platform 10.4:
make -k check-objc check-obj-c++ RUNTESTFLAGS=--target_board=unix\{-
m32,-m32/-fabi-version=1,-m64\}
duh.. I should check my typing before
... So, how do you test with -m32 ? ...
make -k check-obj-c++ RUNTESTFLAGS=--target_board=unix'{-m32,-m64}'
On x86_64-apple-darwin10 (Mac OS X 10.6) -m64 is the default and
could be omitted after the comma, but the above works also on ppc
for which the default is -m32.
The revamped patch in
On Jun 6, 2011, at 12:22 PM, Nicola Pero wrote:
This patch fixes PR obj-c++/48275. It's a routine parser ingenuity.
OK to commit ?
Ok.
On Jun 6, 2011, at 1:32 PM, Dominique Dhumieres wrote:
The revamped patch in attach should fix them. :-)
It does, thanks,
Ok, Iain chimed in that he's ok with it going in sooner, and since -m32 now
works, I think this work can go in now, thanks. Thanks for the testing help
Dominique!
Removed all of the pth code with the exception of pth_save_token_cache
and pth_load_token_cache and their respective closure.
The renaming of the remaining functions to pph will be done in a separate patch.
The patch was tested with a full bootstrap build and regression testing.
Note: There
On 6 Jun 2011, at 22:37, Mike Stump wrote:
On Jun 6, 2011, at 1:32 PM, Dominique Dhumieres wrote:
The revamped patch in attach should fix them. :-)
It does, thanks,
Ok, Iain chimed in that he's ok with it going in sooner, and since -
m32 now works, I think this work can go in now,
1 - 100 of 105 matches
Mail list logo