On Mon, Nov 5, 2012 at 8:33 AM, Gopalasubramanian, Ganesh
wrote:
> Couple of changes done with respect to the review comments.
>
> 1. sseshuf type attribute is handled in unit attribute calculation.
> 2. sseadd1 instruction attribute is handled in the new scheduler descriptions.
>
> The patch is a
This is a case where we had:
(set (reg:HI foo) (plus:HI (reg:HI sp) (const_int X)))
(clobber (reg:CC FLAGS_REG))
and the splitters decided to convert it to an LEA:
(set (reg:SI foo) (plus:SI (subreg:SI (reg:HI sp) 0) (const_int X)))
But this fails to match, because ix86_address_subreg_ope
On Mon, Nov 5, 2012 at 9:30 AM, Richard Sandiford
wrote:
> This is a case where we had:
>
> (set (reg:HI foo) (plus:HI (reg:HI sp) (const_int X)))
> (clobber (reg:CC FLAGS_REG))
>
> and the splitters decided to convert it to an LEA:
>
> (set (reg:SI foo) (plus:SI (subreg:SI (reg:HI sp) 0) (c
Hello,
This patch adds support for SH's rotcl instruction.
While working on it, I've noticed that the DImode left shift by one insn
was not used anymore, and instead ended up as 'x + x'. This
transformation was happening before/during RTL expansion. The fix for
it was to adjust the costs for DIm
Uros Bizjak wrote:
> 2012-11-04 Vladimir Yakovlev
> Uros Bizjak
>
> * mode-switching.c (create_pre_exit): Added code for
> maybe_builtin_apply case.
>
> Patch was bootstrapped and regression tested on x86_64-pc-linux-gnu,
> with vzeroupper patch [1] applied.
>
> I have ad
Hellow, Kaz
I've updated copyright. Is it Ok?
Thanks,
Vladimir
--- a/gcc/mode-switching.c
+++ b/gcc/mode-switching.c
@@ -1,6 +1,6 @@
/* CPU mode switching
Copyright (C) 1998, 1999, 2000, 2001, 2002, 2003, 2004, 2005, 2007, 2008,
- 2009, 2010 Free Software Foundation, Inc.
+ 2009, 2010,
On Mon, Nov 5, 2012 at 1:01 AM, Uros Bizjak wrote:
> On Mon, Nov 5, 2012 at 9:30 AM, Richard Sandiford
> wrote:
>> This is a case where we had:
>>
>> (set (reg:HI foo) (plus:HI (reg:HI sp) (const_int X)))
>> (clobber (reg:CC FLAGS_REG))
>>
>> and the splitters decided to convert it to an LEA:
Hello!
2012-11-05 Uros Bizjak
* gcc.dg/tree-ssa/cunroll-9.c: Dump cunrolli details.
Fix scan-tree-dump-times directive.
Tested on x86_64-pc-linux-gnu, committed to mainline SVN.
Uros.
Index: gcc.dg/tree-ssa/cunroll-9.c
=
On 4 November 2012 16:01, Jonathan Wakely wrote:
> On 3 November 2012 19:05, Paolo Carlini wrote:
>>
>> On 11/03/2012 03:43 PM, François Dumont wrote:
>>>
>>> Note that I run exception tests to validate it and the recently introduced
>>> 23_containers/forward_list/allocator/noexcept.cc test doesn't
Hi!
Depending on branch cost, e.g. on ppc v != d && v != e
isn't folded in the FE. On x86_64/i686 we have in *.original
for range1:
return (x != 0 && (unsigned int) v != 2) && (unsigned int) v + 4294967293 > 1;
but on ppc
return ((x != 0 && (unsigned int) v != 2) && (unsigned int) v != 3) &&
(un
Marc Glisse writes:
> * simplify-rtx.c (simplify_subreg): For vectors, create a VEC_SELECT.
Probably not helpful, sorry, but a subreg->vec_select transformation
feels like it's going in the wrong direction. Going from vec_select
to subreg would be OK from a "layering" perspective (although
Andreas Schwab writes:
> * final.c (final_scan_insn) [HAVE_cc0]: Handle all comparison
> codes in non-jump and cmove insn.
OK, thanks.
Richard
Hello!
Just another instance of testsuite PR51128.
2012-11-05 Uros Bizjak
PR testsuite/51128
* gcc.dg/torture/pr55018.c: Skip if -fno-fat-lto-objects was passed.
Tested on x86_64-pc-linux-gnu, committed to mainline SVN.
Uros.
Index: gcc.dg/torture/pr55018.c
> Done and committed.
That's great, thanks!
--
Eric Botcazou
Hi!
Apparently some targets have DWARF2_ADDR_SIZE < 4, for dw_val_class_addr
when DW_FORM_addr can't be used value_format already handles
DWARF2_ADDR_SIZE 1, 2, 4 and 8, so I've committed this change as obvious.
2012-11-05 Jakub Jelinek
PR target/55194
* dwarf2out.c (value_for
> Hi,
>
> the following patch adds streaming ofcgraph_node.ipa_transforms_to_apply
> so that transformation phases of IPA passes are run in LTO too. It is
> done by simple streaming of pass.static_pass_number and then looking
> it up among all_regular_ipa_passes.
>
> Bootstrapped and tested on x
Hi,
this testcase tests that we are about to bound number of iterations of
for (k = 3; k < NA_1; k++)
for (i = 3; i < MA_1; i++)
for (j = 3; j < MB_1; j++)
{
t = T[i][j];
T[i][j] = t+2+A[i][k]*B[j][k];
}
based on
#define MB 100
#define NA 450
#def
Thomas Schwinge, le Mon 05 Nov 2012 07:09:43 +0100, a écrit :
> Samuel, is there any way you can unpack all Debian source packages on a
> Debian machine, and run a recursive grep command (I can work out a
> suitable regex) to see where removing the MACH or __MACH built-in
> preprocessor macros migh
Hi,
in 4.6 timeframe I limited early inlier growth to apply only for leaf functions.
This does not work really well, because with less propagation of address
expressions
we are really not 100% succesfull on detecting C++ forwarders and predicting
them
zero cost. This patch simply makes the cost
Hi,
while proofreading niter and ivcanon code I noticed that we give up on exits in
inner
loops. I discussed this with Zdenek and it was done only because at the time
it was implemented we did nothing useful on outer loops. We can now unloop
them, so
I removed the restriction.
I also noticed t
> Those compiler generated statements do not have source origins, but
> they need to have debug location information attached so that
> information collected for them can be correlated with program
> constructs (such as CFG). One of the natural way is to use the source
> location associated with th
On Mon, 5 Nov 2012, Richard Sandiford wrote:
Marc Glisse writes:
* simplify-rtx.c (simplify_subreg): For vectors, create a VEC_SELECT.
Probably not helpful, sorry, but a subreg->vec_select transformation
feels like it's going in the wrong direction. Going from vec_select
to subreg w
> As Richard suggested, if a location is UNKNOWN_LOCATION, instead of
> change_scope to DECL_INITIAL (cfun->decl), we should let it inherit
> location from its previous instructions. This is implemented in the
> attached patch.
>
> Bootstrapped and passed gcc regression tests and gdb regression te
On Mon, 2012-10-29 at 18:56 +0100, Jakub Jelinek wrote:
> Status
> ==
>
> I'd like to close the stage 1 phase of GCC 4.8 development
> on Monday, November 5th. If you have still patches for new features you'd
> like to see in GCC 4.8, please post them for review soon. Patches
> posted before
On Mon, Nov 05, 2012 at 06:41:47AM -0600, Peter Bergner wrote:
> On Mon, 2012-10-29 at 18:56 +0100, Jakub Jelinek wrote:
> > I'd like to close the stage 1 phase of GCC 4.8 development
> > on Monday, November 5th. If you have still patches for new features you'd
> > like to see in GCC 4.8, please p
On Mon, Nov 5, 2012 at 2:21 AM, H.J. Lu wrote:
> On Mon, Nov 5, 2012 at 1:01 AM, Uros Bizjak wrote:
>> On Mon, Nov 5, 2012 at 9:30 AM, Richard Sandiford
>> wrote:
>>> This is a case where we had:
>>>
>>> (set (reg:HI foo) (plus:HI (reg:HI sp) (const_int X)))
>>> (clobber (reg:CC FLAGS_REG))
Hi,
this patch implements new badness metric for inliner. It is now based on
relative speedup, but not of calee, but of the caller + callee combo. This
nicely encompasses the edge frequency and other parameters simplifying the
actual badness code. I re-added code considering growth after inlining
On 11/04/2012 11:54 AM, Richard Biener wrote:
On Thu, Nov 1, 2012 at 2:10 PM, Richard Sandiford
wrote:
Kenneth Zadeck writes:
I would like you to respond to at least point 1 of this email. In it
there is code from the rtl level that was written twice, once for the
case when the size of the
> Hmm, like you said originally in the PR, I think generating the invalid
> PLUS is the bug here. I agree (as Richard B said) that an invalid PLUS
> shouldn't cause us to generate wrong code, but an assert seems better
> than a check. Eric, what do you think?
I think that this boils down to deci
> But, for cris-elf (and reasonably the same for other targets)
> there might not be such a constant-pool entry in the first
> place: the vectors are too short to rule out piecewise
> initialization as optimal for size (counting the vectors once
> per use). Let's increase them. Twice might just b
On 10/31/2012 01:12 PM, Richard Sandiford wrote:
> OK with those changes for the rtl bits. Can't approve the generator
> stuff though.
That's also OK.
Bernd
On Mon, 2012-11-05 at 13:53 +0100, Jakub Jelinek wrote:
> On Mon, Nov 05, 2012 at 06:41:47AM -0600, Peter Bergner wrote:
> > I'd like to post later today (hopefully this morning) a very minimal
> > configure patch that adds the -mcpu=power8 and -mtune=power8 compiler
> > options to gcc. Currently,
On Mon, Nov 05, 2012 at 08:40:00AM -0600, Peter Bergner wrote:
> Well we also patch config.in and configure.ac/configure. If those are
> acceptable to be patched later too, then great. If not, the patch
That is the same thing as config.gcc bits.
> isn't really very large. We did do this for po
Hi!
I've noticed that while I'm including ../nop.h header, I was using "NOP"
and so the inclusion was useless. It makes a difference only on
ia64/s390*/mmix where I didn't test, anyway, this should fix that.
Committed as obvious.
2012-11-05 Jakub Jelinek
PR debug/54970
PR deb
Hi!
I've backported 3 patches of mine to 4.7 branch, and committed
them after bootstrap/regtest on x86_64-linux and i686-linux.
Jakub
2012-11-05 Jakub Jelinek
Backported from mainline
2012-10-10 Jakub Jelinek
PR tree-optimization/54877
* tree-vect-l
> OK, thanks. What happens if you go one step farther and always continue?
Because we do want location and its block to be tightly coupled. I may
want to add an assertion that if location is known, the block should
*not* be NULL.
Dehao
>
> --
> Eric Botcazou
> No, there is nothing natural (and this can even be wrong). The statements
> must have the source location corresponding to the source construct they are
> generated for, which may be totally different from that of the insertion
> point. Yes, that can generate jumpiness in GDB, but this is far b
On Sat, Nov 03, 2012 at 12:03:45AM +0100, Dodji Seketeli wrote:
> + int fallthrough_probability =
> +then_more_likely_p
> +? PROB_VERY_UNLIKELY
> +: PROB_ALWAYS - PROB_VERY_UNLIKELY;
Just a formatting nit, I think = needs to go on the next line, so
int fallthrough_probability
=
> 2012-10-31 Martin Jambor
>
> * ipa-prop.c (ipa_get_param_decl_index_1): New function.
> (ipa_get_param_decl_index): Just call ipa_get_param_decl_index_1.
> (ipa_populate_param_decls): Accept descriptors parameter rather
> than the whole info.
> (load_from_unmodif
> Hi,
>
> the patch below disables generation of aggregate jump functions from
> bit-field stores because currently we depend on type size of the value
> to determine the size of the stored value and that does not work with
> bit-fields, making it impossible for IPA-CP to organize them in their
>
On Wed, 2012-10-31 at 11:13 +0100, Richard Biener wrote:
> On Mon, Oct 29, 2012 at 6:56 PM, Jakub Jelinek wrote:
> > Status
> > ==
> >
> > I'd like to close the stage 1 phase of GCC 4.8 development
> > on Monday, November 5th. If you have still patches for new features you'd
> > like to see i
The following patch fixes PR55151. The patch affects a sensitive
part of LRA code. So it took some time to find a PR solution. For the
test there were to many reloads into hard registers for an insn and LRA
failed to assign hard registers to all reload pseudos. There is another
more costl
Hi,
this patch removes RTL loop peeling code and makes tree-cunroll pass to also
perform simple peeling (i.e. one done with profile feedback when loop is
expected to iterate just few times).
The motivation is
1) I want to do some re-tunning of tree peeling heuristics first week(s) of
stage 3.
On 11/03/2012 10:11 AM, Alexandre Oliva wrote:
On Nov 2, 2012, Alexandre Oliva wrote:
On Nov 1, 2012, Jakub Jelinek wrote:
Even for stmt frontiers it is IMHO undesirable to duplicate
(perhaps many times) the whole sequence, as one would then reply the var
changing sequence in the debugger
Jakub and Richi,
At this point I have decided to that i am not going to get the rest of
the wide-int patches into a stable enough form for this round. The
combination of still living without power at my house and some issues
that i hit with the front ends has made it impossible to get this
fi
For the example I listed, the new statement is generated for source
construct at program point (2). However unlike simple code motion, (2)
is not going away after PRE. How would setting the location of the new
statement at the insertion point break coverage? Besides, the new
statement won't create
fill_simple_delay_slots would call optimize_skip with a conditional return,
which would cause a segfault when it calls next_active_insn (RETURN).
Other places in reorg.c check that they are not operating on a return insn
before doing branch-insn only processing, so I added this check there too.
On Sat, Nov 03, 2012 at 07:01:04PM -0400, David Edelsohn wrote:
> On Thu, Nov 1, 2012 at 5:17 PM, Michael Meissner
> wrote:
>
> > This patch adds 4 additional configuration switches, that allow the person
> > building the compiler to add additional prefixes to search for additional
> > executable
This patch from Shenghou Ma supports the os.Stat function on some more
operating systems. Bootstrapped and ran Go testsuite on
x86_64-unknown-linux-gnu. Committed to mainline.
Ian
diff -r 5d8ca2f79dba libgo/Makefile.am
--- a/libgo/Makefile.am Fri Nov 02 16:37:20 2012 -0700
+++ b/libgo/Makefile.
On Mon, Nov 05, 2012 at 05:20:46PM +0100, Jan Hubicka wrote:
> > Hi,
> >
> > the patch below disables generation of aggregate jump functions from
> > bit-field stores because currently we depend on type size of the value
> > to determine the size of the stored value and that does not work with
> >
This patch from Shenghou Ma fixes gccgo on libgo by not losing the TLS
pointer. This works around what I would describe as a bug in the NetBSD
libc. Bootstrapped on x86_64-unknown-linux-gnu, not that that proves
much. Committed to mainline.
Ian
diff -r df61836f495f libgo/runtime/proc.c
--- a/l
On 11/02/2012 01:14 PM, Paolo Carlini wrote:
On 11/02/2012 01:09 PM, Florian Weimer wrote:
I looked at this again and made a new copy of the test case instead.
It has been successfully tested on x86_64-redhat-linux-gnu.
Is this okay for trunk?
Looks very nice to me, and after all the issue s
Hello all,
Since nobody has commented on bug 54805, and I'm pretty sure this is
valid (and obvious), I'm just submitting it to the list.
This removes warnings about implicit declarations and fixes one of the
function calls in vxlib-tls.c for vxworks targets.
I got the old prototypes from
h
> This sequence breaks assumption in mode-switching.c, that final
> function return register load operates in MODE_EXIT mode and triggere
> following code:
>
> for (j = n_entities - 1; j >= 0; j--)
> {
> int e = entity_map[j];
>
On 11/02/2012 12:16 AM, David Edelsohn wrote:
> I would like to introduce filename-based shared library versioning (known as
> the "soname" in ELF world) for AIX, activated by the '--enable-aix-soname'
> configure flag.
> As discussed in the Bugzilla, I think this is a good feature and a
> nice t
On Mon, Nov 5, 2012 at 1:10 PM, Michael Haubenwallner
wrote:
> Well, as long as the old sharedlibs were not created as standalone shared
> objects (lib.so), this is similar to a normal "soname"-bump on AIX, in that
> it is still possible for the package manager to transfer the old shared
> object
On Mon, Nov 5, 2012 at 7:05 PM, Eric Botcazou wrote:
>> This sequence breaks assumption in mode-switching.c, that final
>> function return register load operates in MODE_EXIT mode and triggere
>> following code:
>>
>> for (j = n_entities - 1; j >= 0; j--)
>> {
> Because we do want location and its block to be tightly coupled. I may
> want to add an assertion that if location is known, the block should
> *not* be NULL.
Yes, that would be a first step towards always continuing. But that's 4.9
material, let's install the posted patch as-is for 4.8.
--
Greetings,
This patch is for google/gcc-4_7 branch.
Thanks,
2012-11-05 Paul Pluzhnikov
* contrib/testsuite-management/powerpc-grtev3-linux-gnu.xfail:
extend expiration date for pr54127.
Index: contrib/testsuite-management/powerpc-grtev3-linux-gnu.xfail
=
On Mon, Nov 5, 2012 at 7:34 PM, Uros Bizjak wrote:
> On Mon, Nov 5, 2012 at 7:05 PM, Eric Botcazou wrote:
>>> This sequence breaks assumption in mode-switching.c, that final
>>> function return register load operates in MODE_EXIT mode and triggere
>>> following code:
>>>
>>> for
OK.
Ollie
On Mon, Nov 5, 2012 at 12:40 PM, Paul Pluzhnikov wrote:
> Greetings,
>
> This patch is for google/gcc-4_7 branch.
>
> Thanks,
>
>
> 2012-11-05 Paul Pluzhnikov
>
> * contrib/testsuite-management/powerpc-grtev3-linux-gnu.xfail:
> extend expiration date for pr54127.
>
Uros Bizjak writes:
> On Mon, Nov 5, 2012 at 9:30 AM, Richard Sandiford
> wrote:
>> gcc/
>> PR target/55204
>> * config/i386/i386.c (ix86_address_subreg_operand): Remove stack
>> pointer check.
>> (print_reg): Use true_regnum rather than REGNO.
>> (ix86_pri
On Mon, Nov 5, 2012 at 7:43 PM, Uros Bizjak wrote:
As discussed above, modes of loads, generated from __builtin_apply
have no connection to function return mode. mode-switching.c does
detect __builtin_apply situation and raises maybe_builtin_apply flag,
but doesn't use it to s
Hi Andrew,
Andrew Pinski writes:
> On Fri, Oct 5, 2012 at 8:43 PM, Andrew Pinski
> wrote:
>> On Sun, Aug 19, 2012 at 10:13 AM, Richard Sandiford
>> wrote:
>>> Andrew Pinski writes:
Right now we only produce ins when a zero_extract is used on the
right hand side. We can do better b
The test gcc.dg/torture/mips-sdata-1.c fails when compiled with
-fno-fat-lto-objects because when that option is used no assembly code
is output and thus the scan does not find the '.sdata' line.
Checked with the mips-mti-elf target. OK to checkin?
Steve Ellcey
sell...@mips.com
2012-11-05 Stev
On Mon, 2012-11-05 at 19:19 +, Richard Sandiford wrote:
> Hi Andrew,
>
> Andrew Pinski writes:
> > On Fri, Oct 5, 2012 at 8:43 PM, Andrew Pinski
> > wrote:
> >> On Sun, Aug 19, 2012 at 10:13 AM, Richard Sandiford
> >> wrote:
> >>> Andrew Pinski writes:
> Right now we only produce in
Hi,
tested x86_64-linux, committed to mainline (and soon 4_7-branch too).
Thanks,
Paolo.
//
2012-11-05 Paolo Carlini
PR libstdc++/55215
* include/bits/random.tcc (mersenne_twister_engine<>::seed(_Sseq&)):
Assign state_size to _M_p.
* testsuite
This broke the bootstrap on sparc:
/home/davem/src/GIT/GCC/build-sparc32-linux/./prev-gcc/g++
-B/home/davem/src/GIT/GCC/build-sparc32\
-linux/./prev-gcc/ -B/usr/local/sparc-unknown-linux-gnu/bin/ -nostdinc++
-B/home/davem/src/GIT/GCC\
/build-sparc32-linux/prev-sparc-unknown-linux-gnu/libstdc++-
> Unfortunately the proposed patch fails the testcase from PR41993:
>
> --cut here--
> short retframe_short (void *rframe)
> {
> __builtin_return (rframe);
> }
OK, so that's not only an issue with the mode of the return register, but with
the return register itself. Then the original patch
Hello!
> 2012-11-04 Vladimir Yakovlev
>
>* mode-switching.c (create_pre_exit): Added code for
> maybe_builtin_apply case.
The part above is already committed.
>* config/i386/i386-protos.h (emit_i387_cw_initialization): Deleted.
>(emit_vzero): Added prototype.
>
2012-11-05 Harshit Chopra
* gcc/c-family/c-common.c
(handle_always_patch_for_instrumentation_attribute): Handle
always_patch_for_instrumentation attribute and turn inlining off for the
function.
(handle_never_patch_for_instrumentation_attribute): Handle
never_patch_for_ins
Thanks David for the review. My comments are inline.
On Sat, Nov 3, 2012 at 12:38 PM, Xinliang David Li wrote:
>
> Harshit, Nov 5 is the gcc48 cutoff date. If you want to have the x-ray
> instrumentation feature into this release, you will need to port your
> patch and submit for trunk review no
Jakub Jelinek writes:
> On Sat, Nov 03, 2012 at 12:03:45AM +0100, Dodji Seketeli wrote:
>> + int fallthrough_probability =
>> +then_more_likely_p
>> +? PROB_VERY_UNLIKELY
>> +: PROB_ALWAYS - PROB_VERY_UNLIKELY;
>
> Just a formatting nit, I think = needs to go on the next line, so
>
>
It does not hurt to submit the patch for review -- you need to provide
more background and motivation for this work
1) comparison with -finstrument-functions (runtime overhead etc)
2) use model difference (production binary ..)
3) Interesting examples of use cases (with graphs).
thanks,
David
On
"Steve Ellcey " writes:
> diff --git a/gcc/testsuite/gcc.dg/torture/mips-sdata-1.c
> b/gcc/testsuite/gcc.dg/torture/mips-sdata-1.c
> index 8ffd4d8..53c9e4f 100644
> --- a/gcc/testsuite/gcc.dg/torture/mips-sdata-1.c
> +++ b/gcc/testsuite/gcc.dg/torture/mips-sdata-1.c
> @@ -1,6 +1,7 @@
> /* Check
Richard Sandiford writes:
> "Steve Ellcey " writes:
>> diff --git a/gcc/testsuite/gcc.dg/torture/mips-sdata-1.c
>> b/gcc/testsuite/gcc.dg/torture/mips-sdata-1.c
>> index 8ffd4d8..53c9e4f 100644
>> --- a/gcc/testsuite/gcc.dg/torture/mips-sdata-1.c
>> +++ b/gcc/testsuite/gcc.dg/torture/mips-sdata-
Currently the following testcases are failing on x86_64-apple-darwin11/12...
FAIL: libgomp.c++/pr24455.C -O0 (test for excess errors)
WARNING: libgomp.c++/pr24455.C -O0 compilation failed to produce executable
FAIL: libgomp.c++/pr24455.C -O1 (test for excess errors)
WARNING: libgomp.c++/pr24
On 11/02/2012 02:34 AM, Bin Cheng wrote:
Also I don't understand why the bogus patch can catch more hoist
opportunities and improve code size, so please help if you have any idea
about this.
Well, perturbing the code, particularly in a way which is supposed to
change the amount of register pre
I apply the attached patch.
2012-10-05 François Dumont
* include/ext/throw_allocator.h (__throw_value_base): Add move
semantic, not throwing.
(__throw_value_limit): Likewise.
(__throw_value_random): Likewise.
* testsuite/util/exception/safety.h: Add validation of C++11
This updates the debug-mode and profile-mode forward_list code to
match my recent allocator changes to the default mode. It also adds
checking for unswappable allocators, and fixes some tests that those
new checks showed were broken.
* include/profile/forward_list: Update to meet allocato
On Mon, 2012-11-05 at 20:38 +, Richard Sandiford wrote:
> This sort of thing should usually be handled automatically by
> scan-assembler, and is for me:
>
> /foo/gcc/xgcc -B/foo/gcc/ /bar/gcc/testsuite/gcc.dg/torture/mips-sdata-1.c
> -fno-diagnostics-show-caret
> -O2 -flto -fuse-linker-plu
Hi,
this is a regression present on the 4.7 branch and caused by my fix for
another regression: http://gcc.gnu.org/ml/gcc-patches/2012-07/msg00825.html
It turns out that canonicalize_constructor_val has side effects: on the 4.7
branch, it calls add_referenced_var on the base variable of an addr
On Mon, Nov 05, 2012 at 10:16:31PM +0100, Eric Botcazou wrote:
> Bootstrapped/regtested on x86_64-suse-linux, OK for mainline and 4.7 branch?
>
>
> 2012-11-05 Eric Botcazou
>
> PR tree-optimization/54986
> * gimple-fold.c (canonicalize_constructor_val): Strip again all no-op
>
Steve Ellcey writes:
> On Mon, 2012-11-05 at 20:38 +, Richard Sandiford wrote:
>> This sort of thing should usually be handled automatically by
>> scan-assembler, and is for me:
>>
>> /foo/gcc/xgcc -B/foo/gcc/ /bar/gcc/testsuite/gcc.dg/torture/mips-sdata-1.c
>> -fno-diagnostics-show-caret
On Mon, 2012-11-05 at 21:28 +, Richard Sandiford wrote:
> Does it work for you if you run it separately? E.g. with:
>
> make check-gcc RUNTESTFLAGS=dg-torture.exp=mips-sdata-1.c
>
> Richard
Hm, I configured GCC with '--enable-languages=c,c++', I think this might
set ENABLE_LTO to '0' a
Missing exports for unordered_* containers in debug mode.
The patches between mainline/4.7 are a slight bit different, but
equivalent.
tested x86/linux
tested x86/linux --enable-symvers=gnu-versioned-namespace
2012-11-05 Benjamin Kosnik
Oleg Smolsky
PR libstdc++/55028
* con
Hello Joseph,
Here is the fixed patch with all your changes and the ChangeLog entries
below.
gcc/ChangeLog
2012-11-05 Balaji V. Iyer
* Makefile.in (C_COMMON_OBJS): Added c-family/array-notation-common.o.
* doc/passes.texi (Cilk Plus Transformation): Documented array
On Mon, Nov 5, 2012 at 8:43 AM, Vladimir Makarov wrote:
> The following patch fixes PR55151. The patch affects a sensitive part of
> LRA code. So it took some time to find a PR solution. For the test there
> were to many reloads into hard registers for an insn and LRA failed to
> assign hard
On Sat, Nov 3, 2012 at 1:14 AM, H.J. Lu wrote:
> Hi,
>
> The testcase shows -O -mx32 -maddress-mode=long -fPIC -S generates;
>
> x.i:22:37: internal compiler error: in plus_constant, at explow.c:88
> info[0x6eff - dyn->d_tag + 12] = dyn;
>
> expand_expr_real_2 has
>
> /* No sense savin
Cilk Plus branch was merged with trunk at revision 193151. Committed as
revision 193192.
Thanks,
Balaji V. Iyer.
Hello,
I hadn't expected that cfganal.c's reverse-CFG DFS would actually
depend on the order of the basic blocks. The attached patch fixes that
small oversight...
Bootstrapped&tested on powerpc64-unknown-linux-gnu. OK for trunk?
Ciao!
Steven
* cfganal.c (connect_infinite_loops_to_exit):
Hi all,
the attached patch implements support for polymorphic arguments to
TRANSFER. For details and discussion see the PR.
The patch was regtested successfully on x86_64-unknown-linux-gnu. Ok for trunk?
Btw, as part of the PR is a regression in 4.7 and trunk, I would like
to backport the target
>
> This broke the bootstrap on sparc:
>
> /home/davem/src/GIT/GCC/build-sparc32-linux/./prev-gcc/g++
> -B/home/davem/src/GIT/GCC/build-sparc32\
> -linux/./prev-gcc/ -B/usr/local/sparc-unknown-linux-gnu/bin/ -nostdinc++
> -B/home/davem/src/GIT/GCC\
> /build-sparc32-linux/prev-sparc-unknown-linu
> From: Eric Botcazou
> Date: Mon, 5 Nov 2012 15:29:11 +0100
> > But, for cris-elf (and reasonably the same for other targets)
> > there might not be such a constant-pool entry in the first
> > place: the vectors are too short to rule out piecewise
> > initialization as optimal for size (counting
From: Jan Hubicka
Date: Mon, 5 Nov 2012 23:19:16 +0100
> Is line 784 for you "gcc_checking_assert (uninlined_call_time >= 0);"?
Yes.
On Mon, 2012-11-05 at 13:40 -0800, Steve Ellcey wrote:
> Hm, I configured GCC with '--enable-languages=c,c++', I think this might
> set ENABLE_LTO to '0' and cause check_effective_target_lto to return
> FALSE and cause the -ffat-lto-objects flag to not be added. Of course
> this raises the questi
Steve Ellcey writes:
> I am not sure how ld is supposed to find crt0.o in the uninstalled
> setup, it exists in obj-mips-mti-elf/newlib/mips-mti-elf/libgloss/mips.
The newlib and libgloss flags are usually added by the dejagnu
baseboard file. E.g. for mips-sim.exp:
set_board_info cflags "[libgl
Hi,
The attached patch is to solve PR target/41993 which will affect
targets using MODE_EXIT.
Without it, we can't find all return registers for __builtin_return
in mode-switching.c:create_pre_exit. See the trail #4 by Uros in
the PR for the details. The patch is tested with bootstrap and
regtes
On Mon, 2012-11-05 at 22:55 +, Richard Sandiford wrote:
> Steve Ellcey writes:
> > I am not sure how ld is supposed to find crt0.o in the uninstalled
> > setup, it exists in obj-mips-mti-elf/newlib/mips-mti-elf/libgloss/mips.
>
> The newlib and libgloss flags are usually added by the dejagnu
"H.J. Lu" writes:
> On Sat, Nov 3, 2012 at 1:14 AM, H.J. Lu wrote:
>> Hi,
>>
>> The testcase shows -O -mx32 -maddress-mode=long -fPIC -S generates;
>>
>> x.i:22:37: internal compiler error: in plus_constant, at explow.c:88
>> info[0x6eff - dyn->d_tag + 12] = dyn;
>>
>> expand_expr_real_2 ha
1 - 100 of 124 matches
Mail list logo
