Re: [PATCH, ARM] stop changing signedness in PROMOTE_MODE

2015-07-08 Thread Jeff Law
On 07/07/2015 10:29 AM, Jim Wilson wrote: My years at Cisco didn't give me a chance to work on the SSA passes, so I don't know much about how they work. But looking at this, I see that PHI nodes are eventually handed by emit_partition_copy in tree-outof-ssa.c, which calls convert_to_mode, so it

[patch] Flatten flags.h

2015-07-08 Thread Andrew MacLeod
While looking the initial results of an include reduction, there were a couple of problematic files that are relatively easy to resolve. This patch addresses flags.h. It includes both options.h and flag-types.h. options.h already includes flag-types.h so including both is kind of silly :-). f

Re: [PATCH], Add IEEE 128-bit floating point to PowerPC, patch #2b

2015-07-08 Thread Michael Meissner
David asked me to change the definition of FLOAT128_IBM_P and FLOAT128_IEEE_P so that they were more consistant. In doing so, I realized that I didn't need to test TARGET_LONG_DOUBLE_128 in those macros, since the mode would never be TFmode, IFmode, or KFmode unless TARGET_LONG_DOUBLE_128 was true

Re: [PATCH], Add IEEE 128-bit floating point to PowerPC, patch #2b

2015-07-08 Thread David Edelsohn
On Wed, Jul 8, 2015 at 8:01 PM, Michael Meissner wrote: > David asked me to change the definition of FLOAT128_IBM_P and FLOAT128_IEEE_P > so that they were more consistant. In doing so, I realized that I didn't need > to test TARGET_LONG_DOUBLE_128 in those macros, since the mode would never be >

Re: [patch] Flatten flags.h

2015-07-08 Thread Jeff Law
On 07/08/2015 05:12 PM, Andrew MacLeod wrote: While looking the initial results of an include reduction, there were a couple of problematic files that are relatively easy to resolve. This patch addresses flags.h. It includes both options.h and flag-types.h. options.h already includes flag-type

Re: [PATCH 15/16][fold-const.c] Fix bigendian HFmode in native_interpret_real

2015-07-08 Thread Jeff Law
On 07/08/2015 03:43 AM, Richard Biener wrote: On Wed, Jul 8, 2015 at 12:07 AM, Jeff Law wrote: On 07/07/2015 06:37 AM, Alan Lawrence wrote: As per https://gcc.gnu.org/ml/gcc-patches/2015-04/msg01346.html. Fixes FAIL of advsimd-intrinsics vcreate.c on aarch64_be-none-elf from previous patch.

Re: [patch 0/9] Flattening and initial module rebuilding

2015-07-08 Thread Andrew MacLeod
On 07/08/2015 06:43 PM, Jeff Law wrote: On 07/07/2015 05:53 PM, Andrew MacLeod wrote: yes, with a bit of tweaking and enhancement they can be generally useful. They are all in python. And no one is allowed to make comments like "OMG thats so inefficient" or "what a horrible way to do that" :-)

Re: [PATCH] Simplify vector compare-not-select sequence

2015-07-08 Thread Jeff Law
On 07/06/2015 07:41 AM, Bill Schmidt wrote: Hi, Due to specifics of the POWER architecture, some forms of a vector compare followed by a vector select are represented in RTL as a compare, followed by a logical NOT, followed by the select. This tends to end up generating an extra instruction. T

Re: Drop -Wswitch-bool warning in function.c

2015-07-08 Thread Kito Cheng
Hi Jeff: Thanks your review and approve, however I don't have commit right yet, can you help me to commit it :) thanks On Thu, Jul 9, 2015 at 5:00 AM, Jeff Law wrote: > On 07/08/2015 02:49 AM, Kito Cheng wrote: >> >> Bootstrapped & regression-tested on x86_64-linux-gnu :) >> >> 2015-07-08 Kito

Re: [PING][PATCH, 1/2] Merge rewrite_virtuals_into_loop_closed_ssa from gomp4 branch

2015-07-08 Thread Jeff Law
On 07/07/2015 09:58 AM, Tom de Vries wrote: [Big snip] 0001-Add-rewrite_virtuals_into_loop_closed_ssa.patch Add rewrite_virtuals_into_loop_closed_ssa 2015-07-07 Tom de Vries * tree-cfg.c (get_virtual_phi): New function. * tree-cfg.h (get_virtual_phi): Declare. * tre

Re: [PATCH C/C++/middle-end] move Wunused-parameter to the FEs

2015-07-08 Thread Jeff Law
On 06/29/2015 09:48 AM, Manuel López-Ibáñez wrote: Wunused-parameter warnings are given from cgraph::finalize_function, which is the middle-end. This is an oddity compared to other -Wunused-* warnings. Moreover, Fortran has its own definition of -Wunused-parameter that conflicts with the middle-e

Re: Patch to make dwarf2 tests pass when dwarf-2 is not default

2015-07-08 Thread Jeff Law
On 07/08/2015 08:53 AM, Carlos Sánchez de La Lama wrote: 2015-07-08 Carlos Sánchez de La Lama * g++.dg/debug/dwarf2/*.C: generate dwarf-2 debug information even when dwarf-2 is not the default. Thanks. Installed on the trunk after verifying they didn't change the results of an

Re: Drop -Wswitch-bool warning in function.c

2015-07-08 Thread Jeff Law
On 07/08/2015 09:25 PM, Kito Cheng wrote: Hi Jeff: Thanks your review and approve, however I don't have commit right yet, can you help me to commit it :) Committed to the trunk. jeff

Re: [PATCH] PR target/66817: Check int_size_in_bytes in ix86_return_in_memory

2015-07-08 Thread Uros Bizjak
On Wed, Jul 8, 2015 at 11:16 PM, H.J. Lu wrote: > ix86_return_in_memory should check negative return from int_size_in_bytes, > similar to other ports. > > Tested on Linux/x86-64. OK for trunk? > > Thanks. > > > H.J. > --- > gcc/ > > PR target/66817 > * config/i386/i386.c (ix86_retu

Re: [PATCH] PR target/66818: Define ATTRIBUTE_ALIGNED_VALUE to 32 for IA MCU

2015-07-08 Thread Uros Bizjak
On Thu, Jul 9, 2015 at 12:53 AM, H.J. Lu wrote: > attribute ((aligned)) should align to the minimum of BIGGEST_ALIGNMENT, > which is 4 bytes for -miamcu. > > Tested on Linux/x86-64. OK for trunk? > > Thanks. > > H.J. > --- > gcc/ > > PR target/66818 > * config/i386/i386.h (ATTRIBU

Re: [PR25530] Convert (unsigned t / 2) * 2 into (unsigned t & ~1)

2015-07-08 Thread Paolo Bonzini
On 07/07/2015 11:08, Richard Biener wrote: > Also I am not sure ceil_div and floor_div can be handled this way. > (5 /[ceil] 2) * 2 == 6 but you compute it as 4. So I am only convinced > trunc_div works this way. Of course also floor_div for unsigned arguments. For signed arguments, ceil_div w

Re: [PATCH] S390: Support -mtune=native and -march=native.

2015-07-08 Thread Dominik Vogt
On Wed, Jul 08, 2015 at 04:18:47PM -0400, DJ Delorie wrote: > > > Version 2 of the patch to enable the configure options > > --with-arch=native and --with-tune=native. > > This patch broke cross-compiling with --target=s390-* > > s390_host_detect_local_cpu is only defined if the --host is s390-*

Re: Drop -Wswitch-bool warning in function.c

2015-07-08 Thread Richard Sandiford
Kito Cheng writes: > Yes, I know it's non-fatal warning, but I think gcc should build with > --enable-werror-always by it's self > and it's the *ONLY* warning in trunk now. Yeah, but it should only build with --enable-werror-always if your host compiler is the same version as the one you're build

<    1   2