Re: Make tests less istreambuf_iterator implementation dependent

On 28/09/2017 23:56, Jonathan Wakely wrote: On 28/09/17 21:59 +0200, François Dumont wrote: The current istreambuf_iterator implementation capture the current streambuf state each time it is tested for eof or evaluated. This is why I considered those tests as fragile. Yes, and I think

[PATCH] [graphite] translate reads and writes in a single traversal of memory ops

The patch moves the code that translates reads and writes to isl representation in a same loop. This is to avoid traversing the scop blocks and arrays with memory operations 3 times. * graphite-dependences.c (scop_get_reads): Move code to... (scop_get_must_writes): Move code

Re: [PING][PATCH] Output DIEs for outlined OpenMP functions in correct lexical scope

On Fri, 22 Sep 2017 18:08:39 +0200 Jakub Jelinek wrote: > On Thu, Sep 21, 2017 at 01:08:01PM -0700, Kevin Buettner wrote: > > Ping. > > Ok, thanks. Committed. Kevin > > On Mon, 7 Aug 2017 17:51:38 -0700 > > Kevin Buettner wrote: > > > > > On Wed,

Re: [PATCH] x32: Add and use libgnarl/s-taprop__x32.adb

On 10/1/17, Arnaud Charlet wrote: >> s-taprop.adb failed to compile for x32: >> >> s-taprop.adb:341:29: operator for type "System.Linux.time_t" is not >> directly visible >> s-taprop.adb:341:29: add with_clause and use_clause for "Linux" >> >> which is caused by >> >>

Re: [PATCH] Fix fortran/81509

On Sun, Oct 01, 2017 at 07:42:09PM +0200, Janus Weil wrote: > 2017-09-29 16:08 GMT+02:00 Steve Kargl : > > On Fri, Sep 29, 2017 at 11:15:47AM +0200, Janus Weil wrote: > >> Hi Steve, > >> > >> > As aside effect, the patch removes a questionable GNU Fortran > >> >

MAINTAINERS - maintainership also covers web pages, docs, testsuite

When talking with David (Malcolm) at the Tools Cauldron, he hinted and/or I realized that documenting the below in our MAINTAINERS file might be a good idea. I went ahead and applied the patch below. (Mike, this is also based on some comments of yours.) Thanks! Gerald 2017-10-01 Gerald

Re: [PATCH] Fix fortran/81509

On Sun, Oct 1, 2017 at 8:42 PM, Janus Weil wrote: > 2017-09-29 16:08 GMT+02:00 Steve Kargl : >> On Fri, Sep 29, 2017 at 11:15:47AM +0200, Janus Weil wrote: >>> Hi Steve, >>> >>> > As aside effect, the patch removes a questionable GNU Fortran

Re: [PATCH] Fix fortran/81509

2017-09-29 16:08 GMT+02:00 Steve Kargl : > On Fri, Sep 29, 2017 at 11:15:47AM +0200, Janus Weil wrote: >> Hi Steve, >> >> > As aside effect, the patch removes a questionable GNU Fortran >> > extension that allowed arguments to IAND, IOR, and IEOR to have >> >

Re: [patch, libfortran] Fix thead sanitizer issue with libgfortran

Hi Bernd, I believe that all omp threads are created in detached state, so pthread_join should be undefined on them, just tsan*thinks* otherwise? When I look further on the libgomp sources, I see there are two completely different implementations of the mutexes, barriers, etc. One using posix

Re: Don't query the frontend for unsupported types

Richard Biener writes: > On Fri, Sep 22, 2017 at 6:42 PM, Richard Sandiford > wrote: >> Richard Biener writes: >>> On Thu, Sep 21, 2017 at 2:56 PM, Richard Sandiford >>> wrote:

Fix mismatched precisions in tree arithmetic

The tree wi:: decompose routine wasn't asserting that the requested precision matched the tree's precision. This could make a difference for unsigned trees that are exactly N HWIs wide and that have the upper bit set, since we then need an extra zero HWI when extending it to wider precisions (as

[committed][PATCH] Simplify relationals into simple equality conditionals in DOM

A short while ago Martin Liska posted a patch that lowered certain switch statements into cascading conditionals. His work tripped two regressions in the testsuite, both cases where we did not optimize as well as we should have. Upon investigation I realized a simple improvement to DOM would

Re: [patch, libfortran] Fix thead sanitizer issue with libgfortran

On 10/01/17 15:41, Thomas Koenig wrote: > Am 01.10.2017 um 10:59 schrieb Bernd Edlinger: >> maybe there is a way how you could explicitly join >> all running threads? > > Yes, that seems to do the trick. Thanks! > Oh, that is really very surprising... I believe that all omp threads are created

Re: [Patch, fortran] PR 82312 - [7/8 Regression] Pointer assignment to component of class variable results wrong vptr for the variable

Hi Paul, Bootstraps and regtests on FC23/x86_64 - OK for trunk and 7 branch? OK for both. Thanks for the patch! Regards Thomas

Re: [patch, libfortran] Fix thead sanitizer issue with libgfortran

Am 01.10.2017 um 10:59 schrieb Bernd Edlinger: maybe there is a way how you could explicitly join all running threads? Yes, that seems to do the trick. Thanks! Here is a patch which appears to work. It does hit a snag with static linking, though, because it calls __gthread_self (), and that

Re: [Patch, fortran] PR77296 - [F03] Compiler Error with allocatable string and associate

Hi Paul, The attached patch fixes the PR and most of the remaining, if not all, problems associated with deferred string length targets in the associate construct. Bootstraps and regtests on FC23/x86_64 - OK for trunk? Yes. Thanks a lot for working on this! Regards Thomas

Re: [Patch, fortran] PR77296 - [F03] Compiler Error with allocatable string and associate

Dear Paul, > The attached patch fixes the PR and most of the remaining, if not all, > problems associated with deferred string length targets in the > associate construct. The patch works as expected. It also fixes pr60458 and its duplicate pr65187. Thanks for working on this issue, Dominique

Re: [PATCH gfortran v2] PR 61450: ICE in gfc_global_used()

Committed as revision r253328 with the change logs 2017-10-01 Dominique d'Humieres PR fortran/61450 * parse.c (gfc_global_used): Replace the gfc_internal_error with an error. 2017-10-01 Dominique d'Humieres PR

Re: isl scheduler and spatial locality (Re: [PATCH][GRAPHITE] More TLC)

On Sat, Sep 30, 2017 at 07:47:43PM +0200, Richard Biener wrote: > On September 29, 2017 9:58:41 PM GMT+02:00, Sebastian Pop > wrote: > >On Fri, Sep 29, 2017 at 2:37 PM, Sven Verdoolaege > > wrote: > >> [Sorry for the resend; I used the wrong email

Re: [PATCH] x32: Add and use libgnarl/s-taprop__x32.adb

> s-taprop.adb failed to compile for x32: > > s-taprop.adb:341:29: operator for type "System.Linux.time_t" is not > directly visible > s-taprop.adb:341:29: add with_clause and use_clause for "Linux" > > which is caused by > > 2017-09-25 Doug Rupp > > *

Re: [patch, libfortran] Fix thead sanitizer issue with libgfortran

Hi, I think this might be a false positive tsan warning. Maybe tsan does not see why a desctructor function can not execute before the last detached thread terminates. I created a small test case that receives a warning which is similar to the one you tried to fix with your patch: cat test.c

[PATCH] x32: Add and use libgnarl/s-taprop__x32.adb

s-taprop.adb failed to compile for x32: s-taprop.adb:341:29: operator for type "System.Linux.time_t" is not directly visible s-taprop.adb:341:29: add with_clause and use_clause for "Linux" which is caused by 2017-09-25 Doug Rupp * libgnarl/s-taprop__linux.adb

Re: [PATCH gfortran v2] PR 61450: ICE in gfc_global_used()

Hi Dominique, The patch is OK for trunk. Thanks! Thomas

[PATCH][PR 81376] Remove unnecessary float casts in comparisons

Hi all, (Previous mail was sent with spurious HTML, sorry!) This patch gets rid of float casts in comparisons when all values of casted integral type are exactly representable by the float type (https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81376). Bootstrapped and regtested on x64_64. Ok to

Re: [patch, libfortran] Fix thead sanitizer issue with libgfortran

On Fri, Sep 29, 2017 at 9:53 PM, Thomas Koenig wrote: > Am 29.09.2017 um 10:03 schrieb Janne Blomqvist: > >> >> 1) I'm confused why fbuf_destroy is modified. The fbuf structure is >> part of gfc_unit, and should be accessed with the same locking rules >> as the rest of the