In this bug, the underlying problem was that we had a COMPLEX_EXPR
representing a complex constant rather than a COMPLEX_CST. There was
also the issue that 4.5 didn't deal with this very well, but fixing the
testcase to use COMPLEX_CST (by folding the COMPLEX_EXPR case like we do
everything
On Wed, Apr 20, 2011 at 12:00 AM, Nathan Froyd froy...@codesourcery.com wrote:
On Tue, Apr 05, 2011 at 05:55:33PM +0200, Michael Matz wrote:
I have a preference in having just one DECL_RTL field for conceptual
reasons:
Most DECLs are actually objects (there are some prominent exceptions, but
On Wed, 2011-04-20 at 16:26 +0800, Carrot Wei wrote:
On Tue, Apr 19, 2011 at 8:55 PM, Richard Earnshaw rearn...@arm.com wrote:
On Tue, 2011-04-19 at 17:41 +0800, Guozhi Wei wrote:
Reload pass tries to determine the stack frame, so it needs to check the
push/pop lr optimization
On Wed, Apr 20, 2011 at 2:42 AM, Easwaran Raman era...@google.com wrote:
Hi,
This patch allows variables whose type transitively contains a union to
share stack slots if -fno-strict-aliasing is used.
Bootstraps on x86_64 with no test regressions. Also tested by changing
I will try this method for trunk later.
thanks
Carrot
On Wed, Apr 20, 2011 at 4:48 PM, Richard Earnshaw rearn...@arm.com wrote:
On Wed, 2011-04-20 at 16:26 +0800, Carrot Wei wrote:
On Tue, Apr 19, 2011 at 8:55 PM, Richard Earnshaw rearn...@arm.com wrote:
On Tue, 2011-04-19 at 17:41 +0800,
2011-04-19 Easwaran Raman era...@google.com
* gcc/testsuite/gcc.dg/stack-layout-1.c: New
* gcc/cfgexpand.c (add_alias_set_conflicts): Add conflicts
with a variable containing union type only with
-fstrict-aliasing.
You need an entry for each relevant ChangeLog,
Denis Chertykov schrieb:
2011/4/17 Denis Chertykov cherty...@gmail.com:
2011/4/15 Georg-Johann Lay a...@gjlay.de:
Finally, I exposed alternative #3 of the insns to the register
allocator, because it is not possible to distinguish between
overlapping or non-overlapping regs, and #3 does not
There are still a couple of open issues with libgo on IRIX 6.5. This
patch fixes some of them.
* As documented in the autoconf manual, some versions of sed and grep
cannot handle \? and/or \(, \). The Solaris 8 tools are among them,
so this patch avoids those constructs.
* test $mips_abi
2011/4/20 Georg-Johann Lay a...@gjlay.de:
Denis Chertykov schrieb:
2011/4/17 Denis Chertykov cherty...@gmail.com:
2011/4/15 Georg-Johann Lay a...@gjlay.de:
Finally, I exposed alternative #3 of the insns to the register
allocator, because it is not possible to distinguish between
overlapping
Hi,
In gcc.dg/vect/slp-3.c and gcc.dg/vect/no-vfa-pr29145.c vectorization is
expected to fail on targets vect_no_align. But no realignment is necessary
here except for having the array bases aligned. This patch removes xfail
for vect_no_align (and increases a loop bound in slp-3.c to prevent
Mike Stump mikest...@comcast.net wrote:
I'd pre-approve hoisting these up into the lib/.exp files and checking a
generic target requirement... :-)
-/* { dg-skip-if unaligned access { sparc*-*-* } * } */
+/* { dg-skip-if unaligned access { sparc*-*-* sh*-*-* } * } */
I've thought the
This patch removes some redundant code that caused me some confusion.
It's not possible to construct a constant from multiple ORN
instructions, just as it's not possible to do it with multiple AND
instructions.
OK?
Andrew
2011-04-20 Andrew Stubbs a...@codesourcery.com
gcc/
*
This fixes PR48965, not handling mem-refs properly within
aliasing_component_refs_p. We disregarded the offset for the
decl base.
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk.
Richard.
2011-04-20 Richard Guenther rguent...@suse.de
PR middle-end/48695
Hi,
On Wed, 20 Apr 2011, Richard Guenther wrote:
+ /* A hashtable of trees that potentially refer to variables or functions
+ that must be replaced with their prevailing variant. */
+ static GTY((if_marked (ggc_marked_p), param_is (union tree_node)))
htab_t
+ tree_with_vars;
+
Hi,
On Sun, Apr 17, 2011 at 06:34:59PM +0200, Jan Hubicka wrote:
Hi,
this patch drops vtable_method filed. I never understood what it is
about but reading PR20991 I am convinced that we hit the same
problem with work on new devirutalization code. I implemented what
Mark describe as ideal
This makes us treat a == a in operand_equal_p in case the addresses
have TREE_CONSTANT set, even in the case if a has TREE_SIDE_EFFECTS
set (such as for volatile vars). Simply special-casing DECLs in the
ADDR_EXPR case wouldn't handle a.b == a.b.
I think it's important that operand_equal_p (a,
Denis Chertykov schrieb:
2011/4/19 Anatoly Sokolov ae...@post.ru:
Please use
ASM_OUTPUT_ALIGNED_DECL_COMMON and ASM_OUTPUT_ALIGNED_DECL_LOCAL macros
here.
Confused. These macros are used.
Johann
Sorry... Im look on function name not on macro definition.
I agree with the patch. Please
Hi,
On Tue, 19 Apr 2011, Easwaran Raman wrote:
That is correct but is also what the use of stack_vars[u].representative
achieves alone, ...
I am adding a check to that effect.
... without any check.
@@ -596,7 +581,8 @@
if (vb-conflicts)
{
Hi,
On Tue, Apr 19, 2011 at 09:07:35AM +0200, Jakub Jelinek wrote:
On Mon, Apr 18, 2011 at 03:33:18PM -0700, Jason Merrill wrote:
On 04/18/2011 02:40 PM, Jakub Jelinek wrote:
If TREE_BINFO has BV_VCALL_INDEX set, this needs to be dynamically
adjusted, but none of the callers are prepared
Hi Richard,
This is the gcc patch for the 24K errata. Does this look okay to commit?
Thanks,
Catherine
2011-04-20 Catherine Moore c...@codesourcery.com
* config/mips/mips.opt (mfix-24k): New.
* config/mips/mips.h (ASM_SPEC): Handle -mfix-24k.
* config/mips/mips.md
Hi,
On Tue, Apr 19, 2011 at 02:15:18AM +0200, Jan Hubicka wrote:
Actually what happens here is that CCP devirtualize by propagating the
constructors and due to Richard's new code to drop OBJ_TYPE_REF we finally get
a direct call. This is all good and desirable.
I think good solution would
This patch doesn't change the compiler behaviour; it merely moves the
support for MOVW's 16-bit immediate constant to const_ok_for_op.
This patch is broken out of my previous (rejected) Thumb2-constants
patch. I'll be posting v2 of that patch soon, and this clean up will be
required then.
Hi,
when gnu-ld is used, we do not need the extra vms-dwarf2.o file - which is
needed only for the native vms linker.
Committed on trunk.
Tristan.
2011-04-20 Tristan Gingold ging...@adacore.com
* config/alpha/vms.h (LINK_SPEC): Do not use vms-dwarf2.o for gnu-ld.
Hi,
On Tue, Apr 19, 2011 at 02:15:18AM +0200, Jan Hubicka wrote:
Actually what happens here is that CCP devirtualize by propagating the
constructors and due to Richard's new code to drop OBJ_TYPE_REF we finally
get
a direct call. This is all good and desirable.
I think good
Catherine Moore c...@codesourcery.com writes:
+Work around the 24K E48 Lost Data on Stores during Refill errata.
I think this should either be:
Work around the 24K E48 (@cite{Lost Data on Stores During Refill}) errata.
or:
Work around the 24K E48 (lost data on stores during refill)
On Tue, Apr 19, 2011 at 20:32, Easwaran Raman era...@google.com wrote:
The revised patch has a comment that this should be used with an
assembler wrapper that can recognize --save-temps.
Thanks. Will commit after testing finishes.
Diego.
Hi,
This patch fixes a compiler crash for alpha-vms.
Back-ends should not lie to the middle-end by defining macros to plain abort
since the middle-end is entitled to infer properties from their existence.
The correct thing to do is not to define the macros in the first place.
Committed on
http://codereview.appspot.com/4433054/diff/1/gcc/cp/pph-streamer.c
File gcc/cp/pph-streamer.c (right):
http://codereview.appspot.com/4433054/diff/1/gcc/cp/pph-streamer.c#newcode144
gcc/cp/pph-streamer.c:144: return;
+ if ((type == PPH_TRACE_TREE || type == PPH_TRACE_CHAIN)
+ !data
On Wed, 2011-04-20 at 23:06 +0800, Chung-Lin Tang wrote:
On 2011/4/20 09:24 PM, Richard Sandiford wrote:
Hi Chung-Lin,
I'm seeing an ICE with this patch, specifically;
Chung-Lin Tang clt...@codesourcery.com writes:
+ if (coproc_p)
+ low = SIGN_MAG_LOW_ADDR_BITS (val, 10);
Hi,
On Wed, 20 Apr 2011, Richard Guenther wrote:
If t is a type, why fix up its field if it may not be the canonical
variant?
Because type merging to work sometimes requires already canonicalized
fields, at least that's what I found in investigating why some types
weren't merged
On Wed, 20 Apr 2011, Jan Hubicka wrote:
Hi,
On Tue, Apr 19, 2011 at 02:15:18AM +0200, Jan Hubicka wrote:
Actually what happens here is that CCP devirtualize by propagating the
constructors and due to Richard's new code to drop OBJ_TYPE_REF we
finally get
a direct call. This
Hello,
well the bonus points might gain somebody else ... But this adds a
missing optimization
for tree level implemented in fold-const.
ChangeLog gcc/
2011-04-20 Kai Tietz
* fold-const.c (fold_binary_loc): Add handling for
(X ~Y) | (~X Y) and (X !Y) | (!X Y) optimization
Hi,
On Wed, Apr 20, 2011 at 04:38:25PM +0200, Jan Hubicka wrote:
Hi,
On Tue, Apr 19, 2011 at 02:15:18AM +0200, Jan Hubicka wrote:
Actually what happens here is that CCP devirtualize by propagating the
constructors and due to Richard's new code to drop OBJ_TYPE_REF we
finally get
I committed a patch to remove the now-empty directory
libgo/go/crypto/block.
Ian
This patch adds basic support for the Thumb ADDW and SUBW instructions.
The patch permits the compiler to use the new instructions for constants
that can be loaded with a single instruction (i.e. 16-bit unshifted),
but does not support use of addw with split-constants; I have a patch
for that
On 2011/4/20 11:12 PM, Richard Earnshaw wrote:
On Wed, 2011-04-20 at 23:06 +0800, Chung-Lin Tang wrote:
On 2011/4/20 09:24 PM, Richard Sandiford wrote:
Hi Chung-Lin,
I'm seeing an ICE with this patch, specifically;
Chung-Lin Tang clt...@codesourcery.com writes:
+ if (coproc_p)
+
On Wed, 2011-04-20 at 13:55 +0100, Andrew Stubbs wrote:
This patch removes some redundant code that caused me some confusion.
It's not possible to construct a constant from multiple ORN
instructions, just as it's not possible to do it with multiple AND
instructions.
OK?
Andrew
OK.
On 04/20/2011 08:22 AM, Kai Tietz wrote:
+ if (TREE_CODE (arg0) == BIT_AND_EXPR
+TREE_CODE (arg1) == BIT_AND_EXPR)
+{
+ tree a0, a1, l0, l1, n0, n1;
+
+ a0 = fold_convert_loc (loc, type, TREE_OPERAND (arg1, 0));
+ a1 = fold_convert_loc (loc, type,
On 20/04/11 16:27, Andrew Stubbs wrote:
(*arm_subsi3_insn): Add subw support.
Oh, I should probably say that I've added subw support to arm_subsi3
even though it's not obvious that anything will ever use this.
The existing implementation of arm_subsi3 (sans 'w') supports
On Wed, 2011-04-20 at 15:20 +0100, Andrew Stubbs wrote:
This patch doesn't change the compiler behaviour; it merely moves the
support for MOVW's 16-bit immediate constant to const_ok_for_op.
This patch is broken out of my previous (rejected) Thumb2-constants
patch. I'll be posting v2 of
This fixes PR47892, we are failing to if-convert function calls,
even those we can vectorize. This includes pow() which we
canonicalize x*x to with -ffast-math (yeah, I know ...).
No reason to not if-convert at least const builtins.
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied
2011/4/20 Richard Henderson r...@redhat.com:
On 04/20/2011 08:22 AM, Kai Tietz wrote:
+ if (TREE_CODE (arg0) == BIT_AND_EXPR
+ TREE_CODE (arg1) == BIT_AND_EXPR)
+ {
+ tree a0, a1, l0, l1, n0, n1;
+
+ a0 = fold_convert_loc (loc, type, TREE_OPERAND (arg1, 0));
On 04/20/2011 08:50 AM, Kai Tietz wrote:
+ if (TREE_CODE (arg0) == TREE_CODE (arg1)
+TREE_CODE (arg1) == TRUTH_AND_EXPR)
Ok with these both explicitly testing TRUTH_AND_EXPR now.
r~
2011/4/20 Jakub Jelinek ja...@redhat.com:
On Wed, Apr 20, 2011 at 05:22:31PM +0200, Kai Tietz wrote:
--- gcc.orig/gcc/fold-const.c 2011-04-20 17:10:39.478091900 +0200
+++ gcc/gcc/fold-const.c 2011-04-20 17:11:22.901039400 +0200
@@ -10660,6 +10660,28 @@ fold_binary_loc (location_t loc,
Hi,
as noted in the bug trail the fix for PR48207 broke compilation of C++
programs with -g. This variant fixes the bug too without breaking -g.
Basically we have to set assembler names early also for TYPE_DECLs, we
can't rely on the frontends langhook to do that after free_lang_data.
Okay
Hi!
This splitter allows us to optimize (x {* {2,4,8}, {1,2,3}}) {|,^} y
for constant integer y = {1ULL,3ULL,7ULL} using lea{l,q} (| or ^ in
that case, when the low bits are known to be all 0, is like plus).
Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk?
2011-04-20 Jakub
2011/4/20 Richard Henderson r...@redhat.com:
On 04/20/2011 08:50 AM, Kai Tietz wrote:
+ if (TREE_CODE (arg0) == TREE_CODE (arg1)
+ TREE_CODE (arg1) == TRUTH_AND_EXPR)
Ok with these both explicitly testing TRUTH_AND_EXPR now.
r~
Committed at revision 172776 with explicit
This would work if there is a way to set Werror=coverage-mismatch
without having to explicitly set the option classification as
DK_ERROR. Does this mechanism exist?
Thanks,
David
On Tue, Apr 19, 2011 at 12:52 AM, Richard Guenther
richard.guent...@gmail.com wrote:
On Tue, Apr 19, 2011 at 9:13
This patch changes most of the uses of build_function_type in the Ada to
use build_function_type_list. There are a handful of
build_function_type calls left; replacing those will have to wait until
we get a build_function_type_{n,vec} interface.
Tested on x86_64-unknown-linux-gnu. OK to commit?
Just as $SUBJECT suggests. All the other uses of
build_function_type_list are tied up with get_arg_type_list and will
therefore have to wait for a better FUNCTION_TYPE builder.
Tested on x86_64-unknown-linux-gnu. IIUC the changes to
objc-next-runtime-abi-02.c would not be tested on that
On 4/20/11, dnovi...@google.com dnovi...@google.com wrote:
http://codereview.appspot.com/4433054/diff/1/gcc/cp/pph-streamer.c
File gcc/cp/pph-streamer.c (right):
http://codereview.appspot.com/4433054/diff/1/gcc/cp/pph-streamer.c#newcode144
gcc/cp/pph-streamer.c:144: return;
+ if ((type ==
This solves some missed optimization that can be seen when moving
around bits.
There are 4 combiner patterns that operate on regs and one that uses
them as intermediate patterns and works on I/O. Even if just an
intermediate pattern matches it's still an improvement because avoid
of shift.
On 04/19/2011 06:40 AM, Eric Botcazou wrote:
- if (SUPPORTS_DISCRIMINATOR discriminator != 0)
+ if (SUPPORTS_DISCRIMINATOR
+discriminator != 0
+(dwarf_version = 4 || !dwarf_strict))
fprintf (asm_out_file, discriminator %d, discriminator);
How is this not
On Wed, Apr 20, 2011 at 6:53 AM, Michael Matz m...@suse.de wrote:
Hi,
On Tue, 19 Apr 2011, Easwaran Raman wrote:
That is correct but is also what the use of stack_vars[u].representative
achieves alone, ...
I am adding a check to that effect.
... without any check.
@@ -596,7
On 04/18/2011 02:35 PM, Jakub Jelinek wrote:
PR tree-optimization/48611
* tree-eh.c (note_eh_region_may_contain_throw): Don't propagate
beyond ERT_MUST_NOT_THROW region.
Ok.
r~
FYI, I've just pushed the following two change sets.
I verified that make check on x86_64 produced the same set of 92
failures without as with my changes. However, when I ran
make check MALLOC_PERTURB_=0 MALLOC_CHECK_=0, I saw only 91 failures.
(normally those MALLOC_*_ variables are set to
On Wed, Apr 20, 2011 at 2:12 AM, Eric Botcazou ebotca...@adacore.com wrote:
2011-04-19 Easwaran Raman era...@google.com
* gcc/testsuite/gcc.dg/stack-layout-1.c: New
* gcc/cfgexpand.c (add_alias_set_conflicts): Add conflicts
with a variable containing union type only with
How is this not redundant with the existing
/* The discriminator column was added in dwarf4. Simplify the below
by simply removing it if we're not supposed to output it. */
if (dwarf_version 4 dwarf_strict)
discriminator = 0;
check near the top of the function?
Obviously
Please review the latest patch. SPEC2k FDO testing pass.
Thanks,
David
On Wed, Apr 20, 2011 at 11:22 AM, Xinliang David Li davi...@google.com wrote:
Here is the revised patch. Basic FDO testing went fine. I still saw
the ipa-inline assertion in SPEC FDO. Will run it when the regression
is
As $SUBJECT suggests. Tested with cross to alpha-elf. OK to commit?
-Nathan
* config/alpha/alpha.c (alpha_init_builtins): Call
build_function_type_list instead of build_function_type.
diff --git a/gcc/config/alpha/alpha.c b/gcc/config/alpha/alpha.c
index 5e85e2b..237e9b3
As $SUBJECT suggests. Tested with cross to bfin-elf. OK to commit?
-Nathan
* config/bfin/bfin.c (bfin_init_builtins): Call
build_function_type_list instead of build_function_type.
diff --git a/gcc/config/bfin/bfin.c b/gcc/config/bfin/bfin.c
index 5d08437..03a833d 100644
---
As $SUBJECT suggests. Tested with cross to frv-elf. OK to commit?
-Nathan
* config/frv/frv.c (frv_init_builtins): Delete `endlink' variable.
Call builtin_function_type_list instead of builtin_function_type.
(UNARY, BINARY, TRINARY, QUAD): Likewise.
diff --git
As $SUBJECT suggests. There's still one use of build_function_type;
replacing that type will have to wait for an improved
FUNCTION_TYPE-building interface.
Tested on x86_64-unknown-linux-gnu. OK to commit?
-Nathan
* config/i386/i386.c (ix86_code_end): Call build_function_type_list
As $SUBJECT suggests. Tested with cross to ia64-linux-gnu. OK to
commit?
-Nathan
* config/ia64/ia64.c (ia64_init_builtins): Call
build_function_type_list instead of builtin_function_type.
diff --git a/gcc/config/ia64/ia64.c b/gcc/config/ia64/ia64.c
index 5f22b17..166ec43
As $SUBJECT suggests. Tested with cross to iq2000-elf. OK to commit?
-Nathan
* config/iq2000/i2000.c (iq2000_init_builtins): Call
build_function_type_list instead of build_function_type.
Delete `endlink' variable.
diff --git a/gcc/config/iq2000/iq2000.c
As $SUBJECT suggests. Tested with cross to mips-elf. OK to commit?
-Nathan
* config/mips/mips.c (mips16_build_function_stub): Call
build_function_type_list instead of build_function_type.
(mips16_build_call_stub): Likewise.
diff --git a/gcc/config/mips/mips.c
* config/mep/mep.c (mep_init_builtins): Call build_function_type_list
instead of build_function_type.
Ok.
The problem in this testcase was that we were recognizing a local const
variable with a constant initializer as a constant expression, but we
weren't doing the necessary adjustments to convert the initializer to
the type of the variable.
But some of the other bits of cp_finish_decl caused
As $SUBJECT suggests. Tested with cross to s390-linux-gnu. OK to
commit?
-Nathan
* config/s390/s390.c (s390_init_builtins): Call
build_function_type_list instead of build_function_type.
diff --git a/gcc/config/s390/s390.c b/gcc/config/s390/s390.c
index caee077..adacfa3 100644
As $SUBJECT suggests. Tested with cross to xtensa-elf. OK to commit?
-Nathan
* config/xtensa/xtensa.c (xtensa_init_builtins): Call
build_function_type_list instead of build_function_type.
diff --git a/gcc/config/xtensa/xtensa.c b/gcc/config/xtensa/xtensa.c
index
Nathan Froyd froy...@codesourcery.com writes:
* config/mips/mips.c (mips16_build_function_stub): Call
build_function_type_list instead of build_function_type.
(mips16_build_call_stub): Likewise.
OK, thanks, but:
- build_function_type
As $SUBJECT suggests. The only tricky part is in builtin_function_type,
where we fill in unused args with NULL_TREE so that passing extra
arguments to build_function_type_list doesn't matter.
Tested with cross to powerpc-eabi. OK to commit?
-Nathan
* config/rs6000/rs6000.c
As $SUBJECT suggests. Tested with cross to picochip-elf. OK to commit?
-Nathan
* config/picochip/picochip.c (picochip_init_builtins): Call
build_function_type_list instead of build_function_type.
Delete `endlink' variable.
diff --git a/gcc/config/picochip/picochip.c
Hello!
Attached patch enhances the fix for PR target/48678 to generate
pinsr{q,w,d,q} insn when value is inserted into vector register.
2011-04-20 Uros Bizjak ubiz...@gmail.com
PR target/48678
* config/i386/i386.md (insv): Change operand 0 constraint to
As $SUBJECT suggests. Tested with cross to avr-elf. OK to commit?
-Nathan
* config/avr/avr.c (avr_init_builtins): Call
build_function_type_list instead of build_function_type.
diff --git a/gcc/config/avr/avr.c b/gcc/config/avr/avr.c
index 500a5b2..6dbf8b4 100644
---
As $SUBJECT suggests. Tested with cross to hppa-linux-gnu. OK to
commit?
-Nathan
* config/pa/pa.c (pa_init_builtins): Call build_function_type_list
instead of build_function_type.
diff --git a/gcc/config/pa/pa.c b/gcc/config/pa/pa.c
index e05cf19..aeb8061 100644
---
As $SUBJECT suggests. There's one remaining use of build_function_type,
but replace that will have to wait until we have a better
FUNCTION_TYPE-building interface.
Tested with cross to arm-eabi. OK to commit?
-Nathan
* config/arm/arm.c (arm_init_iwmmxt_builtins): Call
On Wed, Apr 20, 2011 at 03:29:19PM -0400, Nathan Froyd wrote:
As $SUBJECT suggests. Tested with cross to ia64-linux-gnu. OK to
commit?
- ftype = build_function_type (float128_type, void_list_node);
- decl = add_builtin_function (__builtin_infq, ftype,
-
* config/pa/pa.c (pa_init_builtins): Call build_function_type_list
instead of build_function_type.
Ok.
Dave
--
J. David Anglin dave.ang...@nrc-cnrc.gc.ca
National Research Council of Canada (613) 990-0752 (FAX: 952-6602)
As $SUBJECT suggests. The only tricky bit is initializing all the args
to NULL_TREE so that we can safely pass all the args to
build_function_type_list.
Tested with cross to spu-elf; I couldn't build all of libgcc, but that
appears to be a pre-existing problem. OK to commit?
-Nathan
*
As $SUBJECT suggests. For safety's sake, we initialize all the
arguments to NULL before passing them to build_function_type_list. This
is not necessary currently, as we always completely fill in the args
array, but it might save some future coder from quite some grief...
Tested with cross to
As $SUBJECT suggests. The only tricky bit is the initialization of
`args' to NULL_TREEs so that we can safely pass all of the relevant args
to build_function_type_list, regardless of whether the function type in
question has that many args.
Tested with cross to sh-elf. OK to commit?
-Nathan
On 04/18/2011 02:30 AM, Richard Sandiford wrote:
Ping for these two changes:
http://gcc.gnu.org/ml/gcc-patches/2011-04/msg00194.html
http://gcc.gnu.org/ml/gcc-patches/2011-04/msg00195.html
Both ok.
r~
On 04/20/2011 12:09 PM, Eric Botcazou wrote:
How is this not redundant with the existing
/* The discriminator column was added in dwarf4. Simplify the below
by simply removing it if we're not supposed to output it. */
if (dwarf_version 4 dwarf_strict)
discriminator = 0;
Hello list,
while trying to build gcc-4.6.0 on my sparcstation, I got gengtype OOM
killed. That's when I noticed that its RAM usage peaks at 150MB, which is
a bit excessive for parsing a ~500K text file.
The attached patch fixes the leak and gengtype now uses a peak of 4MB
heap. Hopefully I
On Apr 20, 2011, at 10:27 AM, Nathan Froyd wrote:
Tested on x86_64-unknown-linux-gnu. IIUC the changes to
objc-next-runtime-abi-02.c would not be tested on that platform, so it
would be helpful to have a Darwin tester double-check my work.
Just check http://gcc.gnu.org/regtest/HEAD/ after
Ping^2. This patch
http://gcc.gnu.org/ml/gcc-patches/2011-04/msg00130.html is still pending
review. This version applies cleanly to current trunk.
2011-04-20 Joseph Myers jos...@codesourcery.com
* config/alpha/alpha.c (struct machine_function): Use rtx, not
struct rtx_def
On Wed, 2011-04-20 at 13:03 -0700, Nathan Froyd wrote:
On Wed, Apr 20, 2011 at 03:29:19PM -0400, Nathan Froyd wrote:
As $SUBJECT suggests. Tested with cross to ia64-linux-gnu. OK to
commit?
- ftype = build_function_type (float128_type, void_list_node);
- decl =
Ping. This patch
http://gcc.gnu.org/ml/gcc-patches/2011-04/msg00919.html is pending
review.
--
Joseph S. Myers
jos...@codesourcery.com
The m32c one is OK
On Wed, Apr 20, 2011 at 02:09:49PM -0700, Steve Ellcey wrote:
I am not sure what the patch would look like then. You removed the
assignment to decl, so what are you putting in ia64_builtins? Can you
send the full correct patch.
Sure. Updated patch below, which probably looks somewhat more
On Wed, 2011-04-20 at 17:25 -0400, Nathan Froyd wrote:
On Wed, Apr 20, 2011 at 02:09:49PM -0700, Steve Ellcey wrote:
I am not sure what the patch would look like then. You removed the
assignment to decl, so what are you putting in ia64_builtins? Can you
send the full correct patch.
On 04/12/2011 11:49 AM, Lawrence Crowl wrote:
This patch is available for review at http://codereview.appspot.com/4378056
I tried to comment there, but it didn't seem to be working; looking at
the side-by-side diffs didn't show any changes, and double-clicking on a
line in the patch form
Hi,
I wrote:
Basically we have to set assembler names early also for TYPE_DECLs, we
can't rely on the frontends langhook to do that after free_lang_data.
Okay for trunk assuming regstrapping on x86_64-linux works?
Patch retracted, doesn't even survive testsuite. The problem is that we
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
On 04/20/11 15:08, Dimitrios Apostolou wrote:
Hello list,
while trying to build gcc-4.6.0 on my sparcstation, I got gengtype OOM
killed. That's when I noticed that its RAM usage peaks at 150MB, which
is a bit excessive for parsing a ~500K text
On Apr 20, 2011, at 5:22 AM, Kaz Kojima wrote:
Mike Stump mikest...@comcast.net wrote:
I'd pre-approve hoisting these up into the lib/.exp files and checking a
generic target requirement... :-)
-/* { dg-skip-if unaligned access { sparc*-*-* } * } */
+/* { dg-skip-if unaligned access {
On Wed, 20 Apr 2011, Jeff Law wrote:
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
On 04/20/11 15:08, Dimitrios Apostolou wrote:
Hello list,
while trying to build gcc-4.6.0 on my sparcstation, I got gengtype OOM
killed. That's when I noticed that its RAM usage peaks at 150MB, which
is a bit
On Wed, Apr 20, 2011 at 3:49 PM, Nathan Froyd froy...@codesourcery.com wrote:
As $SUBJECT suggests. The only tricky part is in builtin_function_type,
where we fill in unused args with NULL_TREE so that passing extra
arguments to build_function_type_list doesn't matter.
Tested with cross to
On Apr 20, 2011, at 5:09 PM, Joseph S. Myers wrote:
Ping^2. This patch
http://gcc.gnu.org/ml/gcc-patches/2011-04/msg00130.html is still pending
review. This version applies cleanly to current trunk.
...
pdp11 is fine. Thanks!
paul
The test and-1.c has wrong logic.
In the formula:
y ~(y -y)
The part (y -y) is always a mask with one bit set, which corresponds
to the least significant 1 bit in y.
The final result is that bit, is set to zero (y ~mask)
There is no boolean simplification possible, and the compiler always
1 - 100 of 105 matches
Mail list logo