Andi Kleen a...@firstfloor.org writes:
From: Andi Kleen a...@linux.intel.com
Use the Linux MADV_DONTNEED call to unmap free pages in the garbage
collector.Then keep the unmapped pages in the free list. This avoid
excessive memory fragmentation on large LTO bulds, which can lead
to gcc
On 12 October 2011 17:54, Jakub Jelinek ja...@redhat.com wrote:
Hi!
Hi,
This patch allows vectorization of some loops that use
bool (which is especially important now that we use bool more often
even for stmts that weren't originally using bool in the sources),
in particular (when bool is
gcc-patches-ow...@gcc.gnu.org wrote on 14/10/2011 04:43:48 PM:
All,
The attached patch corrects the expected output of the
gcc.dg/vect/pr30858.c testcase.
Historically it has expected the output Unknown def-use cycle pattern.
just once.
However, recent changes to GCC for ARM targets
Hi Steve,
resolve_ref() calls resolve_substring() but failed to check
its return value. This patch does the check.
Looks pretty obvious. Please commit.
Thanks for the patch,
Janus
2011-10-15 Steven G. Kargl ka...@gcc.gcu.org
* resolve.c (resolve_ref): Check return value of
Hi,
this patch makes cgrpahunit to output all thunks and aliases after the function,
not before. Outputting before leads to better code layout and we might omit
tailjump when thunk is just one, but it breaks one pass assembler and comdat
group symbols.
This is the easiest fix and I will play
On 10/14/2011 12:00 PM, Richard Guenther wrote:
On Fri, Oct 14, 2011 at 1:12 AM, Tom de Vries tom_devr...@mentor.com wrote:
On 10/12/2011 02:19 PM, Richard Guenther wrote:
On Wed, Oct 12, 2011 at 8:35 AM, Tom de Vries vr...@codesourcery.com
wrote:
Richard,
I have a patch for PR50672.
On Sun, Oct 16, 2011 at 5:03 AM, Paolo Carlini paolo.carl...@oracle.com wrote:
Hi,
in this simple documentation PR, Tom noticed that we have a (very long
standing) inconsistency between the default value of -fmessage-length for
C++ as documented and as implemented: in fact it's 0 in
On Sun, Oct 16, 2011 at 1:12 AM, Andi Kleen a...@firstfloor.org wrote:
From: Andi Kleen a...@linux.intel.com
Some external tools special case __gnu_lto* symbols, and the new
__gnu_slim_lto
was the only LTO symbol not matching this pattern. Since I don't think
there are any users rename it
On 10/16/2011 12:28 PM, Gabriel Dos Reis wrote:
On Sun, Oct 16, 2011 at 5:03 AM, Paolo Carlinipaolo.carl...@oracle.com wrote:
Hi,
in this simple documentation PR, Tom noticed that we have a (very long
standing) inconsistency between the default value of -fmessage-length for
C++ as documented
On Sun, Oct 16, 2011 at 7:30 AM, Andi Kleen a...@firstfloor.org wrote:
Andi Kleen a...@firstfloor.org writes:
From: Andi Kleen a...@linux.intel.com
Use the Linux MADV_DONTNEED call to unmap free pages in the garbage
collector.Then keep the unmapped pages in the free list. This avoid
Hi,
This patch allows to vectorize a subchain of interleaved loads in
basic block SLP (in loop vectorization this would be more complicated
because of loop peeling). This patch also swaps operands if necessary
(and possible) to make operations isomorphic.
Bootstrapped and tested on
On Sun, Oct 16, 2011 at 5:42 AM, Richard Guenther
richard.guent...@gmail.com wrote:
On Sun, Oct 16, 2011 at 12:31 PM, Paolo Carlini
paolo.carl...@oracle.com wrote:
On 10/16/2011 12:28 PM, Gabriel Dos Reis wrote:
On Sun, Oct 16, 2011 at 5:03 AM, Paolo Carlinipaolo.carl...@oracle.com
wrote:
This fixes a bootstrap failure on trunk with objc enabled, caused by a
recent change which introduced for-loops that compare a size_t index
variable with a TREE_VEC_LENGTH limit. TREE_VEC_LENGTH returns a signed
integer, resulting in a signed/unsigned comparison. The C++ front-end
fails to catch
Reorder chunks in link.c so that the correct configuration is selected when
compiling with --with-gnu-ld on AIX.
No change in behaviour.
Note that in particular it's important that we use native AIX switches even
when using GNU ld. This is important with response files as they must be
The declaration of the Implementation_Defined pragma was moved from the outer
scope to the scope of the nested package Implementation.
Tested on x86_64-pc-linux-gnu, committed on trunk
2011-10-16 Matthew Heaney hea...@adacore.com
* a-cusyqu.ads, a-cbsyqu.ads, a-cuprqu.ads,
Hi all,
here is a patch which fixes the regression in comment #2 of the PR in
the subject line. What it does is setting the 'ts.is_c_interop' flag
correctly for constants with kind-parameter specification (such as
'0.0_c_double'), as is already being done for variables.
Regtested on
On Sun, Oct 16, 2011 at 09:56:57AM +0200, Ira Rosen wrote:
by moving?
Yeah.
Could you please add some more explanations here? I found it very
difficult to follow. It would be nice to have an example here (similar
to vect_recog_bool_pattern) to illustrate what these statements and
operands
-moptimize-noreturn-calls
Optimize noreturn calls. This might make debugging harder but
will save storing the return address when calling roreturn
functions.
Enabled at levels -O2, -O3, -Os.
But the makes debugging harder clause is true for almost any
Paolo Bonzini schrieb:
-moptimize-noreturn-calls
Optimize noreturn calls. This might make debugging harder but
will save storing the return address when calling roreturn
functions.
Enabled at levels -O2, -O3, -Os.
But the makes debugging harder clause is true for
Hi,
This patch fixes another occurrence of the same bug as in PR 50635.
Bootstrapped and tested on powerpc64-suse-linux.
Committed.
Ira
ChangeLog:
PR tree-optimization/50727
* tree-vect-patterns.c (vect_operation_fits_smaller_type): Add
DEF_STMT to the list of
Hi!
gimplify_parameters uses create_tmp_reg, but sometimes it decides to make
it addressable (if the PARM_DECL is addressable). If so, it must not be
DECL_GIMPLE_REG_P.
Alternatively we could call create_tmp_reg only if !TREE_ADDRESSABLE and
call create_tmp_var instead for TREE_ADDRESSABLE (+
Looking at the new gcc-4.6/porting_to.html last week, I noticed that
in the original version there as well as the older instance for GCC 4.4
we would skip h2 headers and directly move from h1 to h3.
I addressed this when committing the GCC 4.6 flavor, and the patch
below which I committed does
Dear Janus,
This is OK for trunk. Thanks fo rthe patch.
Cheers
Paul
On Sun, Oct 16, 2011 at 2:58 PM, Janus Weil ja...@gcc.gnu.org wrote:
Hi all,
here is a patch which fixes the regression in comment #2 of the PR in
the subject line. What it does is setting the 'ts.is_c_interop' flag
On Sun, Oct 16, 2011 at 12:38:16PM +0200, Richard Guenther wrote:
On Sun, Oct 16, 2011 at 7:30 AM, Andi Kleen a...@firstfloor.org wrote:
Andi Kleen a...@firstfloor.org writes:
From: Andi Kleen a...@linux.intel.com
Use the Linux MADV_DONTNEED call to unmap free pages in the garbage
On Sun, Oct 16, 2011 at 8:09 AM, Uros Bizjak ubiz...@gmail.com wrote:
As explained in length in the PR [1], we fail to mark signal frames
correctly, leading to the abort in EH support library.
As suggested by Eric, attached patch marks fs-signal_frame in the
same way as other dwarf2 targets.
On Wed, Sep 28, 2011 at 11:47 AM, Alan Modra amo...@gmail.com wrote:
* config/rs6000/rs6000.c (rs6000_make_savres_rtx): Delete unneeded
declaration.
(rs6000_emit_stack_reset): Only return insn emitted when it adjusts sp.
(rs6000_make_savres_rtx): Rename to
Hi,
in this diagnostic PR the issue it that for:
struct Base{ };
struct Concrete : Base
{
void setValue();
};
int main()
{
Concrete d;
d.Base::setValue();
}
the error message is:
48489.C:11:11: error: ‘struct Derived’ has no member named ‘setValue’
instead of:
48489.C:11:11: error:
while working on PR50547, I noticed some strange things about
resolve_formal_arglist, so I decided to clean it up a little. The
attached patch does a couple of things:
Regtested on x86_64-unknown-linux-gnu. Ok for trunk?
OK. Thanks for the cleanup.
Thanks. Committed as r180061. (I almost
Hello world,
working on PR 50690, I noticed a few things which are
not quite right in front-end optimization statement
walking:
- BLOCKs were walked via their namespaces, not directly.
This caused out-of-order walking, which was confusing.
- current_ns was not marked static.
- ASSOCIATE lists
This patch corrects these things. Regression-tested.
OK for trunk? (I will not be able to commit for a few
days due to business travel, unless somebody is _really_
fast :-)
Looks OK to me. Fast enough for you?
Ciao!
Steven
[...]
Does the attached testcase checked what you mention ?
... with the testcase attached.
--
Fabien
// { dg-do run }
template class T
struct A
{
int f() { return 1; }
};
template class T
struct B : AT
{
int f() { return 2; }
using AT::f;
void g()
{
if (AT::f() != 1 )
On 10/16/2011 09:12 PM, Paolo Carlini wrote:
48489.C:11:11: error: ‘struct Derived’ has no member named ‘setValue’
of course there is a typo here, should read
48489.C:11:11: error: ‘struct Concrete’ has no member named ‘setValue’
Paolo.
On Oct 16, 2011, at 4:28 AM, Mikael Pettersson wrote:
This fixes a bootstrap failure on trunk with objc enabled
Ok for trunk?
Ok.
Factor the vector and chain output routines to remove boolean control
parameters. The functions pph_out_tree_vec_1 and pph_out_chain_1 split
their conditional parts of their implementation into their use cases,
calling each other as needed.
pph_out_tree_vec - nothing special
OK.
Jason
35 matches
Mail list logo