A while ago I've tried doing that for SH (implementing bswap16 with RTL
combine). It was like an explosion of patterns, because combine would
try out a lot of things depending on the surrounding code around the
actual bswap16. In the end I decided to drop that stuff for the most
part.
OK,
On Mon, 2012-09-24 at 09:15 +0200, Eric Botcazou wrote:
A while ago I've tried doing that for SH (implementing bswap16 with RTL
combine). It was like an explosion of patterns, because combine would
try out a lot of things depending on the surrounding code around the
actual bswap16. In
On Fri, 21 Sep 2012, Aldy Hernandez wrote:
On 09/21/12 08:52, Richard Guenther wrote:
This bug happens because GIMPLE_TRANSACTION is a statement
without real but with virtual operands. The short-cut in gimple_copy
thus is overly optimistic. Fixed as follows.
Thank you so much for
On Sat, 22 Sep 2012, Jack Howarth wrote:
The attached patch eliminates the failures in linkage of the new
gcc.dg/torture/pr53922.c testcase on darwin by passing
-Wl,-undefined,dynamic_lookup
to the linker as recommended by the darwin linker developer...
On Sun, Sep 23, 2012 at 2:01 AM, Iyer, Balaji V balaji.v.i...@intel.com wrote:
Hello Everyone,
Attached, please find a patch that will implement Cilk Plus Array
Notations for the C compiler. Array notations are indented to allow
programmers to directly express parallelism in their
Hi,
although the type merging machinery deals with TYPE_NONALIASED_COMPONENT, the
flag is actually not streamed, which can result in strange performance drops
for Ada applications in LTO mode. The attached patch plugs this hole.
Tested on x86_64-suse-linux and LTO-bootstrapped. I also
On Mon, Sep 24, 2012 at 12:05 PM, Eric Botcazou ebotca...@adacore.com wrote:
Hi,
although the type merging machinery deals with TYPE_NONALIASED_COMPONENT, the
flag is actually not streamed, which can result in strange performance drops
for Ada applications in LTO mode. The attached patch
On Wednesday 05 September 2012 13:47:19 Steven Bosscher wrote:
On Wed, Sep 5, 2012 at 1:25 PM, Matthew Gretton-Dann wrote:
+ /* If the two blocks are in different partitions we do not want to mark
+ this as a fallthru edge. */
+ if (BB_PARTITION (b) != BB_PARTITION (c))
+
On Sun, Sep 23, 2012 at 10:46 AM, Eric Botcazou ebotca...@adacore.com wrote:
Hi,
we have run into optimization regressions in Ada caused by the interaction
between the new GIMPLE clobbers and SRA: on the one hand GIMPLE clobbers
create artificial EH handlers for aggregate variables, on the
On Sat, Sep 22, 2012 at 4:08 AM, Dehao Chen de...@google.com wrote:
Hi,
The problem is due to a bug when move_block_to_fn. edge-goto_block
should be updated even when its locus is unknown. This patch also
fixes the way to reset block for expr.
I think the machinery is much more fragile
On Sat, Sep 22, 2012 at 11:17 PM, Marc Glisse marc.gli...@inria.fr wrote:
Hello,
this patch lets forwprop fold bit_field_refs of constructors. It is
particularly useful in forwprop4, which happens after the pass that lowers
unsupported vector operations. Indeed, that pass keeps breaking
This fixes PR54632 - we can end up with garbage collected BLOCKs
(those not part of any BLOCK tree) being still referenced from
DECL_DEBUG_EXPRs. We try to handle that in clear_unused_block_pointer
but fail to walk all local decls as source of them (but only catch
those that are still actively
This fixes PR54684 where optimizing __builtin_unreachable () during
builtin folding fails to update SSA operands properly.
Bootstrapped on x86_64-unknown-linux-gnu, testing in progress.
Richard.
2012-09-24 Richard Guenther rguent...@suse.de
PR tree-optimization/54684
*
This fixes PR53663 - for redundant store removal we may not
use VN translation, but the native encode / interpret trick
is of such a kind because it looks through type changes.
Thus, mark it so.
Bootstrap and regtest queued on x86_64-unknown-linux-gnu.
Richard.
2012-09-24 Richard Guenther
Hi,
some time ago Jon filed this diagnostic enhancement PR: explaining it
requires a bit, please be patient, if you can ;)
Essentially, we are comparing a note emitted for
templatetypename T
struct A {
templatetypename U
void f() { }
};
int main() {
Avoid a;
a.f(0);
}
vs
I think dump_function_decl should strip TFF_TEMPLATE_NAME from flags
before passing flags along to other functions, like dump_template_parms
does.
Jason
OK.
Jason
Hi Richard,
Please see my comments embedded below.
-Original Message-
From: Richard Guenther [mailto:richard.guent...@gmail.com]
Sent: Monday, September 24, 2012 5:53 AM
To: Iyer, Balaji V
Cc: gcc-patches@gcc.gnu.org; al...@redhat.com; r...@redhat.com;
l...@redhat.com
Subject: Re:
On Tue, 18 Sep 2012, Richard Sandiford wrote:
Have you had time to think about this some more? I am not sure I can
guess how you'd like me to fix this patch now without some more specific
review and/or suggestions about where the optimization should happen and
what cases it should be
On 09/24/2012 04:53 PM, Jason Merrill wrote:
I think dump_function_decl should strip TFF_TEMPLATE_NAME from flags
before passing flags along to other functions, like
dump_template_parms does.
I see. The below also works on the note and passes the testsuite.
Note: I'm adding to the existing
OK.
Jason
1) parser.o was missing a dependency on decl.h.
2) The assignment to fn_ptr_type was dead.
Tested x86_64-pc-linux-gnu, applying to trunk.
commit 330ecdbe393e3bd068c14c6639a5ace9764d2191
Author: Jason Merrill ja...@redhat.com
Date: Thu Sep 20 09:52:42 2012 -0400
* Make-lang.in
Hi,
On Tue, Sep 18, 2012 at 03:35:45PM +0200, Jan Hubicka wrote:
Hi,
this patch reorganize lto-symtab to work across symtab's symbol table instead
of building its own. This simplifies things a bit and with the previous
changes it is rather straighforward - i.e. replace all uses of
On 09/21/12 20:32, Richard Henderson wrote:
On 09/21/2012 02:00 PM, Aldy Hernandez wrote:
+ /* ?? For TM_* builtin replacements, can we set this to FALSE??
+Otherwise, do we need to propagate the may_irr bit? */
+ node-local.tm_may_enter_irr = true;
Yes we can.
On Mon, Sep 24, 2012 at 8:50 AM, Martin Jambor mjam...@suse.cz wrote:
Hi,
On Tue, Sep 18, 2012 at 03:35:45PM +0200, Jan Hubicka wrote:
Hi,
this patch reorganize lto-symtab to work across symtab's symbol table instead
of building its own. This simplifies things a bit and with the previous
Hi All!
After recent commit
(http://gcc.gnu.org/ml/gcc-cvs/2012-08/msg00576.html, patch -
http://gcc.gnu.org/ml/gcc-patches/2012-08/msg01512.html), which sets
size of long double equal to double for Bionic, several tests required
some adjustment to pass on Android.
Tested both for Linux and
On Sep 24, 2012, at 9:56 AM, Igor Zamyatin izamya...@gmail.com wrote:
After recent commit
(http://gcc.gnu.org/ml/gcc-cvs/2012-08/msg00576.html, patch -
http://gcc.gnu.org/ml/gcc-patches/2012-08/msg01512.html), which sets
size of long double equal to double for Bionic, several tests required
Hello, gentle maintainer.
This is a message from the Translation Project robot. (If you have
any questions, send them to coordina...@translationproject.org.)
A new POT file for textual domain 'cpplib' has been made available
to the language teams for translation. It is archived as:
Hello, gentle maintainer.
This is a message from the Translation Project robot. (If you have
any questions, send them to coordina...@translationproject.org.)
A new POT file for textual domain 'gcc' has been made available
to the language teams for translation. It is archived as:
Ping...
paul
Begin forwarded message:
From: paul_kon...@dell.com
Date: September 20, 2012 4:55:05 PM EDT
To: gcc-patches@gcc.gnu.org
Subject: Re: [patch] pr/54508: fix incomplete debug information for class
Attached below is an update to the testcase file, to fix the
On Wed, Sep 19, 2012 at 3:03 PM, de...@google.com wrote:
FYI.
Dehao
Thanks. Responses below. New patch being uploaded shortly.
Teresa
https://codereview.appspot.com/6489092/diff/4001/gcc/common.opt
File gcc/common.opt (right):
This patch fixes PR54690; it is rather obvious and has been also
approved by Paul in the PR.
Tobias
Index: gcc/fortran/ChangeLog
===
--- gcc/fortran/ChangeLog (Revision 191675)
+++ gcc/fortran/ChangeLog (Arbeitskopie)
@@ -1,3 +1,9
OK.
Jason
On 09/20/2012 10:01 AM, Dodji Seketeli wrote:
This is because in cplus_decl_attributes, save_template_attributes
makes so that the 'unused' attribute is applied to its appertaining
entity only at instantiation time. But then at parsing time
maybe_warn_unused_local_typedefs checks for TREE_USED
On 09/24/2012 09:02 AM, Aldy Hernandez wrote:
PR middle-end/53850
* trans-mem.c (expand_call_tm): Handle late built built-ins.
Looks good. Typo in the changelog.
r~
Ping.
Revised patch to add a new dump flag that dumps PMU profile information using
the -pmu dump option. (Was issue 6489092, creating new issue since I don't own
that one.)
Ok for google/main?
Passes bootstrap and regression tests.
Teresa
2012-09-24 Teresa Johnson tejohn...@google.com
On Fri, 21 Sep 2012, Dehao Chen wrote:
This patch moves location_adhoc_data into GC, and also rebuild the
hash table when reading in the PCH. After the patch, PCH can work as
expected.
Bootstrapped and passed gcc regression tests on x8664_linux.
If you have a moment to consider improvements
Maciej W. Rozycki ma...@codesourcery.com writes:
On Tue, 18 Sep 2012, Richard Sandiford wrote:
Have you had time to think about this some more? I am not sure I can
guess how you'd like me to fix this patch now without some more specific
review and/or suggestions about where the
Hi,
This patch is for google branches only.
It fixes the unsatified symbol in lipo-use build.
Tested with SPEC and google internal benchmarks.
Thanks,
-Rong
2012-09-24 Rong Xu x...@google.com
* gcc/l-ipo.c (cgraph_is_aux_decl_external): output comdat
virtual functions when
Hi,
This patch is for google branches only.
It prohibits the grouping of modules with diffent flag_rtti value.
otherwise it may cause profile mismatch, undefined symbols and some other
internal errors.
Tested with google internal benchmarks.
Thanks,
-Rong
2012-09-24 Rong Xu x...@google.com
Hi,
This is for google branches only.
It fix the lino number checksum mismatch during LIPO-use build.
Tested with SPEC and google internal banchmarks.
Thanks,
-Rong
2012-09-24 Rong Xu x...@google.com
* gcc/coverage.c (coverage_checksum_string): strip out LIPO
specific
ok.
David
On Mon, Sep 24, 2012 at 2:40 PM, Rong Xu x...@google.com wrote:
Hi,
This patch is for google branches only.
It fixes the unsatified symbol in lipo-use build.
Tested with SPEC and google internal benchmarks.
Thanks,
-Rong
2012-09-24 Rong Xu x...@google.com
*
Ok.
David
On Mon, Sep 24, 2012 at 2:41 PM, Rong Xu x...@google.com wrote:
Hi,
This patch is for google branches only.
It prohibits the grouping of modules with diffent flag_rtti value.
otherwise it may cause profile mismatch, undefined symbols and some other
internal errors.
Tested with
Hello world,
not much to say, the patch is pretty straightforward.
Regression-tested. OK for trunk?
Thomas
2012-09-24 Thomas König tkoe...@gcc.gnu.org
PR fortran/52724
* list_read.c (next_char): Handle kind=4 characters.
* unix.c (open_internal4): Correct
On Mon, 24 Sep 2012, Richard Sandiford wrote:
From the context I am assuming none of this matters for the 74K (and
presumably the 24KE/34K) and a MULT $0, $0 is indeed faster, but overall
isn't it something that should be decided based on instruction costs from
DFA schedulers? Is
On 09/24/12 15:08, Richard Henderson wrote:
On 09/24/2012 09:02 AM, Aldy Hernandez wrote:
PR middle-end/53850
* trans-mem.c (expand_call_tm): Handle late built built-ins.
Looks good. Typo in the changelog.
r~
Actually, that was not a typo. Late built built-ins... as in
cpplib-4.7.2.vi.po.gz
Description: Binary data
The Translation Project robot, in the
name of your translation coordinator.
coordina...@translationproject.org
Hello, gentle maintainer.
This is a message from the Translation Project robot.
A revised PO file for textual domain 'cpplib' has been submitted
by the Vietnamese team of translators. The file is available at:
http://translationproject.org/latest/cpplib/vi.po
(This file,
The current patterns will never match. Fix that. Also, merge the
SI and DI variants of each. Also, remove an unnecessary earlyclobber.
And add a pattern for what combine considers the canonical form of
one of these patterns.
Bootstrapped and regression checked on powerpc64-linux. Also tested
The generated code is now:
myswaps16:
rlwinm 10,3,8,16,23
rlwinm 9,3,24,24,31
or 9,9,10
extsh 3,9
blr
myswapu16:
rlwinm 10,3,8,16,23
rlwinm 9,3,24,24,31
or 9,9,10
rlwinm 3,9,0,0x
blr
While it was (without my
51 matches
Mail list logo