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
2012-05-24 H.J. Lu hongjiu...@intel.com
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
Paolo,
The fix is mainly in the Ada tools, but there is a hitch: when you pass a
response file to the linker by means of the -Wl,@ option, it is intercepted
by collect2. And collect2 creates another response file for the linker
proper only if HAVE_GNU_LD is set to 1. Now HAVE_GNU_LD is set
Il 18/05/2012 23:05, Eric Botcazou ha scritto:
Hi,
this is a regression present on mainline and 4.7 branch or, more precisely,
if
you use recent GNU linkers with LTO plugin. During the link phase, gnatlink
can decide to use a response file to pass a very long line to the linker. It
Hello David,
On May 25, 2012, at 03:25 , David Edelsohn wrote:
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
Tom Tromey tro...@redhat.com writes:
Dodji == Dodji Seketeli do...@redhat.com writes:
I didn't see a gcc-patches CC...?
Woops, indeed. Sorry. I have just bounced the original messages to the
list now.
Dodji When building the compiler on altivec, it appears that an unused local
Dodji
I initially forgot to add the gcc-patches list in CC, and tried
unsuccessfully to bounce the message to the list. I am thus forwarding
it now.
---BeginMessage---
Hello,
When building the compiler on altivec, it appears that an unused local
typedef is lurking in a function of libcpp, breaking
On May 22, 2012, at 7:17 PM, Richard Henderson wrote:
On 05/15/12 02:10, Tristan Gingold wrote:
2012-05-15 Tristan Gingold ging...@adacore.com
* config/i386/i386.c (struct ix86_frame): Remove unused frame field.
(ix86_compute_frame_layout): Fix type of stack_alignment_needed
On May 22, 2012, at 7:17 PM, Richard Henderson wrote:
On 05/15/12 01:52, Tristan Gingold wrote:
2012-05-14 Tristan Gingold ging...@adacore.com
* common/config/ia64/ia64-common.c (ia64_except_unwind_info): Fix typo.
Ok.
Thanks, committed.
Tristan.
Hello,
Ping for the non-back-end parts of
http://gcc.gnu.org/ml/gcc-patches/2012-04/msg01668.html
David approved the rs6000 parts already (with adjustments to
comments, http://gcc.gnu.org/ml/gcc-patches/2012-05/msg00368.html)
Thanks much in advance for your feedback,
Olivier
On Apr 26, 2012,
Hi,
We hit an ICE using lto with neon intrinsics that could be traced back
to an invalid call to convert during expand with LTO . Fixed by
backporting Richi's fix from mainline - checking he had no objections
to this on IRC , tested cross on arm-linux-gnueabi with qemu for no
regressions.
Hi,
I have replaced expand_mult to expand_widening_mult and removed
all direct references to DImode, SImode modes in the
expand_mult_highpart_optab funtion. The attached patch was tested on
arm-7l, mips-32r2 (74k), i686 without new regressions. Richard, do you
think it is ready now?
This is a follow-up to PR lto/52178: there are more type mismatch in component
reference issues related to variably_modified_type_p when you try to build
the gnattools with -flto:
1. Instances of a record type with fixed size at top level aren't merged if
it has a field with
On Fri, May 25, 2012 at 12:27 PM, Eric Botcazou ebotca...@adacore.com wrote:
This is a follow-up to PR lto/52178: there are more type mismatch in
component
reference issues related to variably_modified_type_p when you try to build
the gnattools with -flto:
1. Instances of a record type
ARM is pleased to announce a port of GCC to its AArch64 architecture.
Please note that while the compiler has been used to build a large
body of software, it cannot yet be considered complete. We expect
there to be a number of as yet undiscovered bugs and other issues that
will need to be
This patch adds an implementation of integer iterators.
Index: gcc/ChangeLog.aarch64
* read-rtl.c (rtx_list): New data structure.
(int_iterator_mapping): New data structure.
(int_iterator_data): New. List of int iterator details.
(num_int_iterator_data): New.
On Fri, May 25, 2012 at 12:19:01PM +0100, Marcus Shawcroft wrote:
...
* config/aarch64/aarch64-simd.md: New file.
...
* config/aarch64/iterators.md: New file.
With the above in the same ChangeLog entry
* config/aarch64/aarch64-simd.md
(reduc_smax_v4sf,
Hi Ian,
Thanks. I committed the patch to mainline and 4.7 branch.
Thanks.
Testsuite results are still
terrible since many tests timeout (with timeout doubled to 600s to
account for the slow 250 MHz MIPS R10k CPUs on my test machine), and
others not even hitting the timeout at all:
On 25/05/12 12:27, Jakub Jelinek wrote:
On Fri, May 25, 2012 at 12:19:01PM +0100, Marcus Shawcroft wrote:
...
* config/aarch64/aarch64-simd.md: New file.
...
* config/aarch64/iterators.md: New file.
With the above in the same ChangeLog entry
*
On Fri, May 25, 2012 at 1:25 AM, Jakub Jelinek ja...@redhat.com wrote:
Static functions are also not acceptable as template arguments, so
this patch externalizes the functions. To avoid potential name
conflicts, the function names have been prefixed.
Ugh. I guess the C++ way around this
On Thu, May 24, 2012 at 4:55 PM, Diego Novillo dnovi...@google.com wrote:
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
Hi,
On Fri, 25 May 2012, Jakub Jelinek wrote:
+ /* Return the current size of this hash table. */
+
+ size_t size()
+ {
+return htab-size;
+ }
(and various other places) - formatting is wrong, missing space between (.
And it doesn't start at the first column, and type
On Wed, May 2, 2012 at 1:08 PM, Jason Merrill ja...@redhat.com wrote:
I've been working on a patch to split out the DWARF debug info for COMDAT
functions into COMDAT CUs to go in the same group. One problem I ran into
was that modified versions of function-local types (pointers, cv-qualified
Alternatively you could put the static assertion with the comment
into the l field, i.e.
/* Statically assert that N is 2 or 4. */
unsigned long l[(N == 2 || N == 4) ? N : -1];
I certainly prefer this alternative (the use of extern for that purpose
being extremely confusing
The attached patch solves PR53453. The upcoming dsymutil release will now
issue
a warning when no debug notes are emitted for an object file on linkage. This
causes
several thousand failures in the FSF gcc testsuite. The origin of this problem
is
that the darwin linker requires the presence
The attached patch limits the linkage of -lcrt1.10.6.o to darwin10 and
darwin11
since its usage is deprecated in the 10.8sdk. The patch also solves
radr://11491405,
-pg broken for -mmacosx-version-min=10.8...
19-May-2012 11:10 PM Jack Howarth:
Summary: The default -mmacosx-version-min=10.8
On Fri, May 25, 2012 at 09:03:02AM -0400, Jack Howarth wrote:
DEFHOOKPOD
+(force_at_comp_dir,
+ True if the @code{AT_comp_dir} attribute should be emitted for each \
It is DW_AT_comp_dir, so at least the comment should get it right.
Jakub
On Fri, May 25, 2012 at 1:19 PM, Marcus Shawcroft
marcus.shawcr...@arm.com wrote:
This patch adds an implementation of integer iterators.
Index: gcc/ChangeLog.aarch64
* read-rtl.c (rtx_list): New data structure.
(int_iterator_mapping): New data structure.
The attached patch solves PR53453. The upcoming dsymutil release will now
issue
a warning when no debug notes are emitted for an object file on linkage. This
causes
several thousand failures in the FSF gcc testsuite. The origin of this problem
is
that the darwin linker requires the presence
On Fri, May 25, 2012 at 1:19 PM, Marcus Shawcroft
marcus.shawcr...@arm.com wrote:
* config/aarch64/aarch64.md: New file.
Index: gcc/config/aarch64/aarch64.md
===
--- gcc/config/aarch64/aarch64.md (revision 0)
+++
This is a patch from Patrick, based on an earlier patch by Dave
Boutcher. Thanks folks.
In the failing testcase below we have a transaction_safe function being
accessed indirectly, but for -O1 and above, the corresponding clone is
not generated because we think it is unused. Fixed by
On Fri, May 25, 2012 at 7:17 AM, Michael Matz m...@suse.de wrote:
Hi,
On Fri, 25 May 2012, Jakub Jelinek wrote:
+ /* Return the current size of this hash table. */
+
+ size_t size()
+ {
+ return htab-size;
+ }
(and various other places) - formatting is wrong, missing
Hi,
On Fri, 25 May 2012, Gabriel Dos Reis wrote:
(and various other places) - formatting is wrong, missing space
between (.
And it doesn't start at the first column, and type isn't on a separate
line. I realize that this is a member method, hence indenting and C
GNU coding
On Sun, May 20, 2012 at 7:47 AM, H.J. Lu hongjiu...@intel.com wrote:
Hi,
This patch allows -mpreferred-stack-boundary=3 on x86-64 when SSE is
disabled. Since this option changes ABI, I also added a warning for
-mpreferred-stack-boundary=3. OK for trunk?
Thanks.
H.J.
PR
On Fri, May 25, 2012 at 3:51 PM, Michael Matz m...@suse.de wrote:
Hi,
On Fri, 25 May 2012, Gabriel Dos Reis wrote:
(and various other places) - formatting is wrong, missing space
between (.
And it doesn't start at the first column, and type isn't on a separate
line. I realize that
On Fri, May 25, 2012 at 03:51:25PM +0200, Michael Matz wrote:
On Fri, 25 May 2012, Gabriel Dos Reis wrote:
That's one of my fears, namely that those used to the libstdc++ style
impose that on the compiler source base. Because IMHO the libstdc++ style
isn't very appealing.
Seconded.
Hi,
On Fri, 25 May 2012, Steven Bosscher wrote:
Note also the almost 2 decades of C++ style practice in our libstdc++
implementation.
That's one of my fears, namely that those used to the libstdc++ style
impose that on the compiler source base. Because IMHO the libstdc++
style
Hi Uros,
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
On 12-05-25 09:56 , Jakub Jelinek wrote:
On Fri, May 25, 2012 at 03:51:25PM +0200, Michael Matz wrote:
On Fri, 25 May 2012, Gabriel Dos Reis wrote:
That's one of my fears, namely that those used to the libstdc++ style
impose that on the compiler source base. Because IMHO the libstdc++ style
General question: I suppose you expect to submit patches soon for
other toolchain components (such as binutils, GDB, glibc) and the
Linux kernel, if you haven't done so yet?
Index: config.guess
===
--- config.guess (revision
On Fri, May 25, 2012 at 8:57 AM, Michael Matz m...@suse.de wrote:
Hi,
On Fri, 25 May 2012, Steven Bosscher wrote:
Note also the almost 2 decades of C++ style practice in our libstdc++
implementation.
That's one of my fears, namely that those used to the libstdc++ style
impose that on
On Thu, May 24, 2012 at 07:11:53PM -0400, David Edelsohn wrote:
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,
On Fri, 25 May 2012, Marcus Shawcroft wrote:
Index: gcc/testsuite/gcc.target/aarch64/aapcs64/func-ret-4.x
===
--- gcc/testsuite/gcc.target/aarch64/aapcs64/func-ret-4.x (revision 0)
+++
On Fri, May 25, 2012 at 9:15 AM, Diego Novillo dnovi...@google.com wrote:
I don't care how ugly coding conventions look. I've dealt with many and
they all have their ugly spots. I mostly care about consistency.
Agreed that consistency is very important.
(At at single time in a week, I deal
On Fri, May 25, 2012 at 10:15:54AM -0400, Diego Novillo wrote:
Lawrence, Ian and Gaby have been working on the proposed coding
guidelines for C++ (http://gcc.gnu.org/wiki/CppConventions).
That page is quite inconsistent. E.g. it first talks about retaining space
before ( (which I really hope
Hi guys,
This issue has not been fixed in latest Linaro release (4.6.2012.04) yet.
Can you suggest whether this patch is a correct fix for the pr51020.
If so, should it be applied to the corresponding gcc branch?
On Tue, May 01, 2012 at 11:21:28PM +0400, Alexey Kravets wrote:
On Mon, Apr 30,
On Fri, 25 May 2012, Diego Novillo wrote:
I don't care how ugly coding conventions look. I've dealt with many and they
all have their ugly spots. I mostly care about consistency.
I don't think we should deviate much from the established GNU standards (which
are hideous, btw). Mostly
On 05/25/2012 04:15 PM, Diego Novillo wrote:
On 12-05-25 09:56 , Jakub Jelinek wrote:
On Fri, May 25, 2012 at 03:51:25PM +0200, Michael Matz wrote:
On Fri, 25 May 2012, Gabriel Dos Reis wrote:
That's one of my fears, namely that those used to the libstdc++ style
impose that on the compiler
Hi,
I've noticed that libitm is always rebuild with a non-bootstrap tree even
with merely a sequence of two makes. The reason turns out to be that
installation of unwind.h from libgcc, which is always done with a simple
make:
# make
# make -d
...
dest=../.././gcc/include/tmp$$-unwind.h; \
Hi,
On Fri, May 25, 2012 at 7:17 AM, Michael Matz m...@suse.de wrote:
Hi,
On Fri, 25 May 2012, Jakub Jelinek wrote:
+ /* Return the current size of this hash table. */
+
+ size_t size()
+ {
+return htab-size;
+ }
(and various other places) - formatting is wrong, missing
On 21.05.2012 11:16, Christophe Lyon wrote:
I tried applying your patch but ran into trouble with patch not liking
this . My suspicion is mailer munging white spaces in some form -
Could you send the patch as an attachment please rather than inline in
your mail ?
regards,
Ramana
Here it is,
Hello Michael,
On May 25, 2012, at 16:30 , Michael Matz wrote:
I've noticed that libitm is always rebuild with a non-bootstrap tree even
with merely a sequence of two makes. The reason turns out to be that
installation of unwind.h from libgcc, which is always done with a simple
make:
:
* gimple.dg/20120523-1.gimple: Add expected error.
* gimple.dg/20120525-1.gimple: New.
diff --git a/gcc/gimple/parser.c b/gcc/gimple/parser.c
index 4b29333..3f3eb96 100644
--- a/gcc/gimple/parser.c
+++ b/gcc/gimple/parser.c
@@ -152,10 +152,10 @@ gl_token_as_text (const
Hi,
On Fri, 25 May 2012, Diego Novillo wrote:
That's one of my fears, namely that those used to the libstdc++
style impose that on the compiler source base. Because IMHO the
libstdc++ style isn't very appealing.
Seconded.
I don't care how ugly coding conventions look.
Perhaps
On 12-05-25 11:06 , Michael Matz wrote:
Hi,
On Fri, 25 May 2012, Diego Novillo wrote:
That's one of my fears, namely that those used to the libstdc++
style impose that on the compiler source base. Because IMHO the
libstdc++ style isn't very appealing.
Seconded.
I don't care how ugly
Hello!
2012-05-25 Uros Bizjak ubiz...@gmail.com
PR target/53474
* config/i386/i386.c (ix86_print_operand) case 'O': Print '.' here.
case 'C', case 'c', case 'F', case 'f': Print '.' only for C and c.
Tested on i386-pc-solaris2.10 by Rainer, and on x86_64-pc-linux-gnu
Hi,
On Fri, 25 May 2012, Olivier Hainque wrote:
I've noticed that libitm is always rebuild with a non-bootstrap tree even
with merely a sequence of two makes. The reason turns out to be that
installation of unwind.h from libgcc, which is always done with a simple
make:
This should
On May 25, 2012, at 17:17 , Michael Matz wrote:
This should have been fixed by rev 187839
...
Super, yes, that works.
Great :)
Though I still wonder if the whole copy-over-to-gcc business shouldn't be
dependend on anything newly built. I can't see the use in copying
over the same
On Fri, May 25, 2012 at 11:07 AM, Diego Novillo dnovi...@google.com wrote:
On 12-05-25 11:06 , Michael Matz wrote:
Hi,
On Fri, 25 May 2012, Diego Novillo wrote:
That's one of my fears, namely that those used to the libstdc++
style impose that on the compiler source base. Because IMHO the
On Fri, May 25, 2012 at 9:33 AM, Paolo Carlini pcarl...@gmail.com wrote:
Hi,
On Fri, May 25, 2012 at 7:17 AM, Michael Matz m...@suse.de wrote:
Hi,
On Fri, 25 May 2012, Jakub Jelinek wrote:
+ /* Return the current size of this hash table. */
+
+ size_t size()
+ {
+ return
On 05/25/2012 06:25 AM, Aldy Hernandez wrote:
OK? Would this be acceptable for the 4.7 branch as well?
curr
PR middle-end/53008
* trans-mem.c (ipa_tm_create_version_alias): Output new_node if
accessed indirectly.
(ipa_tm_create_version): Same.
Ok everywhere.
Uros Bizjak ubiz...@gmail.com a écrit:
As shown in the PR, ivar-invalid-type-1 ICEs in constructor_name_p,
due to accessor on NULL type argument.
The one-liner patch fixes the ICE by adding a guard that checks that
current_class_type is non-NULL before calling constructor_name_p.
OK.
Jason
On 05/24/2012 10:57 PM, Paolo Carlini wrote:
+ error_at (decl_spec_token_start-location,
+ a storage class is not allowed);
Let's give more explanation here so that the user knows that the problem
is a storage class on an anonymous union/struct in class
On 05/25/2012 08:46 AM, Dominique Dhumieres wrote:
I certainly prefer this alternative (the use of extern for that purpose
being extremely confusing for a fortraner;-). However, if my test is valid,
if N is not equal to 2 or 4, one gets the following error
error: size of array 'l' is negative
On Fri, May 25, 2012 at 11:44 AM, Paweł Sikora pl...@agmk.net wrote:
so, why you just don't use the hash table implementation from libstdc++?
we have agreed on C++03 as a bootstrap compiler.
There is unfortunately no hash table in C++03.
The page about not defining a new template was just
On Fri, May 25, 2012 at 12:50 PM, Gabriel Dos Reis
g...@integrable-solutions.net wrote:
On Fri, May 25, 2012 at 11:44 AM, Paweł Sikora pl...@agmk.net wrote:
so, why you just don't use the hash table implementation from libstdc++?
we have agreed on C++03 as a bootstrap compiler.
There is
On Friday 25 of May 2012 11:50:13 Gabriel Dos Reis wrote:
On Fri, May 25, 2012 at 11:44 AM, Paweł Sikora pl...@agmk.net wrote:
so, why you just don't use the hash table implementation from libstdc++?
we have agreed on C++03 as a bootstrap compiler.
There is unfortunately no hash table in
On 5/25/12, Diego Novillo dnovi...@google.com wrote:
Lawrence, Ian and Gaby have been working on the proposed coding
guidelines for C++ (http://gcc.gnu.org/wiki/CppConventions).
Lawrence, have you had a chance to update them with your latest
edits?
Not yet; they're on the desk in front of me.
On 5/25/12, Paweł Sikora pl...@agmk.net wrote:
On Friday 25 of May 2012 10:15:54 Diego Novillo wrote:
Lawrence, Ian and Gaby have been working on the proposed coding
guidelines for C++ (http://gcc.gnu.org/wiki/CppConventions).
on this page we can read:
The following features of the C++
Lawrence == Lawrence Crowl cr...@google.com writes:
Lawrence Should I add that to my patch to gdbinit.in?
I think it would be helpful.
Tom
Mike == Mike Stump mikest...@comcast.net writes:
Mike Yeah, I kinda think the gdb people are wimping out by not just
Mike implementing __extension__ and ({}), which, I think get us most of the
Mike way there. Shh, don't tell them I said that.
We eagerly await your patch.
Tom
This patch to libgo fixes a cast error in the new file print.c that
shows up on 32-bit systems. Bootstrapped and ran Go testsuite on
x86_64-unknown-linux-gnu. Committed to mainline and 4.7 branch.
Ian
diff -r ae91b408310f libgo/runtime/print.c
--- a/libgo/runtime/print.c Thu May 24 14:06:08
Diego == Diego Novillo dnovi...@google.com writes:
Diego +struct GTY(()) vec_prefix
Diego +{
Diego + unsigned num;
Diego + unsigned alloc;
Diego +};
Diego +
Diego +/* Vector type, user visible. */
Diego +templatetypename T
Diego +struct GTY(()) vec_t
Diego +{
Diego + vec_prefix prefix;
Diego
On Fri, May 25, 2012 at 1:27 PM, Tom Tromey tro...@redhat.com wrote:
Diego == Diego Novillo dnovi...@google.com writes:
Diego +struct GTY(()) vec_prefix
Diego +{
Diego + unsigned num;
Diego + unsigned alloc;
Diego +};
Diego +
Diego +/* Vector type, user visible. */
Diego
On May 25, 2012, at 10:50 AM, Lawrence Crowl wrote:
Diego and I looked long and hard at this issue. It all came down
to a sequence of problems. First, libstdc++ isn't rigged for GTY,
If portability to other C++ compilers wasn't a concern, we could extend out g++
to make supporting GTY
On May 25, 2012, at 7:29 AM, Bernd Schmidt wrote:
Can emacs handle the indentation rules automatically?
I hope that any style picked, will have a no local mods required for emacs as
the corner stone. One thing I hate about .md files, is they don't (last I
checked) fulfill this requirement.
On May 25, 2012, at 6:06 AM, Jack Howarth wrote:
The attached patch limits the linkage of -lcrt1.10.6.o to darwin10 and
darwin11
since its usage is deprecated in the 10.8sdk.
Ok.
On May 25, 2012, at 6:03 AM, Jack Howarth wrote:
The attached patch solves PR53453.
Darwin bits are Ok.
On 05/25/2012 05:27 PM, Jason Merrill wrote:
On 05/25/2012 08:46 AM, Dominique Dhumieres wrote:
I certainly prefer this alternative (the use of extern for that purpose
being extremely confusing for a fortraner;-). However, if my test is valid,
if N is not equal to 2 or 4, one gets the
On 25/05/12 15:18, Joseph S. Myers wrote:
General question: I suppose you expect to submit patches soon for
other toolchain components (such as binutils, GDB, glibc) and the
Linux kernel, if you haven't done so yet?
There is work going on to get other components ready for community
review. I
On Fri, May 25, 2012 at 10:22 AM, Michael Meissner
meiss...@linux.vnet.ibm.com wrote:
Yes, the second patch just does not build the library or use the
-mstrict-align
option. It is much simpler, but there may be somebody out there that depends
on the functionality. I really don't know, one
OK.
Jason
On Fri, 25 May 2012, Marcus Shawcroft wrote:
We have a long list of intrinsics which we want to move into RTL, only some of
these have been moved so far. However, that said, point noted that we can
usefully exploit TARGET_FOLD_BUILTIN in preference to RTL.
I think the order of preference is
On 12-05-25 14:27 , Tom Tromey wrote:
Diego == Diego Novillodnovi...@google.com writes:
Diego +struct GTY(()) vec_prefix
Diego +{
Diego + unsigned num;
Diego + unsigned alloc;
Diego +};
Diego +
Diego +/* Vector type, user visible. */
Diego +templatetypename T
Diego +struct GTY(())
Ok. Please make sure to verify LTO bootstrap on the branch still
works after this.
Thanks. I already verified it with --enable-checking=yes,rtl.
--
Eric Botcazou
When using the gold linker, the linker will adjust the split-stack
prologue to call __morestack_non_split if it sees that the function
calls a different function compiled without -fsplit-stack. The
__morestack_non_split function has an optimization I committed
2011-12-20 to avoid splitting the
On 5/25/12, Michael Matz m...@suse.de wrote:
On Fri, 25 May 2012, Jakub Jelinek wrote:
+ /* Return the current size of this hash table. */
+
+ size_t size()
+ {
+return htab-size;
+ }
(and various other places) - formatting is wrong, missing space
between (.
Yes,
This patch to the Go frontend changes it to not produce a closure for an
embedded function if one is not needed. Earlier, when Go permitted
comparisons of function types, it was necessary to always create a
closure for an embedded function, so that each embedded function would
be reliably
On 5/25/12, Jakub Jelinek ja...@redhat.com wrote:
On Fri, May 25, 2012 at 10:15:54AM -0400, Diego Novillo wrote:
Lawrence, Ian and Gaby have been working on the proposed coding
guidelines for C++ (http://gcc.gnu.org/wiki/CppConventions).
That page is quite inconsistent. E.g. it first talks
On 5/24/12, Jakub Jelinek ja...@redhat.com wrote:
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
This patch to libgo makes the implementation of function trampolines,
used for nested functions that refer to variables in an enclosing
function scope, much more efficient. Previously we were allocating a
separate page for each trampoline. This patch packs them on a single
page as much as
On 5/25/12, Mike Stump mikest...@comcast.net wrote:
On May 25, 2012, at 10:50 AM, Lawrence Crowl wrote:
Diego and I looked long and hard at this issue. It all came
down to a sequence of problems. First, libstdc++ isn't rigged
for GTY,
If portability to other C++ compilers wasn't a
This patch is for the google/gcc-4_6 branch.
It fixes a problem where we were still trying to output entries
in the .debug_addr table for location lists that were removed,
and fixes a problem where we were getting an ICE while trying
to output a pubname for a member function of a struct.
Tested
Hi,
On 05/25/2012 06:25 PM, Jason Merrill wrote:
On 05/24/2012 10:57 PM, Paolo Carlini wrote:
+error_at (decl_spec_token_start-location,
+ a storage class is not allowed);
Let's give more explanation here so that the user knows that the
problem is a storage class on an
On Fri, May 25, 2012 at 4:16 PM, Lawrence Crowl cr...@google.com wrote:
On 5/25/12, Jakub Jelinek ja...@redhat.com wrote:
On Fri, May 25, 2012 at 10:15:54AM -0400, Diego Novillo wrote:
Lawrence, Ian and Gaby have been working on the proposed coding
guidelines for C++
On Fri, May 25, 2012 at 4:52 PM, Lawrence Crowl cr...@google.com wrote:
Personally, I would rather see if we can take advantage of C++
features to reduce garbage and then use the Boehm collector.
There is too much manual management with GTY, and I'd rather the
compiler leverage mainstream
I've merged gcc-4_7-branch revision 187900 to the gccgo branch.
Ian
OK.
Jason
Hello,
The following patch was necessary to make check_makefile_deps.sh work
on powerpc64-unknown-linux-gnu. Is this OK?
Ciao!
Steven
Index: contrib/ChangeLog
===
--- contrib/ChangeLog (revision 187901)
+++ contrib/ChangeLog
1 - 100 of 109 matches
Mail list logo
