On Wed, 23 May 2012, H.J. Lu wrote:
On Wed, May 23, 2012 at 5:00 AM, Richard Guenther rguent...@suse.de wrote:
This finally switches us to not record global vars in referenced-vars.
For this to work I had to re-engineer how we handle global var removal
from local-decls in
On Wed, 23 May 2012, H.J. Lu wrote:
On Wed, May 23, 2012 at 7:25 AM, H.J. Lu hjl.to...@gmail.com wrote:
On Wed, May 23, 2012 at 5:00 AM, Richard Guenther rguent...@suse.de wrote:
This finally switches us to not record global vars in referenced-vars.
For this to work I had to re-engineer
On Wed, May 23, 2012 at 7:58 PM, Nathan Sidwell nat...@acm.org wrote:
On 05/22/12 15:12, Richard Guenther wrote:
But I wonder why CONSTRUCTORs do not inherit TREE_SIDE_EFFECTS
properly ...
the attached patch fixes the ICE and causes no regressions on
i686-pc-linux-gnu.
ok?
Looks ok to
On Wed, May 23, 2012 at 9:49 PM, Diego Novillo dnovi...@google.com wrote:
Part 2 of the VEC C++ conversion. This patch implements the gengtype
changes.
I extended gengtype to understand templated types. These changes are
not as ugly as I thought they would be. Gengtype has some hardwired
On Thu, 24 May 2012, Richard Guenther wrote:
On Wed, 23 May 2012, H.J. Lu wrote:
On Wed, May 23, 2012 at 7:25 AM, H.J. Lu hjl.to...@gmail.com wrote:
On Wed, May 23, 2012 at 5:00 AM, Richard Guenther rguent...@suse.de
wrote:
This finally switches us to not record global vars in
Hi,
this patch removes code handling alias pairs after they was taken over by
symbol table.
It also streamlines how things are output - do_assemble_alias is called when
alias should
be output now, while assemble_alias is the function registering new alias into
symbol
table.
I think I will
This removes redundant calls to create_var_ann. A future patch
will make this function private.
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied.
Richard.
2012-05-24 Richard Guenther rguent...@suse.de
* varpool.c (add_new_static_var): Remove call to create_var_ann.
Hi,
the issue here is that we don't diagnose jumps into the try of a
function-try-block. The reason is quite simple: from
cp_parser_function_try_block we call
cp_parser_ctor_initializer_opt_and_function_body which calls
cp_parser_function_body, which always passes false to
I'm bootstrapping and testing the following patch that inlines
create_var_ann into its single caller and removes calls to
add_referenced_vars when the var is global (which would be a no-op
and in a future patch will assert).
Bootstrap and regtest ongoing on x86_64-unknown-linux-gnu.
Richard.
On 21/05/12 15:47, Richard Earnshaw wrote:
On 17/05/12 14:23, Jim MacArthur wrote:
Sorry for the delay in responding to this, I had a few problems with
end_hard_regno. Here's a new version of the patch, which adds to
in_hard_reg_set_p the assert and a check for the hardness of end_regno.
Hi!
As discussed in the PR and on IRC, when UNDEFINED meets some vr with
equivalences in it, we need to drop the equivalences, at least when
the equivalenced SSA_NAME definition doesn't dominate the PHI stmt.
The change in vrp_visit_phi_node is needed because otherwise vrp_meet
would drop the
On Thu, May 24, 2012 at 1:41 PM, Jakub Jelinek ja...@redhat.com wrote:
Hi!
As discussed in the PR and on IRC, when UNDEFINED meets some vr with
equivalences in it, we need to drop the equivalences, at least when
the equivalenced SSA_NAME definition doesn't dominate the PHI stmt.
The change
This fixes PR53406, calling execute_fixup_cfg may need a subsequent
cfg-cleanup run.
Profiledbootstrap and regtest running on x86_64-unknown-linux-gnu.
Richard.
2012-05-24 Richard Guenther rguent...@suse.de
PR middle-end/53460
* tree-profile.c (tree_profiling): Cleanup the
On May 24, 2012, at 06:18 , Ian Lance Taylor wrote:
(install-unwind_h): Reinstate, copy to user install destination
for include files, not to the internal gcc object directory one.
(install-leaf): Depend on it.
The effect is that every time libgcc is built, unwind.h
Hi!
Ping.
On Wed, 16 May 2012 19:14:45 +0200, I wrote:
Ping.
On Wed, 09 May 2012 10:01:55 +0800, I wrote:
On Fri, 27 Apr 2012 10:29:06 +0200, Jakub Jelinek ja...@redhat.com wrote:
On Fri, Apr 27, 2012 at 12:42:41PM +0800, Thomas Schwinge wrote:
GET_MODE_BITSIZE (lmode)« (8 bits).
Committed.
Richard.
2012-05-24 Richard Guenther rguent...@suse.de
PR bootstrap/53466
* g++.dg/debug/pr53466.C: New testcase.
Index: gcc/testsuite/g++.dg/debug/pr53466.C
===
---
On Thu, May 24, 2012 at 02:29:18PM +0200, Thomas Schwinge wrote:
Hi!
Ping.
Ok.
* fold-const.c (optimize_bit_field_compare): Abort early in the strict
volatile bitfields case.
Index: fold-const.c
===
---
OK.
Jason
OK, thanks.
Jason
On Wed, May 23, 2012 at 2:48 PM, Diego Novillo dnovi...@google.com wrote:
This series of 3 patches re-implement vec.[ch] in C++.
this is fantastic! The changes look good to me.
-- Gaby
On Thu, May 24, 2012 at 3:16 AM, Richard Guenther
richard.guent...@gmail.com wrote:
Some client code changes were needed:
1- The allocation names 'heap', 'stack' and 'gc' are not embedded in
the type name anymore. They are enum values used as a template
argument for functions like
On Thu, May 24, 2012 at 3:24 AM, Richard Guenther
richard.guent...@gmail.com wrote:
On Wed, May 23, 2012 at 9:49 PM, Diego Novillo dnovi...@google.com wrote:
Part 2 of the VEC C++ conversion. This patch implements the gengtype
changes.
I extended gengtype to understand templated types.
On Thu, May 24, 2012 at 12:12 AM, Richard Guenther rguent...@suse.de wrote:
On Wed, 23 May 2012, H.J. Lu wrote:
On Wed, May 23, 2012 at 5:00 AM, Richard Guenther rguent...@suse.de wrote:
This finally switches us to not record global vars in referenced-vars.
For this to work I had to
Hi,
configure checks if contrib/compare-debug actually works for comparing a
debug and non-debug .o file before enabling bootstrap-debug. But some
compilers encode some command line options (among them -g) into a special
.comment section. This patch removes .comment section before comparing
Il 24/05/2012 15:32, Olivier Hainque ha scritto:
Hello Ian,
On May 24, 2012, at 14:22 , Olivier Hainque wrote:
libgcc/
* Makefile.in: move dependency on install-unwind_h from
install-leaf to install.
Testing went fine for me. Here is what I checked:
For a pristine
On 12-05-24 04:20 , Andrew Pinski wrote:
On Thu, May 24, 2012 at 1:16 AM, Richard Guenther
richard.guent...@gmail.com wrote:
I believe this was because of aliasing - you may dig in the svn history
to find out what we miscompiled when not doing this and whether it is
still necessary or not.
On 12-05-24 04:24 , Richard Guenther wrote:
and we could manually provide specializations for selected types
(well, the templates). Add a GTY((template)) marker to make gengtype
not emit the specializations itself (or simply make it recognize template
types and do nothing for them).
On 12-05-24 04:16 , Richard Guenther wrote:
On Wed, May 23, 2012 at 9:48 PM, Diego Novillodnovi...@google.com wrote:
This series of 3 patches re-implement vec.[ch] in C++.
The main goal of this first step is to minimize changes to client
code. I tried very hard to maintain the existing API.
On Wed, May 23, 2012 at 11:48 AM, Pat Haugen
pthau...@linux.vnet.ibm.com wrote:
The following patch fixes existing code that tried to prevent load-hit-store
(LHS) from being in the same dispatch group. The main problem was use of the
wrong dependency list in is_costly_group(), but I also added
I have committed these three patches as a single revision r187836 on
cxx-conversion.
I will address further comments to the patch in subsequent changes.
Andrew, if you have some time, could you see if you can recreate that
LTO aliasing failure you were describing earlier?
I may change the
On May 24, 2012, at 16:01 , Paolo Bonzini wrote:
(*) For this sequence to work, I had to change libgcc_tm.h into
libgcc_tm.stamp
in the list of files removed by clean: in libgcc/Makefile.in.
I think you need to remove both files?
Agreed. The .h is generated and needs to be removed by
Il 24/05/2012 16:31, Olivier Hainque ha scritto:
On May 24, 2012, at 16:01 , Paolo Bonzini wrote:
(*) For this sequence to work, I had to change libgcc_tm.h into
libgcc_tm.stamp
in the list of files removed by clean: in libgcc/Makefile.in.
I think you need to remove both files?
This patch fixes an ordering problem with deferred string lengths. For
str = str2(:nn)
where nn is a something a tad more complicated than a local variable
(e.g. a non-VALUE dummy argument), the result was wrong: the temporary
variable with the string length was used before it was set.
Il 24/05/2012 16:01, H.J. Lu ha scritto:
configure checks if contrib/compare-debug actually works for comparing a
debug and non-debug .o file before enabling bootstrap-debug. But some
compilers encode some command line options (among them -g) into a special
.comment section. This patch
On 05/23/2012 10:43 PM, Janne Blomqvist wrote:
the attached patch allows the use of clock_gettime() with
CLOCK_PROCESS_CPUTIME_ID or CLOCK_THREAD_CPUTIME_ID if the target
doesn't have getrusage() or times().
It's not completely clear to me whether CLOCK_PROCESS_CPUTIME_ID or even
On Thu, May 24, 2012 at 7:35 AM, Paolo Bonzini bonz...@gnu.org wrote:
Il 24/05/2012 16:01, H.J. Lu ha scritto:
configure checks if contrib/compare-debug actually works for comparing a
debug and non-debug .o file before enabling bootstrap-debug. But some
compilers encode some command line
On May 24, 2012, at 16:32 , Paolo Bonzini wrote:
libgcc/
* Makefile.in (clean): Remove libgcc_tm.stamp as well.
Use a separate command for stamp removals.
Ok, thanks.
Committed. The other one also, on the 4.7 branch as well.
Thanks for for prompt feedback. And thanks Ian
On Thu, May 24, 2012 at 2:46 AM, Jan Hubicka hubi...@ucw.cz wrote:
Hi,
this patch removes code handling alias pairs after they was taken over by
symbol table.
It also streamlines how things are output - do_assemble_alias is called when
alias should
be output now, while assemble_alias is
Jason Merrill ja...@redhat.com writes:
On 05/22/2012 05:04 AM, Dodji Seketeli wrote:
The problem is that cpp_get_token_1 can be called when we are at the
beginning of a macro expansion (inside enter_macro_expansion, called
from cpp_get_token_1), *before* context-c.macro is set. This happens
PING: http://gcc.gnu.org/ml/gcc-patches/2012-05/msg01003.html
Dodji Seketeli do...@redhat.com writes:
Hello,
As discussed previously, the unwinder for macro expansion is quite
verbose [1]. This patch proposes to address that shortcoming.
Consider this test case:
$ cat -n test.c
On Thu, May 24, 2012 at 11:07 AM, Dodji Seketeli do...@redhat.com wrote:
PING: http://gcc.gnu.org/ml/gcc-patches/2012-05/msg01003.html
Sorry, this slipped under my radar.
Patch is OK.
-- Gaby
Hello!
Currently gcc fails to compile following test from the testsuite [1]:
FAIL: gcc.target/i386/pr53249.c (test for excess errors)
We are trying to compile X32 specific test, but the special pattern
that was introduced to handle certain sun assembler limitation, is not
prepared to handle
On 05/24/2012 12:03 PM, Dodji Seketeli wrote:
+ if (macro != NULL)
+pfile-about_to_expand_macro_p = false;
The approach sounds good, but why do this in the push_token_context
functions rather than in enter_macro_context?
Jason
On 23/02/12 20:36, Andrew Stubbs wrote:
On 21/02/12 15:23, Andrew Stubbs wrote:
On 06/02/12 13:13, Andrew Stubbs wrote:
This patch adds DImode shift support in NEON registers/instructions.
The patch causes delays any lowering until the split2 pass, after the
register allocator has chosen
On Mon, Dec 19, 2011 at 3:50 AM, Richard Guenther rguent...@suse.de wrote:
This fixes another case of PR51572 - we need to properly stream
TYPE_DECLs in TYPE_FIELDS.
LTO Boostrap and regtest running on x86_64-unknown-linux-gnu,
SPEC 2k6 build scheduled.
Richard.
2011-12-19 Richard
Jason Merrill ja...@redhat.com writes:
On 05/24/2012 12:03 PM, Dodji Seketeli wrote:
+ if (macro != NULL)
+pfile-about_to_expand_macro_p = false;
The approach sounds good, but why do this in the push_token_context
functions rather than in enter_macro_context?
Because, it's not only in
On Wed, May 23, 2012 at 10:59:10PM +, Joseph S. Myers wrote:
On Wed, 23 May 2012, Michael Meissner wrote:
An alternative would be for the powerpc64-linux case, should we just delete
the
software floating emulation multilib and stop using the -mstrict-align,
since
Linux only runs
Hi,
This is for google branches only.
It changes the format of the temp function name so that they
can be demangled.
Tested with regression tests.
Google ref b/5733865.
Thanks,
2012-05-24 Rong Xu x...@google.com
* l-ipo.c (create_unique_name): Make temp names demanglable.
Index:
Ok.
David
On Thu, May 24, 2012 at 11:38 AM, Rong Xu x...@google.com wrote:
Hi,
This is for google branches only.
It changes the format of the temp function name so that they
can be demangled.
Tested with regression tests.
Google ref b/5733865.
Thanks,
2012-05-24 Rong Xu
On 05/24/2012 01:41 PM, Dodji Seketeli wrote:
Jason Merrillja...@redhat.com writes:
The approach sounds good, but why do this in the push_token_context
functions rather than in enter_macro_context?
Because, it's not only in enter_macro_context that I'd have to put it,
but also in
Jason Merrill ja...@redhat.com writes:
On 05/24/2012 01:41 PM, Dodji Seketeli wrote:
[...]
Another way of seeing it is to say that, from the beginning of
enter_macro_context, we are in a state of about to expand a macro
until we actually push the macro context. So it seems to make sense to
Gabriel Dos Reis g...@integrable-solutions.net writes:
On Thu, May 24, 2012 at 11:07 AM, Dodji Seketeli do...@redhat.com wrote:
PING: http://gcc.gnu.org/ml/gcc-patches/2012-05/msg01003.html
Sorry, this slipped under my radar.
No problem.
Patch is OK.
Thank you. Applied to trunk, revision
On 12-05-22 21:56 , Cary Coutant wrote:
[Revised to address review comments and to fix a bug we found late:
We've changed want_pubnames to a static inline function, and changed
the pubnames output to include (ironically enough) inline functions.]
This patch is for the google/gcc-4_6 branch.
On May 24, 2012, at 6:18 PM, Ian Lance Taylor wrote:
* Makefile.in: move dependency on install-unwind_h from
install-leaf to install.
I don't see the final patch, but it sounds promising.
:) Testing was good on my side and Paolo approved
so this was checked in, rev 187839.
On Thu, May 24, 2012 at 5:37 PM, Tobias Burnus bur...@net-b.de wrote:
On 05/23/2012 10:43 PM, Janne Blomqvist wrote:
the attached patch allows the use of clock_gettime() with
CLOCK_PROCESS_CPUTIME_ID or CLOCK_THREAD_CPUTIME_ID if the target
doesn't have getrusage() or times().
It's not
This patch to libgo copies the implementation of runtime_printf from the
other Go library, and use that instead of calling the C library printf
function. This removes some unnecessary differences in shared files,
and removes some unnecessary stack splits. However, the main reason I'm
doing it is
Rainer Orth r...@cebitec.uni-bielefeld.de writes:
The current 4.7 branch fails to build on IRIX 6.5:
/vol/gcc/src/hg/gcc-4.7-branch/local/libgo/runtime/go-caller.c:51:1: error:
conflicting types for '__go_file_line'
In file included from
OK with a couple of nits I found on the second read.
+ if (!(pp-flags pp_c_flag_gnu_v3) TREE_CODE (t) != POINTER_TYPE)
+ pp_c_type_qualifier_list (pp, t);
You can use 'code' here instead of TREE_CODE(t). Either that, or remove the
declaration of 'code' above.
- if (TREE_CODE (t)
This patch makes the function runtime.Stack actually work, rather than
simply hang the program. Bootstrapped and ran Go testsuite on
x86_64-unknown-linux-gnu. Committed to mainline and 4.7 branch.
Ian
Index: libgo/runtime/mprof.goc
Dodji Seketeli do...@redhat.com writes:
Jason Merrill ja...@redhat.com writes:
On 05/24/2012 01:41 PM, Dodji Seketeli wrote:
[...]
Another way of seeing it is to say that, from the beginning of
enter_macro_context, we are in a state of about to expand a macro
until we actually push the
On Thu, May 24, 2012 at 9:53 AM, H.J. Lu hjl.to...@gmail.com wrote:
On Thu, May 24, 2012 at 7:43 AM, H.J. Lu hjl.to...@gmail.com wrote:
On Thu, May 24, 2012 at 7:35 AM, Paolo Bonzini bonz...@gnu.org wrote:
Il 24/05/2012 16:01, H.J. Lu ha scritto:
configure checks if contrib/compare-debug
On Thu, May 24, 2012 at 02:32:59PM -0700, H.J. Lu wrote:
This patch works on openSUSE 12.1. OK to install?
Can't you do that only if the first cmp failed?
In that case strip and cmp again...
2012-05-24 H.J. Lu hongjiu...@intel.com
PR bootstrap/53472
* contrib/compare-debug
On 5/24/12, Gabriel Dos Reis g...@integrable-solutions.net wrote:
On May 24, 2012 Lawrence Crowl cr...@google.com wrote:
Add a type-safe hash table, typed_htab. Uses of this table
replace uses of libiberty's htab_t. The benefits include less
boiler-plate code, full type safety, and
Hello,
Ping # 2 for http://gcc.gnu.org/ml/gcc-patches/2012-04/msg00298.html
Thanks in advance,
Olivier
(sorry for a possible empty version of that message
sent by mistake a few minutes ago)
On Apr 5, 2012, at 17:30 , Olivier Hainque wrote:
...
With a previous version of the compiler, we
On 12-05-24 17:52 , Lawrence Crowl wrote:
That said, I'll let y'all decide how much to put in any one piece.
I favour the approach we are taking now. Each patch to cxx-conversion
is a small incremental step.
When we merge into trunk we'll merge the final product of each change,
of
Hello,
Ping # 2 for http://gcc.gnu.org/ml/gcc-patches/2012-04/msg00833.html
Thanks in advance,
Olivier
(sorry for a possible duplicate or empty
version of that message a few minutes ago)
On Apr 13, 2012, at 15:06 , Olivier Hainque wrote:
For several years now, Ada has support for a
Hello,
Ping #2 for http://gcc.gnu.org/ml/gcc-patches/2012-04/msg00808.html
Thanks in advance,
Olivier
(sorry for a duplicate or empty version of this
message a few minutes ago)
On Apr 13, 2012, at 10:08 , Olivier Hainque wrote:
Relocation troubles (24bit reloc overflows) might show up when
On Thu, May 24, 2012 at 2:43 PM, Jakub Jelinek ja...@redhat.com wrote:
On Thu, May 24, 2012 at 02:32:59PM -0700, H.J. Lu wrote:
This patch works on openSUSE 12.1. OK to install?
Can't you do that only if the first cmp failed?
In that case strip and cmp again...
Like this? OK to install?
Add basic testsuite support for gimple.
Running the gimple testsuite is the same as running any other
testsuite. From bld/gcc, run:
$ make check-gimple
I have added a single .gimple test which is currently segfaulting the
parser. This means that you will get the following output from
the
On 5/24/12, Diego Novillo dnovi...@google.com wrote:
On 12-05-24 04:16 , Richard Guenther wrote:
On May 23, 2012 Diego Novillodnovi...@google.com wrote:
Some client code changes were needed:
1- The allocation names 'heap', 'stack' and 'gc' are not embedded in
the type name anymore. They
On Wed, May 23, 2012 at 6:36 PM, Michael Meissner
meiss...@linux.vnet.ibm.com wrote:
On powerpc64-linux systems that run on IBM servers, the 32-bit software
emulation library is not built with the Red Hat and SUSE distributions, but
the
FSF sources still list it as a multilib. This patch
On Thu, May 24, 2012 at 01:45:41PM -0400, Michael Meissner wrote:
This alternative patch just disables building the 32-bit softfloat multlib,
and
removes the -mstrict-align in the powerpc64-linux case.
I have bootstrapped it and had no regressions.
David, which patch do you prefer?
libgcc/
* config/rs6000/vxworks/tramp.S (trampoline_setup): Use a longcall
sequence in the non pic case on VxWorks.
+ addis 11, 0,JUMP_TARGET(abort)@ha
Why do you use the addis mnemonic instead of lis mnemonic? Yes, lis
X,Y is an alias for addis X,0,Y, but the
Hi,
another simple issue, this one remained assigned to me for a while ;)
Anyway, we are not rejecting storage classes for anonymous unions in
class scope. Details: I'm handling anonymous structs in the same way,
for consistency (but in principle being an extension we could do
nothing); the
74 matches
Mail list logo