Hi,
Checked in
Thanks, K
On Mon, Feb 18, 2013 at 10:28 AM, Igor Zamyatin izamya...@gmail.com wrote:
Gerald,
Thanks a lot for your remarks!
Below is updated patch which will be checked in.
Thanks,
Igor
On Mon, Feb 18, 2013 at 3:07 AM, Gerald Pfeifer ger...@pfeifer.com wrote:
On Fri,
OK (it is a tuning patch).
Hi,
Checked in: http://gcc.gnu.org/ml/gcc-cvs/2013-02/msg00540.html
Thanks, K
On Tue, Feb 19, 2013 at 4:10 PM, Michael Matz m...@suse.de wrote:
Hi,
from IRC:
[15:45:21] richi ick - lookup_constraint for multi-letter constraints
is quite expensive ... strncmp is not expanded inline for some reason
Instead of fiddling with strncmp inlining, simply generate better code
On Tue, 19 Feb 2013, Jakub Jelinek wrote:
Hi!
On the following patch gcc ICEs because malloc memory is corrupted.
The problem is that const_val array is allocated at the start of the pass,
but during the execution of ccp some new SSA_NAMEs are created
(update_call_from_tree if I remember
Gerald Pfeifer wrote:
On Mon, 18 Feb 2013, Tobias Burnus wrote:
How about the following patch? If it is okay, I will add the remaining
index.html for 4.6 and 4.7 and update gcc-4.6/index.html – and then
commit it.
Looks good to me.
Perhaps you can add a note for the release manager how to
On Wed, Feb 20, 2013 at 10:52:52AM +0100, Richard Biener wrote:
*** get_value (tree var)
*** 295,301
{
prop_value_t *val;
! if (const_val == NULL)
return NULL;
val = const_val[SSA_NAME_VERSION (var)];
--- 296,303
{
prop_value_t *val;
Hunting for the we're getting slower bits I noticed that
TODO_remove_unused_locals is a big part of execute_function_todo
(and accounts for 1% of compile-time of ac.f90).
The following patch removes most of the remove_unused_locals
calls based on the fact that with anonymous SSA names now
On 19/02/13 22:26, Tilman Sauerbeck wrote:
I don't get why relaxing the restrictions for the
andsi3_compare0_scratch pattern results in a mismatch for the
zeroextractsi_compare0_scratch one.
Any ideas?
Because of the way combine works. It first tries to find a pattern that
doesn't have a
Am 18.02.2013 18:49, schrieb John David Anglin:
This patch removes the broken powf hack. This problem is now fixed in
the PA backend.
Tested on hppa2.0w-hp-hpux11.11 and hppa64-hp-hpux11.11.
OK for trunk?
OK. Thanks for the patch and for fixing the problem more properly.*
Tobias
Hi All,
This patch is aimed to recognize (A C) ^ (B C) - (A ^ B) C
pattern in simpify_bitwise_binary for short integer types.
The fix is very simple - we simply turn off short type sinking at the
first pass of forward propagation allows to get
+10% speedup for important benchmark Coremark 1.0
On Wed, 20 Feb 2013, Richard Biener wrote:
Hunting for the we're getting slower bits I noticed that
TODO_remove_unused_locals is a big part of execute_function_todo
(and accounts for 1% of compile-time of ac.f90).
The following patch removes most of the remove_unused_locals
calls based on
On Wed, Feb 20, 2013 at 1:00 PM, Yuri Rumyantsev ysrum...@gmail.com wrote:
Hi All,
This patch is aimed to recognize (A C) ^ (B C) - (A ^ B) C
pattern in simpify_bitwise_binary for short integer types.
The fix is very simple - we simply turn off short type sinking at the
first pass of
This fixes an ICE because gimple_bb of a default def stmt is NULL.
Just don't do anything here, we're not going to adjust anything
anyway for them.
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied.
Richard.
2013-02-20 Richard Biener rguent...@suse.de
PR
Hi,
when debugging a PR I noticed that dumped numbers do not correspond to
changed PARAM_IPA_CP_EVAL_THRESHOLD and defaults are hard-wired to the
fprintf. Fixed by the patch below, which bootstraps and tests fine on
x86_64-linux. Unless there are objections, I will commit it tomorrow
as
Hi,
this patch fixes an omission in IPA-CP's agg_replacements_to_vector
which needs to filter the vector by index and offset and a typo in
intersect_aggregates_with_edge which in one call passed the wrong
index to agg_replacements_to_vector. This combined lead to empty
intersections which were
In the following test, the first statement of a relaxed transaction is
an inline asm:
__transaction_relaxed { __asm__(); }
Since we bypass inserting BUILT_IN_TM_IRREVOCABLE at the beginning of
transactions that are sure to be irrevocable, later when we try to
expand the transaction, we ICE
Richard,
First of all, your proposal to move type sinking to the end of
function does not work since we handle each statement in function and
we want that 1st type folding of X C will not happen.
Note that we have the following sequence of gimple before forwprop1:
x.0_10 = (signed char) x_8;
Hi,
in the testcase bellow we get an ICE in ipa_make_edge_direct_to_target. There
is virtual call that gets devirtualized only while inlining functions called
once. At this point however we already removed bodies for virtual functions
from the callgraph, so we need to update it and re-create its
This patch from Janne Snabb should fix PR 56320 about random failures on
Solaris x86_64. Bootstrapped and ran Go testsuite on
x86_64-unknown-linux-gnu. Committed to mainline.
Ian
diff -r 29b742a2ed37 -r 3b1c3cceaf02 libgo/runtime/lfstack.c
--- a/libgo/runtime/lfstack.c Fri Feb 15 10:54:51 2013
This patch, mainly from Rainer Orth, in PR 56171 fixes passing a file
descriptor on Solaris. Bootstrapped and ran Go testsuite on
x86_64-unknown-linux-gnu. Committed to mainline.
Ian
diff -r 3b1c3cceaf02 libgo/Makefile.am
--- a/libgo/Makefile.am Wed Feb 20 11:41:05 2013 -0800
+++
Hi,
this patch fixes an omission in IPA-CP's agg_replacements_to_vector
which needs to filter the vector by index and offset and a typo in
intersect_aggregates_with_edge which in one call passed the wrong
index to agg_replacements_to_vector. This combined lead to empty
intersections which
* testsuite/23_containers/unordered_set/55043.cc: Add missing
namespace qualification.
* testsuite/23_containers/unordered_multiset/55043.cc: Likewise.
Tested x86_64-linux, committed to trunk.
commit a6d9aa71b453ad2cf7b1cbd581fa04603728bad6
Author: Jonathan Wakely
Hi all,
here is a straightforward patch which fixes a regression with
procedure-pointer components which have an allocatable result.
Regtests cleanly on x86_64-unknown-linux-gnu. Ok for trunk/4.7/4.6?
[In absence of any reviews I will commit as obvious on the weekend.]
Cheers,
Janus
* doc/html/faq.html: Fix spelling.
* doc/xml/faq.xml: Likewise.
* include/bits/basic_ios.h: Likewise.
* include/bits/regex.h: Likewise.
* include/std/istream: Likewise.
* include/std/streambuf: Likewise.
Tested x86_64-linux, committed to trunk
This removes an unclosed @{ group marker so that Doxygen doesn't put
half the members of basic_streambuf in the Friends section (see
http://stackoverflow.com/q/14988997/981959)
I also changed uses of __streambuf_type to basic_streambuf, which
makes the Doxygen page look better because it doesn't
On Fri, Feb 15, 2013 at 3:29 PM, Mike Stump mikest...@comcast.net wrote:
No. Counter proposal, let's handle the cases that don't work. So, you said
in your original email that armeb-unknown-eabi doesn't work.
So, in the existing case statement for:
arm*-*-eabi*)
let's just add:
On Sat, Feb 16, 2013 at 7:45 AM, Bernhard Reutner-Fischer
rep.dot@gmail.com wrote:
Sounds like a DUP of http://gcc.gnu.org/bugzilla/show_bug.cgi?id=16350
Is the missing hunk in by now (cannot look myself right now)?
The commit you describe appears to be in trunk but not in gcc-4_7-branch.
On Tue, Feb 19, 2013 at 2:59 PM, Ramana Radhakrishnan
ramana@googlemail.com wrote:
This is not the correct form of a changelog entry.
Sorry - another attempt below.
Ok to backport provided no regressions when running the testsuite in
big endian mode.
Sorry to say I'm not sure how to run
Hi,
This issue was originally raised here:
http://gcc.gnu.org/ml/gcc-patches/2012-12/msg00863.html
We have a shortcoming in GCC in that we only allow the use half of the FP
MADD instruction subset (MADD.fmt and MSUB.fmt) in the 64-bit/32-register
mode (CP0.Status.FR == 1) on MIPS32r2
Hello,
The attached patch splits a new function df_insn_info_delete from
df_insn_delete. The original motivation was to get rid of the silly
deleting insn with uid = ... messages when re-scanning an insn,
because the mentioned insn isn't deleted at all (it's just rescanned).
But it turns out that
On 02/19/2013 02:40 PM, Janne Blomqvist wrote:
Hi,
attached is an attempt to fix writing formatted sequential I/O to a
pipe (The PR was reopened in comment #22, which refers to formatted
I/O so the PR title is incorrect). I think the underlying reason was
that the introduction of the ssize()
On Mon, 28 Jan 2013, nick clifton wrote:
Also, could you close its duplicates, bugs 36798 and 36966?
Sorry no. I do not actually own these PRs, so I cannot close them. :-(
Sorry if I misinterpret, but it seems a reminder is in order:
magic powers are attached to whome...@gcc.gnu.org accounts
Hi!
If an input operand of inline-asm doesn't allow registers, but allows
memory, we expand it with EXPAND_MEMORY modifier (the only case of using
that modifier). But the movmisalign code added for 4.6 and especially
the extract_bit_field code added for 4.8 results in getting a REG from
Hi!
Currently it is not possible to bootstrap gcc with texinfo 5.0.
This patch attempts to fix the errors that prevent bootstrap, there are tons
of warnings this doesn't address and would be good if somebody more TeXinfo
knowledgeable looked at it.
Bootstrapped/regtested on x86_64-linux and
34 matches
Mail list logo