[patch, libfortran] Fix EOF handling in array I/O

2019-11-23 Thread Thomas Koenig
Hello world, the attached patch fixes a case where transforming do j = 1,1000 read (20,*,end=1)(tdat(j,k),k=1,10) end do 1 continue which on straight transformation yields DO main:j=1 1000 1 READ UNIT=20 FMT=-1 DO main:k=1 10 1 TRANSFER main:tdat(main:j , main:k)

Re: [PATCH] Fix ICE with inline asm "=@cc.." (PR target/92615)

2019-11-23 Thread Uros Bizjak
On Sat, Nov 23, 2019 at 2:09 AM Jakub Jelinek wrote: > > Hi! > > The following testcase ICEs, because ix86_md_asm_adjust emits an invalid > insn that isn't matched. setcc_qi output is nonimmediate_operand and so is > fine, but the problem is if we decide to do a ZERO_EXTEND, because >

RE: Ping: [PATCH V6] Extend IPA-CP to support arithmetically-computed value-passing on by-ref argument (PR ipa/91682)

2019-11-23 Thread Tamar Christina
Hi Honza, > > > > > > > I checked update_jump_functions_after_inlining(), and found one > > > suspicious place: > > > > > > > > for (i = 0; i < count; i++) > > > > { > > > > struct ipa_jump_func *dst = ipa_get_ith_jump_func (args, i); > > > > if (!top) > > > > { > > > >

Re: Ping: [PATCH V6] Extend IPA-CP to support arithmetically-computed value-passing on by-ref argument (PR ipa/91682)

2019-11-23 Thread Jan Hubicka
> Hi Honza, > > > > > > > > > > I checked update_jump_functions_after_inlining(), and found one > > > > suspicious place: > > > > > > > > > > for (i = 0; i < count; i++) > > > > > { > > > > > struct ipa_jump_func *dst = ipa_get_ith_jump_func (args, i); > > > > > if (!top) > > >

Convert inliner to new param infrastructure

2019-11-23 Thread Jan Hubicka
Hi, This patch adds opt_for_fn for all cross module params used by inliner so they can be modified at function granuality. With inlining almost always there are three functions to consider (callee and caller of the inlined edge and the outer function caller is inlined to). I always use the outer

Committed test case for PR 92442

2019-11-23 Thread Thomas Koenig
Hi, the PR is a gcc-9 only regression, for which I just committed a test case to trunk. We can then check what fixed this... 2019-11-23 Thomas Koenig PR fortran/92442 * gfortran.dg/bounds_check_21.f90: New test. ! { dg-do compile } ! { dg-options "-Warray-bounds -O2" } !

[committed] ipa comment spelling fixes

2019-11-23 Thread Jakub Jelinek
Hi! This fixes various comment spelling errors aspell -c found. Committed as obvious to trunk. 2019-11-23 Jakub Jelinek * ipa-fnsummary.c: Fix comment typos. * ipa-ref.h: Likewise. * ipa-predicate.h: Likewise. * ipa-split.c: Likewise. *

Re: Committed test case for PR 92442

2019-11-23 Thread Thomas Koenig
Actually, that was 92422. ChangeLog changed correspondingly.

Re: [patch, libfortran] Fix EOF handling in array I/O

2019-11-23 Thread Thomas Koenig
Am 23.11.19 um 14:15 schrieb Thomas Koenig:     * gfortran.dg/eof_4.f90: New test. This should be eof_6.f90 (and will be on commit). Regards Thomas

Re: [PATCH 2/4] The main m68k cc0 conversion

2019-11-23 Thread John Paul Adrian Glaubitz
Hi Jeff! > On 11/13/19 6:23 AM, Bernd Schmidt wrote: >> Once more with patch. >> >> >> Bernd >> >> >> m68k-2.diff >> >> PR target/91851 >> * config/m68k/m68k-protos.h (output-dbcc_and_branch): Adjust >> declaration. >> (m68k_init_cc): New

Re: Add a new combine pass

2019-11-23 Thread Segher Boessenkool
On Sat, Nov 23, 2019 at 06:01:28PM -0500, Nicholas Krause wrote: > Please just CC to this conversation as I keep getting removed. Everyone who was on Cc: for this thread still is. This is how email works. If you want to see everything on the list, subscribe to the mailing list? Segher

Re: Prevent all uses of DFP when unsupported (PR c/91985)

2019-11-23 Thread Jeff Law
On 11/22/19 9:40 AM, Joseph Myers wrote: > Code that directly uses _Decimal* types on architectures not > supporting DFP is properly diagnosed ("error: decimal floating-point > not supported for this target"), via a call to > targetm.decimal_float_supported_p, if the _Decimal32, _Decimal64 or >

Re: Properly handle C2x attributes on types

2019-11-23 Thread Jeff Law
On 11/21/19 7:42 PM, Joseph Myers wrote: > attribs.c has code to ignore all scoped attributes appertaining to > types except when they are part of the definition of that type. > > I think the premise of that code is incorrect, and its presence is a > bug; such attributes are clearly valid in both

Re: Add a new combine pass

2019-11-23 Thread Segher Boessenkool
Hi! On Mon, Nov 18, 2019 at 05:55:13PM +, Richard Sandiford wrote: > Richard Sandiford writes: > > (It's 23:35 local time, so it's still just about stage 1. :-)) > > Or actually, just under 1 day after end of stage 1. Oops. > Could have sworn stage 1 ended on the 17th :-( Only realised >

Re: Add a new combine pass

2019-11-23 Thread Nicholas Krause
On 11/23/19 5:34 PM, Segher Boessenkool wrote: Hi! On Mon, Nov 18, 2019 at 05:55:13PM +, Richard Sandiford wrote: Richard Sandiford writes: (It's 23:35 local time, so it's still just about stage 1. :-)) Or actually, just under 1 day after end of stage 1. Oops. Could have sworn stage

Re: [PATCH 2/4] The main m68k cc0 conversion

2019-11-23 Thread Jeff Law
On 11/23/19 10:14 AM, John Paul Adrian Glaubitz wrote: > Hi Jeff! > >> On 11/13/19 6:23 AM, Bernd Schmidt wrote: >>> Once more with patch. >>> >>> >>> Bernd >>> >>> >>> m68k-2.diff >>> >>> PR target/91851 >>> * config/m68k/m68k-protos.h (output-dbcc_and_branch): Adjust >>>

Re: [PATCH 2/4] The main m68k cc0 conversion

2019-11-23 Thread Oleg Endo
On Sat, 2019-11-23 at 10:36 -0700, Jeff Law wrote: > > > Any news on this? I would be in favor of merging these patches as I > > have > > tested them successfully on Debian by building the gcc-snapshot > > package > > with the patches applied. I used all four patches plus the > > additional one >

Re: [PATCH 2/4] The main m68k cc0 conversion

2019-11-23 Thread Bernd Schmidt
On 11/23/19 6:36 PM, Jeff Law wrote: > Not really. I've already indicated to Bernd that he should go ahead and > commit the changes and we can iterate on any problems that arise. In the meantime I've made an aranym setup in addition to the qemu setup I had, and I've not been able to reproduce

Re: Add a new combine pass

2019-11-23 Thread Nicholas Krause
On 11/23/19 6:09 PM, Segher Boessenkool wrote: On Sat, Nov 23, 2019 at 06:01:28PM -0500, Nicholas Krause wrote: Please just CC to this conversation as I keep getting removed. Everyone who was on Cc: for this thread still is. This is how email works. If you want to see everything on the

[PATCH] Fix libstdc++ compiling for an aarch64 multilib with big-endian.

2019-11-23 Thread apinski
From: Andrew Pinski Hi if we have a aarch64 compiler that has a big-endian multi-lib, it fails to compile libstdc++ because simd_fast_mersenne_twister_engine is only defined for little-endian in ext/random but ext/opt_random.h thinks it is defined always. OK? Built an aarch64-elf toolchain

Re: [PATCH 2/4] The main m68k cc0 conversion

2019-11-23 Thread Bernd Schmidt
On 11/23/19 9:53 PM, Bernd Schmidt wrote: > I'll spend a few more days trying to see if I can do something about the > bootstrap failure Mikael saw (currently trying to do a two-stage cross > build rather than a really slow bootstrap). Whew, I think I have it. One tst instruction eliminated when