Steven Bosscher stevenb@gmail.com writes:
Index: doc/tm.texi
===
--- doc/tm.texi (revision 188182)
+++ doc/tm.texi (working copy)
@@ -5847,6 +5847,10 @@ value is 0.
@end deftypevr
@deftypefn {Target Hook}
On Wed, Jun 06, 2012 at 10:49:56PM -0700, Ian Lance Taylor wrote:
This patch to the Go frontend fixes an embarrassing and serious bug in
which unsafe.Sizeof returns the wrong value for a named struct that
has fields that are themselves named structs. Bootstrapped and ran Go
testsuite on
This patch is for the google/gcc-4_6 branch.
In porting the earlier patch to use constant forms for DW_AT_high_pc,
the code in dwarf2out_finish was inadvertently changed to use an
indirect form for DW_AT_low_pc in the main compile_unit DIE. This
patch adds the force_direct parameter used in
Jakub Jelinek ja...@redhat.com writes:
On Wed, Jun 06, 2012 at 10:49:56PM -0700, Ian Lance Taylor wrote:
This patch to the Go frontend fixes an embarrassing and serious bug in
which unsafe.Sizeof returns the wrong value for a named struct that
has fields that are themselves named structs.
Go strings are not necessarily NUL terminated. The stack dump code was
erroneously using %s to print the contents of a Go string. The
runtime_printf function that it uses supports %S to print a Go string.
This patch fixes the code to use %S. Without this a stack dump
sometimes prints random
Updated patch based on feedback. Original description:
This patch adds new program summary information to the gcov
profile files that indicate how many profiled counts compose
the majority of the program's execution time. This is used to
provide an indication of the overall code size of the
This is the google/4_6 version of the patch to add new program summary
information to the gcov profile files to use as a estimate of code size
for guiding unrolling.
Bootstrapped and tested on x86_64-unknown-linux-gnu. Ok for google/4_6?
Thanks,
Teresa
Index: doc/invoke.texi
Hi!
omp-low.c isn't prepared to see invalidly nested worksharing
etc., but diagnosed them just with a warning and let them go further.
Fixed by making it an error (and nopping out the problematic constructs),
bootstrapped/regtested on x86_64-linux and i686-linux, committed to trunk.
Backport
Hi!
On the following testcase we ICE, because shrink-wrapping ends up
calling force_nonfallthru_and_redirect on a BB that ends with
asm goto that has some labels point to the fallthru block as well
(i.e. asm goto (... : : : : lab); lab:;). The problem was that in
that case the labels weren't
other than the update issue, good for google branch.
David
http://codereview.appspot.com/6282045/diff/4003/libgcc/libgcov.c
File libgcc/libgcov.c (right):
http://codereview.appspot.com/6282045/diff/4003/libgcc/libgcov.c#newcode1127
libgcc/libgcov.c:1127: cs_prg-num_hot_counters =
On Wed, Jun 6, 2012 at 10:58 PM, Sharad Singhai sing...@google.com wrote:
Sorry about the delay. I have finally incorporated all the suggestions
and reorganized the dump infrastructure a bit. The attached patch
updates vectorizer passes so that instead of accessing global
dump_file directly,
This patch to the Go compiler fixes the case of taking the address of a
field of a local variable. Previously taking the address of the field
was not treated as taking the address of the variable. But, of course,
it is. This patch fixes the problem. Bootstrapped and ran Go testsuite
on
Hi,
The patch adds arm-linux-gnueabihf triplet support.
No regression for arm-linux-gnueabi tests.
There are some differences between testsuite results on softfp Natty
builders and the new hard float Precise builders. But none are due to
the change in triplet.
Thanks!
-Zhenqiang
Thomas Koenig wrote:
his rather simple patch makes sure that only memset is used for
assigning empty strings when front-end optimization is used.
Regression-tested. OK for trunk?
OK, thanks for the quick patch.
Tobias
2012-06-06 Thomas König tkoe...@gcc.gnu.org
PR fortran/52861
On Thu, Jun 7, 2012 at 8:16 AM, Andreas Schwab sch...@linux-m68k.org wrote:
Steven Bosscher stevenb@gmail.com writes:
Index: doc/tm.texi
===
--- doc/tm.texi (revision 188182)
+++ doc/tm.texi (working copy)
@@
On Thu, Jun 7, 2012 at 8:35 AM, Ian Lance Taylor i...@google.com wrote:
Go strings are not necessarily NUL terminated. The stack dump code was
erroneously using %s to print the contents of a Go string. The
runtime_printf function that it uses supports %S to print a Go string.
This patch
Hi Steven,
This is the patch to deal with #ident. It removes the ASM_OUTPUT_IDENT
and IDENT_ASM_OP target macros, and replaces them with a single target
hook.
The RX parts of this patch are approved.
Cheers
Nick
On Thu, Jun 7, 2012 at 8:16 AM, Andreas Schwab sch...@linux-m68k.org wrote:
Steven Bosscher stevenb@gmail.com writes:
Index: doc/tm.texi
===
--- doc/tm.texi (revision 188182)
+++ doc/tm.texi (working copy)
@@
Hi,
Here is new version of patch based up on Paolo review, again tested on
arm-7l, mips-32r2 (74k), i686 without new regressions.
thanks, Dinar.
On Sat, May 26, 2012 at 4:45 PM, Paolo Bonzini bonz...@gnu.org wrote:
Il 26/05/2012 14:35, Paolo Bonzini ha scritto:
Steven Bosscher stevenb@gmail.com writes:
I am not sure what you mean with misplaced.
A nested @deftypefn doesn't look right. Try make info. You need to add
a @hook in tm.texi.in at the right place (output_ident has nothing to do
with anchors).
Andreas.
--
Andreas Schwab,
Hi Maintainers,
Please find the patch below that backports PR target/52908 to GCC 4.7.
The patch passed bootstrap and regression test.
Ok to commit?
regards,
Venkat.
Index: ChangeLog
===
--- ChangeLog (revision 187449)
+++
On Thu, Jun 7, 2012 at 12:42 PM, Andreas Schwab sch...@linux-m68k.org wrote:
Steven Bosscher stevenb@gmail.com writes:
I am not sure what you mean with misplaced.
A nested @deftypefn doesn't look right. Try make info. You need to add
a @hook in tm.texi.in at the right place
On Thu, Jun 07, 2012 at 06:07:18AM -0500, venkataramanan.ku...@amd.com wrote:
Please find the patch below that backports PR target/52908 to GCC 4.7.
The patch passed bootstrap and regression test.
Ok to commit?
Please wait with it until 4.7.1 is released.
+2012-06-07 Venkataramanan
So that it doesn't happen again. Committed as obvious.
Andreas.
* genhooks.c (main): Set progname.
(emit_documentation): Remove variable found_start, always bail out
when a place is missing.
* doc/tm.texi.in (C++ ABI): Add @hook
On Mon, 4 Jun 2012, Steven Bosscher wrote:
How about this:
Looks good to me. Do I understand correctly that this is a change
in GCC 4.7.1 and later (but not GCC 4.7.0)? If that's the case, can
you note this, for example by prepending [GCC 4.7.1 and later] to
your snippet?
Gerald
Il 07/06/2012 12:21, Dinar Temirbulatov ha scritto:
oh, I found typo in comment in the end of patch. fixed.
Great improvement, thanks!
Unfortunately we're not there yet, but much closer! I could understand
the new code much better so I suggest some more improvements below, both
to the comments
Hello world,
my recent patch introduced a regression with deferred-length
characters. I have committed the attached patch as obvious
to fix this.
Regards
Thomas
2012-06-07 Thomas König tkoe...@gcc.gnu.org
PR fortran/52861
* frontend-passes.c (optimize_assignment):
On Wed, Jun 6, 2012 at 11:23 PM, Cary Coutant ccout...@google.com wrote:
This patch is for the google/gcc-4_6 branch.
In porting the earlier patch to use constant forms for DW_AT_high_pc,
the code in dwarf2out_finish was inadvertently changed to use an
indirect form for DW_AT_low_pc in the
On Thu, Jun 7, 2012 at 12:05 AM, davi...@google.com wrote:
other than the update issue, good for google branch.
David
http://codereview.appspot.com/6282045/diff/4003/libgcc/libgcov.c
File libgcc/libgcov.c (right):
On Thu, Jun 7, 2012 at 7:52 PM, Richard Sandiford
rdsandif...@googlemail.com wrote:
Steven Bosscher stevenb@gmail.com writes:
On Wed, Jun 6, 2012 at 7:15 PM, Paolo Bonzini bonz...@gnu.org wrote:
Il 06/06/2012 09:15, Steven Bosscher ha scritto:
+ add_asm_printf (%s\n\t.ascii \%s\\n,
+
Steven Bosscher stevenb@gmail.com writes:
On Thu, Jun 7, 2012 at 7:52 PM, Richard Sandiford
rdsandif...@googlemail.com wrote:
Steven Bosscher stevenb@gmail.com writes:
On Wed, Jun 6, 2012 at 7:15 PM, Paolo Bonzini bonz...@gnu.org wrote:
Il 06/06/2012 09:15, Steven Bosscher ha scritto:
On Sun, 3 Jun 2012, Tom G. Christensen wrote:
Latest results for 4.6.x
-tgc
Testresults for 4.6.3:
i386-pc-solaris2.8
i386-pc-solaris2.10
x86_64-unknown-linux-gnu
Thanks, Tom! Online now.
Gerald
The following patch mostly fixes spec2000 mesa code generation failure
on x86 triggered by committing hard reg splitting patch.
The patch was successfully bootstrapped on x86-64.
Committed as rev. 188316.
2012-06-07 Vladimir Makarov vmaka...@redhat.com
* lra-constraints.c
Hello!
Attached patch rewrites x86 soft-FP as an out-of-line function that
gets conditionally called when exception occurs. This rewrite removes
17 instances of the same code from libgcc. In addition, the patch
unifies a lot of code between 32bit and 64bit x86 target, and also
introduces SSE
On Mon, Apr 11, 2011 at 7:25 AM, Richard Guenther rguent...@suse.de wrote:
This is another try at making sizetype behavior more consistent with
other integral types. In particular this series will eventually
succeed in making TYPE_UNSIGNED tell the truth for sizetypes ...
This first patch
The problem here was that when we instantiate G::bar, we want to
instantiate F, and end up improperly pushing the declaration inside G,
which leads to the ICE. Adjusting the call to pushclass is enough to
fix this, which is what I propose doing for the 4.7 branch. For 4.8 I'm
going to do
Hi,
This patch is for google-4_6 branch only.
It implements the multi-target indirect-call promotion as a regular IPA
passes. I took a shortcut by opening and modifying the IR in IPA execute stage.
The early version pushes the candidates to obstack and makes cgraph_node clones
, and performs
See the PR. Note well, there is no effect on the emitted
assembly code; the failure is in this pass but fixed in another.
Thus it does not pass the work/benefit threshold for adding a
separate test-case from the code in the PR, just for xfailing
this one; and for every xfail there should be a PR.
On 2012/06/01 17:58:41, saugustine wrote:
The enclosed patch updates the earlier patch to address all of the
feedback I
have gotten regarding generating pubnames. It fixes the offset problem
in
the pubtypes table; switches DW_AT_pubtypes to a flag and so on.
It also adds and documents a
http://codereview.appspot.com/6306054/diff/1/cgraph.h
File cgraph.h (right):
http://codereview.appspot.com/6306054/diff/1/cgraph.h#newcode410
cgraph.h:410:
Why not putting this into value-prof.h?
http://codereview.appspot.com/6306054/diff/1/ipa-split.c
File ipa-split.c (right):
will send the new patch set in a few minutes.
http://codereview.appspot.com/6306054/diff/1/cgraph.h
File cgraph.h (right):
http://codereview.appspot.com/6306054/diff/1/cgraph.h#newcode410
cgraph.h:410:
They are referenced in lto-cgraph.c etc.
On 2012/06/07 21:46:53, davidxl wrote:
Why not
On Thu, Jun 7, 2012 at 3:22 PM, x...@google.com wrote:
will send the new patch set in a few minutes.
http://codereview.appspot.com/6306054/diff/1/cgraph.h
File cgraph.h (right):
http://codereview.appspot.com/6306054/diff/1/cgraph.h#newcode410
cgraph.h:410:
They are referenced in
Hi,
This patch is for google-4_6 branch only.
Patch Set 2.
Tested is undergoing.
2012-06-07 Rong Xu x...@google.com
* lto-symtab.c (lto_cgraph_replace_node): indirect-call promotion
support in streaming LIPO.
(Entry_BB_profile_count): Ditto.
ok for google branches -- LTO specific bugs should be isolated and
submitted upstream.
David
http://codereview.appspot.com/6306054/diff/5001/value-prof.c
File value-prof.c (right):
http://codereview.appspot.com/6306054/diff/5001/value-prof.c#newcode1720
value-prof.c:1720: direct_call2 = 0;
Ok
(CC to ARM maintainer approving the original patch.)
I'm listing this under caveats rather than improvements and
before the current top ARM-related caveat (as this one is more
important :) because I don't see performance figures in the
context of the original patch (r178852) backing up this as an
From: Hans-Peter Nilsson h...@axis.com
Date: Fri, 1 Jun 2012 01:38:22 +0200
gcc:
Fix configure test for stack protector support in target C library.
* configure.ac (test_prefix, test_exec_prefix): Move setting from
inside sysroot handling to before and outside it.
*
From: Hans-Peter Nilsson h...@axis.com
Date: Wed, 16 May 2012 02:24:02 +0200
gcc:
* doc/tm.texi.in (Addressing Modes) TARGET_LEGITIMIZE_ADDRESS:
Mention that this hook needs to be defined for native TLS.
* doc/tm.texi: Regenerate.
On 8 June 2012 12:15, Hans-Peter Nilsson hans-peter.nils...@axis.com wrote:
(CC to ARM maintainer approving the original patch.)
I'm listing this under caveats rather than improvements and
before the current top ARM-related caveat (as this one is more
important :) because I don't see
From: Michael Hope michael.h...@linaro.org
Date: Fri, 8 Jun 2012 04:42:30 +0200
On 8 June 2012 12:15, Hans-Peter Nilsson hans-peter.nils...@axis.com wrote:
The some source
codes was in the analyzed case a strcpy of a five-byte string
(busybox/libbb/procps.c:365 'strcpy(filename_tail,
On 8 June 2012 15:20, Hans-Peter Nilsson hans-peter.nils...@axis.com wrote:
From: Michael Hope michael.h...@linaro.org
Date: Fri, 8 Jun 2012 04:42:30 +0200
On 8 June 2012 12:15, Hans-Peter Nilsson hans-peter.nils...@axis.com wrote:
The some source
codes was in the analyzed case a strcpy of
From: Michael Hope michael.h...@linaro.org
Date: Fri, 8 Jun 2012 05:50:52 +0200
On 8 June 2012 15:20, Hans-Peter Nilsson hans-peter.nils...@axis.com wrote:
So the default for ALIGNMENT_TRAP changed in 3.1?
ALIGNMENT_TRAP is on by default but the early boot time trap is now
conditional on
From: Hans-Peter Nilsson h...@axis.com
Date: Fri, 8 Jun 2012 06:29:04 +0200
From: Michael Hope michael.h...@linaro.org
Date: Fri, 8 Jun 2012 05:50:52 +0200
The combination of
older Linux ARM kernels and GCC 4.7 gives a faulty kernel.
We're in agreement!
Oh wait sorry, my bad, I
52 matches
Mail list logo