On Mon, Aug 20, 2012 at 7:03 PM, Dimitrios Apostolou ji...@gmx.net wrote:
Hello,
my last attempt on improving something serious was about three weeks ago,
trying to keep all lengths of all strings parsed in the frontend for the
whole compilation phase until the assembly output. I was hoping
On Mon, Aug 20, 2012 at 10:29 PM, Jan Hubicka hubi...@ucw.cz wrote:
On Mon, Aug 20, 2012 at 6:27 PM, Jan Hubicka hubi...@ucw.cz wrote:
Xinliang David Li davi...@google.com writes:
Process level synchronization problems can happen when two processes
(running the instrumented binary)
This is useful for large applications with a long tail. The
instruction working set for those applications are very large, and
inliner and unroller need to be aware of that and good heuristics can
be developed to throttle aggressive code bloat transformations. For
inliner, it is kind of the
On Tue, Aug 21, 2012 at 3:31 AM, Lawrence Crowl cr...@google.com wrote:
On 8/20/12, H.J. Lu hjl.to...@gmail.com wrote:
The C++ merge caused:
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54332
GCC memory usage is more than doubled from = 3GB to = 10GB.
Is this a known issue?
The two memory
Hi,
this patch adds a hint that if inlining makes bounds on loop iterations known,
it is probably good idea. This is primarely targetting Fortran's array
descriptors, but should be generally useful.
Fortran will still need a bit more work. Often we disregard inlining because we
think the call
Gerald Pfeifer wrote:
I went ahead and made some smaller changes, patch below.
Thanks.
I noticed you are using q.../q, as in qcodee/code/q,
which we usually don't. Why that?
My impression was that a one-letter code didn't stand out enough and
looked rather odd; if you think it improves
On Mon, Aug 20, 2012 at 11:33 PM, Jan Hubicka hubi...@ucw.cz wrote:
This is useful for large applications with a long tail. The
instruction working set for those applications are very large, and
inliner and unroller need to be aware of that and good heuristics can
be developed to throttle
Teresa has done some tunings for the unroller so far. The inliner
tuning is the next step.
What concerns me that it is greatly inaccurate - you have no idea how many
instructions given counter is guarding and it can differ quite a lot. Also
inlining/optimization makes working sets
On Fri, Aug 17, 2012 at 9:13 AM, Ian Lance Taylor i...@google.com wrote:
Looks fine to me.
Ian
Will backport to arm/embedded-4_7-branch. No sure if appropriate for
4.7 branch since it is not a stability problem.
- Joey
Solaris 9/x86 bootstrap was broken after the cxx-conversion merge:
In file included from /vol/gcc/src/hg/trunk/local/gcc/gengtype.c:957:
/vol/gcc/src/hg/trunk/local/gcc/rtl.def:347: error: expected identifier before n
umeric constant
/vol/gcc/src/hg/trunk/local/gcc/rtl.def:347: error: expected
On Tue, Aug 21, 2012 at 10:53 AM, Rainer Orth
r...@cebitec.uni-bielefeld.de wrote:
Solaris 9/x86 bootstrap was broken after the cxx-conversion merge:
In file included from /vol/gcc/src/hg/trunk/local/gcc/gengtype.c:957:
/vol/gcc/src/hg/trunk/local/gcc/rtl.def:347: error: expected identifier
Hello,
This adds support for SH2A's movu.w insn for memory addressing cases as
described in the PR.
Tested on rev 190546 with
make -k check RUNTESTFLAGS=--target_board=sh-sim
\{-m2/-ml,-m2/-mb,-m2a/-mb,-m4/-ml,-m4/-mb,-m4a/-ml,-m4a/-mb}
and no new failures.
OK?
Cheers,
Oleg
ChangeLog:
On Mon, 20 Aug 2012, Richard Guenther wrote:
This shrinks TREE_VEC from 40 bytes to 32 bytes and SSA_NAME from
80 bytes to 72 bytes on a 64bit host. Both structures suffer
from the fact they need storage for an integer (length and version)
which leaves unused padding. Both data structures
On 21 August 2012 10:58, Richard Guenther rguent...@suse.de wrote:
Index: trunk/gcc/tree.h
===
*** trunk.orig/gcc/tree.h 2012-08-20 12:47:47.0 +0200
--- trunk/gcc/tree.h2012-08-21 10:32:47.717394657 +0200
On Sat, Aug 18, 2012 at 9:56 AM, Richard Guenther
richard.guent...@gmail.com wrote:
On Sat, Aug 18, 2012 at 6:17 AM, Andrew Pinski pins...@gmail.com wrote:
Hi,
I implemented this patch almost 6 years ago when the df branch was
being worked on. It adds valgrind support to alloc-pool.c to
I don't think there are any callers out there, but let's fix this for
completeness.
A compiler emitting code to call this function would still have to
perform overflow checks for the new T[n][m] case, so this interface is
not as helpful as it looks at first glance.
Tested on
Hello,
This mainly converts the asm outputs to multi-line strings and uses tab
chars instead of '\\t' in the asm strings, in the hope to make stuff
easier to read and a bit more consistent.
Tested on rev 190546 with
make -k check RUNTESTFLAGS=--target_board=sh-sim
This fixes a few more heap leaks.
Bootstrapped on x86_64-unknown-linux-gnu, testing in progress.
Richard.
2012-08-21 Richard Guenther rguent...@suse.de
* tree-ssa-loop-im.c (tree_ssa_lim_finalize): Properly free
the affine expansion cache.
* tree-ssa-dom.c
Testing in progress.
Richard.
2012-08-21 Richard Guenther rguent...@suse.de
* alloc-pool.c (pool_alloc): Fix valgrind annotation.
* tree.h: Complete flags documentation.
(CLEANUP_EH_ONLY): Check documented allowed tree codes.
Index: gcc/alloc-pool.c
On Tue, 21 Aug 2012, Richard Guenther wrote:
Testing in progress.
Richard.
2012-08-21 Richard Guenther rguent...@suse.de
* alloc-pool.c (pool_alloc): Fix valgrind annotation.
* tree.h: Complete flags documentation.
(CLEANUP_EH_ONLY): Check documented allowed tree
On Wed, Aug 15, 2012 at 05:21:04PM +0200, Martin Jambor wrote:
Hi,
On Fri, Aug 10, 2012 at 04:57:41PM +0200, Eric Botcazou wrote:
- ada/gcc-interface/utils.c:rest_of_subprog_body_compilation calls
dump_function which in turns calls dump_function_to_file which calls
push_cfun. But
On Tue, Aug 21, 2012 at 1:27 PM, Martin Jambor mjam...@suse.cz wrote:
On Wed, Aug 15, 2012 at 05:21:04PM +0200, Martin Jambor wrote:
Hi,
On Fri, Aug 10, 2012 at 04:57:41PM +0200, Eric Botcazou wrote:
- ada/gcc-interface/utils.c:rest_of_subprog_body_compilation calls
dump_function which
Just as the title says: gcc.dg/fixed-point/convert.c is much too big to run on
embedded targets like AVR.
Note that embedded systems are a main audience of ISO/IEC TR 18037,
and that these systems might have limited resources.
The original convert.c inflates to thousands of functions and set
On 20 August 2012 16:45, Joseph S. Myers jos...@codesourcery.com wrote:
On Mon, 20 Aug 2012, Simon Baldwin wrote:
OPT_* for Fortran options only exist when the Fortran front-end is in the
source tree (whether or not enabled). I think we try to avoid knowingly
breaking use cases where
Now that I have had a chance to talk to Richard, I have now done
everything that he requested in his email.
Here is the new patch and changelog. Everything was tested on x86-64.
2012-08-21 Kenneth Zadeck zad...@naturalbridge.com
* alias.c (rtx_equal_for_memref_p): Convert constant
On 2012-08-20 22:41 , Sriraman Tallam wrote:
Hi Gerald / Diego,
I have made all the mentioned changes. I also shortened the
description like Diego mentioned by removing all the strings but kept
the caveats. I have not added a reference to the documentation because
i do not know what link
On Mon, Aug 20, 2012 at 3:18 AM, Dehao Chen de...@google.com wrote:
ping
Conceptually I like the change. Can a libcpp maintainer please have a 2nd
look?
Dehao, did you do any compile-time and memory-usage benchmarks?
Thanks,
Richard.
Thanks,
Dehao
On Tue, Aug 14, 2012 at 10:13 AM,
This backports the obvious heap leak fixes that have accumulated sofar.
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied.
Richard.
2012-08-21 Richard Guenther rguent...@suse.de
Backport from mainline
2012-08-16 Richard Guenther rguent...@suse.de
PR
The issue here is holding lock for all the files (that can be many) versus
number of locks limits possibilities for deadlocking (mind that updating
may happen in different orders on the same files for different programs built
from same objects)
lockf typically has a deadlock detector, and
Hi,
Thanks for the feedback. I respond here to the remaining issues:
Index: gcc/doc/extend.texi
===
--- gcc/doc/extend.texi (revision 187870)
+++ gcc/doc/extend.texi (working copy)
@@ -935,7 +935,8 @@
Not all
Hi,
long double may not be 80-bit on i386. We can't use
REAL_VALUE_TO_TARGET_LONG_DOUBLE for XFmode. This patch replaces
REAL_VALUE_TO_TARGET_LONG_DOUBLE with real_to_target. OK to install?
Thanks.
H.J.
---
2012-08-21 H.J. Lu hongjiu...@intel.com
PR target/54347
*
-Original Message-
From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-
ow...@gcc.gnu.org] On Behalf Of Joseph S. Myers
Sent: 25 May 2012 15:24
To: Marcus Shawcroft
Cc: gcc-patches@gcc.gnu.org
Subject: Re: [AARCH64] [PATCH 2/3] AArch64 Port
On Fri, 25 May 2012, Marcus
On Tue, 21 Aug 2012, Simon Baldwin wrote:
Index: gcc/doc/options.texi
===
--- gcc/doc/options.texi (revision 190535)
+++ gcc/doc/options.texi (working copy)
@@ -468,4 +468,8 @@ of @option{-@var{opt}}, if not explicitl
Hi Jing, the crosstool test passed. You can start the review, thanks! -Han
On Wed, Aug 15, 2012 at 3:11 PM, Han Shen(沈涵) shen...@google.com wrote:
Hi Jing, ping?
On Mon, Aug 13, 2012 at 10:58 AM, Han Shen(沈涵) shen...@google.com wrote:
Hi, the google/gcc-4_7 fails to linking anything (on
I've merged gcc 4.7 branch revision 190560 to the gccgo branch.
Ian
2012/8/13 Georg-Johann Lay a...@gjlay.de:
Denis Chertykov wrote:
2012/8/11 Georg-Johann Lay a...@gjlay.de:
Weddington, Eric schrieb:
From: Georg-Johann Lay
The first step would be to bisect and find the patch that lead to
PR53923. It was not a change in the avr BE, so the question goes
Kenneth Zadeck zad...@naturalbridge.com writes:
I plan to commit this in a few days unless someone has some comments.
This is a mostly trivial patch and the changes from that are Richard
Sandiford's and he is an rtl maintainer.
Please don't do this. Patches need to be sent for review in
Richard Sandiford rdsandif...@googlemail.com writes:
switch (code)
{
-case CONST_INT:
-case CONST_DOUBLE:
-case CONST_FIXED:
+CASE_CONST_UNIQUE:
case SYMBOL_REF:
case CONST:
case LABEL_REF:
This was suppsoed to be CASE_CONST_ANY too. The omission
it would have been tough without the second snippit
On 08/21/2012 01:02 PM, Richard Sandiford wrote:
Richard Sandiford rdsandif...@googlemail.com writes:
switch (code)
{
-case CONST_INT:
-case CONST_DOUBLE:
-case CONST_FIXED:
+CASE_CONST_UNIQUE:
case SYMBOL_REF:
On Aug 21, 2012, at 4:32 AM, Georg-Johann Lay wrote:
The patch breaks up convert.c in parts so that an AVR ATmega103 device
with 128KiB for executable code (.text + .data + .rodata) can run them.
Ok for trunk?
Ok, but watch out for any comments from the fixed-point or the C front-end
folks.
I am certainly not going to check it in if there are any issues with the
patch. However, this was basically a trivial lexicographical cleanup,
and if no one has any comments on it after a reasonable amount of time,
then i do feel this is ok.Obviously if anyone has any comments. that
is
On Tue, Aug 21, 2012 at 12:34 AM, Jan Hubicka hubi...@ucw.cz wrote:
Teresa has done some tunings for the unroller so far. The inliner
tuning is the next step.
What concerns me that it is greatly inaccurate - you have no idea how many
instructions given counter is guarding and it can
Committed after making the changes.
One small problem, I am not sure how to fix this:
The hyper link I referenced is :
http://gcc.gnu.org/onlinedocs/gcc/X86-Built_002din-Functions.html#X86-Built_002din-Functions
whereas the committed changes.html is pointing to:
On Wed, Feb 8, 2012 at 1:23 AM, Jason Merrill ja...@redhat.com wrote:
More traffic on PR 51675 demonstrates that my earlier patch didn't fix the
whole problem. This patch improves handling of user-defined constructors.
Tested x86_64-pc-linux-gnu, applying to trunk.
This caused:
The following patch mostly removes -flra option by defining a
machine-dependent hook lra_p. If the hook returns true, LRA is used.
Otherwise, reload pass is used. By default the hook returns false. It
returns true for 8 targets, lra was ported (i386, rs6000, arm, s390,
ia64, sparc, mips,
Hello,
Two more bitmap obstacks, this time in tree-ssa-coalesce.c.
The advantage isn't so much in having the bitmaps on the non-default
obstack, but more in that the bitmaps can be free'ed all at once by
simply releasing the obstack.
Bootstrappedtested on x86_64-unknown-linux-gnu. OK for trunk?
Kenneth Zadeck zad...@naturalbridge.com writes:
I named it this way CASE_CONST_SCALAR_INTEGER because i need to
introduce in the next patch a predicate that looks like
/* Predicate yielding true iff X is an rtx for a integer const. */
#if TARGET_SUPPORTS_WIDE_INT == 1
#define
This patch is a followup to the addition of support for
-fstrict-volatile-bitfields (required by the ARM EABI); see this thread
http://gcc.gnu.org/ml/gcc-patches/2010-10/msg01889.html
for discussion of the original patch.
That patch only addressed the behavior when extracting the value of a
Sandra Loosemore san...@codesourcery.com writes:
In config/mips/mips.h, there is presently this comment:
/* ??? 16-bit offsets can overflow in large functions. */
#define TARGET_MIPS16_SHORT_JUMP_TABLES TARGET_MIPS16_TEXT_LOADS
A while ago we had a bug report where a big switch statement
Hi!
The recent change in find_assert_locations from XCNEWVEC to XNEWVEC
caused a valgrind warning, because bb_rpo[ENTRY_BLOCK] used to
be accessed, but was never initialized.
Fixed by ignoring edges from ENTRY_BLOCK altogether.
The rest are a couple of memory leak fixes.
Bootstrapped/regtested
When a -fsplit-stack function calls a non-split-stack function, the gold
linker automatically redirects the call to __morestack to call
__morestack_non_split instead. I wrote __morestack_non_split to always
allocate at least 0x4000 bytes. However, that was unclear thinking;
0x4000 bytes is
Hi,
This patch restores df_free_collection_rec call inside the insn traversal
loop and removes the stack allocation check in vec_reserve. It has
been approved in
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54332#c25
It has been tested on Linux/x86-64 and checked in.
Thanks.
H.J.
---
I merged revision 190574 from the gcc 4.7 branch to the gccgo branch.
Ian
On Aug 11, 2012, at 10:39 AM, Senthil Kumar Selvaraj wrote:
This patch allows cflags set in board config files using
set_board_info cflags to be used in the selectors of
dg-skip-if and other dejagnu commands that use the check-flags
proc.
Ok.
Background: There is currently a memory leakage cleanup in the middle
end – and fixing PR 54332 would probably have been also easier without
FE leaks.
I think we should join in an try to remove some leakage - and try to not
introduce new ones.
* * *
Committed: For EXPR_END_PROCEDURE, I
On 17 August 2012 07:29, Julian Brown jul...@codesourcery.com wrote:
On Thu, 16 Aug 2012 19:56:52 +0100
Ramana Radhakrishnan ramra...@arm.com wrote:
On 07/24/12 13:27, Julian Brown wrote:
On Fri, 20 Jul 2012 11:15:27 +0100
Julian Brown jul...@codesourcery.com wrote:
Anyway: this
Oleg Endo oleg.e...@t-online.de wrote:
This mainly converts the asm outputs to multi-line strings and uses tab
chars instead of '\\t' in the asm strings, in the hope to make stuff
easier to read and a bit more consistent.
Tested on rev 190546 with
make -k check
Oleg Endo oleg.e...@t-online.de wrote:
This adds support for SH2A's movu.w insn for memory addressing cases as
described in the PR.
Tested on rev 190546 with
make -k check RUNTESTFLAGS=--target_board=sh-sim
\{-m2/-ml,-m2/-mb,-m2a/-mb,-m4/-ml,-m4/-mb,-m4a/-ml,-m4a/-mb}
and no new failures.
On Tue, Aug 21, 2012 at 6:25 AM, Richard Guenther
richard.guent...@gmail.com wrote:
On Mon, Aug 20, 2012 at 3:18 AM, Dehao Chen de...@google.com wrote:
ping
Conceptually I like the change. Can a libcpp maintainer please have a 2nd
look?
Dehao, did you do any compile-time and
As discussed in
http://gcc.gnu.org/ml/gcc-patches/2012-08/msg01219.html, it is
desirable for the libgcc build with inhibit_libc defined and
--disable-shared to be similar enough to that build without
inhibit_libc and --enable-shared to be usable to build glibc,
producing the same results as if
On Tue, Aug 21, 2012 at 5:33 PM, Joseph S. Myers
jos...@codesourcery.com wrote:
2012-08-21 Joseph Myers jos...@codesourcery.com
* Makefile.in (vis_hide, gen-hide-list): Do not make definitions
depend on --enable-shared.
($(lib1asmfuncs-o)): Use %.vis files
Hi,
Due to the impact of ARM UAL, the Thumb1 and Thumb2 mode use LSRS
instruction while the ARM mode uses MOVS instruction. So the following case
is updated accordingly. Is it OK to trunk?
BR,
Terry
2012-08-21 Terry Guo terry@arm.com
* gcc.target/arm/combine-movs.c: Check movs
I can go ahead with the histogram approach. There is some roundoff
error from the working set scaling approach that can affect different
merging orders as you note, although I think this only really affects the
small counter values. The other place where saving/merging the histogram
Do
On Jul 19, 2012, at 11:24 AM, Richard Henderson wrote:
+# genopinit produces two files.
+insn-opinit.c insn-opinit.h: s-opinit ; @true
+s-opinit: $(MD_DEPS) build/genopinit$(build_exeext) insn-conditions.md
+ $(RUN_GEN) build/genopinit$(build_exeext) $(md_file) \
+
On Tue, Aug 21, 2012 at 6:56 PM, Jan Hubicka hubi...@ucw.cz wrote:
I can go ahead with the histogram approach. There is some roundoff
error from the working set scaling approach that can affect different
merging orders as you note, although I think this only really affects the
small
Committed as Rev. 190586 after successful regtesting.
That's the version I also had attached to
http://gcc.gnu.org/ml/fortran/2012-08/msg00118.html; as written there:
The patch is incomplete, e.g. argss of gfc_conv_procedure_call is not
(or not always) freed. Ditto for rss of
66 matches
Mail list logo