Re: [PATCH 3/3] Handle const_vector in mulv4si3 for pre-sse4.1.

2012-06-19 Thread Uros Bizjak
On Mon, Jun 18, 2012 at 10:06 PM, Richard Henderson wrote: >> Please note that you will probably hit PR33329, this is the reason >> that we expand multiplications after reload. Please see [1] for >> further explanation. There is gcc.target/i386/pr33329.c test to cover >> this issue, but it is not

Re: [PR49888, VTA] don't keep VALUEs bound to modified MEMs

2012-06-19 Thread Alexandre Oliva
On Jun 16, 2012, "H.J. Lu" wrote: > If I understand it correctly, the new approach fails to handle push > properly. It's actually cselib that didn't deal with push properly, so it thinks incoming stack arguments may be clobbered by them. But that's not the whole story, unfortunately. I still d

[PR debug/53682] avoid crash in cselib promote_debug_loc

2012-06-19 Thread Alexandre Oliva
When promote_debug_loc was first introduced, it would never be called with a NULL loc list. However, because of the strategy of temporarily resetting loc lists before recursion introduced a few months ago in alias.c, the earlier assumption no longer holds. This patch adusts promote_debug_loc to d

RE: [PATCH, ARM] New CPU support for Marvell PJ4 cores

2012-06-19 Thread Yi-Hsiu Hsu
marvell-pj4 is added to BE8_LINK_SPEC. Modified patch is attached. Thanks! B.R. Yi-Hsiu, Hsu -Original Message- From: Ramana Radhakrishnan [mailto:ramana.radhakrish...@linaro.org] Sent: Thursday, June 14, 2012 2:19 AM To: Yi-Hsiu Hsu Cc: gcc-patches@gcc.gnu.org Subject: Re: [PATCH, ARM

Re: [PING ARM Patches] PR53447: optimizations of 64bit ALU operation with constant

2012-06-19 Thread Michael Hope
On 18 June 2012 22:17, Carrot Wei wrote: > Hi > > Could ARM maintainers review following patches? > > http://gcc.gnu.org/ml/gcc-patches/2012-06/msg00497.html > 64bit add/sub constants. > > http://gcc.gnu.org/ml/gcc-patches/2012-05/msg01834.html > 64bit and with constants. > > http://gcc.gnu.org/ml

Re: [PATCH] Take branch misprediction effects into account when RTL loop unrolling (issue6099055)

2012-06-19 Thread Teresa Johnson
Ping. Teresa On Fri, May 18, 2012 at 7:21 AM, Teresa Johnson wrote: > Ping? > Teresa > > On Fri, May 11, 2012 at 6:11 AM, Teresa Johnson wrote: >> Ping? >> Teresa >> >> On Fri, May 4, 2012 at 3:41 PM, Teresa Johnson wrote: >>> >>> On David's suggestion, I have removed the changes that rename ni

Re: [patch] Remove NO_IMPLICIT_EXTERN_C target macro

2012-06-19 Thread Hans-Peter Nilsson
On Mon, 18 Jun 2012, Steven Bosscher wrote: > The attached patch removes NO_IMPLICIT_EXTERN_C, and replaces its sole > user with IMPLICIT_EXTERN_C to avoid the double negations (#ifndef > NO_IMPLICIT_EXTERN_C, etc.). > > Bootstrapped and tested on x86_64-unknown-linux-gnu. OK for trunk? I saw it w

C++ PATCH for c++/53651 (ICE with ill-formed use of decltype)

2012-06-19 Thread Jason Merrill
A decltype doesn't have a name. Tested x86_64-pc-linux-gnu, applying to trunk and 4.7. commit bab2f5e9e77bd41b91ca6eae34483eb159307519 Author: Jason Merrill Date: Thu Jun 14 17:28:08 2012 -0700 PR c++/53651 * name-lookup.c (constructor_name_p): Don't try to look at the name of

Re: [PATCH] Unify emit_{pre,post}_atomic_barrier across Alpha, ARM, MIPS and TileGX

2012-06-19 Thread Maxim Kuvyrkov
On 15/06/2012, at 11:16 AM, Richard Henderson wrote: > On 2012-06-14 16:06, Maxim Kuvyrkov wrote: >> 2012-06-15 Maxim Kuvyrkov >> >> * emit-rtl.c (need_atomic_barrier_p): New function. >> * emit-rtl.h (need_atomic_barrier_p): Declare it. >> * config/alpha/alpha.c (alpha_{pre,pos

Re: [PATCH, MIPS] Add most common atomic patterns

2012-06-19 Thread Maxim Kuvyrkov
I've now checked these patches. Tom, thanks for great optimizing sync and atomic builtins for MIPS and XLP, and, Richard, thanks for the reviews and education on writing good .md descriptions. -- Maxim Kuvyrkov CodeSourcery / Mentor Graphics On 13/06/2012, at 5:50 PM, Maxim Kuvyrkov wrote:

Re: User directed Function Multiversioning via Function Overloading (issue5752064)

2012-06-19 Thread Sriraman Tallam
Ping. On Thu, Jun 14, 2012 at 1:13 PM, Sriraman Tallam wrote: > +cc c++ front-end maintainers > > Hi, > >   C++ Frontend maintainers, Could you please take a look at the > front-end part when you find the time? > >   Honza, your thoughts on the callgraph part? > >   Richard, any further comments/

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

2012-06-19 Thread Jason Merrill
On 06/19/2012 10:12 AM, Sterling Augustine wrote: + /* If we're putting types in their own .debug_types sections, +the .debug_pubtypes table will still point to the compile +unit (not the type unit), so we want to use the offset of +the skeleton DIE (if

Re: [RFC 0/3] Stuff related to pr53533

2012-06-19 Thread Richard Henderson
On 2012-06-19 15:55, Matt wrote: > On 2012-06-15 13:57, Richard Henderson wrote: >> > Bootstrapped and tested on x86_64, but I'll leave some time for >> > comment before committing any of this. > >> Patches now committed. > > Hey Richard, > > Thanks for taking on some of these issues. I'm not se

Re: [patch] Deal with #ident without

2012-06-19 Thread Hans-Peter Nilsson
On Tue, 19 Jun 2012, Steven Bosscher wrote: > I've now committed this, see r188791. Breaking cris-elf. Just try rebuilding cc1: ./gcc/gcc/../libdecnumber/dpd -I../libdecnumber\ /tmp/hpautotest-gcc1/gcc/gcc/config/cris/cris.c -o cris.o /tmp/hpautotest-gcc1/gcc/gcc/config/cr

[cxx-conversion] Remove option to build without a C++ compiler (issue6296093)

2012-06-19 Thread Diego Novillo
Remove option to build without a C++ compiler. This patch removes all the configuration code that allowed GCC to build without a C++ compiler. After this patch the following configuration flags are no longer valid: --enable-build-with-cxx --enable-build-poststage1-with-cxx

Re: [RFC 0/3] Stuff related to pr53533

2012-06-19 Thread Matt
On 2012-06-15 13:57, Richard Henderson wrote: > Bootstrapped and tested on x86_64, but I'll leave some time for > comment before committing any of this. Patches now committed. Hey Richard, Thanks for taking on some of these issues. I'm not seeing much of an improvement yet when manually ap

[patch][ARM] Do not include output.h in arm-c.c

2012-06-19 Thread Steven Bosscher
Hello, Only a few front-end files to go that need output.h, and some of them are in the c_target_objs: arm, mep, m32c, and rl78. This patch tackles the ARM case. arm-c.c needs output.h because EMIT_EABI_ATTRIBUTE wants to print to asm_out_file. Solved by replacing EMIT_EABI_ATTRIBUTE with a func

[patch committed testsuite] Tweak gcc.dg/stack-usage-1.c on SH

2012-06-19 Thread Kaz Kojima
Hi, I've applied the attached patch which is a tiny SH specific change of gcc.dg/stack-usage-1.c test. Tested on sh-linux and i686-pc-linux-gnu. Regards, kaz -- 2012-06-19 Kaz Kojima * gcc.dg/stack-usage-1.c: Use sh*-*-* instead of sh-*-*. --- ORIG/trunk/gcc/testsuite/gcc.dg

Re: Fix e500 vector ICE with string constants

2012-06-19 Thread David Edelsohn
On Tue, Jun 19, 2012 at 5:56 PM, Joseph S. Myers wrote: > 2012-06-19  Joseph Myers   > >        * config/rs6000/spe.md (*mov_si_e500_subreg0): Rename to >        mov_si_e500_subreg0. >        (*mov_si_e500_subreg0_elf_low) >        (*mov_si_e500_subreg4_elf_low): New patterns. > > testsuite: > 20

[patch][PCH] Do not write/read asm_out_file, take 2

2012-06-19 Thread Steven Bosscher
Hello, The attached patch removes one more #include output.h, this time from c-family/c-pch.c. Anything written out to asm_out_file between pch_init and c_common_write_pch is read back in by c_common_write_pch and dumped to the PCH that's being written out. In c_common_read_pch this data is writt

Fix e500 vector ICE with string constants

2012-06-19 Thread Joseph S. Myers
On some tests involving storing a pointer to a string constant in a vector, on powerpc with SPE vectors, an ICE occurs of the form: t2.c: In function 'f': t2.c:7:1: error: unrecognizable insn: } ^ (insn 9 8 10 2 (set (subreg:SI (reg:V2SI 125 [ D.1618 ]) 4) (lo_sum:SI (reg:SI 126)

Re: [PATCH] Fix PR53708

2012-06-19 Thread Iain Sandoe
On 19 Jun 2012, at 22:41, Mike Stump wrote: > On Jun 19, 2012, at 12:22 PM, Iain Sandoe wrote: >> On 19 Jun 2012, at 13:53, Dominique Dhumieres wrote: >> >>> On Tue, 19 Jun 2012, Richard Guenther wrote: > Richard Guenther writes: >> We are too eager to bump alignment of some decl

Re: [testsuite] profopt.exp and friends: use expected list of options

2012-06-19 Thread Mike Stump
On Jun 18, 2012, at 4:51 PM, Janis Johnson wrote: > There are tests in g++.tree-prof that have non-unique lines in test > summaries for scan-*-dump checks. Investigation showed that these tests > were being run multiple times, for a list of options that had leaked > over from another set of profi

Re: [PATCH] Fix PR53708

2012-06-19 Thread Mike Stump
On Jun 19, 2012, at 5:53 AM, domi...@lps.ens.fr (Dominique Dhumieres) wrote: > On Tue, 19 Jun 2012, Richard Guenther wrote: >> >>> Richard Guenther writes: We are too eager to bump alignment of some decls when vectorizing. The fix is to not bump alignment of decls the user explicitely >

Re: [PATCH] Fix PR53708

2012-06-19 Thread Mike Stump
On Jun 19, 2012, at 12:22 PM, Iain Sandoe wrote: > On 19 Jun 2012, at 13:53, Dominique Dhumieres wrote: > >> On Tue, 19 Jun 2012, Richard Guenther wrote: >>> Richard Guenther writes: > We are too eager to bump alignment of some decls when vectorizing. > The fix is to not bump align

Re: [patch] Fix failing nested-3.C on ARM.

2012-06-19 Thread Mike Stump
On Jun 19, 2012, at 2:18 AM, Richard Earnshaw wrote: > The regexp in nested-3.C has to parse the machine-specific comment > character; on ARM that is '@'. > > Tested on arm-eabi, where this test now passes. > > OK? Ok.

Re: [PATCH] Fix PR tree-optimization/53636 (SLP generates invalid misaligned access)

2012-06-19 Thread Mikael Pettersson
Richard Guenther writes: > On Fri, Jun 15, 2012 at 5:00 PM, Ulrich Weigand wrote: > > Richard Guenther wrote: > >> On Fri, Jun 15, 2012 at 3:13 PM, Ulrich Weigand > >> wrote: > >> > However, there is a second case where we need to check every pass: if > >> > we're not actually vectorizing

Re: [PATCH] Bad code generation: incorrect folding of TARGET_MEM_REF into a constant

2012-06-19 Thread Jiří Hruška
On Tue, Jun 19, 2012 at 8:59 PM, Andreas Schwab wrote: > Declare them as arrays. > extern const int __start_foo[], __stop_foo[]; Thanks, that's a good suggestion, cleans the code nicely! (Though, of course, both ways work here and the strange things happen only in the 3rd testcase, which does not

Re: [patch] Use IDENTIFIER_LENGTH instead of strlen(IDENTIFIER_POINTER) in a few places

2012-06-19 Thread Mike Stump
On Jun 18, 2012, at 8:55 AM, Steven Bosscher wrote: > Obvious enough > > objc/ >* objc-encoding.c (encode_aggregate_fields): Use IDENTIFIER_LENGTH >instead of strlen(IDENTIFIER_POINTER). >(encode_aggregate_within): Likewise. Ok.

Re: [patch] Fix PR48109 using artificial top-level asm statements (darwin/objc)

2012-06-19 Thread Mike Stump
On Jun 18, 2012, at 10:51 AM, Steven Bosscher wrote: > This patch started as an attempt to remove #include "output.h" from > objc/: Instead of writing references directly to asm_out_file, the > references are output as top-level asm statements. > OK for trunk? Ok.

Re: [patch] Deal with #ident without

2012-06-19 Thread Steven Bosscher
On Thu, Jun 7, 2012 at 11:22 AM, Richard Guenther wrote: > On Thu, Jun 7, 2012 at 8:16 AM, Andreas Schwab wrote: >> Steven Bosscher writes: >> >>> Index: doc/tm.texi >>> === >>> --- doc/tm.texi       (revision 188182) >>> +++ doc/tm

Re: [PATCH] Fix PR53708

2012-06-19 Thread Iain Sandoe
On 19 Jun 2012, at 13:53, Dominique Dhumieres wrote: > On Tue, 19 Jun 2012, Richard Guenther wrote: >> >>> Richard Guenther writes: We are too eager to bump alignment of some decls when vectorizing. The fix is to not bump alignment of decls the user explicitely aligned or that ar

Re: [PATCH] Improve pattern recognizer for division by constant (PR tree-optimization/51581)

2012-06-19 Thread Richard Henderson
On 2012-06-18 22:46, Jakub Jelinek wrote: > On Mon, Jun 18, 2012 at 04:44:21PM -0700, Richard Henderson wrote: >> On 2012-06-14 13:58, Jakub Jelinek wrote: >>> + if (!supportable_widening_operation (WIDEN_MULT_EXPR, last_stmt, >>> + vecwtype, vectype, >>> +

Re: [RFC 0/3] Stuff related to pr53533

2012-06-19 Thread Richard Henderson
On 2012-06-15 13:57, Richard Henderson wrote: > Bootstrapped and tested on x86_64, but I'll leave some time for > comment before committing any of this. Patches now committed. r~

Re: [PATCH] backport darwin12 fixes to gcc-4_7-branch

2012-06-19 Thread Mike Stump
Ok.

Re: [PATCH] Bad code generation: incorrect folding of TARGET_MEM_REF into a constant

2012-06-19 Thread Andreas Schwab
Jiří Hruška writes: > #include > > __attribute__((section("foo"))) const int foo1 = 1; > __attribute__((section("foo"))) const int foo2 = 2; > __attribute__((section("foo"))) const int foo3 = 3; > > extern const int __start_foo, __stop_foo; Declare them as arrays. extern const int __start_foo[

Re: [PATCH 2/3] Use synth_mult for vector multiplies vs scalar constant

2012-06-19 Thread Richard Henderson
On 2012-06-16 04:19, Eric Botcazou wrote: >> @@ -179,7 +179,11 @@ extern const unsigned char >> mode_class[NUM_MACHINE_MODES]; >> >> extern CONST_MODE_SIZE unsigned char mode_size[NUM_MACHINE_MODES]; >> #define GET_MODE_SIZE(MODE)((unsigned short) mode_size[MODE]) >> -#define GET_MODE_BITSIZE

Re: [PATCH, i386]: Introduce FIST_ROUNDING int iterator

2012-06-19 Thread Uros Bizjak
Hello! 2012-06-19 Uros Bizjak * config/i386/i386.md (FIST_ROUNDING): New int iterator. (rounding): Handle UNSPEC_FIST_{FLOOR,CEIL}. (ROUNDING): Ditto. (*fist2__1): Macroize insn from *fist2_{floor,ceil}_1 using FIST_ROUNDING int iterator. (fistdi

Re: [PATCH 2/3] Add XLP-specific atomic instructions and tweaks.

2012-06-19 Thread Richard Sandiford
Maxim Kuvyrkov writes: > The only other change that I made that was not in your comments is the > addition of "b" mips_print_operand specifier. The LDADD and SWAP > instructions accept their address as a plain register without > parenthesis, Ouch. > so I've added the specifier to skip outputtin

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

2012-06-19 Thread Sterling Augustine
On Wed, Jun 13, 2012 at 10:47 PM, Jason Merrill wrote: > On 06/13/2012 04:26 PM, Sterling Augustine wrote: >>> >>> I lean toward -g myself, since there doesn't seem to be a strong rule one >>> way or the other. >> >> >> Unless there are further comments, I'll stick with -g then. >> >> I think that

Re: [PATCH] Bad code generation: incorrect folding of TARGET_MEM_REF into a constant

2012-06-19 Thread Jiří Hruška
On Tue, Jun 19, 2012 at 10:54 AM, Richard Guenther wrote: > The issue is that your testcase is invalid. >    int x = ret(*(&fooS + i)); > this access is only ever valid for i == 0 as otherwise you are creating > a pointer that points outside of the object fooS. Richard, thanks for your reply. T

Re: [Patch] Adjustments for Windows x64 SEH

2012-06-19 Thread Richard Henderson
On 2012-06-18 05:22, Tristan Gingold wrote: > + /* Win64 SEH, very large frames need a frame-pointer as maximum stack > + allocation is 4GB (add a safety guard for saved registers). */ > + if (TARGET_64BIT_MS_ABI && get_frame_size () + 4096 > SEH_MAX_FRAME_SIZE) > +return true; Elsewher

Re: [PATCH, gdc] - Merging gdc (GNU D Compiler) into gcc

2012-06-19 Thread Joseph S. Myers
On Mon, 18 Jun 2012, Iain Buclaw wrote: > http://www.gdcproject.org/files/gdc_testsuite.patch.gz I have no comments on this patch for now. -- Joseph S. Myers jos...@codesourcery.com

Re: [PATCH, gdc] - Merging gdc (GNU D Compiler) into gcc

2012-06-19 Thread Joseph S. Myers
On Mon, 18 Jun 2012, Iain Buclaw wrote: > http://www.gdcproject.org/files/gdc_libphobos.patch.gz Same comments as before about FSF postal addresses. Although runtime libraries need not be assigned to the FSF (as per the GCC Mission Statement), all significant files should still have copyright a

[PATCH, i386]: Introduce FRNDINT_ROUNDING int iterator

2012-06-19 Thread Uros Bizjak
Hello! 2012-06-19 Uros Bizjak * config/i386/i386.md (FRNDINT_ROUNDING): New int iterator. (rounding): New int attribute. (ROUNDING): Ditto. (frndintxf2_): Macroize insn from frndintxf2_{floor,ceil,trunc} using FRNDINT_ROUNDING int iterator. (frnd

Re: [PATCH, gdc] - Merging gdc (GNU D Compiler) into gcc

2012-06-19 Thread Joseph S. Myers
On Mon, 18 Jun 2012, Iain Buclaw wrote: > [PATCH 1/4]: > The D compiler frontend > - gcc/d Only selectively reviewed, but here are some comments: > diff -Naur gcc-4.8-20120617/gcc/d/asmstmt.cc gcc-4.8/gcc/d/asmstmt.cc > --- gcc-4.8-20120617/gcc/d/asmstmt.cc 1970-01-01 01:00:00.0 +010

Re: [PATCH, gdc] - Merging gdc (GNU D Compiler) into gcc

2012-06-19 Thread Steven Bosscher
Hello, I had a very quick look through the gdc_frontend patch. Below are a couple of comments on it: > http://www.gdcproject.org/files/gdc_frontend.patch.gz > > [PATCH 1/4]: > The D compiler frontend >  -  gcc/d How did you test this? You include rtl.h/expr.h in d-builtins.c and d-gcc-includes.h

Re: [PATCH, libcpp]: Use x86 __builtin_ia32_pcmpestri128 instead of asm.

2012-06-19 Thread Richard Henderson
On 2012-06-18 23:38, Uros Bizjak wrote: > On Tue, Jun 19, 2012 at 12:07 AM, Richard Henderson wrote: >> On 2012-06-18 13:19, Uros Bizjak wrote: >>>/* ??? The builtin doesn't understand that the PCMPESTRI read from >>>memory need not be aligned. */ >>> - __asm ("%vpcmpestri $0

[PATCH] Fix vrp68 testcase

2012-06-19 Thread Richard Guenther
This fixes the testcase to match reality - and update the comments appropriately in it. Tested on x86_64-unknown-linux-gnu, applied. Richard. 2012-06-19 Richard Guenther * gcc.dg/tree-ssa/vrp68.c: Adjust testcase. Index: gcc/testsuite/gcc.dg/tree-ssa/vrp68.c ===

Re: [PATCH 4/4] - Merging gdc (GNU D Compiler) into gcc

2012-06-19 Thread Joseph S. Myers
On Mon, 18 Jun 2012, Iain Buclaw wrote: > --- gcc-4.8-20120617/gcc/doc/install.texi 2012-05-29 15:14:06.0 > +0100 > +++ gcc-4.8/gcc/doc/install.texi 2012-06-18 20:39:45.058591380 +0100 > @@ -1360,12 +1360,12 @@ their runtime libraries should be built. > grep language= */config-l

Re: [PATCH][AARCH64]: Invent new regclass - FP low regs.

2012-06-19 Thread Marcus Shawcroft
On 19/06/12 15:03, Tejas Belagod wrote: Hi, The attached patch invents a new register class V0 - V15 that is needed for some lane variants of AdvSIMD instructions that can only take V0 - V15 as their indexed register when working on half-word type. Regression tests are happy. OK? OK /Marcus

Re: [PATCH, gdc] - Merging gdc (GNU D Compiler) into gcc

2012-06-19 Thread Joseph S. Myers
On Mon, 18 Jun 2012, Iain Buclaw wrote: > These series of patches are for the D compiler frontend for inclusion into > GCC. > > http://www.gdcproject.org/files/gdc_frontend.patch.gz > http://www.gdcproject.org/files/gdc_libphobos.patch.gz > http://www.gdcproject.org/files/gdc_testsuite.patch.gz

[PATCH][7/n] VRP and anti-ranges

2012-06-19 Thread Richard Guenther
And here is the union_ranges part. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied. Richard. 2012-06-19 Richard Guenther * tree-vrp.c (union_ranges): New function. (vrp_meet_1): Use union_ranges. (vrp_meet): Dump what we union and call vrp_meet_1. Index

[testsuite] Clear hwcap_2 with Sun ld

2012-06-19 Thread Rainer Orth
In recent Solaris 11 Update 1 builds, the Sun assembler tags AVX2 object files with a hardware capability that isn't cleared by the current gcc/testsuite/gcc.target/i386/clearcap.map file. There are some new capabilities in in AT_SUN_CAP_HW2, but unfortunately the old linker map syntax has no sup

Re: [PATCH] AIX pthread.h fixincludes

2012-06-19 Thread Bruce Korb
Hi David, On Tue, Jun 19, 2012 at 7:16 AM, David Edelsohn wrote: > Okay? Okay. Cheers - Bruce

Re: [PATCH] Add vector cost model density heuristic

2012-06-19 Thread Richard Guenther
On Tue, 19 Jun 2012, William J. Schmidt wrote: > On Tue, 2012-06-19 at 14:48 +0200, Richard Guenther wrote: > > On Tue, 19 Jun 2012, William J. Schmidt wrote: > > > > > I remember having this discussion, and I was looking for it to check on > > > the details, but I can't seem to find it either in

Re: [PATCH] Add vector cost model density heuristic

2012-06-19 Thread William J. Schmidt
On Tue, 2012-06-19 at 14:48 +0200, Richard Guenther wrote: > On Tue, 19 Jun 2012, William J. Schmidt wrote: > > > I remember having this discussion, and I was looking for it to check on > > the details, but I can't seem to find it either in my inbox or in the > > archives. Can you please point me

[PATCH] AIX pthread.h fixincludes

2012-06-19 Thread David Edelsohn
AIX 5.2 pthread.h uses the wrong number of braces for more of the PTHREAD initializers. This patch extends the earlier patch to fix the other broken macros. * inclhack.def (aix_mutex_initializer_1, aix_cond_initializer_1, aix_rwlock_initializer): New. * fixincl.x: Regenerat

[PATCH][AARCH64]: Invent new regclass - FP low regs.

2012-06-19 Thread Tejas Belagod
Hi, The attached patch invents a new register class V0 - V15 that is needed for some lane variants of AdvSIMD instructions that can only take V0 - V15 as their indexed register when working on half-word type. Regression tests are happy. OK? Thanks, Tejas Belagod. ARM. Changelog: 2012-06-19

[arm] Remove obsolete FPA support (7/n): Tidy up attributes

2012-06-19 Thread Richard Earnshaw
This patch cleans up some more of the resulting fall-out from removing the FPA and maverick co-processors. In particular it covers: - Removing the redundant states from the type attributes - Removing some now redundant UNSPEC values. - Removing some state from the generic scheduler description th

[PATCH][5/n] VRP and anti-ranges

2012-06-19 Thread Richard Guenther
This adjusts intersect_ranges to match what will become union_ranges (but in a separate patch). Bootstrapped and tested on x86_64-unknown-linux-gnu, applied. Richard. 2012-06-19 Richard Guenther * tree-vrp.c (intersect_ranges): Handle more cases. (vrp_intersect_ranges): Dump

Re: [PATCH] Fix PR53708

2012-06-19 Thread Dominique Dhumieres
On Tue, 19 Jun 2012, Richard Guenther wrote: > > > Richard Guenther writes: > > > We are too eager to bump alignment of some decls when vectorizing. > > > The fix is to not bump alignment of decls the user explicitely > > > aligned or that are used in an unknown way. > > > > I thought attribute(

Re: [PATCH] Add vector cost model density heuristic

2012-06-19 Thread Richard Guenther
On Tue, 19 Jun 2012, William J. Schmidt wrote: > On Tue, 2012-06-19 at 12:10 +0200, Richard Guenther wrote: > > On Mon, 18 Jun 2012, William J. Schmidt wrote: > > > > > On Mon, 2012-06-18 at 13:49 -0500, William J. Schmidt wrote: > > > > On Mon, 2012-06-11 at 13:40 +0200, Richard Guenther wrote:

Re: [PATCH] Add vector cost model density heuristic

2012-06-19 Thread William J. Schmidt
On Tue, 2012-06-19 at 12:10 +0200, Richard Guenther wrote: > On Mon, 18 Jun 2012, William J. Schmidt wrote: > > > On Mon, 2012-06-18 at 13:49 -0500, William J. Schmidt wrote: > > > On Mon, 2012-06-11 at 13:40 +0200, Richard Guenther wrote: > > > > On Fri, 8 Jun 2012, William J. Schmidt wrote: > >

Re: [4.6][ARM] Backport "MCR Not available in Thumb1"

2012-06-19 Thread Richard Earnshaw
On 19/06/12 12:26, Joey Ye wrote: > Oops! Sorry for such a stupid problem. > > 2012-06-18 Joey Ye > > Backported from mainline > 2011-10-14 David Alan Gilbert > > * config/arm/arm.h (TARGET_HAVE_DMB_MCR): MCR Not available in > Thumb1. > OK. R.

Re: [PATCH] Add vector cost model density heuristic

2012-06-19 Thread Richard Guenther
On Tue, 19 Jun 2012, William J. Schmidt wrote: > On Tue, 2012-06-19 at 12:08 +0200, Richard Guenther wrote: > > On Mon, 18 Jun 2012, William J. Schmidt wrote: > > > > > On Mon, 2012-06-11 at 13:40 +0200, Richard Guenther wrote: > > > > On Fri, 8 Jun 2012, William J. Schmidt wrote: > > > > > > >

Re: [PATCH] Add vector cost model density heuristic

2012-06-19 Thread William J. Schmidt
On Tue, 2012-06-19 at 12:08 +0200, Richard Guenther wrote: > On Mon, 18 Jun 2012, William J. Schmidt wrote: > > > On Mon, 2012-06-11 at 13:40 +0200, Richard Guenther wrote: > > > On Fri, 8 Jun 2012, William J. Schmidt wrote: > > > > > > > > > > > Hmm. I don't like this patch or its general ide

RE: [4.6][ARM] Backport "MCR Not available in Thumb1"

2012-06-19 Thread Joey Ye
Oops! Sorry for such a stupid problem. 2012-06-18 Joey Ye Backported from mainline 2011-10-14 David Alan Gilbert * config/arm/arm.h (TARGET_HAVE_DMB_MCR): MCR Not available in Thumb1. Index: gcc/config/arm/arm.h =

Re: RFA: Fix PR53688

2012-06-19 Thread Richard Guenther
On Tue, Jun 19, 2012 at 12:13 PM, Michael Matz wrote: > Hi, > > On Tue, 19 Jun 2012, Richard Guenther wrote: > >> > So, we have to build a memref always and rewrite its type to one >> > representing the real size.  Note that TYPE_MAX_VALUE may be NULL, so we >> > don't need to check for 'len' bein

Re: [PATCH] Improve pattern recognizer for division by constant (PR tree-optimization/51581)

2012-06-19 Thread Jakub Jelinek
On Mon, Jun 18, 2012 at 04:44:21PM -0700, Richard Henderson wrote: > On 2012-06-14 13:58, Jakub Jelinek wrote: > > + if (!supportable_widening_operation (WIDEN_MULT_EXPR, last_stmt, > > + vecwtype, vectype, > > + &dummy, &dummy, &du

Re: [PATCH] Fix PR53708

2012-06-19 Thread Richard Guenther
On Tue, 19 Jun 2012, Richard Sandiford wrote: > Richard Guenther writes: > > We are too eager to bump alignment of some decls when vectorizing. > > The fix is to not bump alignment of decls the user explicitely > > aligned or that are used in an unknown way. > > I thought attribute((__aligned__)

Re: [PATCH] Fix PR53708

2012-06-19 Thread Richard Sandiford
Richard Guenther writes: > We are too eager to bump alignment of some decls when vectorizing. > The fix is to not bump alignment of decls the user explicitely > aligned or that are used in an unknown way. I thought attribute((__aligned__)) only set a minimum alignment for variables? Most usees I

Re: RFA: Fix PR53688

2012-06-19 Thread Michael Matz
Hi, On Tue, 19 Jun 2012, Richard Guenther wrote: > > So, we have to build a memref always and rewrite its type to one > > representing the real size.  Note that TYPE_MAX_VALUE may be NULL, so we > > don't need to check for 'len' being null or not. > > > > This fixes the C testcase (don't know abo

Re: [PATCH] Add vector cost model density heuristic

2012-06-19 Thread Richard Guenther
On Mon, 18 Jun 2012, William J. Schmidt wrote: > On Mon, 2012-06-18 at 13:49 -0500, William J. Schmidt wrote: > > On Mon, 2012-06-11 at 13:40 +0200, Richard Guenther wrote: > > > On Fri, 8 Jun 2012, William J. Schmidt wrote: > > > > > > > > > > > Hmm. I don't like this patch or its general ide

Re: [PATCH] Add vector cost model density heuristic

2012-06-19 Thread Richard Guenther
On Mon, 18 Jun 2012, William J. Schmidt wrote: > On Mon, 2012-06-11 at 13:40 +0200, Richard Guenther wrote: > > On Fri, 8 Jun 2012, William J. Schmidt wrote: > > > > > > > Hmm. I don't like this patch or its general idea too much. Instead > > I'd like us to move more of the cost model detail

[PATCH] Fix PR53708

2012-06-19 Thread Richard Guenther
We are too eager to bump alignment of some decls when vectorizing. The fix is to not bump alignment of decls the user explicitely aligned or that are used in an unknown way. Bootstrapped and tested on i686-darwin9 and x86_64-apple-darwin10 and powerpc-apple-darwin9 by darwin folks, applied. Rich

[patch] Fix failing nested-3.C on ARM.

2012-06-19 Thread Richard Earnshaw
The regexp in nested-3.C has to parse the machine-specific comment character; on ARM that is '@'. Tested on arm-eabi, where this test now passes. OK? R. * g++.dg/debug/dwarf2/nested-3.C: Add ARM comment character to regexp. --- g++.dg/debug/dwarf2/nested-3.C (revision 188750) +++ g

Re: [PATCH, testsuite]: Fix scan-tree-dump-times argument order in gcc.dg/tree-ssa/vrp68.c.

2012-06-19 Thread Richard Guenther
On Mon, Jun 18, 2012 at 10:01 PM, Janis Johnson wrote: > On 06/17/2012 05:03 AM, Richard Guenther wrote: >> On Sun, Jun 17, 2012 at 10:41 AM, Uros Bizjak wrote: >>> Hello! >>> >>> The testcase still fails on x86_64-pc-linux-gnu with: >>> >>> FAIL: gcc.dg/tree-ssa/vrp68.c scan-tree-dump-times vrp1

Re: [PATCH] Bad code generation: incorrect folding of TARGET_MEM_REF into a constant

2012-06-19 Thread Richard Guenther
On Mon, Jun 18, 2012 at 9:51 PM, Jiří Hruška wrote: > Hi all, > > I have tracked down a bug which results in invalid code being > generated for indexed TARGET_MEM_REF expressions during dominator > optimization. > > The conditions are: accessing objects adjacent in memory in a loop (in > order to

Re: [4.6][ARM] Backport "MCR Not available in Thumb1"

2012-06-19 Thread Richard Earnshaw
On 19/06/12 04:03, Joey Ye wrote: > Backporting trunk r179979 > > OK for 4.6? > > Backported from mainline > 2011-10-14 David Alan Gilbert > > PR target/48126 > * config/arm/arm.c (arm_output_sync_loop): Move label before > barrier. > > Index: gcc/config/arm/a

Re: [patch] Fix PR48109 using artificial top-level asm statements (darwin/objc)

2012-06-19 Thread Richard Guenther
On Mon, Jun 18, 2012 at 7:51 PM, Steven Bosscher wrote: > Hello, > > This patch started as an attempt to remove #include "output.h" from > objc/: Instead of writing references directly to asm_out_file, the > references are output as top-level asm statements. It's a bit of a > hack, but it works an

Re: RFA: Fix PR53688

2012-06-19 Thread Richard Guenther
On Mon, Jun 18, 2012 at 4:59 PM, Michael Matz wrote: > Hi, > > now that we regard MEM_EXPR as a conservative approximation for MEM_SIZE > (and MEM_OFFSET) we must ensure that this is really the case.  It isn't > currently for the string expanders, as they use the MEM_REF (whose address > was taken

Re: [Patch] Adjustments for Windows x64 SEH

2012-06-19 Thread Tristan Gingold
On Jun 18, 2012, at 4:28 PM, Kai Tietz wrote: > Hello Tristan, > > patch works for me, too. Just one nit about the patch. > > 2012/6/18 Tristan Gingold : >> @@ -8558,6 +8558,11 @@ ix86_frame_pointer_required (void) >> if (TARGET_32BIT_MS_ABI && cfun->calls_setjmp) >> return true; >> >> +

Re: [v3] PR 53270 fix hppa-linux bootstrap regression

2012-06-19 Thread Jonathan Wakely
On 14 June 2012 23:23, Jonathan Wakely wrote: > > For 4.6.4 and 4.7.2 I plan to make a less intrusive change, #undef'ing > the __GTHREAD_MUTEX_INIT, _GTHREAD_RECURSIVE_MUTEX_INIT and > __GTHREAD_COND_INIT macros on hppa-linux in C++11 mode, so that the > init functions are used instead.  This fixes

Re: [PATCH, libcpp]: Use x86 __builtin_ia32_pcmpestri128 instead of asm.

2012-06-19 Thread Uros Bizjak
On Tue, Jun 19, 2012 at 8:38 AM, Uros Bizjak wrote: > On Tue, Jun 19, 2012 at 12:07 AM, Richard Henderson wrote: >> On 2012-06-18 13:19, Uros Bizjak wrote: >>>        /* ??? The builtin doesn't understand that the PCMPESTRI read from >>>        memory need not be aligned.  */ >>> -      __asm ("%