RE: [PATCH, ARM] Generate conditional compares in Thumb2 state

2011-08-07 Thread Jiangning Liu
In attached new patch, arm_arch_thumb2 is changed to TARGET_THUMB2. I tried that with my patch command line option -mcpu=armv7-a9 doesn't generate IT instruction any longer, unless option "-mthumb" is being added. All of my tests assume command line option -mthumb, while cortex-M0, cortex-M3 corte

[wwwdocs] Announce new ARM/embedded-4_6-branch branch

2011-08-07 Thread Terry Guo
Hi, This new branch intends to provide fixes and enhancements for GCC 4.6 when used with ARM embedded cores. The attached patch adds documentation for this branch. Is it OK to commit? Thanks. BR, Terry svn-arm-embedded-4_6-branch.patch Description: Binary data

[google] Backport r174965 from trunk to google/gcc-4_6 (issue4852046)

2011-08-07 Thread Guozhi Wei
Hi I want to backport r174965 from trunk to google/gcc-4_6, which fixed vect-72.c failure in target arm, as described in http://gcc.gnu.org/ml/gcc-patches/2011-06/msg00927.html Tested with buildit and regression test on arm qemu. OK for google/gcc-4_6 ? thanks Carrot 2011-08-08 Guozhi Wei

Re: PATCH: Add -mavx2 and properly check numbers of mask bits

2011-08-07 Thread Hans-Peter Nilsson
On Sun, 7 Aug 2011, H.J. Lu wrote: > Hi, > > opth-gen.awk has > > print "#define " mask name " (1 << " masknum[vname]++ ")" > > and int has 32bits. We should check > > if (masknum[var] > 32) > > instead of > > if (masknum[var] > 31) IIUC the (int32_t) sign-bit is supposed to be reserved and this i

Re: [PATCH] Fix PR49518

2011-08-07 Thread H.J. Lu
On Tue, Jul 5, 2011 at 2:35 AM, Richard Guenther wrote: > On Tue, 5 Jul 2011, Ira Rosen wrote: > >> Richard Guenther wrote on 04/07/2011 03:30:59 PM: >> > > >> > > Richard Guenther wrote on 04/07/2011 02:38:50 PM: >> > > >> > > > Handling of negative steps broke one of the many asserts in >> > >

Re: [Patch, Fortran, OOP] PR 49638: [OOP] length parameter is ignored when overriding type bound character functions with constant length.

2011-08-07 Thread Janus Weil
>> Alright then. How about this: I'll commit the attached verision of the >> patch (including your suggestions), and we start messing with the >> return values afterwards? Patch is regtested on >> x86_64-unknown-linux-gnu. I hope the test case is sufficient for a >> start. > > This is OK.  Thanks f

PATCH: Add -mavx2 and properly check numbers of mask bits

2011-08-07 Thread H.J. Lu
Hi, opth-gen.awk has print "#define " mask name " (1 << " masknum[vname]++ ")" and int has 32bits. We should check if (masknum[var] > 32) instead of if (masknum[var] > 31) Now, I got #define OPTION_MASK_ISA_X32 (1 << 30) #define OPTION_MASK_ISA_XOP (1 << 31) in options.h. OK for trunk? T

Re: Support -mcpu=native on Solaris/SPARC

2011-08-07 Thread Eric Botcazou
> 2011-07-27 Rainer Orth > > gcc: > * config/sparc/driver-sparc.c: New file. > * config/sparc/x-sparc: New file. > * config.host: Use driver-sparc.o, sparc/x-sparc on > sparc*-*-solaris2*. > * config/sparc/sparc.opt (native): New value for enum > process

PATCH: PR middle-end/49721: convert_memory_address_addr_space may generate invalid new insns

2011-08-07 Thread H.J. Lu
Hi, We transform ptr_extend:DI (plus:SI (FOO:SI) (const_int Y))) to (plus:DI (ptr_extend:DI (FOO:SI)) (const_int Y)) since this is how Pmode != ptr_mode is supported even if the resulting address may overflow/underflow. It is also true for x32 which has zero_extend instead of ptr_extend. I

Re: [Patch, Fortran, OOP] PR 49638: [OOP] length parameter is ignored when overriding type bound character functions with constant length.

2011-08-07 Thread Thomas Koenig
Am 07.08.2011 19:05, schrieb Janus Weil: 2011/8/7 Thomas Koenig: Am 07.08.2011 12:56, schrieb Janus Weil: + /* Check string length. */ + if (proc_target->result->ts.type == BT_CHARACTER +&&proc_target->result->ts.u.cl&&old_target->result->ts.u.cl +&&gfc_dep_compare_expr

Re: [Patch, Fortran, OOP] PR 49638: [OOP] length parameter is ignored when overriding type bound character functions with constant length.

2011-08-07 Thread Janus Weil
2011/8/7 Janus Weil : > 2011/8/7 Thomas Koenig : >> Am 07.08.2011 12:56, schrieb Janus Weil: >>> >>> +      /* Check string length.  */ >>> +      if (proc_target->result->ts.type == BT_CHARACTER >>> +       &&  proc_target->result->ts.u.cl&&  old_target->result->ts.u.cl >>> +       &&  gfc_dep_com

Re: [PATCH] ARM fixed-point support [5.5/6]: argument & return padding for libcalls

2011-08-07 Thread Richard Sandiford
Hi Julian, Julian Brown writes: > This patch allows padding to be specified per-target for libcalls. This > hasn't been traditionally important, because libcalls haven't accepted > quantities which might need padding, but that's no longer true with the > new(-ish) fixed-point support helper funct

Re: [Patch, Fortran, OOP] PR 49638: [OOP] length parameter is ignored when overriding type bound character functions with constant length.

2011-08-07 Thread Janus Weil
2011/8/7 Thomas Koenig : > Am 07.08.2011 12:56, schrieb Janus Weil: >> >> +      /* Check string length.  */ >> +      if (proc_target->result->ts.type == BT_CHARACTER >> +       &&  proc_target->result->ts.u.cl&&  old_target->result->ts.u.cl >> +       &&  gfc_dep_compare_expr (proc_target->result

Re: [RFC PATCH, i386]: Allow zero_extended addresses (+ problems with reload and offsetable address, "o" constraint)

2011-08-07 Thread Uros Bizjak
On Fri, Aug 5, 2011 at 8:51 PM, Uros Bizjak wrote: > As I read this sentence, the RTX is forced into a temporary register, > and reload tries to satisfy "o" constraint with plus ((reg ...) > (const_int ...)), as said at the introduction of "o" constraint a > couple of pages earlier. Unfortunately

Re: [v3] doxygen markup fixes

2011-08-07 Thread Jonathan Wakely
On 7 August 2011 12:53, Jonathan Wakely wrote: > On 7 August 2011 08:30, Benjamin Kosnik wrote: >> >> This fixes markup that gives warnings or errors as part of html/pdf >> documentation generation. Most of this stuff is pretty simple, and is >> just making sure that the documented parameter name

Re: [Patch, Fortran, OOP] PR 49638: [OOP] length parameter is ignored when overriding type bound character functions with constant length.

2011-08-07 Thread Thomas Koenig
Am 07.08.2011 12:56, schrieb Janus Weil: + /* Check string length. */ + if (proc_target->result->ts.type == BT_CHARACTER + && proc_target->result->ts.u.cl&& old_target->result->ts.u.cl + && gfc_dep_compare_expr (proc_target->result->ts.u.cl->length, +

Re: [v3] doxygen markup fixes

2011-08-07 Thread Jonathan Wakely
On 7 August 2011 08:30, Benjamin Kosnik wrote: > > This fixes markup that gives warnings or errors as part of html/pdf > documentation generation. Most of this stuff is pretty simple, and is > just making sure that the documented parameter names exactly match the > signatures. > > tested x86/linux

Re: [PATCH PR43513, 1/3] Replace vla with array - Implementation.

2011-08-07 Thread Tom de Vries
On 07/30/2011 09:21 AM, Tom de Vries wrote: > Hi, > > On 07/28/2011 08:20 PM, Tom de Vries wrote: >> On 07/28/2011 06:25 PM, Richard Guenther wrote: >>> On Thu, 28 Jul 2011, Tom de Vries wrote: >>> On 07/28/2011 12:22 PM, Richard Guenther wrote: > On Wed, 27 Jul 2011, Tom de Vries wrote:

Re: [PATCH] [JAVA] Double.parseDouble(null) throw NullPointerException

2011-08-07 Thread Jie Liu
Hi, When I use gcj on an RTOS(RTEMS), Double.parseDouble(null) throw NumberFormatException, but it should throw NullPointerException. So I add the patch below: Index: natVMDouble.cc === --- natVMDouble.cc (revision 172224) +++ n

Re: [Patch, Fortran, OOP] PR 49638: [OOP] length parameter is ignored when overriding type bound character functions with constant length.

2011-08-07 Thread Janus Weil
>> OK from my side (given Mikael's comments), provided you spell resolve.c with >> two e :-) > > Thanks, guys. Committed as r177545 (including your comments). Ok, with the 'trivial' parts out of the way, the remainder of my previously proposed patch becomes rather compact (see attachment). Thoma

Re: [Patch, Fortran, OOP] PR 49638: [OOP] length parameter is ignored when overriding type bound character functions with constant length.

2011-08-07 Thread Janus Weil
2011/8/7 Thomas Koenig : > Hi Janus, > >> Here is a preparational patch (basically a subset of the >> previous one), which does not do any real changes yet, only some >> preparation and cleanup: >> * It moves check_typebound_override to interface.c and prefixes it >> with gfc_ (I don't like moving

Re: [Patch, Fortran, OOP] PR 49638: [OOP] length parameter is ignored when overriding type bound character functions with constant length.

2011-08-07 Thread Thomas Koenig
Hi Janus, Here is a preparational patch (basically a subset of the previous one), which does not do any real changes yet, only some preparation and cleanup: * It moves check_typebound_override to interface.c and prefixes it with gfc_ (I don't like moving and modifying it at the same time). * It

Re: [patch testsuite]: Adjust gcc.dg/tree-ssa tests for LLP64 target

2011-08-07 Thread Kai Tietz
2011/8/7 Mike Stump : > On Aug 6, 2011, at 6:04 AM, Kai Tietz wrote: >> this adjusts some testcases for LLP64 target x86_64 mingw. > > Ok.  Please watch for any comments on stdarg...  I don't have any, but others > might. Ok, will do. Applied at rev 177543. Thanks, Kai