Removed charlen_03.f90 and charlen_10.f90 testcases

2016-12-17 Thread Steve Kargl
I have committed a patch to remove the charlen_03.f90 and charlen_10.f90 testcases. These have been attached to PR fortran/78746. Index: ChangeLog === --- ChangeLog (revision 243777) +++ ChangeLog (working copy) @@ -1,3 +1,9 @@ +

Re: [Patch, Fortran, committed] PR 78592: [7 Regression] ICE in gfc_find_specific_dtio_proc, at fortran/interface.c:4939

2016-12-17 Thread Janus Weil
2016-12-17 22:49 GMT+01:00 Janus Weil : > Hi Mikael, > >>> I have just committed a completely obvious patch for this PR. All it >>> does is rearrange some expressions to avoid an ICE (see attachment): >>> >> I have made a late review of it, and I think it’s not as innocent as it >> seems. >> With i

Re: [PATCH] fix powerpc64le bootstrap failure caused by r243661 (PR 78817)

2016-12-17 Thread Martin Sebor
On 12/17/2016 01:01 PM, Markus Trippelsdorf wrote: On 2016.12.16 at 18:27 +0100, Jakub Jelinek wrote: On Fri, Dec 16, 2016 at 10:10:00AM -0700, Martin Sebor wrote: No. The first call to sm_read_sector just doesn't exit. So it is warning about dead code. If the code is dead then GCC should e

Re: [Patch, Fortran, committed] PR 78592: [7 Regression] ICE in gfc_find_specific_dtio_proc, at fortran/interface.c:4939

2016-12-17 Thread Janus Weil
Hi Mikael, >> I have just committed a completely obvious patch for this PR. All it >> does is rearrange some expressions to avoid an ICE (see attachment): >> > I have made a late review of it, and I think it’s not as innocent as it > seems. > With it, if the first element’s formal is not properly

Re: [Patch, Fortran, committed] PR 78592: [7 Regression] ICE in gfc_find_specific_dtio_proc, at fortran/interface.c:4939

2016-12-17 Thread Mikael Morin
Hello, Le 30/11/2016 à 10:52, Janus Weil a écrit : Hi all, I have just committed a completely obvious patch for this PR. All it does is rearrange some expressions to avoid an ICE (see attachment): I have made a late review of it, and I think it’s not as innocent as it seems. With it, if the f

Re: [PATCH] fix powerpc64le bootstrap failure caused by r243661 (PR 78817)

2016-12-17 Thread Markus Trippelsdorf
On 2016.12.16 at 18:27 +0100, Jakub Jelinek wrote: > On Fri, Dec 16, 2016 at 10:10:00AM -0700, Martin Sebor wrote: > > > No. The first call to sm_read_sector just doesn't exit. So it is warning > > > about dead code. > > > > If the code is dead then GCC should eliminate it. With it eliminated >

Re: [PATCH] Fix -fcompare-debug sanopt bug (PR sanitizer/78832)

2016-12-17 Thread Richard Biener
On December 16, 2016 9:56:10 PM GMT+01:00, Jakub Jelinek wrote: >Hi! > >The following testcase fails with -fcompare-debug, because we have a bb >containing 2 ASAN_MARK (POISON, ...) calls immediately after each >other, >followed with -g only by debug stmts till end of basic block. >sanitize_asan_

Re: [RFA] [PR tree-optimization/33562] [PATCH 1/4] Byte tracking in DSE

2016-12-17 Thread Richard Sandiford
Jeff Law writes: > This is the first of the 4 part patchkit to address deficiencies in our > DSE implementation. > > This patch addresses the P2 regression 33562 which has been a low > priority regression since gcc-4.3. To summarize, DSE no longer has the > ability to detect an aggregate store

Re: [PATCH 1/2] [ARC] Generating code for profiling.

2016-12-17 Thread Matthias Klose
On 16.12.2016 14:16, Claudiu Zissulescu wrote: > Committed with the suggested mods. > > Thank you for your review, > Claudiu > >> -Original Message- >> From: Andrew Burgess [mailto:andrew.burg...@embecosm.com] >> Sent: Wednesday, December 14, 2016 1:13 PM >> To: Claudiu Zissulescu >> Cc:

[PATCH, i386]: Further cleanups to bitmanip instructions

2016-12-17 Thread Uros Bizjak
The patch moves splitting to after epilogue_completed, so eventual regrename pass won't break register matching requirements for TARGET_AVOID_FALSE_DEP_FOR_BMI fix. Also, remove unneeded expanders and merge a couple of patterns. 2016-12-17 Uros Bizjak * config/i386/i386.md (*tzcnt_1): Mer

[Ada] Fix PR ada/78845

2016-12-17 Thread Simon Wright
The function Ada.Numerics.Generic_Real_Arrays.Inverse is required (ARM G.3.1(72)) to return a matrix with the bounds of the dimension indices swapped, i.e. result'Range(1) == input'Range(2) and vice versa. The present code gets result'Range(1) correct, but result'Range(2) always starts at 1. Of

Re: [RFA] [PR tree-optimization/33562] [PATCH 1/4] Byte tracking in DSE

2016-12-17 Thread Richard Biener
On December 16, 2016 7:43:22 PM GMT+01:00, Jakub Jelinek wrote: >On Fri, Dec 16, 2016 at 06:35:58PM +, Joseph Myers wrote: >> On Thu, 15 Dec 2016, Jeff Law wrote: >> >> > This version attacks the problem by improving DSE to track stores >to memory at >> > a byte level. That allows us to det

Re: [RFA] [PR tree-optimization/33562] [PATCH 1/4] Byte tracking in DSE

2016-12-17 Thread Jeff Law
On 12/16/2016 12:29 AM, Trevor Saunders wrote: On Thu, Dec 15, 2016 at 06:54:43PM -0700, Jeff Law wrote: unsigned cnt = 0; + bitmap live_bytes = NULL; + bitmap orig_live_bytes = NULL; *use_stmt = NULL; + /* REF is a memory write. Go ahead and get its base, size, extent + informat