Re: [PATCH, gfortran] Cleanup the submodule tests

2017-04-15 Thread Dominique d'Humières
I am currently testing the following patch that handle both modules and submodules. It is a little bit clumsy and may not handle all the possible syntax variants. Any comment welcomed!-) Testing in progress. Dominique --- ../_clean/gcc/testsuite/lib/fortran-modules.exp 2017-01-01

[PATCH, testsuite]: Fix some more dg-do directives

2017-04-15 Thread Uros Bizjak
2017-04-15 Uros Bizjak * gcc.dg/pr54669.c: Fix dg-do directive. * gcc.dg/pr78582.c: Ditto. * gcc.dg/pr80020.c: Ditto. * gcc.dg/torture/pr57993-2.cpp: Ditto. * gcc.dg/tree-ssa/builtin-sprintf-3.c: Ditto. * gcc.dg/tree-ssa/builtin-sprintf-4.c: Ditto.

Re: [PATCH] Cleanup dg directives

2017-04-15 Thread Dominique d'Humières
Committed as revision r246940. Dominique > Le 15 avr. 2017 à 16:56, Janus Weil a écrit : > > 2017-04-15 16:46 GMT+02:00 Dominique d'Humières : >> I am planning to commit as obvious the following changes > > Sure! But ... > > >> ---

Re: [PATCH, gfortran] Cleanup the submodule tests

2017-04-15 Thread Janus Weil
2017-04-15 17:54 GMT+02:00 Dominique d'Humières : >>> This is indeed doable, but before I’ld like to improve the module cleanup >>> with the following patch >> >> Yes, looks very useful to me (makes the regexps much more compact & >> readable). In addition, couldn't one use \s

Re: [PATCH, gfortran] Cleanup the submodule tests

2017-04-15 Thread Dominique d'Humières
> Le 15 avr. 2017 à 17:13, Janus Weil a écrit : > > 2017-04-15 15:57 GMT+02:00 Dominique d'Humières : >> >>> Le 15 avr. 2017 à 13:00, Janus Weil a écrit : >>> >>> Hi Dominique, >>> >>> if I'm not mistaken, the cleanup of module file

Re: [PATCH, testsuite] Allow braces around relative line numbers

2017-04-15 Thread Mike Stump
On Apr 15, 2017, at 1:07 AM, Tom de Vries wrote: > I've attached two patches: > > - second patch is the result of running the script. > Second patch OK for stage4/stage1 trunk? Ok. > [ I'm not sure if the first patch is useful enough to commit to contrib. ] Once the

[PATCH 7/7] [D] libiberty: Refactor and add tests for decoding hexdigits.

2017-04-15 Thread Iain Buclaw
Similar to the patch for decoding numbers, puts the handling of character decoding into a designated function with the same style caller interface. Removes one nested level and decoding logic from `dlang_parse_string'. Added some more tests, which brings overall coverage of this code to all but

[PATCH 6/7] [D] libiberty: Fixes for decoding numbers.

2017-04-15 Thread Iain Buclaw
This fixes two main problems found in the use of strtol(). First that it returns `0' if nothing is decoded, and none of the callers checked whether nothing was consumed. Second that it just returns `LONG_MAX' on overflow. Rather than updating each individual call site, have solved [1] by

[PATCH 5/7] [D] libiberty: Fixes for demangling qualified symbol names

2017-04-15 Thread Iain Buclaw
This patch removes `dlang_parse_symbol', and replaces it with `dlang_parse_qualified' and `dlang_parse_mangle'. All callers have been updated to reflect as to whether we expect either a `MangleName' or `QualifiedName' to be the next token we encounter, which matches many clarifications that have

[PATCH 4/7] [D] libiberty: Remove wrongly spec'd mangle rule for encoded integers.

2017-04-15 Thread Iain Buclaw
This updates the implementation to reflect a part of the D ABI spec that has been removed. There should never be a bare integer value encoded into a template argument list. Integers are always prefixed by `i' if they are positive or `N' if they are negative. Have verified this indeed is the

[PATCH 3/7] [D] libiberty: Recognize anonymous symbols names.

2017-04-15 Thread Iain Buclaw
This implements another previously undocumented part of the D ABI spec, where symbols with no name are always encoded into the mangled name. SymbolName: LName TemplateInstanceName 0 // anonymous symbols This has never really been a problem, as strtol()

[PATCH 2/7] [D] libiberty: Add support for demangling template constraints.

2017-04-15 Thread Iain Buclaw
This implements a previously an undocumented part of the D ABI spec, where symbols instantiated inside a template constraint are given a different prefix. In practise however this should never be encountered, as such instantiations are normally considered speculative, and so never make it to

[PATCH 1/7] [D] libiberty: Add support for demangling scope attributes.

2017-04-15 Thread Iain Buclaw
The next version of D adds a new `scope' function attribute. This adds support for demangling them. --- commit 15a0592cf6403fccbf43f3c7dc44f7d22c0f3dfa Author: Iain Buclaw Date: Sat Apr 15 11:15:41 2017 +0200 libiberty/ChangeLog: 2017-04-15 Iain Buclaw

[PATCH 0/7] [D] libiberty: D symbol demangling updates.

2017-04-15 Thread Iain Buclaw
Hi, This patch series includes a new addition to the current iteration of the D language, plus many fixes surrounding the clarification of certain parts of the D ABI specification. --- Iain Buclaw (7): [D] libiberty: Add support for demangling scope attributes. [D] libiberty: Add support

Re: [PATCH, gfortran] Cleanup the submodule tests

2017-04-15 Thread Janus Weil
2017-04-15 15:57 GMT+02:00 Dominique d'Humières : > >> Le 15 avr. 2017 à 13:00, Janus Weil a écrit : >> >> Hi Dominique, >> >> if I'm not mistaken, the cleanup of module file in the testsuite is >> done automatically by now, right? Couldn't one do the same

Re: [PATCH] Cleanup dg directives

2017-04-15 Thread Dominique d'Humières
> Le 15 avr. 2017 à 16:56, Janus Weil a écrit : > > 2017-04-15 16:46 GMT+02:00 Dominique d'Humières : >> I am planning to commit as obvious the following changes > > Sure! But ... > > >> --- ../_clean/gcc/testsuite/gfortran.dg/deferred_character_3.f90

Re: [PATCH] Cleanup dg directives

2017-04-15 Thread Janus Weil
2017-04-15 16:46 GMT+02:00 Dominique d'Humières : > I am planning to commit as obvious the following changes Sure! But ... > --- ../_clean/gcc/testsuite/gfortran.dg/deferred_character_3.f90 > 2015-11-15 15:12:11.0 +0100 > +++

[PATCH] Cleanup dg directives

2017-04-15 Thread Dominique d'Humières
I am planning to commit as obvious the following changes --- ../_clean/gcc/testsuite/gfortran.dg/deferred_character_3.f90 2015-11-15 15:12:11.0 +0100 +++ gcc/testsuite/gfortran.dg/deferred_character_3.f90 2017-04-15 11:43:05.0 +0200 @@ -1,4 +1,4 @@ -! {dg_do run } +! {

Re: [PATCH, gfortran] Cleanup the submodule tests

2017-04-15 Thread Dominique d'Humières
> Le 15 avr. 2017 à 13:00, Janus Weil a écrit : > > Hi Dominique, > > if I'm not mistaken, the cleanup of module file in the testsuite is > done automatically by now, right? Couldn't one do the same also for > submodules? > > Cheers, > Janus > This is indeed doable, but

Re: [PATCH, gfortran] Cleanup the submodule tests

2017-04-15 Thread Janus Weil
Hi Dominique, if I'm not mistaken, the cleanup of module file in the testsuite is done automatically by now, right? Couldn't one do the same also for submodules? Cheers, Janus 2017-04-15 10:34 GMT+02:00 Dominique d'Humières : > Hi! > > Unless someone objects, I am planning

[PATCH, gfortran] Cleanup the submodule tests

2017-04-15 Thread Dominique d'Humières
Hi! Unless someone objects, I am planning to commit the following cleanup --- ../_clean/gcc/testsuite/gfortran.dg/submodule_17.f082016-08-24 21:57:16.0 +0200 +++ gcc/testsuite/gfortran.dg/submodule_17.f08 2017-04-15 10:19:32.0 +0200 @@ -25,3 +25,4 @@ end submodule

Re: [PATCH, testsuite] Allow braces around relative line numbers

2017-04-15 Thread Tom de Vries
On 03/29/2017 12:23 AM, Jakub Jelinek wrote: On Tue, Mar 28, 2017 at 08:27:54AM +0200, Tom de Vries wrote: this patch fixes testsuite PR80220 - "relative line numbers don't work when put between braces". What is the advantage of putting the line numbers between braces? I don't know of any

Re: [wwwdocs,fortran] Update link to CHKSYS

2017-04-15 Thread Gerald Pfeifer
On Fri, 14 Apr 2017, Arjen Markus wrote: > thanks for the suggestions - I agree that a download link which is > more visible would be a good idea. I will adapt the pages with this > in mind. Cool, thanks. For the time being, I'm going ahead with the change below (taking into consideration