> 2012-05-24 H.J. Lu
>
> PR bootstrap/53472
> * contrib/compare-debug (remove_comment): New function.
> Remove any .comment sections if the first cmp failed.
Ok.
Paolo
> diff --git a/contrib/compare-debug b/contrib/compare-debug
> index 010d17f..fb8986d 100755
> --- a/con
On Thu, May 24, 2012 at 09:43:42AM -0700, Lawrence Crowl 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 improved performance.
You haven't looked at the most important p
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 e
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 simp
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?
On Wed, May 23, 2012 at 6:36 PM, Michael Meissner
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 adds a configuration
> opti
On 5/24/12, Diego Novillo wrote:
> On 12-05-24 04:16 , Richard Guenther wrote:
>> On May 23, 2012 Diego Novillo 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
Add basic testsuite support for gimple.
Running the gimple testsuite is the same as running any other
testsuite. From /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 testsuit
On Thu, May 24, 2012 at 2:43 PM, Jakub Jelinek 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?
Thanks.
-
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
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 "Persis
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 course.
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 happ
On 5/24/12, Gabriel Dos Reis wrote:
> On May 24, 2012 Lawrence Crowl 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 improved performance.
>
> Lawrence, is there
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
>
> PR bootstrap/53472
> * contrib/compare-debug (remove_comment):
On Thu, May 24, 2012 at 9:53 AM, H.J. Lu wrote:
> On Thu, May 24, 2012 at 7:43 AM, H.J. Lu wrote:
>> On Thu, May 24, 2012 at 7:35 AM, Paolo Bonzini 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
Dodji Seketeli writes:
> Jason Merrill 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.
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
===
> 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 (
Rainer Orth 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
> /vol/gcc/src/hg/gcc-4.7-branch/local/libgo/runtime/go-caller.c:11:0:
> /vol
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
On Thu, May 24, 2012 at 5:37 PM, Tobias Burnus 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 complet
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 1878
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.
Fis
Gabriel Dos Reis writes:
> On Thu, May 24, 2012 at 11:07 AM, Dodji Seketeli 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 r187845.
--
Dodji
Jason Merrill 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
>> flip
On 05/24/2012 01:41 PM, Dodji Seketeli wrote:
Jason Merrill 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 replace_args, called from e
Ok.
David
On Thu, May 24, 2012 at 11:38 AM, Rong Xu 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
>
>
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
* l-ipo.c (create_unique_name): Make temp names demanglable.
Index: l-ipo.c
===
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
> > Linu
Jason Merrill 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 enter_m
On Mon, Dec 19, 2011 at 3:50 AM, Richard Guenther 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 Guenther
>
On Wed, May 23, 2012 at 10:17:51AM +0200, Richard Guenther wrote:
> On Wed, 23 May 2012, Andreas Jaeger wrote:
>
> > On Wednesday, May 23, 2012 09:56:31 Richard Earnshaw wrote:
> > > [...]
> > > This is a behaviour change. It would need RM approval for a release
> > > branch.
> > >
> > > R.
> >
On Thu, May 24, 2012 at 11:43 AM, Lawrence Crowl 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 improved performance.
Lawrence, is there any chance you could just call
On Thu, May 24, 2012 at 7:43 AM, H.J. Lu wrote:
> On Thu, May 24, 2012 at 7:35 AM, Paolo Bonzini 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 so
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 wheth
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
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 X32.
Olivier Hainque writes:
> OK to apply ?
>
> libgcc/
> * Makefile.in: move dependency on install-unwind_h from
> "install-leaf" to "install".
I don't see the final patch, but it sounds promising. Thanks for
looking at it so quickly.
Ian
On Thu, May 24, 2012 at 11:07 AM, Dodji Seketeli wrote:
> PING: http://gcc.gnu.org/ml/gcc-patches/2012-05/msg01003.html
Sorry, this slipped under my radar.
Patch is OK.
-- Gaby
PING: http://gcc.gnu.org/ml/gcc-patches/2012-05/msg01003.html
Dodji Seketeli 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
>1
Jason Merrill 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
>> e.g, w
On Thu, May 24, 2012 at 2:46 AM, Jan Hubicka 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 the functio
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 tha
On Thu, May 24, 2012 at 7:35 AM, Paolo Bonzini 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 options (amon
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
CLOCK_THREAD_
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 re
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.
T
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 bo
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 rem
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 da
On Wed, May 23, 2012 at 11:48 AM, Pat Haugen
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 code to verify
> the memo
On 12-05-24 04:16 , Richard Guenther wrote:
On Wed, May 23, 2012 at 9:48 PM, Diego Novillo 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. This means
that ve
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).
Patience.
On 12-05-24 04:20 , Andrew Pinski wrote:
On Thu, May 24, 2012 at 1:16 AM, Richard Guenther
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.
Yes it was done for aliasin
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:
>
> Fo
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
debu
On Thu, May 24, 2012 at 12:12 AM, Richard Guenther wrote:
> On Wed, 23 May 2012, H.J. Lu wrote:
>
>> On Wed, May 23, 2012 at 5:00 AM, Richard Guenther 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 g
On Thu, May 24, 2012 at 3:24 AM, Richard Guenther
wrote:
> On Wed, May 23, 2012 at 9:49 PM, Diego Novillo 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
On Thu, May 24, 2012 at 3:16 AM, Richard Guenther
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 VEC_alloc() and VEC_fr
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 tree:
configure --enable-languages=c --disable
On Wed, May 23, 2012 at 2:48 PM, Diego Novillo wrote:
>
> This series of 3 patches re-implement vec.[ch] in C++.
>
this is fantastic! The changes look good to me.
-- Gaby
OK, thanks.
Jason
OK.
Jason
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
> > >
Committed.
Richard.
2012-05-24 Richard Guenther
PR bootstrap/53466
* g++.dg/debug/pr53466.C: New testcase.
Index: gcc/testsuite/g++.dg/debug/pr53466.C
===
--- gcc/testsuite/g++.dg/debug/pr53466.C(revisio
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 wrote:
> > > On Fri, Apr 27, 2012 at 12:42:41PM +0800, Thomas Schwinge wrote:
> > > > > > GET_MODE_BITSIZE (lmode)« (8 bits). (W
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, unwin
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
PR middle-end/53460
* tree-profile.c (tree_profiling): Cleanup the CFG if
exec
On Thu, May 24, 2012 at 1:41 PM, Jakub Jelinek 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 in vrp_visit
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 equi
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.
end_ha
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.
20
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
cp_parser_compound
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
* varpool.c (add_new_static_var): Remove call to create_var_ann.
* tree-ssa-ope
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 renam
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 wrote:
> > > On Wed, May 23, 2012 at 5:00 AM, Richard Guenther
> > > wrote:
> > >>
> > >> This finally switches us to not record global vars in referenced-vars.
> >
On Wed, May 23, 2012 at 9:49 PM, Diego Novillo 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
> knowledge of
On Wed, May 23, 2012 at 7:58 PM, Nathan Sidwell 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 me.
On Wed, 23 May 2012, H.J. Lu wrote:
> On Wed, May 23, 2012 at 7:25 AM, H.J. Lu wrote:
> > On Wed, May 23, 2012 at 5:00 AM, Richard Guenther 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
On Wed, 23 May 2012, H.J. Lu wrote:
> On Wed, May 23, 2012 at 5:00 AM, Richard Guenther 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 remove_unused_locals. I
81 matches
Mail list logo
