Re: [PATCH], PowerPC, Add -mpower9-dform to switches turned on with -mcpu=power9

2016-01-04 Thread David Edelsohn
On Thu, Dec 31, 2015 at 3:41 PM, Michael Meissner wrote: > When I did the inital d-form support for ISA 3.0 (power9) for loading scalar > SF/DF values into Altivec registers, I did not enable -mpower9-dform with the > other ISA 3.0 switches when you used -mcpu=power9.

Re: [PATCH], PowerPC IEEE 128-bit fp, patch #10 (comparison, documentation, conversion, debug)

2015-12-29 Thread David Edelsohn
On Tue, Dec 29, 2015 at 10:43 AM, Michael Meissner wrote: > This is a clean-up patch for IEEE 128-bit floating point support on the > PowerPC. > > The main change is to change the way comparisons are done when IEEE 128-bit > floating is emulated. Previously, I had

Re: [PATCH], PowerPC IEEE 128-bit fp, #11 (enable libgcc conversions)

2015-12-29 Thread David Edelsohn
On Tue, Dec 29, 2015 at 11:38 AM, Michael Meissner wrote: > This patch enables support of the __float128 software emulation functions in > libgcc. The patch is reworked from previous versions of this patch (listed as > patch #8). > > This patch adds support for

Re: [PATCH, rs6000] Add support for lxvx and stxvx P9 instructions

2015-12-28 Thread David Edelsohn
On Sun, Dec 27, 2015 at 6:00 PM, Bill Schmidt wrote: > Hi, > > POWER9 adds endian-neutral load and store vector instructions that > support unaligned accesses. This allows more efficient code generation > than POWER8. With these new instructions, we no longer

Re: [PATCH, rs6000] Fix PR target/68872, make -mcpu=powerpc64le pass correct assembler option

2015-12-22 Thread David Edelsohn
On Tue, Dec 22, 2015 at 2:30 PM, Peter Bergner <berg...@vnet.ibm.com> wrote: > On Tue, 2015-12-22 at 13:36 -0500, David Edelsohn wrote: >> On Tue, Dec 22, 2015 at 12:13 PM, Peter Bergner <berg...@vnet.ibm.com> wrote: >> Ultimately rs6000_file_start ".m

Re: [PATCH, rs6000] Fix PR target/68872, make -mcpu=powerpc64le pass correct assembler option

2015-12-22 Thread David Edelsohn
On Tue, Dec 22, 2015 at 12:13 PM, Peter Bergner wrote: > Currently, -mcpu=powerpc64le correctly sets the TARGET_* flags for an > LE compile, meaning it mimics a -mcpu=power8 compile, but it doesn't > pass the correct -mpower8/-mpwr8 option to the assembler, so we die > with

Re: [PATCH 4/4] Un-XFAIL ssa-dom-cse-2.c for most platforms

2015-12-21 Thread David Edelsohn
On Mon, Dec 21, 2015 at 8:13 AM, Alan Lawrence wrote: > ...the test passes with --param sra-max-scalarization-size-Ospeed. > > Verified on aarch64 and with stage1 compiler for hppa, powerpc, sparc, s390. > > On alpha, tree-optimized is: > > MEM[(int[8] *)] = { 0, 1 }; >

Re: [PATCH] Pr 68805, Fix PowerPC little endian -mvsx-timode

2015-12-16 Thread David Edelsohn
On Wed, Dec 16, 2015 at 6:20 PM, Michael Meissner wrote: > My first mail did not seem to be delivered, so I'm trying again. > > This fixes a bug with the debug switch -mvsx-timode that we would eventually > like to enable by default on PowerPC little endian server

Re: [PATCH] rs6000: Fix a mistake in cstore_si_as_di (PR68865, PR68879)

2015-12-14 Thread David Edelsohn
On Mon, Dec 14, 2015 at 2:04 AM, Segher Boessenkool wrote: > convert_move does not know how to zero-extend a constant integer to the > target mode -- simply because it does not know the target mode. As a > result, 32-bit SImode with the high bit set would be

[PATCH] g++.dg/cpp1y/pr58708.C testcase fix for AIX

2015-12-12 Thread David Edelsohn
I previously updated the assignment to correct the compile-time error, but the runtime check of values also needs to be updated for 16 bit wchar_t. Tested on powerpc-ibm-aix7.1.0.0. Committed. Thanks, David * g++.dg/cpp17/pr58708.C: Test appropriate value for size of wchar_t. Index: pr58708.C

Re: Request permission to delete gcc.dg/vect/costmodel/ppc/costmodel-fast-math-vect-pr29925.c

2015-12-10 Thread David Edelsohn
On Thu, Dec 10, 2015 at 2:23 PM, Bill Schmidt wrote: > Hi, > > The subject test case has been failing as follows: > > FAIL: gcc.dg/vect/costmodel/ppc/costmodel-fast-math-vect-pr29925.c > scan-tree-dump-times vect "vectorization not profitable" 1 > > The test has been

Re: [PATCH 1/2] rs6000: Implement cstore for signed Pmode register compares

2015-12-04 Thread David Edelsohn
On Fri, Dec 4, 2015 at 9:34 AM, Segher Boessenkool wrote: > This implements cstore for the last case we do not yet handle, using > the superopt algo from the venerable CWG. The only integer cases we > do still not handle after this are for -m32 -mpowerpc64. Those

Re: [PATCH 2/2] rs6000: Clean up the cstore code a bit

2015-12-04 Thread David Edelsohn
On Fri, Dec 4, 2015 at 9:34 AM, Segher Boessenkool wrote: > "register_operand" was a bit confusing. Also some other minor cleanups. > > Tested on powerpc64-linux; okay for mainline? > > > Segher > > > 2015-12-04 Segher Boessenkool > > *

[PATCH] PPC sqrtf using rsqrtes (PR 68609)

2015-12-04 Thread David Edelsohn
The PowerPC port provides reciprocal sqrt but doesn't implement the extra incantation to utilize it for sqrtf. The current implementation re-associates terms in the N-R iteration to utilize one constant instead of two, but does not provide a pre-computed estimate multiplied by the source, which

Re: [PATCH] Fix large code model with the ELFv2 ABI

2015-12-02 Thread David Edelsohn
On Wed, Dec 2, 2015 at 2:44 PM, Ulrich Weigand wrote: > Hello, > > this patch fixes support for the large code model with the ELFv2 ABI. > > The global entry point prologue currently assumes that the TOC associated > with a function is less than 2GB away from the function

Re: [PATCH] rs6000: Optimise SImode cstore on 64-bit

2015-12-01 Thread David Edelsohn
On Tue, Dec 1, 2015 at 8:55 PM, Segher Boessenkool wrote: > On 64-bit we can do comparisons of 32-bit values by extending those > values to 64-bit, subtracting them, and then getting the high bit of > the result. For registers this is always cheaper than using the

Re: [PATCH, rs6000] Fix analyze_swaps to handle vperm for large and small code models

2015-12-01 Thread David Edelsohn
> Uli Weigand discovered that the gcc.target/powerpc/swaps-p8-21.c test case fails when large and small code models are used, rather than the default medium code model. This is because analyze_swaps is determining whether the mask used for a vperm insn is loaded from the constant pool, and there

Re: PING: [PATCH] PR target/67215: -fno-plt needs improvements for x86

2015-12-01 Thread David Edelsohn
On Wed, 28 Oct 2015 at 18:14 PM, H.J. Lu wrote: > On Wed, Oct 28, 2015 at 6:11 PM, Bernd Schmidt wrote: >> On 10/29/2015 02:10 AM, H.J. Lu wrote: >>> >>> On Wed, Oct 28, 2015 at 5:23 PM, Jeff Law wrote: So I'll ask again, why did you commit a

Re: [PATCH] rs6000_adjust_cost old thinko

2015-11-30 Thread David Edelsohn
On Mon, Nov 30, 2015 at 4:44 AM, Eric Botcazou wrote: >> Note this also is wrong on PA and one of the SPARC adjust_cost macros. > > Thanks for the heads up, fixed thusly, applied on the mainline > > > PR target/28115 > * config/sparc/sparc.c

[PATCH] rs6000_adjust_cost old thinko

2015-11-29 Thread David Edelsohn
When rs6000_adjust_cost originally was written, the return value from recog_memoized() was handled incorrectly. Note this also is wrong on PA and one of the SPARC adjust_cost macros. Bootstrapped on powrepc-ibm-aix7.1.0.0. PR target/28115 * config/rs6000/rs6000.c (rs6000_adjust_cost): Correct

Re: [PATCH] GCC system.h and Graphite header order

2015-11-27 Thread David Edelsohn
On Fri, Nov 27, 2015 at 11:24 AM, Thomas Schwinge wrote: > Hi! > > On Tue, 24 Nov 2015 10:32:12 +, Alan Lawrence > wrote: >> I note doc/install.texi says that gcc uses "ISL Library version 0.15, >> 0.14, 0.13, or 0.12.2". This patch breaks the

Re: GCC 5.3 Status Report (2015-11-20)

2015-11-25 Thread David Edelsohn
On Wed, Nov 25, 2015 at 11:57 AM, Paolo Bonzini wrote: > Patch committed to upstream libtool, thanks for your understanding. Great! How can I have the patch backported to GCC trunk and 5-branch libtool, and then rebuild configure with the appropriate versions of autoconf? I

[PATCH] AIX declare aliases

2015-11-25 Thread David Edelsohn
The initial rs6000_declare_alais support fort handle AIX function descriptors was no complete. For function descriptors, both the undecorated symbols (the function descriptor) and the "dot" symbol (the function entry point address) must be globalized and renamed. Thanks, David *

Re: [PATCH], Add power9 support to GCC, patch #10 (SFmode/DFmode d-form addressing)

2015-11-24 Thread David Edelsohn
On Tue, Nov 10, 2015 at 4:56 PM, Michael Meissner wrote: > This patch d-form addressing to float/double scalars for the PowerPC that was > added in ISA 3.0 (power9). This patch does not yet turn on D-form addressing > as default. It is likely that patch #11, which

Re: [PATCH] lround for PowerPC

2015-11-24 Thread David Edelsohn
On Tue, Nov 24, 2015 at 1:55 PM, Michael Meissner <meiss...@linux.vnet.ibm.com> wrote: > Whoops, I forgot to update the ChangeLog after reversing Fv and Fv2. > > 2015-11-24 David Edelsohn <dje@gmail.com> > Michael Meissner <meiss...@linux.vnet.ib

Re: [PATCH] rs6000: Fix for and_operand oversight (PR68332, PR67677)

2015-11-24 Thread David Edelsohn
On Tue, Nov 24, 2015 at 2:13 AM, Segher Boessenkool wrote: > Calling rs6000_is_valid_and_mask on a reg instead of on a const_int is > not a good idea, as PR68332 and PR67677 as well as testing with > --enable-checking=yes,rtl show. Fix this. > > Bootstrapped and

Re: GCC 5.3 Status Report (2015-11-20)

2015-11-24 Thread David Edelsohn
On Sun, Nov 22, 2015 at 8:38 AM, Paolo Bonzini <bonz...@gnu.org> wrote: > > > On 20/11/2015 14:14, David Edelsohn wrote: >> On Fri, Nov 20, 2015 at 7:53 AM, Richard Biener <rguent...@suse.de> wrote: >>> >>> Status >>> == >>> >&g

Re: [PATCH] lround for PowerPC

2015-11-23 Thread David Edelsohn
r on x86, but it blew up on a big endian power7 with a segmentation > violation that I need to look into. I'm also building on a little endian > power8 right now, and it has gotten further. > > 2015-11-23 David Edelsohn <dje@gmail.com> > Michael Meissner <meiss.

Re: [PATCH] lround for PowerPC

2015-11-22 Thread David Edelsohn
On Sun, Nov 22, 2015 at 2:34 AM, Richard Biener <richard.guent...@gmail.com> wrote: > On November 22, 2015 2:52:53 AM GMT+01:00, David Edelsohn <dje@gmail.com> > wrote: >>PowerPC was missing a definition of the lroundMN pattern, which can be >>implemented wi

Re: [PATCH] lround for PowerPC

2015-11-22 Thread David Edelsohn
v2 of the patch. Seems to pass the GCC testsuite, although the testsuite doesn't stress FP. There is something wrong with current VSX SFmode constraints. Index: rs6000.md === --- rs6000.md (revision 230723) +++ rs6000.md (working

Re: [PATCH, rs6000, gcc 5 backport] Fix PR target/67808, LRA ICE on double to long double conversion

2015-11-21 Thread David Edelsohn
On Fri, Nov 20, 2015 at 6:47 PM, Michael Meissner wrote: > On Fri, Oct 02, 2015 at 02:04:48PM -0500, Peter Bergner wrote: >> PR67808 exposes a problem with the constraints in the *extenddftf2_internal >> pattern, in that it allows TFmode operands to occupy Altivec

Re: [PATCH] Fix uninitialized src_range within c_expr (Re: libcpp/C FE source range patch committed (r230331))

2015-11-21 Thread David Edelsohn
On Tue, Nov 17, 2015 at 3:12 PM, David Malcolm wrote: > On Tue, 2015-11-17 at 16:24 +0100, Bernd Schmidt wrote: >> On 11/17/2015 04:13 PM, David Malcolm wrote: >> > On Mon, 2015-11-16 at 22:34 +0100, Bernd Schmidt wrote: >> >> >> >> Should c_expr perhaps acquire a constructor

[PATCH] lround for PowerPC

2015-11-21 Thread David Edelsohn
PowerPC was missing a definition of the lroundMN pattern, which can be implemented with VSX instructions available in Power7. Below is a first draft. - David * config/rs6000/rs6000.md (*xsrdpidf2): New define_insn. (lrounddfdi2): New define_expand. diff --git a/gcc/config/rs6000/rs6000.md

Re: [PATCH] Fix uninitialized src_range within c_expr (Re: libcpp/C FE source range patch committed (r230331))

2015-11-21 Thread David Edelsohn
On Sat, Nov 21, 2015 at 3:00 PM, David Malcolm <dmalc...@redhat.com> wrote: > On Sat, 2015-11-21 at 13:54 -0500, David Edelsohn wrote: >> On Tue, Nov 17, 2015 at 3:12 PM, David Malcolm <dmalc...@redhat.com> wrote: >> > On Tue, 2015-11-17 at 16:24 +0100, Bernd Schmidt

[PATCH] GCC system.h and Graphite header order

2015-11-21 Thread David Edelsohn
Graphite relies on the ISL library and includes multiple ISL headers. The ISL headers refer to identifiers that are poisoned for use in GCC. The source files for Graphite were organized to include the ISL headers first, to avoid the identifier poisoning, which breaks some platforms because GCC

Re: GCC 5.3 Status Report (2015-11-20)

2015-11-20 Thread David Edelsohn
On Fri, Nov 20, 2015 at 7:53 AM, Richard Biener wrote: > > Status > == > > We plan to do a GCC 5.3 release candidate at the end of next week > followed by the actual release a week after that. > > So now is the time to look at your regression bugs in bugzilla and > do some

[PATCH] 23_containers/vector/profile/vector.cc

2015-11-20 Thread David Edelsohn
The testcase allocates so much memory that it requires an additional option to enabled higher memory limit on AIX. Bootstrapped on powerpc-ibm-aix7.1.0.0 Thanks, David * testsuite/23_containers/vector/profile/vector.cc: Add maxdata option on AIX. Index: 23_containers/vector/profile/vector.cc

Re: [PATCH 1/2] [graphite] Move codegen related functions to graphite-isl-ast-to-gimple.c

2015-11-19 Thread David Edelsohn
The patch fixed the bootstrap failure. Thanks, David On Thu, Nov 19, 2015 at 3:37 PM, Sebastian Pop wrote: > Fixed in r230625. > > David, please test on your systems, we were not able to reproduce the fails on > x86_64-linux: the linker does optimize away the functions that

Re: [PATCH 1/2] [graphite] Move codegen related functions to graphite-isl-ast-to-gimple.c

2015-11-19 Thread David Edelsohn
This patch broke bootstrap when ISL is not enabled. graphite-isl-ast-to-gimple.c is protected by HAVE_isl but get_false_edge_from_guard_bb() is used outside of Graphite, including sese.c, which is not restricted to HAVE_isl. Please fix. Thanks, David

Re: POWERPC64_TOC_POINTER_ALIGNMENT

2015-11-18 Thread David Edelsohn
On Tue, Nov 17, 2015 at 9:32 PM, Alan Modra wrote: > On Tue, Nov 17, 2015 at 07:53:18PM -0500, Michael Meissner wrote: >> Here is the temporary patch I'm using to get past rs6000.c. But I suspect >> the >> TOC alignment should never be 256. > > Yes, it should be. Recent GNU

PPC bootstrap failure builtin clz

2015-11-18 Thread David Edelsohn
Richard, One of the patches in your recent series caused bootstrap failure on PPC: In file included from /nasfarm/edelsohn/src/src/libgcc/libgcov-driver.c:49:0: /nasfarm/edelsohn/src/src/libgcc/../gcc/gcov-io.c: In function 'gcov_do_dump':

Re: PATCH to shorten_compare -Wtype-limits handling

2015-11-18 Thread David Edelsohn
On Wed, Nov 18, 2015 at 11:26 PM, Jason Merrill wrote: > The rs6000 target was hitting a bootstrap failure due to > -Werror=type-limits. Since warn_tautological_cmp and other warnings avoid > warning if one of the operands comes from a macro, I thought it would make > sense to

Re: [PATCH, VECTOR ABI] Add __attribute__((__simd__)) to GCC.

2015-11-17 Thread David Edelsohn
Kirill, * c-c++-common/attr-simd.c and * c-c++-common/attr-simd-3.c fail on 32 bit systems, e.g., see powerpc64-linux tested in 32 bit mode. - David

[PATCH] g++.dg/cpp1y/pr58708.C wchar_t size

2015-11-17 Thread David Edelsohn
The testcase in the GCC testsuite assumes that wchar_t is 32 bits, which is not correct on AIX. 32 bit AIX maintains 16 bit wchar_t for backward compatibility (64 bit AIX uses 32 bit wchar_t). What is the preferred method to make the testcase safe for smaller wchar_t? The following patch works

Re: C++ PATCH to integrate c++-delayed-folding branch

2015-11-17 Thread David Edelsohn
On Mon, Nov 16, 2015 at 10:47 PM, Jason Merrill <ja...@redhat.com> wrote: > On 11/16/2015 09:39 PM, David Edelsohn wrote: >> >> The PPC port seems to be bootstrapping again, but I'm not sure why. >> Mike Meissner's patch only should have affected long doubl

Re: [PATCH] g++.dg/cpp1y/pr58708.C wchar_t size

2015-11-17 Thread David Edelsohn
On Tue, Nov 17, 2015 at 11:22 AM, Jonathan Wakely <jwakely@gmail.com> wrote: > On 17 November 2015 at 16:04, David Edelsohn wrote: >> The testcase in the GCC testsuite assumes that wchar_t is 32 bits, >> which is not correct on AIX. 32 bit AIX maintains 16 bit wc

Re: [PATCH] g++.dg/init/vbase1.C and g++.dg/cpp/ucn-1.C

2015-11-16 Thread David Edelsohn
On Mon, Nov 16, 2015 at 4:15 AM, Eric Botcazou wrote: >> No RISC architecture can store directly to MEM, so the expected RTL in >> g++.dg/init/vbase1.C is wrong. I am adding XFAIL for PowerPC. This >> probably should be disabled for ARM and other RISC architectures. > >

Re: C++ PATCH to integrate c++-delayed-folding branch

2015-11-16 Thread David Edelsohn
gt; On 11/13/2015 11:46 PM, David Edelsohn wrote: >> >> This seems to have broken bootstrap on PowerPC. I originally saw it >> on AIX, but duplicated it on PPC64LE Linux. > > > Was this a clean build? The gtfiles change doesn't propagate to the > Makefile unless config.status is regenerated. > > Jason >

Re: [PATCH] PR 68192 Export AIX TLS symbols

2015-11-15 Thread David Edelsohn
On Thu, Nov 5, 2015 at 10:53 PM, Alexandre Oliva <aol...@redhat.com> wrote: > On Nov 5, 2015, Paolo Bonzini <bonz...@gnu.org> wrote: > >> On 05/11/2015 17:28, David Edelsohn wrote: >>> [Explicitly copying build maintainers.] >>> >>> Pao

Re: [PATCH] g++.dg/init/vbase1.C and g++.dg/cpp/ucn-1.C

2015-11-14 Thread David Edelsohn
On Sat, Nov 14, 2015 at 8:30 AM, Jakub Jelinek <ja...@redhat.com> wrote: > On Sat, Nov 14, 2015 at 08:09:44AM -0500, David Edelsohn wrote: >> On Sat, Nov 14, 2015 at 5:16 AM, Andreas Schwab <sch...@linux-m68k.org> >> wrote: >> > David Edelsohn <dje

Re: [PATCH] g++.dg/init/vbase1.C and g++.dg/cpp/ucn-1.C

2015-11-14 Thread David Edelsohn
On Sat, Nov 14, 2015 at 8:41 AM, Andreas Schwab <sch...@linux-m68k.org> wrote: > David Edelsohn <dje@gmail.com> writes: > >> On Sat, Nov 14, 2015 at 5:16 AM, Andreas Schwab <sch...@linux-m68k.org> >> wrote: >>> David Edelsohn <dje

[PATCH] testsuite/experimental/random/randint.cc

2015-11-14 Thread David Edelsohn
Testcases that use TLS need additional options on some targets (normall -pthread). This fixes the testcase on AIX and Solaris. Committed as obvious. * testsuite/experimental/random/randint.cc: Add dg-add-options tls. Index: testsuite/experimental/random/randint.cc

Re: [PATCH] g++.dg/init/vbase1.C and g++.dg/cpp/ucn-1.C

2015-11-14 Thread David Edelsohn
On Sat, Nov 14, 2015 at 5:16 AM, Andreas Schwab <sch...@linux-m68k.org> wrote: > David Edelsohn <dje@gmail.com> writes: > >> + int c\u0024c;// { dg-error "not valid in an identifier" { >> target { powerpc-ibm-aix* } } } > > FAIL: g++.dg/c

Re: [patch] GSoC: Implement std::experimental::shared_ptr

2015-11-14 Thread David Edelsohn
The copy_ctor_neg testcase fails on AIX. /nasfarm/edelsohn/src/src/libstdc++-v3/testsuite/experimental/memory/shared_ptr/cons/copy_ctor_neg.cc: In function 'void test02()': /nasfarm/edelsohn/src/src/libstdc++-v3/testsuite/experimental/memory/shared_ptr/cons/copy_ctor_neg.cc:50:44: error: no

Re: libcpp/C FE source range patch committed (r230331).

2015-11-14 Thread David Edelsohn
This patch causes numerous new testsuite failure on AIX caused by the compiler crashing during compilation, e.g. gcc.c-torture/execute/20020206-1.c in GCC libcpp 991 linemap_assert (line >= LINEMAPS_MACRO_LOWEST_LOCATION (set)); (gdb) where #0 _Z11fancy_abortPKciS0_ (

[PATCH] 21_strings/basic_string/capacity/wchar_t/18654.cc

2015-11-13 Thread David Edelsohn
http://www.cplusplus.com/reference/string/basic_string/reserve/ "Note that the resulting string capacity may be equal or greater than n." The current testcase verifies that the capacity is exactly equal to the length of the string or reserve value, but the standard allows the capacity to be

[PATCH] g++.dg/init/vbase1.C and g++.dg/cpp/ucn-1.C

2015-11-13 Thread David Edelsohn
No RISC architecture can store directly to MEM, so the expected RTL in g++.dg/init/vbase1.C is wrong. I am adding XFAIL for PowerPC. This probably should be disabled for ARM and other RISC architectures. Dollar sign is not a valid identifier on AIX, so g++.dg/cpp/ucn-1.C will produce an

Re: C++ PATCH to integrate c++-delayed-folding branch

2015-11-13 Thread David Edelsohn
This seems to have broken bootstrap on PowerPC. I originally saw it on AIX, but duplicated it on PPC64LE Linux. The failure is genoutput seems to be miscompiled by stage1 and produces error messages implying that it did not parse rs6000/vsx.md correctly. I don't know if the miscompilation is

Re: [PATCH] 21_strings/basic_string/capacity/wchar_t/18654.cc

2015-11-13 Thread David Edelsohn
On Fri, Nov 13, 2015 at 1:40 PM, David Edelsohn <dje@gmail.com> wrote: > http://www.cplusplus.com/reference/string/basic_string/reserve/ > > "Note that the resulting string capacity may be equal or greater than n." > > The current testcase verifies tha

Re: [PATCH], Add power9 support to GCC, patch #8 (add integer multiply/add)

2015-11-12 Thread David Edelsohn
On Tue, Nov 10, 2015 at 1:39 PM, Michael Meissner wrote: > This patch adds support for the MADDLD instruciton, which is a fused > multiply/add instruction for integers. At this time, it is for 64-bit > multiplies only. Eventually, we will restructure 128-bit

Re: [PATCH], Add power9 support to GCC, patch #7 (direct move enhancements)

2015-11-12 Thread David Edelsohn
On Sun, Nov 8, 2015 at 7:48 PM, Michael Meissner wrote: > This patch adds support for the new direct move instructions (MFVSRLD and > MTVSRDD) that simplify moving 128-bit data between GPRs and vector registers. > > I have built previous versions of this patch with no

Re: open acc default data attribute

2015-11-12 Thread David Edelsohn
Nathan, The ChangeLog was placed in the wrong files. gcc/ * gimplify.c (oacc_default_clause): New. (omp_notice_variable): Call it. Should go in gcc/ChangeLog without "gcc/" gcc/testsuite/ * c-c++-common/goacc/data-default-1.c: New. should go in

Re: [PATCH], Add power9 support to GCC, patch #6 (IEEE 128-bit hardware support)

2015-11-12 Thread David Edelsohn
On Sun, Nov 8, 2015 at 7:44 PM, Michael Meissner wrote: > This patch adds support for the IEEE 128-bit hardware instructions that are > being added to the PowerPC ISA 3.0 (power9). With this patch, users on power7 > and power8 will use the software emulation

Re: [rs6000] Rotate stack checking loop

2015-11-12 Thread David Edelsohn
On Thu, Nov 12, 2015 at 4:51 PM, Eric Botcazou wrote: > Hi, > > this patch rotates the loop generated in the prologue to do stack checking > when -fstack-check is specified, thereby saving one branch instruction. It > was initially implemented as a WHILE loop to match the

[PATCH] AIX asserts

2015-11-10 Thread David Edelsohn
This patch adds some asserts of cpu and machine to fix an assert testcase and makes the defines consistent with PPC64 Linux. boottrapped on powerpc-ibm-aix7.1.0.0 * config/rs6000/aix.h (TARGET_OS_AIX_CPP_BUILTINS): Add cpu and machine asserts. Update defines for 64 bit. Thanks, David Index:

Re: [PATCH], Add power9 support to GCC, patch #5 (ISA 3.0 fusion)

2015-11-09 Thread David Edelsohn
On Mon, Nov 9, 2015 at 11:57 AM, Segher Boessenkool wrote: > On Mon, Nov 09, 2015 at 12:34:20PM -0500, Michael Meissner wrote: >> > > +(define_insn "*toc_fusionload_" >> > > + [(set (match_operand:QHSI 0 "int_reg_operand" "=,??r") >> > > + (match_operand:QHSI 1

Re: [PATCH], Add power9 support to GCC, patch #3 (scalar count trailing zeros)

2015-11-09 Thread David Edelsohn
On Sun, Nov 8, 2015 at 4:37 PM, Michael Meissner wrote: > This patch adds support for scalar count trailing zeros instruction that is > being added to ISA 3.0 (power9). > > I have built this patch (along with patches #2 and #4) with a bootstrap build > on a power8

Re: [PATCH], Add power9 support to GCC, patch #5 (ISA 3.0 fusion)

2015-11-09 Thread David Edelsohn
On Sun, Nov 8, 2015 at 4:42 PM, Michael Meissner wrote: > This patch adds support for new fusion forms in ISA 3.0 (power9). In > particular, ISA 3.0 can fuse GPR loads of R0, FPR loads, GPR stores, FPR > stores, and some constant generation that ISA 2.07 (power8)

Re: [PATCH 5/6] Simplify rs6000_builtin_vectorized_function

2015-11-09 Thread David Edelsohn
On Mon, Nov 9, 2015 at 8:30 AM, Richard Sandiford wrote: > After the previous patches it's no longer necessary for > TARGET_BUILTIN_VECTORIZED_FUNCTION to return functions that > map to the vector optab of the original operation. We'll use > a vector form of the

Re: [PATCH], Add power9 support to GCC, patch #4

2015-11-09 Thread David Edelsohn
On Sun, Nov 8, 2015 at 4:39 PM, Michael Meissner wrote: > This patch adds support for the EXTSWSLI instruction that is being added to > PowerPC ISA 3.0 (power9). > > I have built this patch (along with patches #2 and #3) with a bootstrap build > on a power8 little

Re: [PATCH], Add power9 support to GCC, patch #1 (revised)

2015-11-09 Thread David Edelsohn
On Sun, Nov 8, 2015 at 4:33 PM, Michael Meissner wrote: > This is patch #1 that I revised. I changed -mfusion-toc to -mtoc-fusion. I > changed the references to ISA 2.08 to 3.0. I added two new debug switches for > code in future patches that in undergoing

Re: [PATCH], Add power9 support to GCC, patch #2 (add modulus instructions)

2015-11-09 Thread David Edelsohn
On Sun, Nov 8, 2015 at 4:36 PM, Michael Meissner wrote: > This is patch #2. It adds support for the new modulus instructions that are > being added in ISA 3.0 (power9): > > I have built this patch (along with patches #3 and #4) with a bootstrap build > on a power8

Re: [PATCH 2/2] rs6000: Extend 20050603-3.c testcase to 64-bit

2015-11-09 Thread David Edelsohn
On Sun, Nov 8, 2015 at 11:33 PM, Segher Boessenkool wrote: > The testcase used to fail on 64-bit, but it was disabled there. > This patch makes it run there, and beefs up the checking of the > generated code a bit. > > Tested on powerpc64-linux

Re: [PATCH], Add power9 support to GCC, patch #5 (ISA 3.0 fusion)

2015-11-09 Thread David Edelsohn
On Mon, Nov 9, 2015 at 2:17 PM, Michael Meissner <meiss...@linux.vnet.ibm.com> wrote: > On Mon, Nov 09, 2015 at 01:11:41PM -0800, David Edelsohn wrote: >> On Mon, Nov 9, 2015 at 11:57 AM, Segher Boessenkool >> <seg...@kernel.crashing.org> wrote: >> > On M

[PATCH] Complete cxa_atexit for AIX

2015-11-07 Thread David Edelsohn
IBM xlC++ compiler provides its own implementation of atexit() to provide correct interaction between atexit() and destructors. GCC needs to provide the same through libgcc. I previously added partial support based on the implementation in GLIBC (copied with FSF permission) and confirmed that

Re: [PATCH] gcc/config.gcc: fix typo for powerpc e6500 cpu_is_64bit

2015-11-06 Thread David Edelsohn
2015-11-06 Arnout Vandecappelle * gcc/config.gcc: fix typo for powerpc e6500 cpu_is_64bit For GCC, please don't send ChangeLog entries as diffs. Applied. Thanks. - David

Re: [PATCH] PR 68192 Export AIX TLS symbols

2015-11-05 Thread David Edelsohn
[Explicitly copying build maintainers.] Paolo and Alexandre, Could you review and help with this patch? TLS symbols in AIX display a new, different symbol type in nm output. Libtool explicitly creates a list of exported symbols for shared libraries using nm and does not recognize the new TLS

Re: [PATCH] PR 68192 Export AIX TLS symbols

2015-11-05 Thread David Edelsohn
<richard.guent...@gmail.com> wrote: > On November 5, 2015 5:38:45 PM GMT+01:00, David Edelsohn <dje@gmail.com> > wrote: >>On Thu, Nov 5, 2015 at 8:34 AM, Paolo Bonzini <bonz...@gnu.org> wrote: >>> >>> >>> On 05/11/2015 17:28, David E

Re: [PATCH] PR 68192 Export AIX TLS symbols

2015-11-05 Thread David Edelsohn
On Thu, Nov 5, 2015 at 8:34 AM, Paolo Bonzini <bonz...@gnu.org> wrote: > > > On 05/11/2015 17:28, David Edelsohn wrote: >> [Explicitly copying build maintainers.] >> >> Paolo and Alexandre, >> >> Could you review and help with this patch? >> &g

[PATCH] Merge from gomp-4_5-branch to trunk

2015-11-05 Thread David Edelsohn
This patch broke bootstrap on AIX. /nasfarm/edelsohn/src/src/libstdc++-v3/src/c++98/ios_init.cc: In static member function 'static bool std::ios_base::sync_with_stdio(bool)': /nasfarm/edelsohn/src/src/libstdc++-v3/src/c++98/ios_init.cc:181:59: internal compiler error: in tree_to_shwi, at

Re: [PATCH] Merge from gomp-4_5-branch to trunk

2015-11-05 Thread David Edelsohn
Sorry for the incorrect blame. I thought the failure was due to GOMP changes, but it appears to be due to the placement new size changes. I am re-testing after Martin's fix. By the way, Martin, the ChangeLog entry is wrong 2015-11-05 Martin Sebor PR c++/67942

Re: [PATCH], PowerPC IEEE 128-bit patch #7 (revised #2), Subpatch #17

2015-11-04 Thread David Edelsohn
On Tue, Oct 27, 2015 at 12:44 PM, Michael Meissner wrote: > This patch adds a test to make sure __float128 and __ibm128 are not allowed to > be combined in binary operations. I re-ran the test suite on power8 little > endian, and this test passed. Once the preceeding

Re: [PATCH] PR 68192 Export AIX TLS symbols

2015-11-03 Thread David Edelsohn
On Tue, Nov 3, 2015 at 9:47 AM, Richard Biener <richard.guent...@gmail.com> wrote: > On Tue, Nov 3, 2015 at 3:23 PM, David Edelsohn <dje@gmail.com> wrote: >> TLS symbols in AIX display a new, different symbol type in nm output. >> Libtool explicitly creates

[PATCH] PR 68192 Export AIX TLS symbols

2015-11-03 Thread David Edelsohn
TLS symbols in AIX display a new, different symbol type in nm output. Libtool explicitly creates a list of exported symbols for shared libraries using nm and does not recognize the new TLS symbols, so those symbols are not exported. This is a regression for TLS support on AIX. This patch updates

Re: [PATCH 2/2] rs6000: Rewrite rs6000_reg_live_or_pic_offset_p

2015-10-31 Thread David Edelsohn
On Sat, Oct 31, 2015 at 1:39 PM, Segher Boessenkool wrote: > This function is quite a puzzle; untangle it. No functional change. > > Tested etc.; okay for trunk? > > > Segher > > > 2015-10-31 Segher Boessenkool > > *

Re: [PATCH 1/2] rs6000: Another PIC LRA fix

2015-10-31 Thread David Edelsohn
On Sat, Oct 31, 2015 at 1:39 PM, Segher Boessenkool wrote: > This one for TARGET_TOC && TARGET_MINIMAL_TOC. Without it, r30 is not > saved in the prologue for functions that do not use r30, but the register > is set later in the prologue anyhow. This made all java

Re: [PATCH], PowerPC IEEE 128-bit patch #7 (revised #2), Subpatch #16

2015-10-29 Thread David Edelsohn
On Fri, Oct 23, 2015 at 2:03 PM, Michael Meissner wrote: > This patch adds a test to make sure __float128 are passed and returned like > vector objects, and not as IBM extended double did. > > This is the last subpatch of patch #7. I have bootstrapped the compiler

Re: [PATCH], PowerPC IEEE 128-bit patch #7 (revised #2), Subpatch #06

2015-10-29 Thread David Edelsohn
On Fri, Oct 23, 2015 at 1:40 PM, Michael Meissner wrote: > This patch sets up all of the emulation functions. > > I have built the compiler with this patch and the previous subpatches (1-4). > I > have bootstrapped the compiler with all 16 subpatches installed, and

Re: [PATCH], PowerPC IEEE 128-bit patch #7 (revised #2), Subpatch #09

2015-10-29 Thread David Edelsohn
On Fri, Oct 23, 2015 at 1:47 PM, Michael Meissner wrote: > This patch is the new patch from the last submission. It sets up a hook so > that > the compiler will not allow IBM extended double and IEEE 128-bit floating > point > to intermix in a binary expression

Re: [PATCH], PowerPC IEEE 128-bit patch #7 (revised #2), Subpatch #07

2015-10-29 Thread David Edelsohn
On Fri, Oct 23, 2015 at 1:43 PM, Michael Meissner wrote: > This patch updates to use the unordered comparison function for IEEE 128-bit > floating point to mimic the behaviour of SFmode/DFmode using the fcmpu > instruction. > > It also restructures the code to allow a

Re: [PATCH], PowerPC IEEE 128-bit patch #7 (revised #2), Subpatch #15

2015-10-29 Thread David Edelsohn
On Fri, Oct 23, 2015 at 2:01 PM, Michael Meissner wrote: > This patch adds the documentation. > > I have built the compiler with this patch and the previous subpatches (1-14). > I have bootstrapped the compiler with all 16 subpatches installed, and there > were no

Re: [PATCH], PowerPC IEEE 128-bit patch #7 (revised #2), Subpatch #04

2015-10-29 Thread David Edelsohn
On Fri, Oct 23, 2015 at 1:36 PM, Michael Meissner wrote: > This patch allows SUBREG's for the reg_or_indexed_operand, which is used when > you have an integral value in a float/vector register, and you want to move > the > value (either via direct move on power8, or

Re: [PATCH], PowerPC IEEE 128-bit patch #7 (revised #2), Subpatch #14

2015-10-29 Thread David Edelsohn
On Fri, Oct 23, 2015 at 2:00 PM, Michael Meissner wrote: > This patch makes TFmode be fully switchable for comparisons. > > I have built the compiler with this patch and the previous subpatches (1-13). > I have bootstrapped the compiler with all 16 subpatches

Re: [PATCH], PowerPC IEEE 128-bit patch #7 (revised #2), Subpatch #05

2015-10-29 Thread David Edelsohn
On Fri, Oct 23, 2015 at 1:39 PM, Michael Meissner wrote: > This patch prevents the compiler from calling the IEEE 128-bit emulation > functions with the vector value in both GPRs and vector registers due to the > fact that the library function did not have a

Re: [PATCH], PowerPC IEEE 128-bit patch #7 (revised #2), Subpatch #02

2015-10-29 Thread David Edelsohn
On Fri, Oct 23, 2015 at 1:30 PM, Michael Meissner wrote: > This patch changes the switch from -mfloat128-software and -mfloat128-none to > be a binary switch, -mfloat128 and -mno-float128. It also provides some of > the > basic setup for IEEE types. It also removes

Re: [PATCH], PowerPC IEEE 128-bit patch #7 (revised #2), Subpatch #01

2015-10-29 Thread David Edelsohn
On Fri, Oct 23, 2015 at 1:26 PM, Michael Meissner wrote: > This patch is the rs6000.h changes. It makes the IEEE 128-bit floating point > type that can go in vector registers a 'vector' type, so that the address code > in rs6000.c that determines whether to use VSX

Re: [PATCH], PowerPC IEEE 128-bit patch #7 (revised #2), Subpatch #11

2015-10-29 Thread David Edelsohn
On Fri, Oct 23, 2015 at 1:52 PM, Michael Meissner wrote: > This patch changes the mangling for __float128. I came to the conclusion that > the current code is so tangled, that it would be better to use U10__float128 > rather than "e". However, if it is felt that we

Re: [PATCH], PowerPC IEEE 128-bit patch #7 (revised #2), Subpatch #13

2015-10-29 Thread David Edelsohn
On Fri, Oct 23, 2015 at 1:58 PM, Michael Meissner wrote: > This patch is the second part to allow TFmode to be IBM extended double or > IEEE > 128-bit floating point depending on switches. > > I have built the compiler with this patch and the previous subpatches

Re: [PATCH], PowerPC IEEE 128-bit patch #7 (revised #2), Subpatch #12

2015-10-29 Thread David Edelsohn
On Fri, Oct 23, 2015 at 1:57 PM, Michael Meissner wrote: > This patch is the first of two rs6000.md patches to straighten out the IFmode, > KFmode, and TFmode support. Part of the change is to change the iterator > names > to be easier to understand, using IEEE128,

Re: [PATCH], PowerPC IEEE 128-bit patch #7 (revised #2), Subpatch #03

2015-10-29 Thread David Edelsohn
On Fri, Oct 23, 2015 at 1:33 PM, Michael Meissner wrote: > This patch defines 3 macros to tell the user whether -mfloat128 is enabled or > not, and whether long double is IBM extended double or IEEE 128-bit floating > point. > > I have built the compiler with this

<    4   5   6   7   8   9   10   11   12   13   >