http://gcc.gnu.org/ml/gcc-patches/2011-03/msg02172.html
The last version:
ChangeLog:
* doc/invoke.texi (preferred-vector-size): Document.
* params.h (PREFERRED_VECTOR_SIZE): Define.
* config/arm/arm.c (arm_preferred_simd_mode): Use param
PREFERRED_VECTOR_SIZE instead of
2011/4/20 Nathan Froyd froy...@codesourcery.com:
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.
Please, commit.
Denis.
Committed as obvious (Rev. 172812).
Tobias
Index: gcc/testsuite/gfortran.dg/coarray_18.f90
===
--- gcc/testsuite/gfortran.dg/coarray_18.f90 (revision 0)
+++ gcc/testsuite/gfortran.dg/coarray_18.f90 (revision 0)
@@ -0,0 +1,39 @@
+! {
On Wed, Apr 20, 2011 at 9:43 PM, Nathan Froyd froy...@codesourcery.com wrote:
As $SUBJECT suggests. Tested with cross to s390-linux-gnu. OK to
commit?
Those kind of patches are pretty obvious, so if $target maintainers do not
comment within 48h consider them approved.
Thanks,
Richard.
On Wed, Apr 20, 2011 at 11:09 PM, Joseph S. Myers
jos...@codesourcery.com 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.
The s/struct rtx_def */rtx/ changes are all ok (in fact I'd say
Nathan Froyd wrote:
for (parm = 1; d-parm[parm] != SPU_BTI_END_OF_PARAMS; parm++)
;
- p = void_list_node;
+ gcc_assert (parm = (SPU_MAX_ARGS_TO_BUILTIN + 1));
+
+ for (i = 0; i ARRAY_SIZE (args); i++)
+ args[i] = NULL_TREE;
+
while (parm 1)
-
Nathan Froyd wrote:
* config/s390/s390.c (s390_init_builtins): Call
build_function_type_list instead of build_function_type.
This is OK.
Bye,
Ulrich
--
Dr. Ulrich Weigand
GNU Toolchain for Linux on System z and Cell BE
ulrich.weig...@de.ibm.com
Status
==
A first release candidate for GCC 4.5.3 is beeing made. The branch
is now frozen until after the final 4.5.3 release. All changes
require explicit release manager approval.
Quality Data
Priority # Change from Last Report
---
To get back to this...
Richard Sandiford richard.sandif...@linaro.org writes:
Richard Guenther richard.guent...@gmail.com writes:
On Thu, Mar 31, 2011 at 3:32 PM, Richard Sandiford
richard.sandif...@linaro.org wrote:
This patch adds an array_mode_supported_p hook, which says whether
On Thursday 21 April 2011 11:30:49 Richard Guenther wrote:
Status
==
A first release candidate for GCC 4.5.3 is beeing made. The branch
is now frozen until after the final 4.5.3 release. All changes
require explicit release manager approval.
Quality Data
Priority
On Thu, Apr 21, 2011 at 11:50 AM, Richard Sandiford
richard.sandif...@linaro.org wrote:
To get back to this...
Richard Sandiford richard.sandif...@linaro.org writes:
Richard Guenther richard.guent...@gmail.com writes:
On Thu, Mar 31, 2011 at 3:32 PM, Richard Sandiford
On Thu, 21 Apr 2011, Richard Guenther wrote:
On Wed, Apr 20, 2011 at 11:09 PM, Joseph S. Myers
jos...@codesourcery.com 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.
The
Joseph,
mingw part is ok, too.
Thanks,
Kai
This patch is a repost of the one I previously posted here:
http://gcc.gnu.org/ml/gcc-patches/2010-12/msg00652.html
As requested, I've broken out the other parts of the original patch, and
those have already been reposted yesterday (and one committed also).
This (final) part is support for
With all returns now having virtual operands we can trivially arrive
at them during alias walks. Instead of always returning true as we
did sofar this patch makes us more precise, also handle the fact
that a function return implicitly is a use for all values that
escape (now, hopefully I will
This changes maybe_build_tarfile to require that the first directory
in the list actually exists. This avoids building a GO tarball on
old branches where neither gcc/go nor libgo exist but libffi does.
I have built the 4.5.3 release candidate with this patch applied.
Looks ok?
Thanks,
On Thu, Apr 21, 2011 at 01:36:53PM +0200, Richard Guenther wrote:
This changes maybe_build_tarfile to require that the first directory
in the list actually exists. This avoids building a GO tarball on
old branches where neither gcc/go nor libgo exist but libffi does.
I have built the
On Thu, 21 Apr 2011, Jakub Jelinek wrote:
On Thu, Apr 21, 2011 at 01:36:53PM +0200, Richard Guenther wrote:
This changes maybe_build_tarfile to require that the first directory
in the list actually exists. This avoids building a GO tarball on
old branches where neither gcc/go nor libgo
Georg-Johann Lay schrieb:
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
On Wed, Apr 20, 2011 at 10:27:36AM -0700, Richard Henderson wrote:
On 04/20/2011 09:09 AM, Jakub Jelinek wrote:
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
Hi,
this simple patch adds a guard to avoid a crash. As -mdebug-main directly
calls dwarf2out, debug_info_level should be checked before.
Committed on trunk.
Tristan.
2011-04-21 Tristan Gingold ging...@adacore.com
* config/ia64/ia64.c (ia64_start_function): Add a guard.
---
Mike,
[Could you please configure your mail client to break lines? It's hard
to reply to messages all on a single line. Thanks.]
On Apr 19, 2011, at 11:12 AM, Rainer Orth wrote:
I've had a closer look now and think it's possible (and desirable) to
define HAVE_GAS_HIDDEN for Darwin, too.
Hi,
vmsdbgout could generate buggy records for long path. This happened while
cross-compiling.
Committed on trunk.
Tristan.
2011-04-21 Tristan Gingold ging...@adacore.com
* vmsdbgout.c (write_srccorr): Compute file length from the string.
(dst_file_info_struct): Remove
Remove INDIRECT_REF times code, don't use the strange SSA_VAR_P
predicate. Simple cleanups, catched a case where we still build
an INDIRECT_REF.
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk.
Richard.
2011-04-21 Richard Guenther rguent...@suse.de
*
Richard Guenther rguent...@suse.de writes:
Perhaps it would be better to use site.tmp to match current git
automake?
I can certainly use any other temporary name, just the tmp0 use
is annoying ;) It's a matter of a single search-and-replace.
Although this isn't exactly part of the
On Thu, Mar 03, 2011 at 05:54:32PM +0100, Jakub Jelinek wrote:
This patch is a WIP patch for OpenMP 3.1 atomics, so far for C FE only.
It handles parsing (I created 3 new tree codes for atomic read,
atomic capture of the old and of the new value), gimplification and
omp expansion thereof, but
On Thu, 21 Apr 2011, Rainer Orth wrote:
Richard Guenther rguent...@suse.de writes:
Perhaps it would be better to use site.tmp to match current git
automake?
I can certainly use any other temporary name, just the tmp0 use
is annoying ;) It's a matter of a single search-and-replace.
Hi,
On Wed, 20 Apr 2011, Michael Matz wrote:
It would have been nice to have the top-level tree merging as a
separate patch, as I am not convinced it is correct, but see below ...
I'll split it out.
Like so (also including the other remarks).
Regstrapping on x86_64-linux in progress.
Am 19.04.2011 20:35, schrieb Thomas Koenig:
Hello world,
this patch fixes the enhancement PR, plus probably a few regressions.
The basic problem was that the code walker got confused when *c, the
pointer to the current gfc_code statement, was changed by inserting
additional code.
Currently
Hi 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.
Approved - please apply.
Cheers
Nick
Hi Nathan,
* config/iq2000/i2000.c (iq2000_init_builtins): Call
build_function_type_list instead of build_function_type.
Delete `endlink' variable.
Approved - please apply.
Cheers
Nick
Hi Nathan,
* config/stormy16/stormy16 (xstormy16_init_builtins): Call
build_function_type_list instead of build_function_type.
Rearrange initialization of `args' to do so.
Approved - please apply.
Cheers
Nick
On Thursday 21 April 2011 16:02:18 Thomas Koenig wrote:
Am 19.04.2011 20:35, schrieb Thomas Koenig:
Hello world,
this patch fixes the enhancement PR, plus probably a few regressions.
The basic problem was that the code walker got confused when *c, the
pointer to the current gfc_code
The attached testcase triggers an ICE during nested functions lowering, a
regression present on the mainline and 4.6 branch. The middle-end is trying
to create an object which must be created by the front-end:
/* If the type is of variable size or a type which must be created by the
This fixes the fallout of not re-setting the set_decl_assembler_name
langhook with -flto.
Bootstrapped and tested on x86_64-unknown-linux-gnu, SPEC2k6 build tested,
installed.
Richard.
2011-04-21 Richard Guenther rguent...@suse.de
PR lto/48703
* tree.c
Looks good to me Nathan. Please go ahead and commit.
Thanks
Hari
On 20/04/11 20:51, Nathan Froyd wrote:
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
On Thu, Apr 21, 2011 at 3:46 PM, Michael Matz m...@suse.de wrote:
Hi,
On Wed, 20 Apr 2011, Michael Matz wrote:
It would have been nice to have the top-level tree merging as a
separate patch, as I am not convinced it is correct, but see below ...
I'll split it out.
Like so (also
This patch does two things:
- centralizes some infrastructure for defining builtin function types
for frontends by providing a common function that
DEF_FUNCTION_TYPE_FOO macros can call; and
- in order to do that well, it also introduces
build{,_varargs}_function_type_array for cases when
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
On 04/20/11 17:35, Dimitrios Apostolou wrote:
Plus a whole page which is preallocated by the obstack, if I understand
correctly. As a result, for each word in the text file we consume 4KB,
which are never freed.
Plausible. Though I always
Hi,
On Wed, 20 Apr 2011, Easwaran Raman wrote:
But you're right - not adding that conflict doesn't actually reduce the
size of bit maps. Reverting back to what was there originally.
Thanks, I have no more issues with the patch. You'll need to find someone
who can formally approve it,
On 04/20/2011 07:52 PM, Segher Boessenkool wrote:
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
On Thu, Apr 21, 2011 at 5:04 PM, Nathan Froyd froy...@codesourcery.com wrote:
This patch does two things:
- centralizes some infrastructure for defining builtin function types
for frontends by providing a common function that
DEF_FUNCTION_TYPE_FOO macros can call; and
- in order to do
On Thu, Apr 21, 2011 at 5:22 PM, Michael Matz m...@suse.de wrote:
Hi,
On Wed, 20 Apr 2011, Easwaran Raman wrote:
But you're right - not adding that conflict doesn't actually reduce the
size of bit maps. Reverting back to what was there originally.
Thanks, I have no more issues with the
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
For some dumb reason I thought handling threading through a SWITCH_EXPR
was hard in VRP; that's definitely not the case, it's no more difficult
than handling a COND_EXPR.
This patch allows tree-vrp.c to thread through a SWITCH_EXPR when we
know the
On Thu, 14 Apr 2011, Nathan Froyd wrote:
On Fri, Apr 08, 2011 at 01:50:24PM -0400, Jason Merrill wrote:
On 03/24/2011 09:15 AM, Nathan Froyd wrote:
+ tree t = make_node (CASE_LABEL_EXPR);
+
+ TREE_TYPE (t) = void_type_node;
+ SET_EXPR_LOCATION (t, input_location);
As jsm and
Hi,
On Wed, 20 Apr 2011, Richard Guenther wrote:
I had occasion to try this today; this inheritance structure doesn't
work. The truncated inheritance tree looks like:
* decl_common
* field_decl
* const_decl
* decl_with_rtl
* label_decl
* result_decl
* parm_decl
On Thu, 14 Apr 2011, Andrew Stubbs wrote:
On 14/02/11 18:20, Joseph S. Myers wrote:
Is there a reason you didn't add these functions to the shared libgcc
(adjust t-bpabi and t-symbian accordingly, add them to libgcc-bpabi.ver at
version GCC_4.6.0)? The GCC-specific names were deliberately
On Thu, 14 Apr 2011, Richard Guenther wrote:
2011-04-14 Richard Guenther rguent...@suse.de
* c-typeck.c (build_unary_op): Do not expand array-refs via
pointer arithmetic. Only adjust qualifiers for function types.
OK.
--
Joseph S. Myers
jos...@codesourcery.com
On Thu, Apr 21, 2011 at 05:54:28PM +0200, Michael Matz wrote:
In particular, FIELD_DECLs have a size, but they have no RTL associated
with them. And LABEL_DECLs have RTL, but no size.
Blaeh. So far about nice clean ideas :) One hacky idea: change my
proposal to this:
decl_common
On Thu, 14 Apr 2011, Gunther Nikl wrote:
However, the link spec seems to be harder.
%{m68020-*|m68040|m68060:-fl libm020}
Do I have to replace every m680x0 option with a matching mcpu= (maybe
even together with a march=) option?
Whatever options you want to match that spec should be
A few selected files must be compiled with special options in Ada, both for the
compiler and the runtime. The attached patch cleans things up a bit in this
area and also fixes various minor related issues.
Bootstrapped/regtested on i586-suse-linux, applied on the mainline.
2011-04-21 Eric
Not strictly related to this patch, but there are other cleanups possible
because of the only-used-on-Tru64 nature of mips-tdump/mips-tfile. In
particular, there are seven target macros (all undocumented) used by those
programs and nowhere else in GCC: ALIGN_SYMTABLE_OFFSET CODE_MASK
To refresh everyone's memory, here is the problem:
struct
{
unsigned int a : 4;
unsigned char b;
unsigned int c: 6;
} var;
void seta(){
var.a = 12;
}
Stores into a cannot touch b, so we can't store with anything wider
(e.g. a 32 bit store) that will touch b. This problem
Hello,
This patch suppresses the missing field initializer warning when a structure
is initialized with ` = { 0 }' in C. Even though the PR author asks
specifically to suppress (at least) only when a trailing comma is included,
results from Google code search suggest that spelling without a
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
On 04/07/11 18:24, Xinliang David Li wrote:
Hi,
the following patch implements the option to fine control the emitted
warnings --
1) allow suppressing warnings for use of values that may be
uninitialized. Definitely uninitialized values that
Bootstrapped and regression tested on x86_64-unknown-linux-gnu. OK for
trunk?
Would this also fix PR18046?
Ciao!
Steven
On Tue, Apr 19, 2011 at 15:47, Xinliang David Li davi...@google.com wrote:
The attached is the revised patch with a warning suggested for cases
when CFG matches, but source locations change.
Ok for trunk?
The tree.c changes are OK.
Diego.
Ping ..
David
On Wed, Apr 20, 2011 at 9:34 AM, Xinliang David Li davi...@google.com wrote:
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,
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
On 04/21/11 11:26, Steven Bosscher wrote:
Bootstrapped and regression tested on x86_64-unknown-linux-gnu. OK for
trunk?
Would this also fix PR18046?
Not right now.
If we look at VRP2 (and this only affects VRP's jump threading) we have:
SSA
On Thu, Apr 21, 2011 at 10:21 AM, Jeff Law l...@redhat.com wrote:
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
On 04/07/11 18:24, Xinliang David Li wrote:
Hi,
the following patch implements the option to fine control the emitted
warnings --
1) allow suppressing warnings for use of values
On Thu, 21 Apr 2011, Laurynas Biveinis wrote:
:( Why don't you get yourself a compile farm account?
http://gcc.gnu.org/wiki/CompileFarm
Thanks Laurynas, I am absolutely thrilled to see such a variety of
hardware! I'll try applying, but I'm not sure I'm eligible, my
contributions to OSS are
On 04/21/2011 05:31 AM, Georg-Johann Lay wrote:
+;; Some combiner patterns dealing with bits.
+;; See PR42210
+
+;; Move bit $3.$4 into bit $0.$4
+(define_insn *movbitqi.1-6.a
...
+(define_insn *movbitqi.1-6.b
...
+(define_insn *movbitqi.0
...
+(define_insn *movbitqi.7
This looks to be
In looking at some improvements to the powerpc, we wanted to change the default
for when a table jump is generated vs. a series of if statements. Now, we
could just add a powerpc specific TARGET_CASE_VALUES_THRESHOLD, but I tend to
think that these should be settable on all/most ports with
On Thu, Apr 21, 2011 at 05:36:42PM +0200, Richard Guenther wrote:
On Thu, Apr 21, 2011 at 5:04 PM, Nathan Froyd froy...@codesourcery.com
wrote:
This patch does two things:
- centralizes some infrastructure for defining builtin function types
for frontends by providing a common function
On Thu, Apr 21, 2011 at 8:43 AM, Richard Guenther
richard.guent...@gmail.com wrote:
On Thu, Apr 21, 2011 at 5:22 PM, Michael Matz m...@suse.de wrote:
Hi,
On Wed, 20 Apr 2011, Easwaran Raman wrote:
But you're right - not adding that conflict doesn't actually reduce the
size of bit maps.
Hi Mikael,
Regression-testing passed. Ping ** 0.25?
Thomas
OK. Thanks
Waiting for Emacs...
Sende fortran/ChangeLog
Sende fortran/frontend-passes.c
Sende testsuite/ChangeLog
Hinzufügen testsuite/gfortran.dg/function_optimize_6.f90
Übertrage Daten
On Mon, 18 Apr 2011, Richard Guenther wrote:
This avoids changing -P output with the PR48248 fix which appearantly
breaks Chrome (with has bogus assumptions on gcc -E -P output).
For 4.7 I think we should instead go with the 2nd patch and make -P
output smaller (which is the whole reason
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
On 04/21/11 11:26, Steven Bosscher wrote:
Bootstrapped and regression tested on x86_64-unknown-linux-gnu. OK for
trunk?
Would this also fix PR18046?
Also note that the assertion machinery doesn't really have the concept
of anti-ranges, much less
2011-04-21 Easwaran Raman era...@google.com
* gcc/cfgexpand.c (stack_var): Remove OFFSET...
(add_stack_var): ...and its reference here...
(expand_stack_vars): ...and here.
(stack_var_cmp): Sort by descending order of size.
(partition_stack_vars): Change
On Tue, 19 Apr 2011, Xinliang David Li wrote:
2011-04-18 Neil Vachharajani nvach...@gmail.com
* flags.c: New flag variable.
* opts.c (common_handle_options): Set flag_werror_set.
* opts-global.c (decode_options): Delay Werror decision
for Wcoverage-mismatch util after
On Wed, Apr 20, 2011 at 18:00, Jason Merrill ja...@redhat.com wrote:
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
On Tue, Apr 19, 2011 at 12:02:08PM -0700, Easwaran Raman wrote:
This makes the gcc driver pass the --save-temps option to the assembler or
assembler wrapper so that post-assembly tools like MAO can be integrated.
Bootstraps on x86_64. Ok for google/main?
2011-04-19 Easwaran Raman
On Thu, 21 Apr 2011, Alexander Monakov wrote:
Hello,
This patch suppresses the missing field initializer warning when a structure
is initialized with ` = { 0 }' in C. Even though the PR author asks
specifically to suppress (at least) only when a trailing comma is included,
results from
@@ -730,6 +726,8 @@ void cgraph_clone_inlined_nodes (struct
void compute_inline_parameters (struct cgraph_node *);
cgraph_inline_failed_t cgraph_edge_inlinable_p (struct cgraph_edge *);
+void cgraph_init_node_map (void);
+void cgraph_del_node_map (void);
Given that you don't even
On Thu, Apr 21, 2011 at 12:30 PM, Eric Botcazou ebotca...@adacore.com wrote:
2011-04-21 Easwaran Raman era...@google.com
* gcc/cfgexpand.c (stack_var): Remove OFFSET...
(add_stack_var): ...and its reference here...
(expand_stack_vars): ...and here.
(stack_var_cmp):
On Thu, Apr 21, 2011 at 1:36 PM, Jan Hubicka hubi...@ucw.cz wrote:
@@ -730,6 +726,8 @@ void cgraph_clone_inlined_nodes (struct
void compute_inline_parameters (struct cgraph_node *);
cgraph_inline_failed_t cgraph_edge_inlinable_p (struct cgraph_edge *);
+void cgraph_init_node_map (void);
I don't really follow the logic here. buffer is allocated to be size of
block+4 and it is expected that gcov_write_words is not executed on size
greater than 4. Since gcov_write_string now seems to be expected to handle
strings of bigger size, I think you acually need to make
On Thu, Apr 21, 2011 at 1:36 PM, Jan Hubicka hubi...@ucw.cz wrote:
@@ -730,6 +726,8 @@ void cgraph_clone_inlined_nodes (struct
void compute_inline_parameters (struct cgraph_node *);
cgraph_inline_failed_t cgraph_edge_inlinable_p (struct cgraph_edge *);
+void cgraph_init_node_map
On Thu, Apr 21, 2011 at 1:43 PM, Jan Hubicka hubi...@ucw.cz wrote:
I don't really follow the logic here. buffer is allocated to be size of
block+4 and it is expected that gcov_write_words is not executed on size
greater than 4. Since gcov_write_string now seems to be expected to handle
On Thu, Apr 21, 2011 at 12:07 PM, Michael Meissner
meiss...@linux.vnet.ibm.com wrote:
On Tue, Apr 19, 2011 at 12:02:08PM -0700, Easwaran Raman wrote:
This makes the gcc driver pass the --save-temps option to the assembler or
assembler wrapper so that post-assembly tools like MAO can be
On Thu, Apr 21, 2011 at 3:18 PM, Easwaran Raman era...@google.com wrote:
When using gcc with a post-assembly tool, we use a wrapper that
invokes the tool and needs to know if the tool's output (another
assembly file) needs to be saved. The wrapper doesn't generate any
additional .o files and
On Thu, Apr 21, 2011 at 12:54 PM, Joseph S. Myers
jos...@codesourcery.com wrote:
On Tue, 19 Apr 2011, Xinliang David Li wrote:
2011-04-18 Neil Vachharajani nvach...@gmail.com
* flags.c: New flag variable.
* opts.c (common_handle_options): Set flag_werror_set.
* opts-global.c
On Thu, 21 Apr 2011, Xinliang David Li wrote:
On Thu, Apr 21, 2011 at 12:54 PM, Joseph S. Myers
jos...@codesourcery.com wrote:
On Tue, 19 Apr 2011, Xinliang David Li wrote:
2011-04-18 Neil Vachharajani nvach...@gmail.com
* flags.c: New flag variable.
* opts.c
Rainer Orth r...@cebitec.uni-bielefeld.de writes:
Here's the error I run into:
/vol/gcc/src/hg/trunk/irix/libgo/go/os/file.go:432:12: error: incompatible
types in assignment (implicit assignment of 'syscall.Timeval' hidden field
'_f0')
On 04/21/2011 07:17 PM, Lawrence Crowl wrote:
@@ -1911,7 +1911,7 @@ ggc_collect (void)
- timevar_push (TV_GC);
+ timevar_start (TV_GC);
Why this change? GC time shouldn't be counted against whatever we
happen to be parsing when it happens.
If not, then code that generates lots of garbage
On Fri, Apr 08, 2011 at 01:50:24PM -0400, Jason Merrill wrote:
On 03/24/2011 09:15 AM, Nathan Froyd wrote:
+ tree t = make_node (CASE_LABEL_EXPR);
+
+ TREE_TYPE (t) = void_type_node;
+ SET_EXPR_LOCATION (t, input_location);
As jsm and richi said, using input_location like this is a
This patch fixes up hot/cold partitioning on ports that don't have long
conditional branches. I'll note that the entire file has lots of other jump
optimizations that are suspect.
Ok?
2011-04-21 Mike Stump mikest...@comcast.net
* reorg.c (relax_delay_slots): Don't delete a jump
Ping?
On Apr 12, 2011, at 4:36 PM, Mike Stump wrote:
This fixes 20020425-1.c when the compiler under test is built with -O0 and
we're on a machine with an 8 meg stack.
Ok?
2011-04-12 Mike Stump mikest...@comcast.net
* c-typeck.c (c_finish_if_stmt): Fold result.
*
Hi,
I have committed this patch to update my email address.
Jie
2011-04-21 Jie Zhang jzhang...@gmail.com
* MAINTAINERS: Update my email address.
Index: MAINTAINERS
===
--- MAINTAINERS (revision 172853)
+++ MAINTAINERS (working
On 04/21/2011 08:50 PM, Nathan Froyd wrote:
On Fri, Apr 08, 2011 at 01:50:24PM -0400, Jason Merrill wrote:
As jsm and richi said, using input_location like this is a regression.
Can we use DECL_SOURCE_LOCATION (label_decl) instead?
I went off and tried that; some callers provide a NULL
On 04/20/2011 03:24 PM, Nathan Froyd wrote:
As $SUBJECT suggests. Tested with cross to bfin-elf. OK to commit?
OK. Thanks!
Jie
-Nathan
* config/bfin/bfin.c (bfin_init_builtins): Call
build_function_type_list instead of build_function_type.
diff --git
On Thu, Apr 21, 2011 at 10:49:05PM -0400, Jason Merrill wrote:
On 04/21/2011 08:50 PM, Nathan Froyd wrote:
On Fri, Apr 08, 2011 at 01:50:24PM -0400, Jason Merrill wrote:
As jsm and richi said, using input_location like this is a regression.
Can we use DECL_SOURCE_LOCATION (label_decl) instead?
On Mon, Apr 18, 2011 at 10:29:17AM -0700, Nathan Froyd wrote:
Could I request that you use FOREACH_FUNCTION_ARGS in these two cases?
The conversion is trivial, and avoiding more exposed TYPE_ARG_TYPES
calls is a good thing.
Committed revision 172855.
--
Alan Modra
Australia Development Lab,
On Fri, Apr 22, 2011 at 01:29:14PM +0930, Alan Modra wrote:
On Mon, Apr 18, 2011 at 10:29:17AM -0700, Nathan Froyd wrote:
Could I request that you use FOREACH_FUNCTION_ARGS in these two cases?
The conversion is trivial, and avoiding more exposed TYPE_ARG_TYPES
calls is a good thing.
Thank you for review, update and commit this patch set!
Jie
On 04/18/2011 10:04 AM, Richard Earnshaw wrote:
On Mon, 2010-10-11 at 15:44 +0800, Jie Zhang wrote:
This patch implements TARGET_BUILTIN_DECL for ARM. With the changes of
the previous two patches, this one is straightforward.
Is it
Please review the new patch.
Thanks,
David
On Thu, Apr 21, 2011 at 3:59 PM, Joseph S. Myers
jos...@codesourcery.com wrote:
On Thu, 21 Apr 2011, Xinliang David Li wrote:
On Thu, Apr 21, 2011 at 12:54 PM, Joseph S. Myers
jos...@codesourcery.com wrote:
On Tue, 19 Apr 2011, Xinliang David Li
96 matches
Mail list logo