Thanks!
K
On Tue, Mar 13, 2012 at 11:14 PM, Uros Bizjak ubiz...@gmail.com wrote:
On Sun, Mar 11, 2012 at 10:16 AM, Kirill Yukhin kirill.yuk...@gmail.com
wrote:
The patch is OK for mainline, if there are no further comments in next 24h.
According to Tobias's input, I've added few lines
On Mar 13, 2012, at 9:57 PM, Gary Funck wrote:
On 03/06/12 14:09:23, Tristan Gingold wrote:
The patch is simple: the C front-end will now calls
c_build_pointer_type (instead of build_pointer_type), which in
turn calls build_pointer_type_for_mode using the right mode.
[...]
Joining this
On Wed, Mar 14, 2012 at 12:13 AM, H.J. Lu hjl.to...@gmail.com wrote:
Similar to tls_symbolic_operand and tls_modbase_operand, we are not
interested in the mode of the operand in the predicate.
2012-03-13 Uros Bizjak ubiz...@gmail.com
* config/i386/predicates.md
Hi,
fixinc.sh is not anymore in gcc/ directory. So I propose to adjust
fixincludes/README.
Ok for trunk ?
Tristan.
fixincludes/
2012-03-14 Tristan Gingold ging...@adacore.com
* README (EXAMPLES OF FIXES): Update.
diff --git a/fixincludes/README b/fixincludes/README
index
On Tue, Mar 13, 2012 at 7:31 PM, Andrew Pinski pins...@gmail.com wrote:
Ping? Rebootstrapped on x86_64-linux-gnu with no regressions.
Zdenek, can you have a look here? I think the patch is reasonable, but
you should have a better idea ;)
Thanks,
Richard.
Thanks,
Andrew Pinski
On Sat, Jan
Ian Lance Taylor i...@google.com writes:
I would like to update the Go support on the 4.7 branch. As I've
mentioned before, Go is working toward a stable Go 1 release. That
release is not complete, but it is quite close. The 4.7 branch was made
at a slightly unstable point in the process.
This fixes PR52571, we should not align DECL_COMMON variables as
they might be pre-empted by a definition with lower alignment.
At LTO/WPA level we might recover from missed optimizations by
promoting DECL_COMMON variables to non-common. Not sure if we
do that already.
Bootstrapped on
I suppose after a decade of being gone, it's time to stop referencing it.
OK for *ALL* active branches.
On Wed, Mar 14, 2012 at 2:36 AM, Tristan Gingold ging...@adacore.com wrote:
Hi,
fixinc.sh is not anymore in gcc/ directory. So I propose to adjust
fixincludes/README.
Ok for trunk ?
On Wed, 14 Mar 2012, Richard Guenther wrote:
This fixes PR52571, we should not align DECL_COMMON variables as
they might be pre-empted by a definition with lower alignment.
At LTO/WPA level we might recover from missed optimizations by
promoting DECL_COMMON variables to non-common. Not
Hi,
highly misleading. Committed to mainline (suitable for 4.7.1 too of course)
Thanks,
Paolo.
///
2012-03-14 Paolo Carlini paolo.carl...@oracle.com
* include/bits/forward_list.h: Fix comments.
Index: include/bits/forward_list.h
Hi!
I've committed the following workaround for PR52582 to 4.7 branch (acked
on IRC by David and Richard) after bootstrapping/regtesting it on
powerpc64-linux (--with-cpu=default32 as well as -m64 defaulted build),
and for trunk committed just the testcase. Richard committed some fix to
the
On 12 March 2012 18:10, DJ Delorie d...@redhat.com wrote:
Looks OK to me.
I was about to commit this into my svn checkout and then realized the
patch p4 didn't have the changes to configure - So I regenerated it
again and then began a journey into the depths of m4 and autoconf for
a bit before
Hi,
the dwarf3 DW_OP_call_frame_cfa implicitly refers to .debug_frame section. But
if we aren't
emitting dwarf unwinding info, it isn't anymore well defined.
There is one target that doesn't emit dwarf unwind info: ia64. And gdb is not
able to deal
with DW_OP_call_frame_cfa on ia64, thus
Hi,
it happens that some system headers on VMS have #pragma between parameters.
This is spotted by building the Ada runtime.
This patch simply handles them.
Manually tested by building for ia64-hp-openvms.
No regressions for c on x86_64-darwin.
Ok for trunk ?
Tristan.
2012-03-14 Tristan
Il 14/03/2012 16:37, Ramana Radhakrishnan ha scritto:
Empirically I spotted this odd behaviour with
gcc_GAS_CHECK_FEATURE and comments - Attached are the 2 alternate
patches that I tried and the difference in the configure scripts
themselves . I am no m4 expert but it does look like the
Hi,
the code to call expand_main_function currently only checks DECL_NAME. This
leads
to a hack in ada/gcc-interface/utils.c to handle the gnatbind generated file
that could
declare:
package ada_main is
…
function my_main
(argc : Integer;
argv : System.Address;
envp :
Jason Merrill ja...@redhat.com writes:
On 03/12/2012 01:41 PM, Joseph S. Myers wrote:
As a *target macro* it makes sense to remove it - reomve it from
defaults.h, make it purely internal to dwarf2out.c. But I think it makes
sense to have it inside dwarf2out.c
Agreed.
Here's the revised
On Wed, 14 Mar 2012, Tristan Gingold wrote:
Hi,
the code to call expand_main_function currently only checks DECL_NAME. This
leads
to a hack in ada/gcc-interface/utils.c to handle the gnatbind generated file
that could
declare:
package ada_main is
…
function my_main
(argc
Richard Henderson r...@redhat.com writes:
On 03/12/12 09:22, Rainer Orth wrote:
There are only two issues:
* In alpha.c (alpha_option_override), it's unclear if the optimize 0
test can be removed completely.
Yeah, I guess leave that for now.
* During testing, I ran into this error:
Hi,
On Mon, Mar 12, 2012 at 03:09:04PM +0100, Martin Jambor wrote:
Hi,
this patch is very similar to the one I posted before
(http://gcc.gnu.org/ml/gcc-patches/2012-02/msg01377.html) except that
it is now adjusted to sit on top of the new one before this and does
not ignore complex
On Wed, 14 Mar 2012, Tristan Gingold wrote:
Hi,
it happens that some system headers on VMS have #pragma between parameters.
This is spotted by building the Ada runtime.
This patch simply handles them.
Manually tested by building for ia64-hp-openvms.
No regressions for c on
Paolo Bonzini bonz...@gnu.org writes:
Yes, the # comment is actually part of the macro argument. If you want
to write a real comment (i.e. at the m4 rather than shell level) use
dnl instead of #.
Actually both are part of the macro argument and act like comment
introducers at the m4 level,
On Wed, 14 Mar 2012, Rainer Orth wrote:
Jason Merrill ja...@redhat.com writes:
On 03/12/2012 01:41 PM, Joseph S. Myers wrote:
As a *target macro* it makes sense to remove it - reomve it from
defaults.h, make it purely internal to dwarf2out.c. But I think it makes
sense to have it
Joseph S. Myers jos...@codesourcery.com writes:
On Wed, 14 Mar 2012, Rainer Orth wrote:
Jason Merrill ja...@redhat.com writes:
On 03/12/2012 01:41 PM, Joseph S. Myers wrote:
As a *target macro* it makes sense to remove it - reomve it from
defaults.h, make it purely internal to
On 03/14/12 09:09, Rainer Orth wrote:
Nearly ok.
+ targetm.asm_file_start_file_directive = 0;
This is default and may be deleted.
r~
The results of running the testsuite in parallel should match the
results when run serially. This patch adds KFAIL counts so that happens.
[There's still a nit that the order of the results don't precisely match,
but that's a separate issue.]
I will check this into the gdb tree if there are no
Hi!
This patch adds ASSERT_EXPRs derived from
tmp_N = var_M cst2;
if (tmp_N cmp val)
where cst2 and val are constants. Bootstrapped/regtested
on x86_64-linux and i686-linux, ok for trunk?
2012-03-14 Jakub Jelinek ja...@redhat.com
PR tree-optimization/52267
* tree-vrp.c
Ian Lance Taylor i...@google.com writes:
Unfortunately, this patch introduced several go and/or libgo testsuite
failures on both Solaris/SPARC and x86 on the 4.7 branch while the
results were clean before. I've filed PR go/52583 for this, but not yet
investigated in detail.
Hmmm, the patch
Joseph S. Myers jos...@codesourcery.com writes:
On Mon, 5 Mar 2012, Rainer Orth wrote:
The only two users of MIPS_DEBUGGING_INFO are on their way out: I've
just submitted a patch to remove the OpenBSD/MIPS configuration, and
IRIX removal will follow soon. There seems to be no point in
On Tue, Mar 13, 2012 at 9:42 PM, Diego Novillo dnovi...@google.com wrote:
On 08/03/12 20:47 , Sandeep Soni wrote:
+/* Splits the token TOKEN into two tokens FIRST_TOKEN and SECOND_TOKEN.
+ Note that the split should work only if the type of the TOKEN is
+ either CPP_RSHIFT or CPP_LSHIFT
This is for the google/gcc-4_6 branch only.
Fix output_indirect_string so that it does not output strings that
have already been written to the .debug_str.dwo section.
This is a backport of part of an unrelated change from upstream 4.7:
http://gcc.gnu.org/ml/gcc-cvs/2011-05/msg00130.html
On 2012/03/14 18:15:25, Cary wrote:
This is for the google/gcc-4_6 branch only.
Fix output_indirect_string so that it does not output strings that
have already been written to the .debug_str.dwo section.
This is a backport of part of an unrelated change from upstream 4.7:
On Wed, 14 Mar 2012, Uros Bizjak wrote:
The patch is technically OK for mainline, Joseph - can you please
review option stuff?
The option changes look fine to me.
--
Joseph S. Myers
jos...@codesourcery.com
On Mar 14, 2012, at 10:21 AM, Doug Evans wrote:
The results of running the testsuite in parallel should match the
results when run serially. This patch adds KFAIL counts so that happens.
[There's still a nit that the order of the results don't precisely match,
but that's a separate issue.]
Hello, gentle maintainer.
This is a message from the Translation Project robot.
A revised PO file for textual domain 'gcc' has been submitted
by the Swedish team of translators. The file is available at:
http://translationproject.org/latest/gcc/sv.po
(This file, 'gcc-4.7-b20120128.sv.po',
Hi,
Manuel provided this patchlet improving his template instantiation notes
work to use a template_backtrace_limit parameter, as requested in
c++/44783. I booted and tested it on x86_64-linux.
Is it Ok for mainline? Anything else we want to do here?
Thanks,
Paolo.
///
This patch overloads GCC option -freorder-functions with -freoder-functions=*
which will invoke the linker plugin libfunction_reordering_plugin.so. For now,
the only accepted option is -freoder-functions=cgedge, where the functions
which are connected by hot callgraph edges are placed closer.
On Wed, Mar 14, 2012 at 1:56 PM, Sriraman Tallam tmsri...@google.com wrote:
This patch overloads GCC option -freorder-functions with -freoder-functions=*
which will invoke the linker plugin libfunction_reordering_plugin.so. For
now, the only accepted option is -freoder-functions=cgedge, where
On Wed, 14 Mar 2012, Uros Bizjak wrote:
I have also added Gerald to CC, to check if everything is OK with
added documentation.
Joseph is better with that than I am, but I'll give it a try. :-)
+@item -maddress-mode=long
+@opindex maddress-mode=long
+Generate code for long address mode. It is
On Tue, 13 Mar 2012, Richard Guenther wrote:
Goes back to rev. 5880 by rms, at which time tsystem.h did not exist.
I'm going to remove those two lines, bootstrap test it and commit as
obvious.
Thanks, Richard. I could not do the tests you suggested yesterday
before seeing this other mail.
Hi,
I'd like to add an SH target testcase which is supposed to check the
usage of the FPUL register when float values are treated as int and vice
versa. Does this make sense?
Tested against rev 185360 with the usual
make -k check RUNTESTFLAGS=--target_board=sh-sim
thanks. This greatly improves the usability of the plugin based
function reordering feature and is in a shape that can be pushed to
trunk.
Regarding the sub-options, cgedge does not seem user friendly. How
about just -freorder-functions=callgraph or
-freorder-function=clustering?
What is the
On Wed, Mar 14, 2012 at 2:57 PM, Gerald Pfeifer ger...@pfeifer.com wrote:
On Wed, 14 Mar 2012, Uros Bizjak wrote:
I have also added Gerald to CC, to check if everything is OK with
added documentation.
Joseph is better with that than I am, but I'll give it a try. :-)
+@item
On Thu, 2012-03-15 at 08:12 +0900, Kaz Kojima wrote:
Oleg Endo oleg.e...@t-online.de wrote:
I'd like to add an SH target testcase which is supposed to check the
usage of the FPUL register when float values are treated as int and vice
versa. Does this make sense?
Tested against rev
On 03/12/2012 05:28 PM, Janne Blomqvist wrote:
Hi,
currently when writing a value of type real or complex using
list-directed output, the G0 edit descriptor, or namelist output,
gfortran chooses the number of significant digits such that a
binary-ascii-binary roundtrip recovers the original
Oleg Endo oleg.e...@t-online.de wrote:
I'd like to add an SH target testcase which is supposed to check the
usage of the FPUL register when float values are treated as int and vice
versa. Does this make sense?
Tested against rev 185360 with the usual
make -k check
Detect one class of invalid merges.
When merging two different EXPRs, we are currently merging some trees
that should not be merged. This patch tries to identify one case
where we can certainly detect that a bad merge is about to happen.
This fixes no tests, but diagnoses quite a few bad merges
Add a GC root for the lexer.
This GC root is required to support running the garbage collector
during lexing. This happens when the lexer loads pre-parsed headers.
Loading these headers calls into the middle-end, which may run the
garbage collector.
This was triggering in some of the bigger
Made the following changes:
* -ffunction-sections turned on with -freorder-functions=
* Change name from cgedge to callgraph.
* The plugin should not dump final layout to file by default.
* cgraphbuild.c (remove_cgraph_callee_edges): Replace
flag_callgraph_profiles_sections with
On Wed, Mar 14, 2012 at 6:56 PM, Sriraman Tallam tmsri...@google.com wrote:
Made the following changes:
* -ffunction-sections turned on with -freorder-functions=
* Change name from cgedge to callgraph.
* The plugin should not dump final layout to file by default.
* cgraphbuild.c
OK.
Jason
51 matches
Mail list logo
