On 06/07/2011 06:37 AM, Douglas B Rupp wrote:
Revised and retested patch attached.
OK to commit?
It should be enough to add AC_USE_SYSTEM_EXTENSIONS to configure.ac
instead (right after AC_PROG_CC).
Paolo
On 7 June 2011 06:50, Basile Starynkevitch wrote:
> Also search short plugins also in a front-end
> specific sub-directory.
That doesn't look like an improvement :)
> I checked on i686-darwin9 with both m32 and m32/abi=1 the only difference I
> see is the XPASSes
> for m64 torture/forward-1.m for gnu runtime.
That is very comforting; I have committed the (revised) patch.
Thanks a lot for your help testing and reviewing. :-)
> I will try to test on darwin8 d
On Mon, 6 Jun 2011, Diego Novillo wrote:
> On Mon, Jun 6, 2011 at 10:27, Diego Novillo wrote:
> > On Mon, Jun 6, 2011 at 04:50, Richard Guenther wrote:
> >
> >> I'd have it done in the loop that computes canonical types, at this
> >> place we do not gain the advantage that the decl register func
Hi,
sanity checked x86_64-linux, committed to mainline.
Paolo.
///
2011-06-07 Paolo Carlini
PR libstdc++/49293
* testsuite/22_locale/time_get/get_weekday/char/38081-1.cc: Tweak
for glibc 2.14.
* testsuite/22_locale/time_get/get_weekday/char/38
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
Hi Tobias,
On 06/06/11 23:40, Tobias Burnus wrote:
> This patch adds incomplete parsing support for the LOCK and UNLOCK
> statement. Missing part 2 is the addition of the LOCK_TYPE of the
> ISO_FORTRAN_ENV.
>
> Build and tested on x86-64-linux.
> OK
On Mon, Jun 6, 2011 at 6:20 PM, Xinliang David Li wrote:
>>
>> Your patch doesn't really improve this but adds to the confusion.
>>
>> + /* Override dump TODOs. */
>> + if (dump_file && (pass->todo_flags_finish & TODO_dump_func)
>> + && (dump_flags & TDF_BEFORE))
>> + {
>> + pass->
On Mon, Jun 6, 2011 at 7:58 PM, Jakub Jelinek wrote:
> Hi!
>
> When an indirect noreturn call has a likely call target, value-prof.c
> ICEs on it, as there is no fallthru edge after the call (nor a join bb).
> Fixed thusly, bootstrapped/regtested on x86_64-linux and i686-linux,
> ok for trunk/4.6?
Hi Carrot,
ChangeLog:
2011-03-29 Wei Guozhi
PR target/45335
* gcc/config/arm/ldmstm.md (ldm2_ia, stm2_ia, ldm2_ib, stm2_ib, ldm2_da,
stm2_da, ldm2_db, stm2_db): Add condition !arm_arch7 to these insns.
(ldrd, ldrd_reg1, ldrd_reg2 and peephole2): New insn patte
On Mon, Jun 6, 2011 at 8:07 PM, William J. Schmidt
wrote:
> It's been some time since I last posted about the address lowering issue from
> PR46556 (http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46556). I've had a basic
> prototype in place for some time, but I ran into some issues that initially
>
On Mon, Jun 6, 2011 at 8:59 PM, William J. Schmidt
wrote:
> Hi Richard,
>
> Here's the reworked patch addressing the noted concerns. Regtested for
> powerpc-linux. OK for trunk?
Ok with ...
> Thanks,
> Bill
>
>
> 2011-06-06 Bill Schmidt
>
> PR tree-optimization/46728
> * tree-
On Mon, Jun 6, 2011 at 6:00 PM, Xinliang David Li wrote:
> On Mon, Jun 6, 2011 at 4:38 AM, Richard Guenther
> wrote:
>> On Thu, Jun 2, 2011 at 9:12 AM, Xinliang David Li wrote:
>>> This is the version of the patch that walks through pass lists.
>>>
>>> Ok with this one?
>>
>> +/* Dump all optimi
On Mon, Jun 6, 2011 at 9:20 PM, Xinliang David Li wrote:
> Please take a look at the revised one.
Ok.
Thanks,
Richard.
> Thanks,
>
> David
>
> On Mon, Jun 6, 2011 at 4:22 AM, Richard Guenther
> wrote:
>> On Wed, Jun 1, 2011 at 7:24 PM, Xinliang David Li wrote:
>>> The attached is the split #1
On Tue, Jun 7, 2011 at 7:39 AM, Jason Merrill wrote:
> In the testcase, fold_indirect_ref_1 won't fold *(T*)(s1+10) to an ARRAY_REF
> because T != unsigned. Even if it were just a typedef to unsigned, that
> isn't close enough, but in this case it's a typedef to const unsigned.
>
> I'm not sure w
On Tue, Jun 7, 2011 at 8:31 AM, Jakub Jelinek wrote:
> Hi!
>
> While for the trunk I hope Michael will finalize a much better fix,
> this patch provides a quick workaround for 4.6 branch.
>
> In particular, I'd like to avoid reverting the
> http://gcc.gnu.org/ml/gcc-patches/2011-01/msg01442.html
>
On Tue, Jun 07, 2011 at 12:19:59PM +0200, Richard Guenther wrote:
> On Tue, Jun 7, 2011 at 7:39 AM, Jason Merrill wrote:
> > In the testcase, fold_indirect_ref_1 won't fold *(T*)(s1+10) to an ARRAY_REF
> > because T != unsigned. Even if it were just a typedef to unsigned, that
> > isn't close eno
On Jun 6, 2011, Richard Guenther wrote:
>> You meant 4.6 stage1, but I missed it. How's it for 4.7 stage1?
>> Regstrapped on x86_64-linux-gnu and i686-linux-gnu.
> Isn't exactly ICEing for num_ssa_operands/delink_stmt_imm_use.
Uhh. Looks like I didn't update the patch per your comments befor
Hi Carrot,
2011-05-06 Guozhi Wei
PR target/47855
* config/arm/thumb2.md (thumb2_movsi_insn): Add length addtribute.
(thumb2_shiftsi3_short and peephole2): Remove 3-register case.
(thumb2_cbz): Refine length computation.
(thumb2_cbnz): Likewise.
Not a
On 02/06/11 10:46, Richard Earnshaw wrote:
OK.
Committed, thanks.
Andrew
On 02/06/11 16:47, Richard Earnshaw wrote:
OK.
Committed, thanks.
Andrew
Hi Carrot,
ChangeLog:
2011-05-16 Wei Guozhi
PR target/46975
* config/arm/arm.md (*addsi3_carryin_compare0_): New pattern.
(peephole2 for conditional move): Generate 16 bit instructions.
ChangeLog:
2010-05-16 Wei Guozhi
PR target/46975
* gcc.target/arm/
On Sun, 29 May 2011, Gerald Pfeifer wrote:
> ...at least having the testsuite and Java separate makes a lot of sense.
>
> The vast majority of users does not need the former and the latter is
If building GCC from source, running tests is generally considered a good
idea...
> on the verge of be
On Sun, 29 May 2011, Richard Guenther wrote:
> For libjava I would argue to split out only libjava/classpath which
> is what we drop in (in a slightly modified form?) from elsewhere.
> So splitting that would eventually make sense (maybe even
> with making java also compile w/o that classpath dir)
On Sun, 29 May 2011, Kaz Kojima wrote:
> Hi,
>
> Rainer Orth wrote:
> > 2010-12-30 Uros Bizjak
> > Rainer Orth
> >
> > * gcc.dg/torture/tls/run-gd.c: New test.
> > * gcc.dg/torture/tls/run-ie.c: New test.
> > * gcc.dg/torture/tls/run-ld.c: New test.
> > * gcc.dg/tort
On Tue, Jun 7, 2011 at 12:27 PM, Jakub Jelinek wrote:
> On Tue, Jun 07, 2011 at 12:19:59PM +0200, Richard Guenther wrote:
>> On Tue, Jun 7, 2011 at 7:39 AM, Jason Merrill wrote:
>> > In the testcase, fold_indirect_ref_1 won't fold *(T*)(s1+10) to an
>> > ARRAY_REF
>> > because T != unsigned. Ev
On Tue, Jun 7, 2011 at 12:38 PM, Alexandre Oliva wrote:
> On Jun 6, 2011, Richard Guenther wrote:
>
>>> You meant 4.6 stage1, but I missed it. How's it for 4.7 stage1?
>>> Regstrapped on x86_64-linux-gnu and i686-linux-gnu.
>
>> Isn't exactly ICEing for num_ssa_operands/delink_stmt_imm_use.
>
>
On Tue, 7 Jun 2011, Richard Guenther wrote:
> On Tue, Jun 7, 2011 at 12:27 PM, Jakub Jelinek wrote:
> > On Tue, Jun 07, 2011 at 12:19:59PM +0200, Richard Guenther wrote:
> >> On Tue, Jun 7, 2011 at 7:39 AM, Jason Merrill wrote:
> >> > In the testcase, fold_indirect_ref_1 won't fold *(T*)(s1+10)
On Mon, Jun 06, 2011 at 10:27:51AM -0300, Alexandre Oliva wrote:
> On Jun 6, 2011, Eric Botcazou wrote:
>
> >> It might be too late for DF to do anything sensible to preserve the
> >> debug info rather than just throw it away, as your partial approval
> >> suggests.
>
> > OK, let me think about
On 06/07/11 07:11, Nathan Sidwell wrote:
> On 06/06/11 17:27, Richard Earnshaw wrote:
>
>> Eh? This is backwards. There is blx , but no bl . If the assembler
>> gets confused with 'bl r0' then it needs to be fixed urgently.
>
> Are you requiring the assembler be fixed before this patch can be com
On Thu, 26 May 2011, Richard Guenther wrote:
>
> In preparation to move all frontends common tree node building to
> the middle-end this makes the Java frontend actually use the
> standard routines. It's still severely broken in overriding
> things with stuff that does not match the targets C AB
"Joseph S. Myers" wrote:
>> gcc.dg/torture/tls/run-gd.c fails on SH because SH assumes
>> -fpic for global and local dynamic tls models. Although a line
>
> That sounds like an SH target bug to me.
Ok, I'd like to fix the SH backend so that those tests pass
without -fpic/-fpie.
Regards,
On Sun, Jun 5, 2011 at 1:46 PM, Uros Bizjak wrote:
> Hello!
>
> 2011-06-05 Uros Bizjak
>
> * config/i386/i386.md (*movdf_internal_rex64) :
> Remove MODE_TI handling. Remove SSE1 handling in attribute "mode"
> calculation.
> (*movdf_internal_rex64) : Remove MODE_TI h
Hi,
On Tue, 7 Jun 2011, Richard Guenther wrote:
> > > fold_convert_loc it to the expected type, while the middle-end has
> > > the notion of useless type conversions, fold-const.c is also used by
> > > FEs and I think it is expected to have the types exactly matching.
> > > So (T)s1[10] instea
On Mon, 30 May 2011, Vladimir Makarov wrote:
> * doc/tm.texi.in (TARGET_REGISTER_BANK): New hook.
Please include the text of documentation for new hooks in target.def
instead of tm.texi.in where possible (so tm.texi.in just has the @hook
line) - that is, unless it's based on older GFDL-on
On Tue, 7 Jun 2011, Michael Matz wrote:
> Hi,
>
> On Tue, 7 Jun 2011, Richard Guenther wrote:
>
> > > > fold_convert_loc it to the expected type, while the middle-end has
> > > > the notion of useless type conversions, fold-const.c is also used by
> > > > FEs and I think it is expected to have
Hi,
On Tue, 7 Jun 2011, Richard Guenther wrote:
> > + tree base = get_base_address (lhs);
>
> Probably easier and more complete to do
>
> if (lhs && TREE_CODE (lhs) != SSA_NAME)
> {
> tree base = get_base_address (lhs);
>
>
> I don't like the
On 06/07/2011 06:19 AM, Richard Guenther wrote:
I _think_ that you can unconditionally change the code to do
TYPE_MAIN_VARIANT (t1) == TYPE_MAIN_VARIANT (t2)
&& TYPE_QUALS (t1) == TYPE_QUALS (t2)
now, I'm not sure if for the testcase T and unsigned differ in qualifiers.
Do they?
Hmm, I
Hi Guys,
I am applying the patch below to tidy up the V850 architecture
selection in config.gcc and to add support for newer V850
architectures (eg V850e2v3).
Cheers
Nick
gcc/ChangeLog
2011-06-07 Nick Clifton
* config.gcc: Unify V850 architecture options and add support for
On Tue, 31 May 2011, Paolo Bonzini wrote:
> > I'd thought about it, but refrained since HAVE_ENABLE_EXECUTE_STACK
> > affects only three cpus. Currently, our documentation of libgcc
> > configury and macros used is close to non-existant. That's probably for
> > someone who has implemented this s
On Tue, Jun 7, 2011 at 3:42 PM, H.J. Lu wrote:
>> 2011-06-05 Uros Bizjak
>>
>> * config/i386/i386.md (*movdf_internal_rex64) :
>> Remove MODE_TI handling. Remove SSE1 handling in attribute "mode"
>> calculation.
>> (*movdf_internal_rex64) : Remove MODE_TI handling.
On Tue, Jun 7, 2011 at 3:55 PM, Jason Merrill wrote:
> On 06/07/2011 06:19 AM, Richard Guenther wrote:
>>
>> I _think_ that you can unconditionally change the code to do
>>
>> TYPE_MAIN_VARIANT (t1) == TYPE_MAIN_VARIANT (t2)
>> && TYPE_QUALS (t1) == TYPE_QUALS (t2)
>>
>> now, I'm not sure if
On Tue, Jun 7, 2011 at 7:02 AM, Uros Bizjak wrote:
> On Tue, Jun 7, 2011 at 3:42 PM, H.J. Lu wrote:
>
>>> 2011-06-05 Uros Bizjak
>>>
>>> * config/i386/i386.md (*movdf_internal_rex64) :
>>> Remove MODE_TI handling. Remove SSE1 handling in attribute "mode"
>>> calculation.
On Tue, Jun 7, 2011 at 4:07 PM, H.J. Lu wrote:
>> OTOH, you are right, this statement can be moved to case 6, without
>> the check for registers.
>>
>
> Like this? OK for trunk with a ChangeLog entry?
>
> Thanks.
>
> --
> H.J.
>
> diff --git a/gcc/config/i386/i386.md b/gcc/config/i386/i386.
On 06/07/2011 10:05 AM, Richard Guenther wrote:
In that case you could do what Jakub suggested - but only for rvalues
of course.
Right, and I need to handle lvalues as well.
I'm not sure if we already avoid calling the folding where we
require lvalues.
No, we don't.
Can't you instead adju
Hans-Peter Nilsson writes:
> I needed a baseline for PR48542 (problems with setjmp/longjmp), but
> building MMIX was broken on trunk. It was "bitten" by the fixups to
> predicate matching done recentlish. The comment above the MMIX
> truncdfsf2 expander complains about having to use an expander
On 05/29/2011 02:07 AM, Gerald Pfeifer wrote:
>
> The vast majority of users does not need the former and the latter is
> on the verge of becoming practically irrelevant. As one datapoint, the
> entire FreeBSD Ports Collection has a single(!) port relying on GCJ.
It's not quite as irrelevant as
On Tue, 2011-06-07 at 12:06 +0200, Richard Guenther wrote:
> On Mon, Jun 6, 2011 at 8:07 PM, William J. Schmidt
> wrote:
> > * If the original expression will be recognized as a "hidden global store"
> > in
> > tree-ssa-sink.c:is_hidden_global_store, but the replacement expression
> > wil
Now that there is a single place left to call set_sizetype we can
remove it and initialize sizetypes properly from the start
(in initialize_sizetypes).
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk.
Richard.
2011-06-07 Richard Guenther
* stor-layout.c (initia
On Tue, Jun 7, 2011 at 4:14 PM, William J. Schmidt
wrote:
>
> On Tue, 2011-06-07 at 12:06 +0200, Richard Guenther wrote:
>> On Mon, Jun 6, 2011 at 8:07 PM, William J. Schmidt
>> wrote:
>
>
>
>> > * If the original expression will be recognized as a "hidden global
>> > store" in
>> > tree-ssa
On Tue, Jun 7, 2011 at 7:12 AM, Uros Bizjak wrote:
> On Tue, Jun 7, 2011 at 4:07 PM, H.J. Lu wrote:
>
>>> OTOH, you are right, this statement can be moved to case 6, without
>>> the check for registers.
>>>
>>
>> Like this? OK for trunk with a ChangeLog entry?
>>
>> Thanks.
>>
>> --
>> H.J.
>> -
"Joseph S. Myers" writes:
> On Tue, 31 May 2011, Paolo Bonzini wrote:
>
>> > I'd thought about it, but refrained since HAVE_ENABLE_EXECUTE_STACK
>> > affects only three cpus. Currently, our documentation of libgcc
>> > configury and macros used is close to non-existant. That's probably for
>> >
On Fri, 3 Jun 2011, Rainer Orth wrote:
> * The gcc/unwind* files and gcc/emutls.c can move, too.
And the other files listed in LIB2ADDEH for various targets, I expect.
arm/libunwind.S
arm/pr-support.c
arm/unwind-arm.c
arm/unwind-arm.h
ia64/fde-vms.c
ia64/unwind-ia64.c
ia64/unwind-ia64.h
rs6000/d
A bug report on IRC pointed out that we were giving the -Wabi warning in
cases that don't affect the ABI at all, namely use of scoped enums in
switch. So this patch limits the warning to the varargs case by
catching scoped enums in perform_integral_promotions so that only
callers that use the
"Joseph S. Myers" writes:
> On Fri, 3 Jun 2011, Rainer Orth wrote:
>
>> * The gcc/unwind* files and gcc/emutls.c can move, too.
>
> And the other files listed in LIB2ADDEH for various targets, I expect.
>
> arm/libunwind.S
> arm/pr-support.c
> arm/unwind-arm.c
> arm/unwind-arm.h
> ia64/fde-vms.c
On Mon, 6 Jun 2011, Rainer Orth wrote:
> Paolo Bonzini writes:
>
> >> * Except for Darwin, the code uses TRAMPOLINE_SIZE. This only exists in
> >>the backend headers. While it could be duplicated somewhere in the
> >>libgcc configury, I'd rather propose that gcc define
> >>__TRAMPO
"Joseph S. Myers" writes:
> On Mon, 6 Jun 2011, Rainer Orth wrote:
>
>> Paolo Bonzini writes:
>>
>> >> * Except for Darwin, the code uses TRAMPOLINE_SIZE. This only exists in
>> >>the backend headers. While it could be duplicated somewhere in the
>> >>libgcc configury, I'd rather prop
On 06/07/2011 05:30 PM, Rainer Orth wrote:
> So my view is that you should define __LIBGCC_TRAMPOLINE_SIZE__, only if
> -fbuilding-libgcc.
I can give it a try if I can figure out how to define -fbuilding-libgcc
via the option handling machinery. I just want to avoid having to
implement too ma
On Sun, 5 Jun 2011, H.J. Lu wrote:
> * config.gcc: Support --enable-x32/--enable-ia32 for x86 Linux
> targets.
>
> * configure.ac: Support --enable-x32/--enable-ia32.
> * configure: Regenerated.
New configure options need documenting in install.texi.
> #undef ASM_SPEC
>
An additional comment on the specs: OPTION_DEFAULT_SPECS needs updating
(-mx32 should use the --with-arch-64 etc. settings).
--
Joseph S. Myers
jos...@codesourcery.com
On 06/06/2011 04:17 AM, Bruce Korb wrote:
It also caused a code analysis tool to wander off into the weeds.
And "x, break;" didn't? :)
(First time I see it, I swear).
Paolo
Hi,
the below starts using it, for correctness wrt move constructors which
can throw. For sure we are missing the optimization which uses the
default-constructor + swap when the type isn't nothrow move
constructible, per Note 5 in n3050, for example.
Tested x86_64-linux, committed.
Paolo.
On Tue, Jun 7, 2011 at 2:36 AM, Richard Guenther
wrote:
> On Mon, Jun 6, 2011 at 6:20 PM, Xinliang David Li wrote:
>>>
>>> Your patch doesn't really improve this but adds to the confusion.
>>>
>>> + /* Override dump TODOs. */
>>> + if (dump_file && (pass->todo_flags_finish & TODO_dump_func)
>>
Hi,
the way we propagate flags in SRA access tree has evolved in an
unfortunate way, this patch simplifies the whole thing while doing
exactly the same thing.
Bootstrapped and tested on x86_64-linux, OK for trunk?
Thanks,
Martin
2011-06-06 Martin Jambor
* tree-sra.c (mark_rw_stat
Ok -- that sounds good.
David
On Tue, Jun 7, 2011 at 3:10 AM, Richard Guenther
wrote:
> On Mon, Jun 6, 2011 at 6:00 PM, Xinliang David Li wrote:
>> On Mon, Jun 6, 2011 at 4:38 AM, Richard Guenther
>> wrote:
>>> On Thu, Jun 2, 2011 at 9:12 AM, Xinliang David Li
>>> wrote:
This is the ver
On Tue, Jun 7, 2011 at 02:36, Richard Guenther
wrote:
>> For one thing, you need to either remember what is the previous pass,
>> or dump all passes which for large files can take very long time. Even
>> with all the dumps, you will need to eyeballing to find the previous
>> pass which may or may
Jakub,
any word on this patch? I think I only need approval for the gcc.c
part.
Thanks.
Rainer
Rainer Orth writes:
> Jakub Jelinek writes:
>
>> On Fri, Jun 03, 2011 at 09:38:31PM +0200, Rainer Orth wrote:
>>> Rainer Orth writes:
>>> Jakub, any suggestion how to properly test for -
Any suggestions on the dump position specification string, before and
after is not enough. How about
start, before, after, and finish?
I.e.
-fdump-tree-pre-start --> dump IR before TODO_start of PRE pass
-fdump-tree-pre-before --> dump IR just before PRE after its TODO start finishes
-fdump-tree
On Tue, Jun 07, 2011 at 06:48:08PM +0200, Rainer Orth wrote:
> any word on this patch? I think I only need approval for the gcc.c
I'm not a maintainer of gcc.c, and I think it is a bad idea.
PIE is just a (slightly) more secure form of an executable, therefore
if a target doesn't support positio
Jakub Jelinek writes:
> On Tue, Jun 07, 2011 at 06:48:08PM +0200, Rainer Orth wrote:
>> any word on this patch? I think I only need approval for the gcc.c
>
> I'm not a maintainer of gcc.c, and I think it is a bad idea.
> PIE is just a (slightly) more secure form of an executable, therefore
> i
On Tue, Jun 7, 2011 at 09:51, Xinliang David Li wrote:
> Any suggestions on the dump position specification string, before and
> after is not enough. How about
>
> start, before, after, and finish?
>
> I.e.
>
> -fdump-tree-pre-start --> dump IR before TODO_start of PRE pass
> -fdump-tree-pre-befor
On Tue, Jun 7, 2011 at 10:01 AM, Diego Novillo wrote:
> On Tue, Jun 7, 2011 at 09:51, Xinliang David Li wrote:
>> Any suggestions on the dump position specification string, before and
>> after is not enough. How about
>>
>> start, before, after, and finish?
>>
>> I.e.
>>
>> -fdump-tree-pre-start
Ping.
Diego, David,
Is this patch OK for google/main?
-Easwaran
On Thu, Jun 2, 2011 at 4:48 PM, Easwaran Raman wrote:
> Ping.
>
> On Sat, May 14, 2011 at 8:01 AM, Easwaran Raman wrote:
>> http://gcc.gnu.org/ml/gcc-patches/2011-05/msg00781.html
>>
>
On Tue, Jun 7, 2011 at 10:11 AM, Easwaran Raman wrote:
> Ping.
>
> Diego, David,
> Is this patch OK for google/main?
Yes -- but I would like to see it in trunk too if possible -- both
llvm and icc do very good job in these cases.
Thanks,
David
>
> -Easwaran
>
> On Thu, Jun 2, 2011 at 4:48 PM,
On Tue, Jun 7, 2011 at 10:06, Xinliang David Li wrote:
> On Tue, Jun 7, 2011 at 10:01 AM, Diego Novillo wrote:
>> On Tue, Jun 7, 2011 at 09:51, Xinliang David Li wrote:
>>> Any suggestions on the dump position specification string, before and
>>> after is not enough. How about
>>>
>>> start, bef
On Fri, 3 Jun 2011, Bernd Schmidt wrote:
> >> Ok. Although I wonder how sel-sched can end up reusing an entry in
> >> h_d_i_d?
Unlike Haifa scheduler, we recompute INSN_LUIDs for each region. However, we
call sched_deps_{init,finish} once per function (like Haifa) and that makes us
reuse entri
On 06/07/11 21:39:57, Alexander Monakov wrote:
> 2011-06-07 Alexander Monakov
>
> * sel-sched.c (move_op): Use correct type for 'res'. Verify that
> code_motion_path_driver returned 0 or 1.
> (sel_region_finish): Clear h_d_i_d.
Alexander, will this patch fix the recently rep
On Mon, Jun 6, 2011 at 16:12, Gabriel Charette wrote:
> Ok removed the tests as well.
>
> Executed the full test suite with no errors (except for currently known
> ones..)
Thanks. Committed as rev 174761.
Diego.
After getting new failures due to an unrelated fix, I think this will
be more trouble than it's worth.
First, we can't get rid of the XPASSes, so those will always be noisy.
Second, some XPASSes will need to be unmarked because we just fixed
the underlying problem.
Third, we are at such an early
We need to stream TREE_TYPE for identifier node.
This fixes some ICEs, but introduces some new assembly mismatch errors.
Here is the testing diff:
47,49d46
< XPASS: g++.dg/pph/x1autometh.cc -fpph-map=pph.map -I. (test for bogus
messages, line )
< XPASS: g++.dg/pph/x1autometh.cc -fpph-map=pph.
On Tue, Jun 7, 2011 at 8:47 AM, Joseph S. Myers wrote:
> On Sun, 5 Jun 2011, H.J. Lu wrote:
>
>> * config.gcc: Support --enable-x32/--enable-ia32 for x86 Linux
>> targets.
>>
>> * configure.ac: Support --enable-x32/--enable-ia32.
>> * configure: Regenerated.
>
> New configu
On Tue, Jun 7, 2011 at 10:09 PM, Gary Funck wrote:
> On 06/07/11 21:39:57, Alexander Monakov wrote:
>> 2011-06-07 Alexander Monakov
>>
>> * sel-sched.c (move_op): Use correct type for 'res'. Verify that
>> code_motion_path_driver returned 0 or 1.
>> (sel_region_finish): Clear
Please review the attached two patches.
In the first patch, gate functions are cleaned up. All the per
function legality checks are moved into the executor and the
optimization heuristic checks (optimize for size) remain in the
gators. These allow the the following overriding order:
common fl
32 --with-arch-32=atom --with-cpu-64=corei7
--enable-cloog-backend=isl --with-ppl-include=/opt/gnu/include
--with-ppl-lib=/opt/gnu/lib64 --with-cloog-include=/opt/gnu/include
--with-cloog-lib=/opt/gnu/lib64 --with-fpmath=sse
Thread model: posix
gcc version 4.7.0 20110607 (experimen
On Tue, 7 Jun 2011, Rainer Orth wrote:
> Right, my plan was to move all files only referenced by LIB2ADDEH*.
>
> Does the plan seem ok otherwise?
Yes, it seems plausible. Thanks for working on this transition.
--
Joseph S. Myers
jos...@codesourcery.com
On Tue, 7 Jun 2011, Rainer Orth wrote:
> > So my view is that you should define __LIBGCC_TRAMPOLINE_SIZE__, only if
> > -fbuilding-libgcc.
>
> I can give it a try if I can figure out how to define -fbuilding-libgcc
> via the option handling machinery. I just want to avoid having to
That should
On Tue, 7 Jun 2011, H.J. Lu wrote:
> I thought about it when I started working on it. But I couldn't find a way
> to do it properly. What we have are
>
> #if TARGET_64BIT_DEFAULT
> #define SPEC_32 "m32"
> #define SPEC_64 "!m32"
> #else
> #define SPEC_32 "!m64"
> #define SPEC_64 "m64"
> #endif
>
On Tue, 7 Jun 2011, H.J. Lu wrote:
> That is -mx32 takes the same default as -m64. It is the correct
> behavior for -mx32.
I think it's only right in the case where -m64 is the default, and the
specs for the case where -m32 is the default need changing to handle -mx32
properly.
--
Joseph S. M
Hi,
I checked this patch into x32 branch.
H.J.
commit d41e1fd1c4d8114532152647e5fc5b6bc3bce5dd
Author: H.J. Lu
Date: Tue Jun 7 11:39:27 2011 -0700
Document --enable-ia32 and --enable-x32.
diff --git a/gcc/ChangeLog.x32 b/gcc/ChangeLog.x32
index 06d931a..3ce5fa1 100644
--- a/gcc/ChangeLog
Hi,
I checked this patch into x32 branch.
H.J.
---
commit e8fa82850eaa92e660e06d4a939a0cc313b5aa3e
Author: H.J. Lu
Date: Tue Jun 7 11:44:34 2011 -0700
Use @option{} on -mx32, -m64 and -mx32.
diff --git a/gcc/ChangeLog.x32 b/gcc/ChangeLog.x32
index 3ce5fa1..5655c61 100644
--- a/gcc/Change
This patch (prepared by me and Iain) fixes almost all failures of the testsuite
on Darwin 8
caused by the large testsuite changes I committed yesterday.
The patch fixes all the failures but 2. These two require a different small
fix that is being
developed/tested and will be submitted separatel
The dump-pass patch with test case.
David
On Tue, Jun 7, 2011 at 11:54 AM, Xinliang David Li wrote:
> Please review the attached two patches.
>
> In the first patch, gate functions are cleaned up. All the per
> function legality checks are moved into the executor and the
> optimization heuristic
On 06/07/2011 07:39 PM, Alexander Monakov wrote:
>
>
> On Fri, 3 Jun 2011, Bernd Schmidt wrote:
>
Ok. Although I wonder how sel-sched can end up reusing an entry in
h_d_i_d?
>
> Unlike Haifa scheduler, we recompute INSN_LUIDs for each region. However, we
> call sched_deps_{init,finis
This patch (written with Iain) fixes all the testsuite failiures on Darwin8.
It includes the previous one.
OK to commit ?
Thanks
Index: ChangeLog
===
--- ChangeLog (revision 174760)
+++ ChangeLog (working copy)
@@ -1,3 +1,18 @
On Jun 7, 2011, at 1:38 PM, Nicola Pero wrote:
> This patch (written with Iain) fixes all the testsuite failiures on Darwin8.
> It includes the previous one.
>
> OK to commit ?
Ok.
On Wed, Jun 1, 2011 at 2:23 AM, Ira Rosen wrote:
> Hi,
>
> The vectorizer expects widening multiplication pattern to be:
>
> type a_t, b_t;
> TYPE a_T, b_T, prod_T;
>
> a_T = (TYPE) a_t;
> b_T = (TYPE) b_t;
> prod_T = a_T * b_T;
>
> where type 'TYPE' is double the size of type
Several tests in gcc.target/arm use dg-options with -mcpu=, which
causes compiler warnings or errors when the multilib flags include
-march=. This patch causes those tests to be skipped. It also
prevents gcc.target/arm/20090811-1.c from running with multilibs that
would override -mcpu or
On Tue, 7 Jun 2011, Janis Johnson wrote:
> Several tests in gcc.target/arm use dg-options with -mcpu=, which
> causes compiler warnings or errors when the multilib flags include
> -march=. This patch causes those tests to be skipped. It also
> prevents gcc.target/arm/20090811-1.c from ru
On Tue, Jun 7, 2011 at 8:44 PM, Gabriel Charette wrote:
> We need to stream TREE_TYPE for identifier node.
That seems unlikely, as identifiers do not have a type. There is some
TREE_TYPE abuse in cp-tree.h, perhaps you should find out what you're
streaming.
Why are you not using accessor macros
This patch removes some obsolete code (which used to be used by the Traditional
API)
and that is no longer used anywhere.
Committed to trunk.
Thanks
Index: class.c
===
--- class.c (revision 174766)
+++ class.c (working copy
Jason Merrill writes:
| I want to add a %S format to the C++ front end for printing a
| template/args pair during template argument deduction substitution. I
| want to print this during print_instantiation_partial_context, which
| uses pp_verbatim. But for some reason the format attribute on
|
1 - 100 of 124 matches
Mail list logo
