Re: Updated to respond to various email comments from Jason, Diego and Cary (issue6197069)

2012-06-23 Thread Dominique Dhumieres
> As I don't have access to a Darwin machine to test a fix, would you > mind updating the test? The failures are gone with the obvious patch: diff -up ../_clean/gcc/testsuite/gcc.dg/pubtypes-2.c gcc/testsuite/gcc.dg/pubtypes-2.c --- ../_clean/gcc/testsuite/gcc.dg/pubtypes-2.c 2009-11-25 18:15:43

Re: [PATCH][C++] Fix PR53605

2012-06-23 Thread H.J. Lu
On Mon, Jun 11, 2012 at 5:15 AM, Richard Guenther wrote: > > When I changed empty arrays domain to use a signed sizetype [0, -1] > domain mangling of a empty-array-type broke because mangling adds > an unsigned one to the signed -1 which causes an ICE (I chose to > do that instead of shifting the

Re: [patch testsuite]: Fix two testcases for x86_64-*-mingw* target

2012-06-23 Thread Mike Stump
On Jun 20, 2012, at 5:10 AM, Kai Tietz wrote: > 2012-06-20 Kai Tietz > >* gcc.target/i386/pr23943.c (size_t): Use compatible type-definition >for LLP64 targets. >* gcc.target/i386/pr38988.c: Likewise > Ok for apply? Ok.

[PATCH 0/3] Fix target/53749

2012-06-23 Thread Richard Henderson
The problem being addressed is that expand_mult expects that any mode that supports multiply also support shift. In practice that's a fairly valid assumption -- I've never seen an ISA for which this didn't hold. However, it means that if you jump through hoops in the backend to provide a (vector)

[PATCH 1/3] i386: Extract the guts of mulv16qi3 to ix86_expand_vecop_qihi

2012-06-23 Thread Richard Henderson
* config/i386/sse.md (mul3): Change from insn_and_split to pure expander; move expansion code ... * config/i386/i386.c (ix86_expand_vecop_qihi): ... here. New function. * config/i386/i386-protos.h: Update. diff --git a/gcc/config/i386/i386-protos.h b/gcc/config/i3

[PATCH 2/3] i386: Pass ix86_expand_sse_unpack operands by value

2012-06-23 Thread Richard Henderson
* config/i386/i386.c (ix86_expand_sse_unpack): Split operands[] parameter into src and dest. * config/i386/sse.md (vec_unpacku_hi_): Update call. (vec_unpacks_hi_): Likewise. (vec_unpacku_lo_): Likewise. (vec_unpacks_lo_): Likewise. * config/i

[PATCH 3/3] PR target/53749

2012-06-23 Thread Richard Henderson
* config/i386/i386.c (ix86_rtx_costs): Add reasonable costs for V*QImode shifts and multiply. (ix86_expand_vecop_qihi): Support shifts. * config/i386/i386.md (any_shift): New code iterator. * config/i386/sse.md (ashlv16qi3): Merge ...

Re: [Patch] PR 51938: extend ifcombine

2012-06-23 Thread Marc Glisse
Hello, thanks for looking into the patch. A couple more details now that I am back from a conference: On Wed, 20 Jun 2012, Marc Glisse wrote: On Wed, 20 Jun 2012, Richard Guenther wrote: On Sun, Jun 10, 2012 at 4:16 PM, Marc Glisse wrote: Hello, currently, tree-ssa-ifcombine handles pai

Coverage unlinking

2012-06-23 Thread Nathan Sidwell
This patch changes the coverage unlinking to be less aggressive. As David pointed out, when experimenting with random optimization options along side -fuse-coverage and -frandom-seed, it is inconvenient for gcc to remove the coverage data file. The reason it was doing so, is because -frandom-s

Re: [gimplefe] creating individual gimple_assign statements

2012-06-23 Thread Sandeep Soni
On Thu, Jun 21, 2012 at 11:33 AM, Diego Novillo wrote: > On 12-06-20 23:04 , Sandeep Soni wrote: >> >> Hi, >> >> This patch creates basic gimple_assign statements. It is a little raw >> not considering all types of gimple_assign statements for which I have >> already started working. >> >> Here is

Re: [Patch] PR 51938: extend ifcombine

2012-06-23 Thread Marc Glisse
On Wed, 20 Jun 2012, Marc Glisse wrote: On Wed, 20 Jun 2012, Richard Guenther wrote: On Sun, Jun 10, 2012 at 4:16 PM, Marc Glisse wrote: Hello, currently, tree-ssa-ifcombine handles pairs of imbricated "if"s that share the same then branch, or the same else branch. There is no particular r

Re: Coverage unlinking

2012-06-23 Thread Xinliang David Li
thanks for the fix. It works fine for me. David On Sat, Jun 23, 2012 at 11:00 AM, Nathan Sidwell wrote: > This patch changes the coverage unlinking to be less aggressive.  As David > pointed out, when experimenting with random optimization options along side > -fuse-coverage and -frandom-seed, i

Re: libgo patch committed: Copy runtime_printf from other library

2012-06-23 Thread Ian Lance Taylor
mich...@chamberlain.net.au writes: > On 25/05/2012 6:45 AM, Ian Lance Taylor wrote: >> This patch to libgo copies the implementation of runtime_printf ... > > The actual patch attached to the message was the one you had earlier > posted for the Go 1 release compatibility with the off-by-1 error in

ping x3 : latch mem to reg before multi-access in convert_move

2012-06-23 Thread Olivier Hainque
Hello, ping # 3 for http://gcc.gnu.org/ml/gcc-patches/2012-04/msg00298.html This is related to convert-move possibly emitting a sequence with multiple accesses to one input, triggering multiple memory accesses when that input happens to be a mem. > The original problem we had with this was the i

ping x2: fix mem+32760 ICE on powerpc - introduce predicates weaker than mode_dependent_address_p

2012-06-23 Thread Olivier Hainque
Hello, Ping #2 for the non-back-end parts of http://gcc.gnu.org/ml/gcc-patches/2012-04/msg01668.html Thanks much in advance, Olivier > David approved the rs6000 parts already (with adjustments to > comments, http://gcc.gnu.org/ml/gcc-patches/2012-05/msg00368.html) > > Thanks much in advance f