Hello Yunfeng,
Thank you for following up, and sorry for me reviewing your patches so
lately. The libcpp changes are coming along nicely, IMHO. I like the
fact that they are getting pretty minimal. I just have a few mostly
cosmetic comments at this point.
[...]
diff -cpr
On Mon, Aug 13, 2012 at 2:34 AM, Diego Novillo dnovi...@google.com wrote:
On 12-08-12 16:16 , Steven Bosscher wrote:
On Sun, Aug 12, 2012 at 10:09 PM, Diego Novillo dnovi...@google.com
wrote:
This patch implements the configuration changes needed to
bootstrap with a C++ compiler by default.
On Sun, Aug 12, 2012 at 11:49 PM, Steven Bosscher stevenb@gmail.com wrote:
Hello,
This patch tried to use non-clearing memory allocation where possible.
This is especially important for very large functions, when arrays of
size in the order of n_basic_blocks or num_ssa_names are allocated
This fixes one issue with copyrename, that it leaks names backward
through a PHI node because it treats a PHI node
_1 = PHI _2, _3, _4
as
_1 = _2;
_1 = _3;
_1 = _4;
at the point of the PHI node which is certainly not what it is
(the assigns exist, one each, on the incoming edges only).
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
to the authors of the respective patch.
Up
Hello,
To get the prefetchw cpuid flag, cpuid
function 0x8001 needs to be called.
Previous to patch, function 0x7 is called.
Bootstrapping and make -k check passes without failures.
Ok for trunk?
Regards
Ganesh
2012-08-13 Ganesh Gopalasubramanian ganesh.gopalasubraman...@amd.com
On 12-08-13 02:51 , Steven Bosscher wrote:
On Mon, Aug 13, 2012 at 2:34 AM, Diego Novillo dnovi...@google.com wrote:
On 12-08-12 16:16 , Steven Bosscher wrote:
On Sun, Aug 12, 2012 at 10:09 PM, Diego Novillo dnovi...@google.com
wrote:
This patch implements the configuration changes needed
On Sun, Aug 12, 2012 at 10:04 PM, Diego Novillo dnovi...@google.com wrote:
I will be sending 6 patches that implement all the changes we
have been making on the cxx-conversion branch. As described in
http://gcc.gnu.org/ml/gcc/2012-08/msg00015.html, these patches
change the default bootstrap
On Sun, Aug 12, 2012 at 11:30 PM, Marc Glisse marc.gli...@inria.fr wrote:
On Sun, 12 Aug 2012, Diego Novillo wrote:
This implements the double_int rewrite.
See http://gcc.gnu.org/ml/gcc-patches/2012-08/msg00711.html for
details.
Diego.
I am taking it as a chance to ask a couple
On Sun, Aug 12, 2012 at 11:30:59PM +0200, Marc Glisse wrote:
+inline double_int
+double_int::operator ++ ()
+{
+ *this + double_int_one;
*this += double_int_one;
would be less confusing.
Do you mean that *this + double_int_one; alone also works, just is
confusing? That would mean
Hi!
I've bootstrapped/regtested on x86_64-linux and i686-linux and committed the
following backport to 4.7 branch.
2012-08-13 Jakub Jelinek ja...@redhat.com
Backported from trunk
2012-07-19 Jakub Jelinek ja...@redhat.com
PR rtl-optimization/53942
*
Hi,
For thumb1, arm-gcc rewrites move insn into subtract of ZERO in peephole2
pass intentionally, then executes
pass_if_after_reload/pass_regrename/pass_cprop_hardreg sequentially.
In this scenario, copy propagation opportunities are missed because:
1. the move insns are re-written.
2.
Ping.
-Original Message-
From: Richard Earnshaw
Sent: Thursday, July 26, 2012 9:19 PM
To: Andrew Pinski
Cc: Bin Cheng; gcc-patches@gcc.gnu.org
Subject: Re: [PATCH]Remove duplicate check on BRANCH_COST in fold-const.c
On 26/07/12 11:27, Andrew Pinski wrote:
On Thu, Jul 26, 2012
On Mon, 13 Aug 2012, Jakub Jelinek wrote:
On Sun, Aug 12, 2012 at 11:30:59PM +0200, Marc Glisse wrote:
+inline double_int
+double_int::operator ++ ()
+{
+ *this + double_int_one;
*this += double_int_one;
would be less confusing.
Do you mean that *this + double_int_one; alone also works,
This is a small patch for sched-rgn that attempts to save DFA state at
the end of a basic block and re-use it in successor blocks. This was a
customer-requested optimization; I've not seen it make much of a
difference in any macro benchmarks.
Bootstrapped and tested on x86_64-linux and also
Yes! Thanks Jakub.
-Original Message-
From: Jakub Jelinek [mailto:ja...@redhat.com]
Sent: Monday, August 13, 2012 3:16 PM
To: Gopalasubramanian, Ganesh
Cc: gcc-patches@gcc.gnu.org
Subject: Re: [PATCH,i386] cpuid function for prefetchw
On Mon, Aug 13, 2012 at 09:29:45AM +,
On Fri, Aug 10, 2012 at 7:00 PM, Jonathan Wakely jwakely@gmail.com wrote:
Let's CC Gaby, who likes to keep an eye on patches involving complex
Thanks Jonathan.
The patch is OK -- though I suspect we should have a documentation note
about the extension of allowing other C99 complex functions
On Mon, Aug 13, 2012 at 5:32 AM, Marc Glisse marc.gli...@inria.fr wrote:
On Mon, 13 Aug 2012, Jakub Jelinek wrote:
On Sun, Aug 12, 2012 at 11:30:59PM +0200, Marc Glisse wrote:
+inline double_int
+double_int::operator ++ ()
+{
+ *this + double_int_one;
*this += double_int_one;
would
On Mon, 13 Aug 2012, Gabriel Dos Reis wrote:
On Fri, Aug 10, 2012 at 7:00 PM, Jonathan Wakely jwakely@gmail.com wrote:
Let's CC Gaby, who likes to keep an eye on patches involving complex
Thanks Jonathan.
The patch is OK -- though I suspect we should have a documentation note
about the
On Mon, Aug 13, 2012 at 6:17 AM, Marc Glisse marc.gli...@inria.fr wrote:
On Mon, 13 Aug 2012, Gabriel Dos Reis wrote:
On Fri, Aug 10, 2012 at 7:00 PM, Jonathan Wakely jwakely@gmail.com
wrote:
Let's CC Gaby, who likes to keep an eye on patches involving complex
Thanks Jonathan.
The
On 12-08-13 05:37 , Richard Guenther wrote:
On Sun, Aug 12, 2012 at 10:04 PM, Diego Novillo dnovi...@google.com wrote:
I will be sending 6 patches that implement all the changes we
have been making on the cxx-conversion branch. As described in
http://gcc.gnu.org/ml/gcc/2012-08/msg00015.html,
On 12-08-13 05:39 , Richard Guenther wrote:
It's an odd thing that you need to touch code replacing - with . (yes, it's
due to the use of references) but not at the same time convert those places
to the new VEC interface.
Yes. I hated this aspect of the initial conversion. It caused many
On Mon, 13 Aug 2012, Gabriel Dos Reis wrote:
On Mon, Aug 13, 2012 at 6:17 AM, Marc Glisse marc.gli...@inria.fr wrote:
On Mon, 13 Aug 2012, Gabriel Dos Reis wrote:
On Fri, Aug 10, 2012 at 7:00 PM, Jonathan Wakely jwakely@gmail.com
wrote:
Let's CC Gaby, who likes to keep an eye on
Hi!
John Regehr discovered a couple of spots in GCC sources that invoke
undefined behavior during bootstrap/regtest, the following patch fixes most
of them. I couldn't reproduce the diagnostic.c failure and would like to
leave the ipa hunk to Honza, I think the probability/frequency code often
On 08/12/2012 10:00 PM, François Dumont wrote:
On 08/11/2012 03:47 PM, Marc Glisse wrote:
On Sat, 11 Aug 2012, François Dumont wrote:
Your remark on using std::move rather than std::forward Marc made
sens but didn't work. I don't understand why but the new test is
showing that
On Mon, Aug 13, 2012 at 11:45:36AM +0200, Jakub Jelinek wrote:
On Mon, Aug 13, 2012 at 09:29:45AM +, Gopalasubramanian, Ganesh wrote:
To get the prefetchw cpuid flag, cpuid
function 0x8001 needs to be called.
Previous to patch, function 0x7 is called.
Bootstrapping and make -k
On Sat, Aug 11, 2012 at 2:25 PM, Marc Glisse marc.gli...@inria.fr wrote:
On Fri, 10 Aug 2012, Marc Glisse wrote:
this patch detects permutations of permutations and merges them. It also
canonicalizes permutations a bit more.
There are several issues with this patch:
1) I am not sure we
On Mon, Aug 13, 2012 at 2:01 PM, Jakub Jelinek ja...@redhat.com wrote:
Hi!
John Regehr discovered a couple of spots in GCC sources that invoke
undefined behavior during bootstrap/regtest, the following patch fixes most
of them. I couldn't reproduce the diagnostic.c failure and would like to
Paolo Carlini paolo.carl...@oracle.com writes:
Marc Glisse marc.gli...@inria.fr ha scritto:
To be honest, I only checked the patch on linux/glibc, so there is a
real
risk on other platforms (which I don't have access to). I also did a
quick
sanity check on freebsd (not a true test).
Ok,
On 13.08.2012 14:32, Bernd Schmidt wrote:
This is a small patch for sched-rgn that attempts to save DFA state at
the end of a basic block and re-use it in successor blocks. This was a
customer-requested optimization; I've not seen it make much of a
difference in any macro benchmarks.
FWIW,
Hi,
I've just merged upstream trunk on the aarch64-branch up to r190335.
Thanks
Sofiane
On Mon, Aug 13, 2012 at 2:10 PM, Jakub Jelinek ja...@redhat.com wrote:
To get the prefetchw cpuid flag, cpuid
function 0x8001 needs to be called.
Previous to patch, function 0x7 is called.
Bootstrapping and make -k check passes without failures.
Ok for trunk?
IMHO you move it to
Accessing loop_depth (bb-loop_father) isn't very expensive. The
following removes the duplicate info in basic-blocks which is not
properly kept up-to-date at the moment.
Bootstrapped on x86_64-unknown-linux-gnu, testing in progress.
Richard.
2012-08-13 Richard Guenther rguent...@suse.de
Bootstrapped on x86_64-unknown-linux-gnu, committed.
Richard.
2012-08-13 Richard Guenther rguent...@suse.de
* tree-cfg.c (print_loop): Avoid ICEing for loops marked for
removal and loops with multiple latches.
Index: gcc/tree-cfg.c
On Mon, 13 Aug 2012, Richard Guenther wrote:
On Sat, Aug 11, 2012 at 2:25 PM, Marc Glisse marc.gli...@inria.fr wrote:
On Fri, 10 Aug 2012, Marc Glisse wrote:
1) I am not sure we always want to combine permutations. Indeed, someone
(user? vectorizer?) may have written 2 permutations to help
On Mon, 13 Aug 2012, Rainer Orth wrote:
Paolo Carlini paolo.carl...@oracle.com writes:
Marc Glisse marc.gli...@inria.fr ha scritto:
To be honest, I only checked the patch on linux/glibc, so there is a
real
risk on other platforms (which I don't have access to). I also did a
quick
sanity
On Mon, Aug 13, 2012 at 5:59 AM, Uros Bizjak ubiz...@gmail.com wrote:
On Mon, Aug 13, 2012 at 2:10 PM, Jakub Jelinek ja...@redhat.com wrote:
To get the prefetchw cpuid flag, cpuid
function 0x8001 needs to be called.
Previous to patch, function 0x7 is called.
Bootstrapping and make
On Mon, Aug 13, 2012 at 3:03 PM, Marc Glisse marc.gli...@inria.fr wrote:
On Mon, 13 Aug 2012, Richard Guenther wrote:
On Sat, Aug 11, 2012 at 2:25 PM, Marc Glisse marc.gli...@inria.fr wrote:
On Fri, 10 Aug 2012, Marc Glisse wrote:
1) I am not sure we always want to combine permutations.
I guess people will complain soon enough if this causes horrible performance
regressions in vectorized code.
Not having looked at your patch in great detail,. surely what we don't
want is a situation where 2 constant permutations are converted into
one generic permute. Based on a quick read of
On Mon, Aug 13, 2012 at 3:12 PM, Ramana Radhakrishnan
ramana.radhakrish...@linaro.org wrote:
I guess people will complain soon enough if this causes horrible performance
regressions in vectorized code.
Not having looked at your patch in great detail,. surely what we don't
want is a situation
On Mon, 13 Aug 2012, Richard Guenther wrote:
On Mon, Aug 13, 2012 at 3:12 PM, Ramana Radhakrishnan
ramana.radhakrish...@linaro.org wrote:
I guess people will complain soon enough if this causes horrible performance
regressions in vectorized code.
Not having looked at your patch in great
On Mon, Aug 13, 2012 at 03:13:26PM +0200, Richard Guenther wrote:
On Mon, Aug 13, 2012 at 3:12 PM, Ramana Radhakrishnan
ramana.radhakrish...@linaro.org wrote:
I guess people will complain soon enough if this causes horrible
performance
regressions in vectorized code.
Not having
On Mon, 13 Aug 2012, Richard Guenther wrote:
+ /* Check that it is only used here. We cannot use has_single_use
+ since the expression is using it twice itself... */
Ah ... so then
|| num_imm_uses (op0) != 2
Ah, ok, that's simpler indeed, but there were such dire warnings
Jonathan == Jonathan Wakely jwakely@gmail.com writes:
$11 = std::unique_ptr containing (datum *) 0x6067d0
Jonathan It's inconsistent with the other printers in that it prints
Jonathan the stored type, unlike e.g. std::vectorint which just says
Jonathan std::vector of length ... but I think
Dear All,
Please find attached a patch and testcase for the above PR. The
comment before generate_component_assignments explains the need for
the patch, which itself is fairly self explanatory.
Bootstrapped and regtested on Fc9/x86_64 - OK for trunk?
Best regards
Paul and Alessandro.
On Sun, Aug 12, 2012 at 11:28 PM, Laurynas Biveinis
laurynas.bivei...@gmail.com wrote:
I'm referring to the very first part of gty.texi, section 22 before
the subsection table of contents:
http://gcc.gnu.org/onlinedocs/gccint/Type-Information.html#Type-Information.
It talks about C, structs
On Mon, 13 Aug 2012, Jakub Jelinek wrote:
On Mon, Aug 13, 2012 at 03:13:26PM +0200, Richard Guenther wrote:
The patch does not do that. It merely assumes that the target knows
how to perform an optimal constant permute and that two constant
permutes never generate better code than a single
On Mon, Aug 13, 2012 at 03:45:00PM +0200, Marc Glisse wrote:
On Mon, 13 Aug 2012, Jakub Jelinek wrote:
On Mon, Aug 13, 2012 at 03:13:26PM +0200, Richard Guenther wrote:
The patch does not do that. It merely assumes that the target knows
how to perform an optimal constant permute and that
On 13 August 2012 14:21, Marc Glisse marc.gli...@inria.fr wrote:
On Mon, 13 Aug 2012, Richard Guenther wrote:
On Mon, Aug 13, 2012 at 3:12 PM, Ramana Radhakrishnan
ramana.radhakrish...@linaro.org wrote:
I guess people will complain soon enough if this causes horrible
performance
On 13 August 2012 14:54, Jakub Jelinek ja...@redhat.com wrote:
On Mon, Aug 13, 2012 at 03:45:00PM +0200, Marc Glisse wrote:
On Mon, 13 Aug 2012, Jakub Jelinek wrote:
On Mon, Aug 13, 2012 at 03:13:26PM +0200, Richard Guenther wrote:
The patch does not do that. It merely assumes that the
On Mon, 13 Aug 2012, Ramana Radhakrishnan wrote:
On 13 August 2012 14:21, Marc Glisse marc.gli...@inria.fr wrote:
On Mon, 13 Aug 2012, Richard Guenther wrote:
On Mon, Aug 13, 2012 at 3:12 PM, Ramana Radhakrishnan
ramana.radhakrish...@linaro.org wrote:
I guess people will complain soon
Hello,
here is a fix for PR47586: missing deep copy for the case:
dt_w_alloc = ptr_func(arg)
where dt_w_alloc is of derived type with allocatable components, and
ptr_func returns a data pointer.
The fix tweaks expr_is_variable so that gfc_trans_scalar_assign is
called with the flag enabling
Hi all,
It is known that LEA splitting is one of the most critical problems
for Atom processors and changes try to improve it through:
1. More aggressive Lea splitting – do not perform splitting if
only split cost exceeds AGU stall .
2. Reordering splitting instructions to get better
On 08/13/2012 06:32 AM, Bernd Schmidt wrote:
This is a small patch for sched-rgn that attempts to save DFA state at
the end of a basic block and re-use it in successor blocks. This was a
customer-requested optimization; I've not seen it make much of a
difference in any macro benchmarks.
On Mon, 13 Aug 2012, Marc Glisse wrote:
On Mon, 13 Aug 2012, Richard Guenther wrote:
If your new predicate would match more places (can you do a quick search?)
You mean: if there are more optimizations that either already check for
double use in the same statement, or could benefit from
On Sun, Aug 12, 2012 at 08:02:30PM +0100, Jonathan Wakely wrote:
This improves the fairly uninformative Operation not supported
message given when std::thread is used without linking to libpthread.
Now you get:
terminate called after throwing an instance of 'std::system_error'
what():
On Sun, Aug 12, 2012 at 12:46 PM, Jack Howarth howa...@bromo.med.uc.edu wrote:
This patch introduces the regressions...
FAIL: g++.dg/debug/dwarf2/pubnames-2.C scan-assembler
.section\t.debug_pubnames
FAIL: g++.dg/debug/dwarf2/pubnames-2.C scan-assembler
_GLOBAL__sub_I__ZN3one3c1vE0+[
Richard Henderson wrote:
Only tested visually, by examining assembly diffs of the
runtime libraries between successive patches. All told it
would appear to be some remarkable code size improvements.
Thanks for having a look at this!
Please test.
Unfortunately GCC crashes during bootstrap,
On 08/13/2012 10:07 AM, Ulrich Weigand wrote:
+/* Check whether a rotate of ROTL followed by an AND of CONTIG is equivalent
+ to a shift followed by the AND. In particular, CONTIG should not overlap
+ the (rotated) bit 0/bit 63 gap. */
+
+bool
+s390_extzv_shift_ok (int bitsize, int
On 13 August 2012 16:47, Joe Buck joe.b...@synopsys.com wrote:
On Sun, Aug 12, 2012 at 08:02:30PM +0100, Jonathan Wakely wrote:
This improves the fairly uninformative Operation not supported
message given when std::thread is used without linking to libpthread.
Now you get:
terminate called
On 13 August 2012 14:31, Tom Tromey wrote:
Jonathan == Jonathan Wakely jwakely@gmail.com writes:
$11 = std::unique_ptr containing (datum *) 0x6067d0
Jonathan It's inconsistent with the other printers in that it prints
Jonathan the stored type, unlike e.g. std::vectorint which just says
Hi, the google/gcc-4_7 fails to linking anything (on x86-generic), by
looking into specs file, it seems that 'link_emulation' section is
missing in specs.
The problem is in config/i386/linux.h, SUBTARGET_EXTRA_SPECS (which is
not empty for chrome x86-generic) is overridden by
On 13 August 2012 12:57, Marc Glisse wrote:
I only modified the xml version. I expect the html version will be updated
the next time someone who knows what they are doing touches the doc...
That's no problem, I tend to regenerate the html fairly frequently.
Thanks.
I noticed recently that while the validator was accepting the
'expire=MMDD' attribute, it was not actually doing anything with
it.
This patch fixes the oversight. Simon, I will be backporting the
patch to google/gcc-4_7.
Committed to trunk.
2012-08-13 Diego Novillo dnovi...@google.com
On 13 August 2012 18:49, Jonathan Wakely wrote:
I suppose EOPNOTSUPP might be better, if it's supported everywhere.
EPERM has the advantage of being a documented error for
pthread_create.
We do define std::errc::operation_not_supported unconditionally on
most platforms, but not mingw or djgpp.
Hello Paul,
I think there are a couple of bugs not triggered by the single component
types in the test. See below.
On 13/08/2012 15:37, Paul Richard Thomas wrote:
+
+ /* Go through the code chain eliminating all but calls to
+ typebound procedures. Since we have been through
+
Hello Dehao,
I have mostly cosmetic comments to make about the libcpp parts.
Dehao Chen de...@google.com writes:
Index: libcpp/include/line-map.h
===
*** libcpp/include/line-map.h (revision 189835)
--- libcpp/include/line-map.h
Hi!
move_unallocated_pseudos apparently relies on no insns being deleted
in between find_moveable_pseudos and itself, which can happen when
delete_trivially_dead_insns removes dead insns and insns that feed them.
This can be fixed either by moving the delete_trivially_dead_insns
call earlier in
On Mon, Aug 13, 2012 at 9:03 PM, Richard Henderson r...@redhat.com wrote:
+ (eq_attr isa fma) (symbol_ref TARGET_FMA)
+ (eq_attr isa fma4)
+(symbol_ref TARGET_FMA4 !TARGET_FMA)
Why the !TARGET_FMA for fma4?
If both ISAs are enabled, I don't see why we couldn't choose
Hi all!
Patch aims to fix instability introduced by first scheduler on x86. In
particular it targets following list:
[1] http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46843
[2] http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46829
[3] http://gcc.gnu.org/bugzilla/show_bug.cgi?id=36680
[4]
Jakub Jelinek ja...@redhat.com writes:
--- gcc/simplify-rtx.c.jj 2012-08-10 15:49:20.0 +0200
+++ gcc/simplify-rtx.c2012-08-13 09:51:43.628508537 +0200
@@ -66,7 +66,7 @@ static rtx simplify_binary_operation_1 (
static rtx
neg_const_int (enum machine_mode mode, const_rtx
On 08/13/2012 02:10 PM, Paolo Carlini wrote:
On 08/12/2012 10:00 PM, François Dumont wrote:
Ok for trunk ?
Ok, thanks!
Paolo.
PS: you may want to remove the trailing blank line of
testsuite_counter_type.h
Attached patch applied.
2012-08-13 François Dumont fdum...@gcc.gnu.org
On 08/13/2012 12:33 PM, Uros Bizjak wrote:
AFAIU fma3 is better than fma4 for bdver2 (the only CPU that
implements both FMA sets). Current description of bdver2 doesn't even
enable fma4 in processor_alias_table due to this fact.
The change you are referring to adds preference for fma3 insn
This fixes an error I inadvertently introduced a few months ago.
2012-08-13 David Adler d.adler.s at gmail dot com
PR libstdc++/54185
* src/c++11/condition_variable.cc (condition_variable): Always
destroy native type in destructor.
*
Dear all,
Attached is the first part of a patch which will implement finalization
support and polymorphic freeing in gfortran.
It addresses two needs:
a) For polymorphic (CLASS) variables, allocatable components have to
be freed; however, at compile time only the allocatable components of
Hello,
This patch adds basic support for utilizing the SH div0s instruction to
simplify some integer sign comparisons such as '(a 0) == (b 0)'.
Tested on rev 190332 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
2012-08-07 Cary Coutant ccout...@google.com
gcc/
* dwarf2out.c (clone_as_declaration): Copy DW_AT_abstract_origin
attribute.
(generate_skeleton_bottom_up): Remove DW_AT_object_pointer attribute
from original DIE.
(clone_tree_hash): Rename to ...
On 8/13/12, Gabriel Dos Reis g...@integrable-solutions.net wrote:
On Aug 13, 2012 Marc Glisse marc.gli...@inria.fr wrote:
On Mon, 13 Aug 2012, Jakub Jelinek wrote:
On Sun, Aug 12, 2012 at 11:30:59PM +0200, Marc Glisse wrote:
+inline double_int
+double_int::operator ++ ()
+{
On 8/12/12, Marc Glisse marc.gli...@inria.fr wrote:
On Sun, 12 Aug 2012, Diego Novillo wrote:
This implements the double_int rewrite.
See http://gcc.gnu.org/ml/gcc-patches/2012-08/msg00711.html for
details.
I am taking it as a chance to ask a couple questions about the coding
On 8/13/12, Richard Guenther richard.guent...@gmail.com wrote:
Increment/decrement operations did not exist, please do not add
them at this point.
Note that I have also added +=, -= and *= operations. Having them
has three advantages. First, it matches expectations on what
numeric types
Hi,
Here, we were setting the LOOKUP_NONVIRTUAL flag wrongly. Actually, we
need to check if the function context is the same than the instance
type -- yes that might happen that they be different in presence of
using-declarations.
It happens that it was working if the call was invoked through a
On Mon, Aug 13, 2012 at 3:50 PM, Jakub Jelinek ja...@redhat.com wrote:
The formatting doesn't match GCC coding conventions in several ways.
You don't have spaces before (, and ( shouldn't be at the end of line if
possible.
Updated patch attached.
Ollie
commit
On Mon, Aug 13, 2012 at 5:41 AM, Richard Guenther
richard.guent...@gmail.com wrote:
*this += double_int_one;
would be less confusing.
Increment/decrement operations did not exist, please do not add them
at this point.
But they are going to be used when the call-sites are converted.
There is
On 08/13/2012 01:22 PM, Lawrence Crowl wrote:
yes, it is just as confusing and a bug as
2.3 + 1;
is in plain C.
Yes, it is a bug. It's a bit disturbing that it wasn't caught
in bootstrap.
You'll recall that I pointed it out last time around as well.
r~
On 8/13/12, Richard Henderson r...@redhat.com wrote:
On 08/13/2012 01:22 PM, Lawrence Crowl wrote:
yes, it is just as confusing and a bug as
2.3 + 1;
is in plain C.
Yes, it is a bug. It's a bit disturbing that it wasn't caught
in bootstrap.
You'll recall that I pointed it out last
On Mon, Apr 25, 2011 at 2:53 PM, Jason Merrill ja...@redhat.com wrote:
In C++0x we can have an initializer that is potentially constant and yet
still type-dependent if it involves a call, so we need to handle that.
For 4.7 I'm explicitly testing for type-dependency; for 4.6 I've made a
Hello!
It is known that LEA splitting is one of the most critical problems
for Atom processors and changes try to improve it through:
1. More aggressive Lea splitting – do not perform splitting if
only split cost exceeds AGU stall .
2. Reordering splitting instructions to get
Hello,
This adds support for the SH2A instructions movu.b and movu.w for
zero-extending mem loads with displacement addressing.
Tested on rev 190332 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?
On 08/03/2012 11:28:57 +0200 Paolo Carlini wrote:
No problem ;) Patch is Ok, thanks!
Hi Paolo,
I see you committed the patch for me. Thanks!! I literally posted
that last patch just minutes before heading out on vacation and didn't
want to commit it just before I left in case it caused any
Hi Carrot, could you take a look at this patch? Thanks!
The modification is in upstream trunk patch revision - 186859.
The same patch has been back ported to google/gcc-4_6
(http://codereview.appspot.com/6206055/), this is to apply on
google/gcc-4_7
Regards,
-Han
2012-08-13 Han Shen
On Sun, Aug 12, 2012 at 12:46 PM, Jack Howarth howa...@bromo.med.uc.edu wrote:
On Sun, Jul 01, 2012 at 09:33:06AM -0500, Gabriel Dos Reis wrote:
On Thu, Jun 28, 2012 at 12:50 PM, Sterling Augustine
saugust...@google.com wrote:
The enclosed patch adds a fix for the pubnames anonymous
On 9/08/2012, at 7:10 AM, Richard Sandiford wrote:
Hmm, yeah, it does look like they should be using mips_linked_madd_p
instead, except that mips_linked_madd_p isn't yet wired up to handle
DSP macs. Rather than pattern-match them all, the easiest thing would
probably be to define a new
On 08/13/2012 02:44 PM, Jakub Jelinek wrote:
Hi!
move_unallocated_pseudos apparently relies on no insns being deleted
in between find_moveable_pseudos and itself, which can happen when
delete_trivially_dead_insns removes dead insns and insns that feed them.
This can be fixed either by moving
On 07/23/12 02:34, Steven Bosscher wrote:
Hello,
While reading up on how gcov/profiling works, I noticed that there are
a lot of places where the notes file is still referred to as the
basic block graph file. Also, the gcov manual has not been updated
for -fprofile-dir. The attached patch
Hi Tobias,
Thanks for your work on this. This is a big step. I would add to your
list the following:
(4) If the entity is of extended type and the parent type has a component
that is finalizable, the parent component's component is finalized.
In ForTrilnos, we need for this to happen even
On Aug 13, 2012, at 4:56 PM, Sterling Augustine wrote:
The enclosed patch adjusts the test so it will pass on darwin. The
issue was that it looked for some elf-specific assembly directives,
which it shouldn't.
OK for mainline?
Ok.
Attached, is a patch to fix PR20020, and three test cases.
This patch improves the code generated for structs that
can be represented in a TImode value on an x86_64 target.
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=20020
The test cases scan the generated RTL and verify
that TImode operations
On 13/08/2012, at 10:32 PM, Bernd Schmidt wrote:
This is a small patch for sched-rgn that attempts to save DFA state at
the end of a basic block and re-use it in successor blocks. This was a
customer-requested optimization; I've not seen it make much of a
difference in any macro benchmarks.
On 4/08/2012, at 12:05 AM, Bernd Schmidt wrote:
This patch allows us to change
rn++
rm=[rn]
into
rm=[rn + 4]
rn++
This is a good scheduler optimization that I wanted to have for quite a while.
Bernd, kudos for implementing it!
I am going to review this patch, and then the
Dear Mikael,
I think there are a couple of bugs not triggered by the single component
types in the test. See below.
Yes, you are right. We should have tested multiple components... my fault!
This could be moved to the only next caller (`previous' doesn't need to
be updated if `this_code'
1 - 100 of 102 matches
Mail list logo