Re: [PATCH 1/3] rs6000: Split rs6000_cpu

2017-12-06 Thread David Edelsohn
On Wed, Dec 6, 2017 at 6:42 AM, Segher Boessenkool wrote: > This splits rs6000_cpu into rs6000_cpu and rs6000_tune. Both are still > initialised identically, so there is no behaviour change. > > Now rs6000_cpu should be set by -mcpu=, and rs6000_tune by -mtune= (but >

Re: [PATCH libstdc++/66689] comp_ellint_3 and ellint_3 return garbage values

2017-12-05 Thread David Edelsohn
On Mon, Nov 20, 2017 at 4:51 PM, Jonathan Wakely <jwak...@redhat.com> wrote: > On 20/11/17 21:07 +, Jonathan Wakely wrote: >> >> On 20/11/17 21:01 +, Jonathan Wakely wrote: >>> >>> On 20/11/17 21:43 +0100, Christophe Lyon wrote: >>>> >

Re: [PATCH] rs6000: Set rs6000_cpu correctly (PR43871)

2017-12-02 Thread David Edelsohn
On Sat, Dec 2, 2017 at 1:56 PM, Andreas Schwab wrote: > configured by ../configure, generated by GNU Autoconf 2.64, > with options " '--prefix=/usr' '--build=powerpc64-suse-linux' > '--enable-checking=release' '--enable-shared' 'CFLAGS=-O2 -g' 'CXXFLAGS=-O2 > -g'

Re: [PATCH libstdc++/66689] comp_ellint_3 and ellint_3 return garbage values

2017-11-20 Thread David Edelsohn
This patch has introduced new regressions on at least PowerPC and AArch64. FAIL: ext/special_functions/hyperg/check_value.cc execution test FAIL: tr1/5_numerical_facilities/special_functions/17_hyperg/check_value.cc execution test Thanks, David

Re: [PATCH #2], make Float128 built-in functions work with -mabi=ieeelongdouble

2017-11-16 Thread David Edelsohn
On Thu, Nov 16, 2017 at 12:48 PM, Michael Meissner wrote: > On Thu, Nov 16, 2017 at 04:48:18AM -0600, Segher Boessenkool wrote: >> On Wed, Nov 15, 2017 at 04:56:10PM -0500, Michael Meissner wrote: >> > David tells me that the patch to enable float128 built-in

Re: [PATCH] RISC-V: Add Jim Wilson as a maintainer

2017-11-07 Thread David Edelsohn
On Tue, Nov 7, 2017 at 2:14 AM, Jakub Jelinek wrote: > On Mon, Nov 06, 2017 at 06:39:20PM -0800, Palmer Dabbelt wrote: >> Jim has recently started working at SiFive, where he'll be contributing >> to our GCC port. Andrew, Kito and I would like him to be a mainatiner. >> My

Re: [PATCH,RFC] collect2 LTO for AIX

2017-11-06 Thread David Edelsohn
On Mon, Nov 6, 2017 at 11:01 AM, Jim Wilson <j...@sifive.com> wrote: > On 10/31/2017 12:11 PM, David Edelsohn wrote: >> >> With your recent removal of SDB and -gcoff support, I would appreciate >> your advice about my patch to incrementally add some preliminary LTO >&

Re: [PATCH,RFC] collect2 LTO for AIX

2017-10-31 Thread David Edelsohn
On Mon, Oct 30, 2017 at 3:13 PM, Jeff Law <l...@redhat.com> wrote: > On 10/13/2017 12:04 PM, David Edelsohn wrote: >> The attached patch is an incremental step toward GCC LTO on AIX. The >> recent Libiberty Simple Object improvements for XCOFF provide more >> capabilit

Re: [PATCH, rs6000] (v3) Gimple folding for vec_madd()

2017-10-27 Thread David Edelsohn
On Fri, Oct 27, 2017 at 11:51 AM, Will Schmidt wrote: > Hi, > V3. :-) > > [PATCH, rs6000] (v2) Gimple folding for vec_madd() > > Add support for gimple folding of the vec_madd() (vector multiply-add) > intrinsics. > Renamed the define_insn of altivec_vmladduhm to

Re: [PATCH v2, middle-end]: Introduce memory_blockage named insn pattern

2017-10-14 Thread David Edelsohn
On Sat, Oct 14, 2017 at 1:29 PM, Uros Bizjak <ubiz...@gmail.com> wrote: > On Sat, Oct 14, 2017 at 5:44 PM, David Edelsohn <dje@gmail.com> wrote: >> This patch has broken bootstrap on AIX and possibly powerpc64-linux. >> Was this patch tested on any

Re: [PATCH v2, middle-end]: Introduce memory_blockage named insn pattern

2017-10-14 Thread David Edelsohn
On Sat, Oct 14, 2017 at 11:44 AM, David Edelsohn <dje@gmail.com> wrote: > This patch has broken bootstrap on AIX and possibly powerpc64-linux. > Was this patch tested on any architecture other than x86? > > /nasfarm/edelsohn/src/src/libgcc/emutls.c: In function '__

Re: [PATCH v2, middle-end]: Introduce memory_blockage named insn pattern

2017-10-14 Thread David Edelsohn
This patch has broken bootstrap on AIX and possibly powerpc64-linux. Was this patch tested on any architecture other than x86? /nasfarm/edelsohn/src/src/libgcc/emutls.c: In function '__emutls_get_address': /nasfarm/edelsohn/src/src/libgcc/emutls.c:139:11: internal compiler error: in invalid_void,

Re: Check that there are no missing probabilities

2017-10-13 Thread David Edelsohn
This patch also caused a huge number of testsuite failures on PowerPC, although it didn't break bootstrap. Thanks, David

[PATCH,RFC] collect2 LTO for AIX

2017-10-13 Thread David Edelsohn
The attached patch is an incremental step toward GCC LTO on AIX. The recent Libiberty Simple Object improvements for XCOFF provide more capabilities for operations on XCOFF object files, which are a prerequisite for GCC LTO functionality. This patch adds the basic LTO scanning pass to the COFF

[PATCH,AIX] rs6000 output_aligned_decl_common fix

2017-10-11 Thread David Edelsohn
GCC toplev.c uses ASM_OUTPUT_ALIGNED_DECL_COMMON with a NULL decl to emit the LTO marker, so the rs6000 implementation needs to test for that situation. config/darwin.c tests similarly. With this patch, gcc -flto produces correct output on AIX. Bootstrapped on powerpc-ibm-aix7.2.0.0 Thanks,

Re: [RFC] overflow safe scaling of 64bit values

2017-10-10 Thread David Edelsohn
> Any ideas for better version? If not I will go ahead with this variant and > increase profile probability base. Why not use GCC wide int? - David

Re: [PATCH,AIX] Fix issue with PRI*64 on AIX.

2017-10-10 Thread David Edelsohn
On Tue, Oct 10, 2017 at 10:31 AM, REIX, Tony wrote: > Hi David, > > Since the file ./gcc/go/go-system.h in GCC source code starts by: > > // go-system.h -- Go frontend inclusion of gcc header files -*- C++ -*- > // Copyright (C) 2009-2017 Free Software Foundation, Inc. > >

Re: [PATCH,AIX] Fix issue with PRI*64 on AIX.

2017-10-10 Thread David Edelsohn
On Tue, Oct 10, 2017 at 5:09 AM, REIX, Tony wrote: > Description: > * This patch enables to build on AIX. > > Tests: > * AIX: Build: SUCCESS >- build made by means of gmake within GCC 8 trunk. > > ChangeLog: > * go-system.h : Enable to build on AIX. > (fix issue with

Re: [PATCH,AIX] Manage Go Closure in libffi for AIX

2017-10-03 Thread David Edelsohn
I committed the import of AIX Go support from libffi repository into the GCC copy of libffi. Thanks, David

Re: [PATCH] Enhance PHI processing in VN

2017-09-27 Thread David Edelsohn
On Wed, Sep 27, 2017 at 6:58 PM, Richard Sandiford <richard.sandif...@linaro.org> wrote: > David Edelsohn <dje@gmail.com> writes: >> On Fri, Sep 15, 2017 at 2:53 AM, Richard Biener <rguent...@suse.de> wrote: >>> On Thu, 14 Sep 2017, David Edelsohn wrote: &g

Re: [Patch, Fortran] PR 82143: add a -fdefault-real-16 flag

2017-09-26 Thread David Edelsohn
On Tue, Sep 26, 2017 at 4:44 AM, Janus Weil <ja...@gcc.gnu.org> wrote: > 2017-09-25 23:23 GMT+02:00 Steve Kargl <s...@troutmask.apl.washington.edu>: >> On Mon, Sep 25, 2017 at 11:14:42PM +0200, Janus Weil wrote: >>> 2017-09-25 17:07 GMT+02:00 David Edelsohn <dje.

Re: [Patch, Fortran] PR 82143: add a -fdefault-real-16 flag

2017-09-25 Thread David Edelsohn
promotion_3.f90 and promotion_4.f90 are failing on at least PowerPC and AArch64. Are these new tests limited to x86 or some long double assumptions? f951: Fatal Error: REAL(KIND=16) is not available for '-fdefault-real-16' option compilation terminated. f951: Fatal Error: REAL(KIND=10) is not

Re: [PATCH] Enhance PHI processing in VN

2017-09-15 Thread David Edelsohn
On Fri, Sep 15, 2017 at 2:53 AM, Richard Biener <rguent...@suse.de> wrote: > On Thu, 14 Sep 2017, David Edelsohn wrote: > >> * tree-ssa-sccvn.c (visit_phi): Merge undefined values similar >> to VN_TOP. >> >> This seems to have regressed >> >> FAIL:

Re: [PATCH] Enhance PHI processing in VN

2017-09-14 Thread David Edelsohn
* tree-ssa-sccvn.c (visit_phi): Merge undefined values similar to VN_TOP. This seems to have regressed FAIL: gcc.dg/tree-prof/time-profiler-2.c scan-ipa-dump-times profile "Read tp_first_run: 0" 2 FAIL: gcc.dg/tree-prof/time-profiler-2.c scan-ipa-dump-times profile "Read tp_first_run: 2" 1 FAIL:

Re: Add support to trace comparison instructions and switch statements

2017-09-07 Thread David Edelsohn
On Thu, Sep 7, 2017 at 6:57 PM, Rainer Orth <r...@cebitec.uni-bielefeld.de> wrote: > Jakub Jelinek <ja...@redhat.com> writes: > >> On Wed, Sep 06, 2017 at 10:08:01PM +0200, David Edelsohn wrote: >>> This change broke bootstrap on AIX because sancov.c now refer

Re: Add support to trace comparison instructions and switch statements

2017-09-06 Thread David Edelsohn
This change broke bootstrap on AIX because sancov.c now references a macro that is defined as a function on AIX. sancov.c needs to include tm_p.h to pull in the target-dependent prototypes. The following patch works for me. Is this okay? * sancov.c: Include tm_p.h. Index: sancov.c

Re: C++ PATCH for c++/80767, unnecessary instantiation of generic lambda

2017-09-01 Thread David Edelsohn
On Thu, Aug 31, 2017 at 1:16 PM, Jason Merrill <ja...@redhat.com> wrote: > On Thu, Aug 31, 2017 at 9:37 AM, David Edelsohn <dje@gmail.com> wrote: >> Jason, >> >> The recent patch for 82030 >> >> PR c++/82030 - ICE inheriting from mu

Re: C++ PATCH for c++/80767, unnecessary instantiation of generic lambda

2017-08-31 Thread David Edelsohn
Jason, The recent patch for 82030 PR c++/82030 - ICE inheriting from multiple lambdas PR c++/80767 * call.c (compare_ics): Handle null candidate. causes bootstrap to fail for AIX due to a segfault ICE building libstdc++ with stage1 compiler. - David

[PATCH] rs6000_expand_binop_builtin

2017-08-30 Thread David Edelsohn
The patch to convert rs6000.c:rs6000_expand_binop_builtin to switch statement broke bootstrap because some of the CODE_FOR_XXX labels are not guaranteed to be defined or have unique values in all configurations. Restores bootstrap on AIX. Committed. Thanks, David *

Re: [PATCH] Fix PR bootstrap/81638

2017-08-02 Thread David Edelsohn
To fix the false-positive uninitialized warning, this patch initializes the "incl" variable. * xcoff.c (xcoff_process_linenos): Initialize incl to NULL. Index: xcoff.c === --- xcoff.c (revision 250806) +++ xcoff.c (working

[PATCH] Fix PR bootstrap/81638

2017-08-02 Thread David Edelsohn
To fix the false-positive uninitialized warning, this patch initializes the "incl" variable.

Re: [PATCH 4/N] Recover GOTO predictor.

2017-07-31 Thread David Edelsohn
This patch breaks bootstrap on AIX and probably is a hidden bug on all targets. With the patch, libbacktrace/xcoff.c fails to compile with the error: /nasfarm/edelsohn/src/src/libbacktrace/xcoff.c: In function 'xcoff_add': /nasfarm/edelsohn/src/src/libbacktrace/xcoff.c:822:13: error: 'incl' may

Re: [PATCH,AIX] Enable libffi for AIX

2017-07-28 Thread David Edelsohn
On Wed, Jul 26, 2017 at 11:17 AM, REIX, Tony wrote: > Description: > * This patch enables libffi on AIX. > > Tests: > * Fedora25/x86_64 + GCC trunk : Configure/Build: SUCCESS >- build made by means of gmake. > > ChangeLog: > * configure.ac, configure: Enable libffi for

Re: [PATCH,AIX] Enable Go for AIX

2017-07-28 Thread David Edelsohn
On Wed, Jul 26, 2017 at 8:48 AM, REIX, Tony wrote: > Description: > * This patch enables Go on AIX. > > Tests: > * Fedora25/x86_64 + GCC trunk : Configure/Build: SUCCESS >- build made by means of gmake. > > ChangeLog: > * configure.ac, configure: Enable Go for AIX > *

Re: [PATCH,AIX] Changes for linking gotools on AIX.

2017-07-28 Thread David Edelsohn
On Thu, Jul 27, 2017 at 11:51 AM, REIX, Tony wrote: > Hi Ian, David, > > On AIX, that is more complicated... > > We have to use -static-libgo when building the libgo tests. Because AIX does > not work like Linux does and because the Go libgo tests are done by > duplicating

Re: [PATCH,AIX] Changes for linking gotools on AIX.

2017-07-26 Thread David Edelsohn
On Wed, Jul 26, 2017 at 12:41 PM, REIX, Tony wrote: > Description: > * This patch adds linker options for gotools for AIX. > > Tests: > * Fedora25/x86_64 + GCC trunk : Configure/Build: SUCCESS >- build remade by means of gmake. >- some test redone in libgo (gmake

[PATCH] dwarf2out.c patch for AIX

2017-07-22 Thread David Edelsohn
Recent versions of the AIX toolchain now support the debug_macinfo section. The section header of the macinfo_section includes a reference to the debug lineptr. As with the earlier adjustment to the debug line reference in the debug_info section, the debug_macinfo section requires a similar

[PATCH] dwarf2asm.c for AIX

2017-07-22 Thread David Edelsohn
dw2_asm_output_nstring() essentially inlines the version of OUTPUT_ASCII for the GNU Assembler ELF file format for the debug_asm case to append the comment and calls OUTPUT_ASCII when not debugging. The DWARF portion of the GCC testsuite relies on the text that appears in the comments, so it needs

Re: Deprecate DBX/stabs?

2017-07-21 Thread David Edelsohn
> Nathan Sidwell writes: > Let's at least deprecate it. I attach a patch to do so. With the > patch, you'll get a note about dbx being deprecated whenever you use > stabs debugging on a system that prefers stabs (thus both -g and -gstabs > will warn). On systems where stabs is not

Re: Add support to trace comparison instructions and switch statements

2017-07-21 Thread David Edelsohn
On Fri, Jul 21, 2017 at 1:38 AM, 吴潍浠(此彼) wrote: > Hi Jeff > > I have signed the copyright assignment, and used the name 'Wish Wu' . > Should I send you a copy of my assignment ? Your assignment now is on file in the FSF Copyright Assignment list where Jeff, I and other

Re: [PATCH] Fix memory leaks in libstdc++ ABI tests

2017-07-10 Thread David Edelsohn
The memory leak cleanup has introduced a testsuite failure on AIX: /nasfarm/edelsohn/src/src/libstdc++-v3/testsuite/abi/pr42230.cc: In function 'int main()': /nasfarm/edelsohn/src/src/libstdc++-v3/testsuite/abi/pr42230.cc:15: error: 'free' is not a member of 'std' compiler exited with status 1

Re: [PATCH] rs6000 branch probability changes

2017-07-01 Thread David Edelsohn
On Sat, Jul 1, 2017 at 9:06 AM, Jan Hubicka wrote: >> >> > * config/rs6000/rs6000.c (emit_unlikely_jump): Adjust to new branch >> >> > probability data type. >> >> > >> >> > Index: rs6000.c >> >> > === >> >> > ---

Re: [PATCH] rs6000 branch probability changes

2017-07-01 Thread David Edelsohn
On Fri, Jun 30, 2017 at 7:18 PM, Segher Boessenkool <seg...@kernel.crashing.org> wrote: > On Fri, Jun 30, 2017 at 10:36:27PM +0100, Ramana Radhakrishnan wrote: >> On Fri, Jun 30, 2017 at 2:36 PM, David Edelsohn <dje@gmail.com> wrote: >> > Convert the rs6000 port

[PATCH] rs6000 branch probability changes

2017-06-30 Thread David Edelsohn
Convert the rs6000 port to use the new API for branch probabilities. Okay? Thanks, David * config/rs6000/rs6000.c (emit_unlikely_jump): Adjust to new branch probability data type. Index: rs6000.c === --- rs6000.c (revision 249839)

Re: [PATCH,AIX] Enable XCOFF in libbacktrace on AIX

2017-06-15 Thread David Edelsohn
Ping. https://gcc.gnu.org/ml/gcc-patches/2017-06/msg00383.html https://gcc.gnu.org/ml/gcc-patches/2017-05/msg01186.html Thanks, David

Re: [PATCH 9/9] rs6000: Comment fixes + some leftovers

2017-06-13 Thread David Edelsohn
On Tue, Jun 13, 2017 at 8:53 AM, Segher Boessenkool wrote: > 2017-06-13 Segher Boessenkool > > * config/rs6000/rs6000.c: Update all comments that mentioned SPE. > (rs6000_expand_builtin): Remove RS6000_BTC_EVSEL. >

Re: [PATCH 7/9] rs6000: Remove FIXED_SCRATCH

2017-06-13 Thread David Edelsohn
On Tue, Jun 13, 2017 at 8:53 AM, Segher Boessenkool wrote: > 2017-06-13 Segher Boessenkool > > * config/rs6000/rs6000.h (FIXED_SCRATCH): Delete. Okay. Thanks, david

Re: [PATCH 8/9] rs6000: Remove VECTOR_SPE

2017-06-13 Thread David Edelsohn
On Tue, Jun 13, 2017 at 8:53 AM, Segher Boessenkool wrote: > 2017-06-13 Segher Boessenkool > > * config/rs6000/rs6000-opts.h (enum rs6000_vector): Delete VECTOR_SPE. > * config/rs6000/rs6000.c (rs6000_debug_vector_unit):

Re: [PATCH 6/9] rs6000: Updates to t-rtems

2017-06-13 Thread David Edelsohn
On Tue, Jun 13, 2017 at 8:53 AM, Segher Boessenkool wrote: > 2017-06-13 Segher Boessenkool > > * config/rs6000/t-rtems: Don't handle SPE. Okay. Thanks, David

Re: [PATCH 5/9] rs6000: Updates to t-linux

2017-06-13 Thread David Edelsohn
On Tue, Jun 13, 2017 at 8:53 AM, Segher Boessenkool wrote: > 2017-06-13 Segher Boessenkool > > * config/rs6000/t-linux: Don't handle SPE. Okay. Thanks, David

Re: [PATCH 4/9] rs6000: Remove eabispe.h

2017-06-13 Thread David Edelsohn
On Tue, Jun 13, 2017 at 8:53 AM, Segher Boessenkool wrote: > 2017-06-13 Segher Boessenkool > > * config/rs6000/eabispe.h: Delete file. Okay. Thanks, David

Re: [PATCH 3/9] rs6000: Remove t-spe

2017-06-13 Thread David Edelsohn
On Tue, Jun 13, 2017 at 8:53 AM, Segher Boessenkool wrote: > 2017-06-13 Segher Boessenkool > > * config/rs6000/t-spe: Delete file. Okay. Thanks, David

Re: [PATCH 2/9] rs6000: Remove SPE_CONST_OFFSET_OK

2017-06-13 Thread David Edelsohn
On Tue, Jun 13, 2017 at 8:53 AM, Segher Boessenkool wrote: > 2017-06-13 Segher Boessenkool > > * config/rs6000/rs6000.c (SPE_CONST_OFFSET_OK): Delete. > (rs6000_legitimate_offset_address_p): Return false for anything in >

Re: [PATCH 1/9] rs6000: Sanitize vector modes

2017-06-13 Thread David Edelsohn
On Tue, Jun 13, 2017 at 8:53 AM, Segher Boessenkool wrote: > This removes the vector modes that were only used by SPE. It also > rearranges things so it is easier to see what is there, and for what. > > > 2017-06-13 Segher Boessenkool >

Re: [PATCH][GCC][PATCHv3] Improve fpclassify w.r.t IEEE like numbers in GIMPLE.

2017-06-08 Thread David Edelsohn
Tamar, This patch has caused a large number of new failures on AIX, including compiler ICEs. Thanks, David

Re: [PATCH,AIX] Enable libiberty to read AIX XCOFF

2017-06-07 Thread David Edelsohn
On Wed, Jun 7, 2017 at 10:22 AM, REIX, Tony wrote: > Hi David, > > I'll fix the code incorrectly indented. I already have fixed the indentation in my copy. > > About your comment about our code looking for TEXT section by looking at > string ".text" , please note that our

Re: [PATCH,AIX] Enable XCOFF in libbacktrace on AIX

2017-06-06 Thread David Edelsohn
Tony, This patch seems like a reasonable start for the minimal libbacktrace framework needed by GCC Go. The libbacktrace maintainer, Ian, needs to approve the patch. https://gcc.gnu.org/ml/gcc-patches/2017-05/msg01186.html I'll defer to Ian on the style of the fileline.c changes. Thanks,

Re: [PATCH,AIX] Enable libiberty to read AIX XCOFF

2017-06-06 Thread David Edelsohn
Tony, This patch generally looks good to me -- it clearly is an incremental improvement. One of the libiberty maintainers, such as Ian, needs to approve the patch. https://gcc.gnu.org/ml/gcc-patches/2017-05/msg01181.html + if (strcmp (name, ".text") == 0) +textptr = scnptr; The

Re: [PATCH 14/14] rs6000: Remove rs6000_nonimmediate_operand

2017-06-06 Thread David Edelsohn
On Tue, Jun 6, 2017 at 11:56 AM, Segher Boessenkool wrote: > Now rs6000_nonimmediate_operand is just nonimmediate_operand. > > > 2017-06-06 Segher Boessenkool > > * config/rs6000/predicates.md (rs6000_nonimmediate_operand):

Re: [PATCH 13/14] rs6000: Remove spe_acc and spefscr

2017-06-06 Thread David Edelsohn
On Tue, Jun 6, 2017 at 11:56 AM, Segher Boessenkool wrote: > We can also remove the two other SPE registers. > > > 2017-06-06 Segher Boessenkool > > * config/rs6000/darwin.h (REGISTER_NAMES): Delete the SPE_ACC and >

Re: [PATCH 12/14] rs6000: Remove SPE high registers

2017-06-06 Thread David Edelsohn
On Tue, Jun 6, 2017 at 11:56 AM, Segher Boessenkool wrote: > Now we can remove the SPE high registers. > > > 2017-06-06 Segher Boessenkool > > * config/rs6000/darwin.h (REGISTER_NAMES): Delete the SPE high > registers. >

Re: [PATCH 11/14] rs6000: Remove type attribute "brinc"

2017-06-06 Thread David Edelsohn
On Tue, Jun 6, 2017 at 11:56 AM, Segher Boessenkool wrote: > Nothing uses it anymore. > > > 2017-06-06 Segher Boessenkool > > * config/rs6000/8540.md (ppc8540_brinc): Delete. > * config/rs6000/e500mc.md (e500mc_brinc):

Re: [PATCH 10/14] rs6000: Remove spe.md, spe.h, linuxspe.h

2017-06-06 Thread David Edelsohn
On Tue, Jun 6, 2017 at 11:56 AM, Segher Boessenkool wrote: > 2017-06-06 Segher Boessenkool > > * config.gcc (powerpc*-*-*): Don't add spe.h to extra_headers. > (powerpc*-linux*spe*): Use ${cpu_type} instead of rs6000. >

Re: [PATCH 09/14] rs6000: Remove reg_or_none500mem_operand

2017-06-06 Thread David Edelsohn
On Tue, Jun 6, 2017 at 11:56 AM, Segher Boessenkool wrote: > 2017-06-06 Segher Boessenkool > > * config/rs6000/predicates.md (reg_or_mem_operand): Reformat. > (reg_or_none500mem_operand): Delete. > *

Re: [PATCH 08/14] rs6000: Remove -mspe options

2017-06-06 Thread David Edelsohn
On Tue, Jun 6, 2017 at 11:56 AM, Segher Boessenkool wrote: > 2017-06-06 Segher Boessenkool > > * config/rs6000/rs6000.c (rs6000_option_override_internal): Delete > handling of SPE flags. > *

Re: [PATCH 07/14] rs6000: Remove TARGET_SPE and TARGET_SPE_ABI and friends

2017-06-06 Thread David Edelsohn
On Tue, Jun 6, 2017 at 11:56 AM, Segher Boessenkool wrote: > 2017-06-06 Segher Boessenkool > > * config/rs6000/rs6000-common.c (rs6000_handle_option): Remove > SPE ABI handling. > * config/rs6000/paired.md

Re: [PATCH 06/14] rs6000: Remove UNSPEC_MV_CR_GT

2017-06-06 Thread David Edelsohn
On Tue, Jun 6, 2017 at 11:56 AM, Segher Boessenkool wrote: > 2017-06-06 Segher Boessenkool > > config/rs6000/rs6000.md (UNSPEC_MV_CR_GT): Delete. Okay. Thanks, David

Re: [PATCH 05/14] rs6000: Remove output_e500_flip_gt_bit

2017-06-06 Thread David Edelsohn
On Tue, Jun 6, 2017 at 11:56 AM, Segher Boessenkool wrote: > 2017-06-06 Segher Boessenkool > > * config/rs6000/rs6000-protos.h (output_e500_flip_gt_bit): Delete. > * config/rs6000/rs6000.c: Ditto. Okay. Thanks, David

Re: [PATCH 04/14] rs6000: Remove rs6000_cbranch_operator

2017-06-06 Thread David Edelsohn
On Tue, Jun 6, 2017 at 11:56 AM, Segher Boessenkool wrote: > rs6000_cbranch_operator now is just comparison_operator, so just use > that directly. > > > 2017-06-06 Segher Boessenkool > > * config/rs6000/predicated.md

Re: [PATCH 03/14] rs6000: Remove -mfloat-gprs

2017-06-06 Thread David Edelsohn
On Tue, Jun 6, 2017 at 11:56 AM, Segher Boessenkool wrote: > This deletes -mfloat-gprs and the variables that go with it. > > > 2017-06-06 Segher Boessenkool > > * config/rs6000/rs6000.c: Remove everything related to -mfloat-gprs.

Re: [PATCH 02/14] rs6000: Remove TARGET_E500_{SINGLE,DOUBLE}

2017-06-06 Thread David Edelsohn
On Tue, Jun 6, 2017 at 11:56 AM, Segher Boessenkool wrote: > Similarly, TARGET_E500_{SINGLE,DOUBLE} is always false now. > > > 2017-06-06 Segher Boessenkool > > * config/rs6000/predicates.md: Replace TARGET_E500_DOUBLE and >

Re: [PATCH 01/14] rs6000: Remove TARGET_FPRS

2017-06-06 Thread David Edelsohn
On Tue, Jun 6, 2017 at 11:56 AM, Segher Boessenkool wrote: > Since rs6000 no longer supports SPE, TARGET_FPRS now always is true. > > This makes TARGET_{SF,DF}_SPE always false. Many patterns in spe.md > can now be deleted; which makes it possible to merge e.g. negdd2

Re: [PATCH,DWARF,v2] AIX dwarf2out label fix

2017-06-01 Thread David Edelsohn
On Thu, Jun 1, 2017 at 3:03 PM, Jason Merrill <ja...@redhat.com> wrote: > On 05/18/2017 06:00 AM, David Edelsohn wrote: >> >> This version adds a macro DWARF_INITIAL_LENGTH_SIZE_STR based on >> DWARF_OFFSET_SIZE to define the string expression to append to the >

Re: [PATCH,DWARF,v2] AIX dwarf2out label fix PING

2017-06-01 Thread David Edelsohn
Ping https://gcc.gnu.org/ml/gcc-patches/2017-05/msg01440.html Thanks, David

[PATCH,DWARF,v2] AIX dwarf2out label fix

2017-05-18 Thread David Edelsohn
This version adds a macro DWARF_INITIAL_LENGTH_SIZE_STR based on DWARF_OFFSET_SIZE to define the string expression to append to the label to correct the offset. Because AIX Assembler inserts the section length, the section label generated by GCC points to the wrong location and must be adjusted

Re: [PATCH rs6000] Fix up dg-options for BMI intrinsic tests

2017-05-17 Thread David Edelsohn
On Wed, May 17, 2017 at 4:56 PM, Steven Munroe wrote: > David pointed out that I my earlier X86 BMI intrinsic header submission > was causing make check failures on on powerpc64le platforms. The patch > below tests out on Linux BE powerpc64/32 and should also resolve

Re: [PATCH,AIX] Enable FFI Go Closure on AIX

2017-05-17 Thread David Edelsohn
On Wed, May 17, 2017 at 9:11 AM, REIX, Tony wrote: > Patch has been submitted to libffi github : > https://github.com/libffi/libffi/pull/308 . Good. There still was no ChangeLog included with the patch. Also, the new functions in the patch are not consistent with the

[PATCH,DWARF] AIX dwarf2out label fix

2017-05-16 Thread David Edelsohn
The AIX Assembler inserts the DWARF section lengths itself. dwarf2out.c previously has been modified to not emit the length on AIX. All well and good until GCC emits a label that it believes references the start of a DWARF section, but actually references an address _after_ the section length

Re: [PATCH,AIX] Enable FFI Go Closure on AIX

2017-05-16 Thread David Edelsohn
On Tue, May 16, 2017 at 10:44 AM, REIX, Tony wrote: > Hi David, > > We'll submit the patch to the libffi project asap. > > We have tested this patch on AIX 6.1 with libffi (master from github) in > 32bit and 64bit with same results (same exact failures) when testing with and

Re: [PATCH,AIX] Enable Stack Unwinding on AIX

2017-05-16 Thread David Edelsohn
On Tue, May 16, 2017 at 9:50 AM, REIX, Tony wrote: > Tests: > > The change has been tested in 32bit and 64bit on AIX 6.1, 7.1 & 7.2 by using > the libbacktrace (with XCOFF support) within a signal handler context (we > used a specific back.c program). > And it has been

Re: [PATCH,AIX] Enable FFI Go Closure on AIX

2017-05-15 Thread David Edelsohn
This patch needs to be submitted to the libffi project. Also, the ChangeLog needs to specify exactly what is being changed not "Implement Go Closures". The patch clearly touches existing parts of the files that affect more than simply Go closures. How was this tested? libffi is used in many

Re: [PATCH,AIX] Enable Stack Unwinding on AIX

2017-05-15 Thread David Edelsohn
Please do not email my IBM Notes address with patches. Please copy this Gmail address for patch submissions. > * libgcc/config/rs6000/aix-unwind.h : Implements stack unwinding on AIX. This ChangeLog entry clearly is wrong because aix-unwind.h already implements ppc_aix_fallback_frame_state.

Re: [PATCH, rs6000] Fold vector logicals in GIMPLE

2017-05-15 Thread David Edelsohn
On Mon, May 15, 2017 at 12:24 PM, David Edelsohn <dje@gmail.com> wrote: > On Mon, May 15, 2017 at 9:09 AM, Will Schmidt <will_schm...@vnet.ibm.com> > wrote: >> On Sat, 2017-05-13 at 18:03 -0700, David Edelsohn wrote: >>> On Thu, May 11, 2017 at 3:0

Re: [PATCH, rs6000] Fold vector logicals in GIMPLE

2017-05-15 Thread David Edelsohn
On Mon, May 15, 2017 at 9:09 AM, Will Schmidt <will_schm...@vnet.ibm.com> wrote: > On Sat, 2017-05-13 at 18:03 -0700, David Edelsohn wrote: >> On Thu, May 11, 2017 at 3:09 PM, Segher Boessenkool >> <seg...@kernel.crashing.org> wrote: >> > On Thu, May 11, 20

Re: [PATCH, rs6000] Add x86 instrinsic headers to GCC PPC64LE taget

2017-05-13 Thread David Edelsohn
On Thu, May 11, 2017 at 12:22 PM, Steven Munroe wrote: > On Thu, 2017-05-11 at 09:39 -0500, Segher Boessenkool wrote: >> On Wed, May 10, 2017 at 12:59:28PM -0500, Steven Munroe wrote: >> > > That is just for the testsuite; I meant what happens if a user tries >> > >

Re: [PATCH, rs6000] Fold vector logicals in GIMPLE

2017-05-13 Thread David Edelsohn
On Thu, May 11, 2017 at 3:09 PM, Segher Boessenkool wrote: > On Thu, May 11, 2017 at 02:36:26PM -0500, Will Schmidt wrote: >> On Thu, 2017-05-11 at 14:15 -0500, Segher Boessenkool wrote: >> > Hi! >> > >> > On Thu, May 11, 2017 at 10:53:33AM -0500, Will Schmidt wrote:

Re: [PATCH] prevent -Wno-system-headers from suppressing -Wstringop-overflow (PR 79214)

2017-05-05 Thread David Edelsohn
> Andreas Schwab wrote: > I see this failure on aarch64 with -mabi=ilp32: > FAIL: gfortran.dg/alloc_comp_auto_array_2.f90 -O3 -fomit-frame-pointer > -funroll-loops -fpeel-loops -ftracer -finline-functions (test for excess > errors) > Excess errors: >

Re: Enable Go for AIX

2017-04-26 Thread David Edelsohn
Hi, Tony Why is enabling Go the *first* patch? It seems that the support should be added before one allows it to be configured. Also, what are you trying to accomplish with the changes to "rs6000" in the target triplets. I think that you are trying to distinguish between different versions of

Re: Who broke options.h?

2017-04-25 Thread David Edelsohn
On Tue, Apr 25, 2017 at 11:03 AM, David Malcolm wrote: > On Tue, 2017-04-25 at 06:59 -0700, Steve Kargl wrote: >> Someone (other than Richard who seems to have fixed his >> bootstrap issue) in the last 3 days has broken bootstrap >> on FreeBSD. The generated file

Re: [PATCH] Real fix for AIX exception handling

2017-03-30 Thread David Edelsohn
On Thu, Mar 30, 2017 at 4:11 AM, Michael Haubenwallner <michael.haubenwall...@ssi-schaefer.com> wrote: > On 03/29/2017 10:21 PM, David Edelsohn wrote: >> On Wed, Mar 29, 2017 at 3:50 PM, Jeff Law <l...@redhat.com> wrote: >>> On 03/27/2017 09:41 AM, David Edelsohn

Re: [PATCH] Real fix for AIX exception handling

2017-03-29 Thread David Edelsohn
On Wed, Mar 29, 2017 at 5:20 PM, Jeff Law <l...@redhat.com> wrote: > On 03/29/2017 02:56 PM, David Edelsohn wrote: >> >> On Wed, Mar 29, 2017 at 4:48 PM, Jeff Law <l...@redhat.com> wrote: >>> >>> On 03/29/2017 02:21 PM, David Edelsohn wrote: >

Re: [PATCH] Real fix for AIX exception handling

2017-03-29 Thread David Edelsohn
On Wed, Mar 29, 2017 at 4:48 PM, Jeff Law <l...@redhat.com> wrote: > On 03/29/2017 02:21 PM, David Edelsohn wrote: >> >> >> The problem is GCC EH tables and static linking. libstdc++ and the >> main application are ending up with two separate copies of th

Re: [PATCH] Real fix for AIX exception handling

2017-03-29 Thread David Edelsohn
On Wed, Mar 29, 2017 at 3:50 PM, Jeff Law <l...@redhat.com> wrote: > On 03/27/2017 09:41 AM, David Edelsohn wrote: >>> >>> As far as I have discovered, the real problem with AIX exception handling >>> is >>> that the exception landing pads are symbo

Re: [PATCH] Real fix for AIX exception handling

2017-03-27 Thread David Edelsohn
> As far as I have discovered, the real problem with AIX exception handling is > that the exception landing pads are symbols that must not (but still are) > exported from shared libraries - even libstdc++. > > I'm wondering if attached libtool(!)-patch would fix even that GDB problem > once

Re: [PATCH] For broken exception handling in GDB on AIX platform

2017-03-26 Thread David Edelsohn
Nitish works for IBM and I can stand in as the contributor. The patch would be very helpful to work around an AIX exception handling problem. Because the patch is a generic option and not target-specific, it needs a build machinery maintainer or Global Reviewer to approve. Thanks, David

Re: [PATCH] PR libstdc++/79789 fix non-reserved names in headers

2017-03-10 Thread David Edelsohn
On Fri, Mar 10, 2017 at 10:20 AM, Jonathan Wakely <jwak...@redhat.com> wrote: > On 09/03/17 19:46 +, Jonathan Wakely wrote: >> >> On 03/03/17 10:47 -0500, David Edelsohn wrote: >>> >>> This patch caused a new regression on AIX. >> >> >>

Re: [PATCH] PR libstdc++/79789 fix non-reserved names in headers

2017-03-03 Thread David Edelsohn
This patch caused a new regression on AIX. - David FAIL: 17_intro/names.cc (test for excess errors) Excess errors: /tmp/GCC/gcc/include-fixed/sys/types.h:600: error: expected unqualified-id before '[' token /tmp/GCC/gcc/include-fixed/sys/types.h:600: error: expected ')' before '[' token

Re: [PATCH] portability fix for gcc.dg/strncmp-2.c testcase

2017-02-14 Thread David Edelsohn
On Tue, Feb 14, 2017 at 2:24 PM, Aaron Sawdey wrote: > On Tue, 2017-02-14 at 13:09 -0600, Segher Boessenkool wrote: >> On Tue, Feb 14, 2017 at 11:56:50AM -0600, Aaron Sawdey wrote: >> > This testcase I added failed to compile on AIX or older linux due >> > to >> > the

Re: [PATCH] rs6000: Mark powerpc*-*-*spe* as obsolete

2017-02-14 Thread David Edelsohn
On Tue, Feb 14, 2017 at 10:22 AM, Segher Boessenkool wrote: > As discussed in . > > Is this okay for trunk? > > > Segher > > > 2017-02-14 Segher Boessenkool > > * config.gcc

Re: [PATCH] Fix gcc-5-branch build with libc++

2017-01-31 Thread David Edelsohn
> As discussed with the FreeBSD gcc ports maintainer, building the > gcc-5-branch with libc++ requires standard C++ headers to be included > *before* gcc/system.h, otherwise the redefinition of abort() to > fancy_abort() will cause trouble. > > In trunk r235362, most gcc C++ sources were updated

<    1   2   3   4   5   6   7   8   9   10   >