ira.c:setup_pressure_classes treats "leaf" classes as pressure classes
even if moves between them are more expensive than moves to or from memory:
if (ira_class_hard_regs_num[cl] != 1
/* A register class without subclasses may contain a few
hard registers and movement
Steve Ellcey writes:
> On Thu, 2012-09-06 at 06:47 +0100, Richard Sandiford wrote:
>> > Is this an 'if-then-else' usage?
>>
>> Yeah, but I typoed, sorry. It should be:
>>
>> %{mips32r2|mips64r2:-msynci;:-mno-synci}
>>
>> Richard
>
> OK, I got that working now. I am still having some issues th
Ping!!
Cheers,
Yi-Hsiu, Hsu
-Original Message-
From: Yi-Hsiu Hsu
Sent: Tuesday, June 26, 2012 1:51 PM
To: 'Chung-Lin Tang'
Cc: Ramana Radhakrishnan; gcc-patches@gcc.gnu.org
Subject: RE: [PATCH, ARM] New CPU support for Marvell PJ4 cores
Hi Chung-Lin,
I think tune_marvell attribute bet
This fixes part of the issue described in PR gcov-profile/54487 where
there were warnings about mismatches due to slight differences in the
merged histograms in different object files. This can happen due to
the truncating integer division in the merge routine, which could result
in slightly differ
From: Andi Kleen
For parallel LTO builds setting -fmem-report does not work very well
because all the LTRANS phases dump it in parallel and typically interleave
it to unreadability.
Since usually the memory bottleneck is WPA add a flag to only dump
the memory report for that.
Passed bootstrap a
Fixed spacing and condensed repeated code into helper.
This patch should be applied to google/main.
Tested with crosstools.
2012-09-06 Chris Manghane
* gcc/doc/invoke.texi: Modified pmu-profile-use option.
* gcc/tree-dump.c: Added new dump flag.
* gcc/tree-pretty-prin
On Sun, Aug 26, 2012 at 7:03 PM, Segher Boessenkool
wrote:
> On all supported targets, plain "nop" is the correct thing
> to use ("cror 31,31,31" was for POWER).
>
> Tested on powerpc64-linux --enable-languages=c,c++,fortran; no
> regressions. Okay for mainline?
>
>
> Segher
>
>
> 2012-08-26 Seg
On Thu, 2012-09-06 at 06:47 +0100, Richard Sandiford wrote:
> > Is this an 'if-then-else' usage?
>
> Yeah, but I typoed, sorry. It should be:
>
> %{mips32r2|mips64r2:-msynci;:-mno-synci}
>
> Richard
OK, I got that working now. I am still having some issues though. My
original patch was setu
On Thu, Sep 6, 2012 at 5:34 PM, Chris Manghane wrote:
>
>
> On Thu, Sep 6, 2012 at 5:08 PM, Teresa Johnson wrote:
>>
>> On Thu, Sep 6, 2012 at 2:49 PM, Chris Manghane wrote:
>> > This patch adds a new dump flag that dumps PMU profile information using
>> > the -pmu dump option.
>> >
>> > This pa
On Thu, Sep 6, 2012 at 2:49 PM, Chris Manghane wrote:
> This patch adds a new dump flag that dumps PMU profile information using
> the -pmu dump option.
>
> This patch should be applied to google/main.
>
> Tested with crosstools.
>
> 2012-09-06 Chris Manghane
>
> * gcc/doc/invoke.texi:
On 09/06/2012 07:07 PM, Joseph S. Myers wrote:
The -t* options added duplicate -mcpu= options; the only existing
precedent appears to be arm-vxworks and I don't think the options are
appropriate for generic PowerPC target files (not specific to an OS port
such as VxWorks with its own special sele
I've committed this trivial comment patch to src/include/dwarf2.def
(and will shortly do the same in gcc/include).
-cary
2012-09-06 Cary Coutant
include/
* dwarf2.def: Edit comment.
diff --git a/include/dwarf2.def b/include/dwarf2.def
index 3c3dfcc..7fe2df1 100644
--- a/include/dwar
I have committed this patch to the gccgo branch to add some cherry
picked bug fixes to the Go library. This is for internal release
purposes. The mainline and 4.7 tree remain at the Go 1.0.2 release. I
expect that most of these patches will be in Go 1.0.3.
The complete list of cherry-picked pat
On 09/06/2012 06:09 PM, Andrew Pinski wrote:
Could you explain why you are changing system.h ?
That was a convenience to me at one point.
It should have been deleted from the patch set.
--
Jim Lemke
Mentor Graphics / CodeSourcery
Orillia Ontario, +1-613-963-1073
On Thu, 6 Sep 2012, James Lemke wrote:
> Attached are the patches for this gcc port.
>
> On a recent checkout (r191027) I have run the DejaGNU suite with no new
> failures for binutils, gas, ld, gcc, g++, gfortran. A bootstrap is in
> progress.
The -t* options added duplicate -mcpu= options; th
On Thu, 6 Sep 2012, Andrew Pinski wrote:
> > You mean this:
> >
> > + POWERPC_E200_MASK = MASK_VLE | MASK_ISEL | MASK_MULTIPLE
> >
> > ? Well, this just marks that the e200 processor supports ISEL regardless
> > of the mode selected (standard vs VLE). Then with -mvle ISEL is supposed
> > to be
Hi,
On 09/06/2012 10:32 PM, howa...@frodo.msbb.uc.edu wrote:
The attached patch eliminates the bootstrap failures in libstdc++-v3
of PR 54419 by adding a define to acinclude.m4, GLIBCXX_CHECK_X86_RDRAND, that
checks for assembler support of the new rdrnd opcode, using this new define
in conf
On Thu, Sep 6, 2012 at 3:39 PM, Maciej W. Rozycki
wrote:
> On Thu, 6 Sep 2012, Andrew Pinski wrote:
>
>> Could you explain why you are changing system.h ?
>> Also seems like TARGET_VLE_ISEL should not be needed TARGET_ISEL is
>> always set for VLE targets.
>
> You mean this:
>
> + POWERPC_E200_M
On Thu, 6 Sep 2012, Andrew Pinski wrote:
> Could you explain why you are changing system.h ?
> Also seems like TARGET_VLE_ISEL should not be needed TARGET_ISEL is
> always set for VLE targets.
You mean this:
+ POWERPC_E200_MASK = MASK_VLE | MASK_ISEL | MASK_MULTIPLE
? Well, this just marks t
Hi,
On 09/06/2012 07:47 PM, Jason Merrill wrote:
On 09/06/2012 12:18 PM, Paolo Carlini wrote:
On 09/06/2012 02:03 AM, Jason Merrill wrote:
but note that for:
template
struct A
{
int select() { return 0; }
};
we have parser->num_template_parameter_lists == 1 and num_templates ==
0. Thus
On Thu, Sep 6, 2012 at 2:55 PM, James Lemke wrote:
> Attached are the patches for this gcc port.
>
> On a recent checkout (r191027) I have run the DejaGNU suite with no new
> failures for binutils, gas, ld, gcc, g++, gfortran. A bootstrap is in
> progress.
Could you explain why you are changing
This patch adds a new dump flag that dumps PMU profile information using
the -pmu dump option.
This patch should be applied to google/main.
Tested with crosstools.
2012-09-06 Chris Manghane
* gcc/doc/invoke.texi: Modified pmu-profile-use option.
* gcc/tree-dump.c: Added new d
Hello!
We can use 32bit address without .code64 directive; this test is valid
for 32bit and 64bit targets.
2012-09-06 Uros Bizjak
* configure.ac (hle prefixes): Remove .code64.
* configure: Regenerated.
Tested on x86_64-pc-linux-gnu, committed to mainline SVN.
Uros.
Index: c
On Thu, Sep 06, 2012 at 01:33:11PM -0700, Benjamin De Kosnik wrote:
> Here's the patch as applied to trunk in rev. 191042. I'll apply it to
> 4.7 this weekend as long as nobody yelps.
> 2012-09-06 Thiago Macieira
>
> PR libstdc++/54172
> * libsupc++/guard.cc (__cxa_guard_acquire)
On quinta-feira, 6 de setembro de 2012 13.33.11, Benjamin De Kosnik wrote:
> Here's the patch as applied to trunk in rev. 191042. I'll apply it to
> 4.7 this weekend as long as nobody yelps.
Thanks. The change to ACQUIRE is also a bugfix.
--
Thiago Macieira - thiago.macieira (AT) intel.com
Soft
Here's the patch as applied to trunk in rev. 191042. I'll apply it to
4.7 this weekend as long as nobody yelps.
-benjamin
2012-09-06 Thiago Macieira
PR libstdc++/54172
* libsupc++/guard.cc (__cxa_guard_acquire): Exit the loop earlier if
we detect that another thread has had
The attached patch eliminates the bootstrap failures in libstdc++-v3
of PR 54419 by adding a define to acinclude.m4, GLIBCXX_CHECK_X86_RDRAND, that
checks for assembler support of the new rdrnd opcode, using this new define
in configure.ac and also defining _GLIBCXX_X86_RDRAND in config.h if sup
On 2012-09-06 15:16 , NightStrike wrote:
Actually, on closer inspection, other places list the minimum version
as 4.3.2, including:
http://gcc.gnu.org/install/prerequisites.html
as well as the two locations that I mention above. Perhaps configure
should do likewise.
It already does (configur
On 2012-09-06 15:13 , NightStrike wrote:
On Thu, Sep 6, 2012 at 2:13 PM, Diego Novillo wrote:
Doug was trying to build with gmp 4.2.1 and ran into this error:
gmp-4.2.1/include/gmp.h:515:12: error: 'std::FILE' has not been declared
This bug was fixed in gmp 4.2.3, so I've bumped the minimum
a
On Thu, Sep 06, 2012 at 03:22:10PM -0400, Jack Howarth wrote:
>The attached patch eliminates the bootstrap failures in libstdc++-v3
> of PR 54419 by having configure check for assembler support of the new
> rdrand opcode and defining _GLIBCXX_X86_RDRAND in config.h if supported.
> Tested on x8
On Thu, Sep 06, 2012 at 03:22:10PM -0400, Jack Howarth wrote:
>The attached patch eliminates the bootstrap failures in libstdc++-v3
> of PR 54419 by having configure check for assembler support of the new
> rdrand opcode and defining _GLIBCXX_X86_RDRAND in config.h if supported.
> Tested on x8
The attached patch eliminates the bootstrap failures in libstdc++-v3
of PR 54419 by having configure check for assembler support of the new
rdrand opcode and defining _GLIBCXX_X86_RDRAND in config.h if supported.
Tested on x86_64-apple-darwin12 against the assembler from Xcode 4.4.1.
Okay for g
On Thu, Sep 6, 2012 at 3:13 PM, NightStrike wrote:
> On Thu, Sep 6, 2012 at 2:13 PM, Diego Novillo wrote:
>> Doug was trying to build with gmp 4.2.1 and ran into this error:
>>
>> gmp-4.2.1/include/gmp.h:515:12: error: 'std::FILE' has not been declared
>>
>> This bug was fixed in gmp 4.2.3, so I'
On Thu, Sep 6, 2012 at 2:13 PM, Diego Novillo wrote:
> Doug was trying to build with gmp 4.2.1 and ran into this error:
>
> gmp-4.2.1/include/gmp.h:515:12: error: 'std::FILE' has not been declared
>
> This bug was fixed in gmp 4.2.3, so I've bumped the minimum
> acceptable version in configure.ac
On Thu, 2012-09-06 at 14:41 +0200, Georg-Johann Lay wrote:
> Oleg Endo schrieb:
> > On Wed, 2012-09-05 at 14:39 -0400, DJ Delorie wrote:
> >> I don't feel the m32c change needs my specific ack, it's a harmless
> >> change that goes with the ack for the feature itself.
> >>
> >> However, I will note
On Thu, 6 Sep 2012, Iyer, Balaji V wrote:
> >Concepts such as "pentium_4" are architecture-specific and have no place in
> >front-end files. This whole mapping from one sort of string to another
> >belongs
> >within the back end.
>
> Please excuse me if I am "beating this horse to death." I am
On Thu, Sep 6, 2012 at 2:40 PM, Jack Howarth wrote:
> Okay for gcc trunk?
One typo:
> * configure.ac: Test for rdrnd support in assembler.
It's rdrand. I wouldn't be pedantic if the opcode wouldn't have
changed from rdrnd to rdrand at some point and using the old name
could be confusin
On Thu, 30 Aug 2012 12:48:34 +0200
Thiago Macieira wrote:
> Hello
>
> The attached patch is a simple improvement to make a thread that
> failed to set the waiting bit to exit the function earlier, if it
> detects that another thread has successfully finished initialising.
> It matches the CAS co
The attached patch eliminates the bootstrap failures in libstdc++-v3
of PR 54419 by having configure check for assembler support of the new
rdrnd opcode and defining _GLIBCXX_X86_RDRAND in config.h if supported.
Tested on x86_64-apple-darwin12 against the assembler from Xcode 4.4.1.
Okay for gc
>-Original Message-
>From: Joseph Myers [mailto:jos...@codesourcery.com]
>Sent: Thursday, September 06, 2012 12:18 PM
>To: Iyer, Balaji V
>Cc: gcc-patches@gcc.gnu.org; Aldy Hernandez (al...@redhat.com); Jeff Law;
>r...@redhat.com
>Subject: RE: [PATCH] Merging Cilk Plus into Trunk (Patch 1
Most bits are stolen from Linux, but there are a few subtle
differences since our assembler is configured to be slightly more
HP-UX-ish.
libgcc/:
2012-09-06 Mark Kettenis
* config.host (hppa-*-openbsd*): New target.
* config/pa/t-openbsd: New file.
gcc/:
2012-09-06 Mark Ke
Hello Richard,
I forgot to answer one of questions. Please see it below:
Thanks,
Balaji V. Iyer.
>+static tree
>+handle_vector_attribute (tree *node, tree name ATTRIBUTE_UNUSED,
>+ tree args ATTRIBUTE_UNUSED,
>+ int ARG_UNUSED (flags), bool *n
Doug was trying to build with gmp 4.2.1 and ran into this error:
gmp-4.2.1/include/gmp.h:515:12: error: 'std::FILE' has not been declared
This bug was fixed in gmp 4.2.3, so I've bumped the minimum
acceptable version in configure.ac
Tested on x86_64 with gmp 4.2.3. Committed to trunk.
Diego.
On 09/06/2012 12:18 PM, Paolo Carlini wrote:
On 09/06/2012 02:03 AM, Jason Merrill wrote:
but note that for:
template
struct A
{
int select() { return 0; }
};
we have parser->num_template_parameter_lists == 1 and num_templates ==
0. Thus it seems that the case 'num_templates + 1' isn't (j
> From: Richard Henderson [mailto:r...@redhat.com]
> On 09/06/2012 08:06 AM, Ian Bolton wrote:
> > If I don't use my split pattern, I could alter combine to remove the
> > requirement that parent is a MEM.
> >
> > What do you think?
>
> I merely question the calling out of CONST as special.
>
> E
On 06/09/12 12:29, Kyrylo Tkachov wrote:
> 2012-09-06 Kyrylo Tkachov
>
> * config/arm/arm.c: Use CONST_INT_P, CONST_DOUBLE_P, REG_P, MEM_P,
> LABEL_P, JUMP_P, CALL_P, NOTE_P, BARRIER_P consistently.
> * config/arm/arm.h: Use REG_P, MEM_P consistently.
> * co
On 06/09/12 17:07, Christophe Lyon wrote:
> On 6 September 2012 10:48, Richard Earnshaw wrote:
>> On 05/09/12 17:01, Christophe Lyon wrote:
>>
>> +(define_insn "*arm_revsh"
>> + [(set (match_operand:SI 0 "s_register_operand" "=r")
>> + (sign_extend:SI (bswap:HI (match_operand:HI 1 "s_regist
Sorry, I didn't see this message. Please see my responses below:
>-Original Message-
>From: Marc Glisse [mailto:marc.gli...@inria.fr]
>Sent: Thursday, September 06, 2012 7:04 AM
>To: gcc-patches@gcc.gnu.org
>Cc: Gabriel Dos Reis; Iyer, Balaji V; Aldy Hernandez (al...@redhat.com); Jeff
>La
On Thu, Sep 6, 2012 at 11:11 AM, Iyer, Balaji V wrote:
>>> On Wed, Sep 5, 2012 at 5:09 PM, Iyer, Balaji V
>>> wrote:
Let's say we have two for loops like this:
int my_func (int x, int y);
For (ii = 0; ii < 1; ii++)
X[ii] = my_func (Y[ii], Z[ii]);
>>
>>I
Hi,
On 09/06/2012 02:03 AM, Jason Merrill wrote:
but note that for:
template
struct A
{
int select() { return 0; }
};
we have parser->num_template_parameter_lists == 1 and num_templates ==
0. Thus it seems that the case 'num_templates + 1' isn't (just) about
member templates...
That's o
On Thu, 6 Sep 2012, Iyer, Balaji V wrote:
> Ok, I was mistaken there. I thought we had to add a changelog entry for
> every function and not every file. I will fix it in the updated patch I
> send soon.
For functions in existing files you do need to mention each function - but
not for new file
On Thu, Sep 6, 2012 at 10:51 AM, Richard Henderson wrote:
> On 09/06/2012 02:37 AM, Richard Guenther wrote:
>> In all this seems unrelated to CILK+ work (even if you make use of this
>> from within CILK+).
>
> While true, we also asked him to split up the work. And this piece,
> done correctly, s
Hello Marc,
Please see my response below.
Thanks for looking at my patch!
Sincerely,
Balaji V. Iyer.
>-Original Message-
>From: Marc Glisse [mailto:marc.gli...@inria.fr]
>Sent: Thursday, September 06, 2012 2:06 AM
>To: Gabriel Dos Reis
>Cc: Iyer, Balaji V; gcc-patches@gcc.gnu.o
On 09/05/2012 03:09 PM, Iyer, Balaji V wrote:
> If we annotate *both* the function declaration and the function with the
> following attribute, the compiler will create a vector and scalar version of
> the function.
>
> __attribute__((vector)) my_func (int x, int y);
>
> __attribute__((vector)
On 6 September 2012 10:48, Richard Earnshaw wrote:
> On 05/09/12 17:01, Christophe Lyon wrote:
>
> +(define_insn "*arm_revsh"
> + [(set (match_operand:SI 0 "s_register_operand" "=r")
> + (sign_extend:SI (bswap:HI (match_operand:HI 1 "s_register_operand"
> "r"]
> + "TARGET_32BIT && arm
Hello Joseph,
Thanks for reviewing my patch. Please see my responses below:
>-Original Message-
>From: Joseph Myers [mailto:jos...@codesourcery.com]
>Sent: Wednesday, September 05, 2012 8:07 PM
>To: Iyer, Balaji V
>Cc: gcc-patches@gcc.gnu.org; Aldy Hernandez (al...@redhat.com); Jef
On 09/06/2012 02:37 AM, Richard Guenther wrote:
> In all this seems unrelated to CILK+ work (even if you make use of this
> from within CILK+).
While true, we also asked him to split up the work. And this piece,
done correctly, seems useful even if the rest of cilk is ignored.
r~
The current test is failing on some architectures because the underlying
gimple has changed. I believe the best way to test the speculative
store data race is with the simulate-thread/ harness. We already have a
speculative store test in the harness, so this will be a nice addition.
I have m
Vtables were causing several different problems for constexpr:
1) Value-initializing a nearly-empty class (that has a vptr but no data)
meant two initializers for a single base. Fixed by not bothering to
zero out a type with no data before calling its constructor.
2) A primary base is alloca
On 09/06/2012 08:06 AM, Ian Bolton wrote:
> If I don't use my split pattern, I could alter combine to remove the
> requirement that parent is a MEM.
>
> What do you think?
I merely question the calling out of CONST as special.
Either you've got some pattern that handles SYMBOL_REF
the same way,
> On 2012-08-31 07:49, Ian Bolton wrote:
> > +(define_split
> > + [(set (match_operand:DI 0 "register_operand" "=r")
> > + (const:DI (plus:DI (match_operand:DI 1 "aarch64_valid_symref"
> "S")
> > + (match_operand:DI 2 "const_int_operand"
> "i"]
> > + ""
> > + [
On 06/09/12 15:31, James Greenhalgh wrote:
>
> Hi,
>
> This patch implements section anchors for the AArch64 port.
>
> OK for aarch64-branch?
>
> Regards,
> James Greenhalgh
>
> --
> 2012-09-06 James Greenhalgh
> Richard Earnshaw
>
> * common/config/aarch64/aarch64-common
Enable the raising of exception in long double soft float and support
for rounding mode.
2012-09-06 Marcus Shawcroft
* config/aarch64/sfp-machine.h (FP_EX_INVALID, FP_EX_DIVZERO)
(FP_EX_OVERFLOW, FP_EX_UNDERFLOW, FP_EX_INEXACT)
(FP_HANDLE_EXCEPTIONS, FP_RND_NEAREST, F
Hi,
This patch implements section anchors for the AArch64 port.
OK for aarch64-branch?
Regards,
James Greenhalgh
--
2012-09-06 James Greenhalgh
Richard Earnshaw
* common/config/aarch64/aarch64-common.c
(aarch_option_optimization_table): New.
(TARGET_OP
Relax the logic that prevents TFmode constants being addressed in the
constant pool.
2012-09-06 Marcus Shawcroft
* config/aarch64/aarch64.c (aarch64_classify_address):
Allow 16 byte modes in constant pool.
diff --git a/gcc/config/aarch64/aarch64.c b/gcc/config/aarch64/aarch6
Hi,
the following patch replaces #define IPA_MAX_AFF_JF_ITEMS with a new
parameter PARAM_IPA_MAX_AGG_ITEMS (yeah, without the type).
Bootstrapped and tested on x86_64-linux, I consider it obvious enough
that I will commit it next week if there are no objections.
Thanks,
Martin
2012-09-05 Mart
rs6000_debug_address_cost() needs to call TARGET_ADDRESS_COST with the
new arguments.
Committed.
- David
* config/rs6000/rs6000.c (rs6000_xcoff_asm_named_section): Add TLS
section.
* config/rs6000/rs6000.c (rs6000_debug_address_cost): Add new
arguments to TARGET_A
The following fixes PR54498 - we may not treat already visited
parts of the CFG as non-aliasing after re-writing the expression
we lookup. Just abort the walk in this case.
Bootstrapped and tested on x86_64-unknown-linux-gnu on the 4.7
branch, trunk testing in progress.
Richard.
2012-09-06 Ri
Oleg Endo schrieb:
On Wed, 2012-09-05 at 14:39 -0400, DJ Delorie wrote:
I don't feel the m32c change needs my specific ack, it's a harmless
change that goes with the ack for the feature itself.
However, I will note that m32c does have different costs for addresses
in different address spaces, a
On Thu, Aug 11, 2011 at 8:31 AM, Richard Guenther wrote:
>
> When adjusting the C++ FE code generation for base type copies
> I noticed the XFAILs on g++.dg/tree-ssa/pr41186.C and investigated
> things a bit. We can fix the testcase easily iff the frontend
> emits the MEM_REF variants, thus the f
This removes the MOVE_NONTEMPORAL tree flag which is nowhere set.
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied.
Richard.
2012-09-06 Richard Guenther
* tree.h (MOVE_NONTEMPORAL): Remove.
* tree-pretty-print.c (dump_generic_node): Remove
MOVE_NONTEMPORA
On Thu, 6 Sep 2012, Marc Glisse wrote:
AFAIU, my_func is defined in a separate library and because of the attribute
on the definition, it will actually export overloads:
int myfunc(int,int);
v2si myfunc(v2si,v2si);
v4si myfunc(v4si,v4si);
etc (where does it stop? seems problematic if the librar
Thank you for the comments.
> -Original Message-
> From: Eric Botcazou [mailto:ebotca...@adacore.com]
> Sent: Wednesday, September 05, 2012 9:39 PM
> To: Zhenqiang Chen
> Cc: gcc-patches@gcc.gnu.org
> Subject: Re: [PATCH] Enable bbro for -Os
>
> > Basic block reordering is disabled for -O
On Thu, Sep 6, 2012 at 1:06 AM, Marc Glisse wrote:
> On Wed, 5 Sep 2012, Gabriel Dos Reis wrote:
>
>> On Wed, Sep 5, 2012 at 5:09 PM, Iyer, Balaji V
>> wrote:
>>>
>>> Let's say we have two for loops like this:
>>>
>>> int my_func (int x, int y);
>>>
>>> For (ii = 0; ii < 1; ii++)
>>>
On Wed, Aug 22, 2012 at 1:54 AM, Dehao Chen wrote:
> On Tue, Aug 21, 2012 at 6:25 AM, Richard Guenther
> wrote:
>> On Mon, Aug 20, 2012 at 3:18 AM, Dehao Chen wrote:
>>> ping
>>
>> Conceptually I like the change. Can a libcpp maintainer please have a 2nd
>> look?
>>
>> Dehao, did you do any
On 6 Sep 2012, at 11:18, domi...@lps.ens.fr (Dominique Dhumieres) wrote:
Could you please commit this (I can't at the moment)?
Sorry, I don't have write access.
OK, then I will commit it later today.
Cheers,
Oleg
On Thu, Sep 6, 2012 at 8:06 AM, Marc Glisse wrote:
> On Wed, 5 Sep 2012, Gabriel Dos Reis wrote:
>
>> On Wed, Sep 5, 2012 at 5:09 PM, Iyer, Balaji V
>> wrote:
>>>
>>> Let's say we have two for loops like this:
>>>
>>> int my_func (int x, int y);
>>>
>>> For (ii = 0; ii < 1; ii++)
>>>
On 06/09/12 09:33, Bin Cheng wrote:
>>>
>>> Yes, it may be feasible to rewrite the instruction in machine reorg
>>> pass, rather than peephole2. But that need bigger change in ARM back
> end.
>>> Hi Ramana, Richard, what's your opinion on this?
>>>
>>> Thanks very much.
>>>
>>>
>>
>> I side with Ri
On Wed, Sep 5, 2012 at 10:40 PM, Steven Bosscher wrote:
> Hi,
>
> bitmap.c:bitmap_last_set_bit() is not used by any code in the current
> GCC trunk, but I'm using it and I noticed it returns an incorrect
> result. This patch rewrites most of the function to return the correct
> result.
>
> Not sur
> Could you please commit this (I can't at the moment)?
Sorry, I don't have write access.
Dominique
On Wed, Sep 5, 2012 at 10:34 PM, Steven Bosscher wrote:
> Hello,
>
> Just some cleanups I did while working on something bigger.
>
> OK for trunk?
Ok.
Thanks,
Richard.
> Ciao!
> Steven
>
> * graphite.c (print_global_statistics): Use EDGE_COUNT instead
> of VEC_length.
>
On Thu, 2012-09-06 at 09:49 +0200, Dominique Dhumieres wrote:
> Oleg,
>
> Bootstrap fails at revision 190996 on powerpc-apple-darwin9 with:
>
> g++ -c -g -DIN_GCC -fno-exceptions -fno-rtti -W -Wall -Wno-narrowing
> -Wwrite-strings -Wcast-qual -Wmissing-format-attribute -pedantic
> -Wno-long
This fixes -fno-tree-pta - noticed it doesn't work when trying
to compile a testcase where PTA uses too much memory.
Bootstrapped & tested on x86_64-unknown-linux-gnu, applied.
Richard.
2012-09-06 Richard Guenther
* passes.c (execute_function_todo): Call compute_may_aliases
On Wed, 5 Sep 2012, Lawrence Crowl wrote:
> On 9/5/12, Richard Guenther wrote:
> > On Tue, 4 Sep 2012, Lawrence Crowl wrote:
> >> Modify gcc/*.[hc] double_int call sites to use the new interface.
> >> This change entailed adding a few new methods to double_int.
> >>
> >> Other changes will happen
On 05/09/12 17:01, Christophe Lyon wrote:
> Hi,
>
> This patch implements __builtin_bswap16() on ARM (v6 and above) using
> revsh with a signed input and rev16 with an unsigned input.
>
> It is pretty much equal to the patch posted some time ago
> http://gcc.gnu.org/ml/gcc-patches/2012-04/msg0096
> >
> > Yes, it may be feasible to rewrite the instruction in machine reorg
> > pass, rather than peephole2. But that need bigger change in ARM back
end.
> > Hi Ramana, Richard, what's your opinion on this?
> >
> > Thanks very much.
> >
> >
>
> I side with Richard on this one. The mid-end should
On 06/09/12 06:41, Bin Cheng wrote:
> Hi Richard,
> Thanks very much for comments.
>
>>> Ping?
>>>
>>> Hi Ramana, could you help me review this patch?
>>> Hi Eric, Richard, could you help me review the change in regcprop.c?
>>
>> Subtraction of zero isn't canonical rtl though. Passes after peeph
On Thu, Sep 06, 2012 at 12:50:44AM -0700, Andrew Pinski wrote:
> > Patch preapproved, but you've attached a different patch.
>
> Sorry about that. Here is the correct one.
>
> Also is this ok for the 4.7 branch?
Yes, thanks.
> --- tree-inline.c (revision 191004)
> +++ tree-inline.c (wo
On Thu, Sep 6, 2012 at 12:07 AM, Jakub Jelinek wrote:
> On Wed, Sep 05, 2012 at 09:10:53PM -0700, Andrew Pinski wrote:
>> The inlininer likes to recreate some MEM_REF, it copies most of the
>> bits (TREE_THIS_NOTRAP, TREE_THIS_VOLATILE, etc.) but forgets about
>> TREE_SIDE_EFFECTS. This causes
Oleg,
Bootstrap fails at revision 190996 on powerpc-apple-darwin9 with:
g++ -c -g -DIN_GCC -fno-exceptions -fno-rtti -W -Wall -Wno-narrowing
-Wwrite-strings -Wcast-qual -Wmissing-format-attribute -pedantic -Wno-long-long
-Wno-variadic-macros -Wno-overlength-strings -fno-common -DHAVE_CONFI
On Wed, Sep 05, 2012 at 09:10:53PM -0700, Andrew Pinski wrote:
> The inlininer likes to recreate some MEM_REF, it copies most of the
> bits (TREE_THIS_NOTRAP, TREE_THIS_VOLATILE, etc.) but forgets about
> TREE_SIDE_EFFECTS. This causes the strlen optimization to think the
> memory store does not
91 matches
Mail list logo
