[PATCH, committed] PR93234 INQUIRE on pre-assigned files of ROUND and SIGN properties

2020-01-17 Thread Jerry
Committed as obvious after regression testing. The master branch has been updated by Jerry DeLisle : https://gcc.gnu.org/g:e2947cfa2d1d4da13bb298b4f36cd745b007d88d commit r10-6060-ge2947cfa2d1d4da13bb298b4f36cd745b007d88d Author: Jerry DeLisle Date: Fri Jan 17 19:36:03 2020 -0800

Re: [patch, fortran] Updated fix PR 92961, ICE on division by zero error in array bounds

2019-12-29 Thread Jerry
This one looks OK Thomas Cheers, Jerry On 12/22/19 7:28 AM, Thomas Koenig wrote: Hello world, here is an update for the fix for PR 92961, which also takes care of the second test case in the PR (included in the first one). The patch itself should be clear enough - make sure that there is a

Re: *ping*[patch, fortran] Fix PR 91541, ICE on valid for INDEX

2019-12-29 Thread Jerry
On 12/29/19 2:16 AM, Thomas Koenig wrote: Am 19.12.19 um 08:23 schrieb Thomas Koenig: Regression-tested. OK for trunk? Ping? This looks good Thomas, Thanks for patch, Jerry

Re: *ping**2 Re: [Patch, Fortran] OpenMP/OpenACC – fix more issues with OPTIONAL

2019-12-29 Thread Jerry
Between Holidays and being short on people that understand this, I would say commit it unless Jakub objects. (When in doubt, make a decision and move forward principle, assuming one is not stupid,) Cheers, Jerry On 12/29/19 2:27 PM, Tobias Burnus wrote: On 12/16/19 9:06 AM, Tobias Burnus

[patch, libfortran] Fortran 2018: Support d0.d, e0.d, es0.d, en0.d, g0.d and ew.d e0 edit descriptors

2019-12-29 Thread Jerry
yet and I propose we commit the patch here and address any DEC stuff as a follow up. (I will be looking at the DEC stuff in the next few days.) OK for trunk? Regards, Jerry diff --git a/gcc/testsuite/gfortran.dg/fmt_zero_width.f90 b/gcc/testsuite/gfortran.dg/fmt_zero_width.f90 index 640b6735c65

Re: [patch, libfortran] Fortran 2018: Support d0.d, e0.d, es0.d, en0.d, g0.d and ew.d e0 edit descriptors

2020-01-01 Thread Jerry
On 12/31/19 2:17 AM, Thomas Koenig wrote: Hi Jerry, OK for trunk? Looks good. I also think that your approach that DEC stuff should be checked later is good.  If it passes the testsuite, that's good enough for a commit. Thanks for the patch! Regards Thomas Committed r279828

Re: [PATCH] Fortran: name conflict between internal procedure and derived type [PR104351]

2023-10-13 Thread Jerry D
... ;-) Used to ICE previously. Regtested on x86_64-pc-linux-gnu. OK for mainline? Thanks, Harald This one is OK Harald. Thanks, Jerry

Re: [PATCH] Fortran: name conflict between internal procedure and derived type [PR104351]

2023-10-13 Thread Jerry D
email. Regardless, both good to go. Thanks, Jerry

Re: [PATCH] Fortran: out of bounds access with nested implied-do IO [PR111837]

2023-10-16 Thread Jerry D
on x86_64-pc-linux-gnu. OK for mainline? As this fixes a regression since 8-release, I plan to backport to all active branches. Thanks, Harald OK for Mainline and backport Thanks Harald Jerry

[patch, committed, fortran] PR82009 [F08] ICE with block construct

2018-07-04 Thread Jerry DeLisle
This patch committed as obvious after regression testing and auditing the code. New test case added. Author: jvdelisle Date: Wed Jul 4 18:08:16 2018 New Revision: 262416 URL: https://gcc.gnu.org/viewcvs?rev=262416&root=gcc&view=rev Log: 2018-07-04 Jerry DeLisle PR fortr

Re: [patch, fortran] Asynchronous I/O, take 3

2018-07-15 Thread Jerry DeLisle
, under Linux, gfortran.dg does not link in pthreads, so the tests would not be executed in parallel, and some of them would fail. So, here is the final version. I would really like to get this into trunk, and out of the way, so Nicolas and I can focus on other things. So, OK? OK, Go for it. Jerry

Re: [patch, fortran] Asynchronous I/O, take 3

2018-07-15 Thread Jerry DeLisle
We can work on the gfortran.dg incantations as a follow-on effort. (maybe a separate PR for it) Jerry

Re: [patch, fortran] Asynchronous I/O, take 3

2018-07-15 Thread Jerry DeLisle
ing. Which linux are you using? I will try it here as well. Jerry

Re: [patch, fortran] Asynchronous I/O, take 3

2018-07-15 Thread Jerry DeLisle
On 07/15/2018 11:46 AM, Rainer Orth wrote: Hi Jerry, Hmm, interesting. Which linux are you using? Fedora 27. Rainer Works for me. Fedora 28. Do not know for other OS's Jerry

[patch, libgfortran] PR98825 Unexpected behavior of FORTRAN FORMAT expressions when suppressing new line with '$'

2021-02-07 Thread Jerry DeLisle
Hello all, Attached patch fixes this and adds a test case. The "$" edit descriptor was being completely ignored when next_record_w is processed. Fixed by adding a simple check. OK for trunk and backport to 10 since it is simple enough? Regards, Jerry libgfortran: Do not emit end

Re: [patch, libgfortran] PR98825 Unexpected behavior of FORTRAN FORMAT expressions when suppressing new line with '$'

2021-02-10 Thread Jerry DeLisle
On 2/7/21 9:07 AM, Thomas Koenig wrote: Hi Jerry, OK for trunk and backport to 10 since it is simple enough? OK for trunk. Because this is a user-visible change (even if we did the wrong thing before) I don't feel that we should backport (but I am open to suggestions otherwise).

[patch, libfortran] PR95647 operator(.eq.) and operator(==) treated differently

2021-02-11 Thread Jerry DeLisle
The attached patch is another provided from Steve Kargle in the PR report. I have created a test case and regression tested the result. OK for trunk? Regards, Jerry libgfortran: Fix PR95647 by changing the interfaces of operators .eq. and .ne. The FE converts the old school .eq. to

Re: [patch, libfortran] PR95647 operator(.eq.) and operator(==) treated differently

2021-02-12 Thread Jerry DeLisle
How do I get permissions set so that I can change status of bug reports and assign to myself.  My permissions got dissolved during some evolution in the last year. also The master branch has been updated by Jerry DeLisle: https://gcc.gnu.org/g:0631e008adc759cc801d0d034224ee6b4bcf31aa commit

[patch, fortran] PR96686 Namelist group objects shall be defined before appearing in namelist

2021-02-16 Thread Jerry DeLisle
an object type is found to not be declared already, an error is issued. One new test case added and one modified to pass. Regression tested. OK for trunk? Regards, Jerry fortran: Object types should be declared before use in NAMELIST. gcc/fortran/ChangeLog:     PR fortran/98686

Re: [Patch] Fortran: Fix DTIO with type ICE [PR99146]

2021-02-19 Thread Jerry DeLisle
On 2/19/21 8:00 AM, Tobias Burnus wrote: In this example, the formal argument is a derived type and not a class – hence, there is an ICE. OK for the trunk? This is OK, could you also check 89219 and 81499 and see if these are the same or similar. Much appreciated. Jerry

Re: *PING**2 Re: [Patch] Fortran: Fix coarray handling for gfc_dep_resolver [PR99010] (was: Re: [Patch] Fortran: Fix Array dependency with local coarrays [PR98913]

2021-02-19 Thread Jerry DeLisle
? Yes OK, thanks for patch. Jerry

Re: [PATCH] PR fortran/99206 - [11 Regression] ICE in add_init_expr_to_sym, at fortran/decl.c:1980

2021-02-22 Thread Jerry DeLisle
Hi Harald, Looks Good to Me. OK Thanks On 2/22/21 1:14 PM, Harald Anlauf via Fortran wrote: Dear all, when simplifying the RESHAPE intrinsic we lost the string length when the resulting array had size zero. The attached patch sets the resulting length from the source. Regtested on x86_64-pc

Re: [Patch] Fortran: Fix -freal-{4,8}-real* handling [PR99355]

2021-03-03 Thread Jerry DeLisle
Yes OK got mainline. On 3/3/21 3:45 AM, Tobias Burnus wrote: Rather obvious change – don't apply replacement multiple times. OK for mainline? Tobias - Mentor Graphics (Deutschland) GmbH, Arnulfstrasse 201, 80634 München Registergericht München HRB 106955, Geschäftsführer: Tho

Re: [PATCH] PR libfortran/99218 - [8/9/10/11 Regression] matmul on temporary array accesses invalid memory

2021-03-03 Thread Jerry DeLisle
Yes, OK, however, have you been able to test performance. I am only curious. There was a test program we used back when this code was first implemented in bugzilla. I do not remember the PR number off hand. Jerry On 2/23/21 1:46 PM, Harald Anlauf via Fortran wrote: Dear all, under certain

Re: [Patch] Fortran: Follow fixes to -freal-{4,8}-real* handling [PR99355,PR57871] (was: Re: [Patch] Fortran: Fix -freal-{4,8}-real* handling [PR99355])

2021-03-04 Thread Jerry DeLisle
OK, thanks Dominique for spotting this. On 3/4/21 9:02 AM, Tobias Burnus wrote: Dominique found an issue ("regression") with this patch – and a testcase omission. The previous patch mishandled the noncommitted testcase of PR57871 – or in other words: It did not promote 1.0_4 or 1.0_8 – as only

[Patch, Fortran] PR90374 Support d0.d, e0.d, es0.d, en0.d, g0.d

2019-11-01 Thread Jerry DeLisle
? Jerry 2019-11-01 Jerry DeLisle PR fortran/90374 * io.c (check_format): Allow zero width for D, E, EN, and ES specifiers as default and when -std=F2018 is given. Retain existing errors when using the -fdec family of flags. 2019-11-01 Jerry DeLisle

Re: [Patch][Fortran] PR91253 fix continuation-line handling with -pre_include

2019-11-07 Thread Jerry DeLisle
IBUTE_UNUSED from get_file's 'reason' as it is used in the linemap_add call. And I have moved the OpenMP/OpenACC comment before if openmp/openacc condition, where in my opinion it belongs to. OK for the trunk? Tobias Yes, OK, thanks for patch. Jerry

[patch, libgfortran] PR90374 Zero width format specifiers.

2020-01-12 Thread Jerry Gmail
it as soon as I include in the test case. Regards, Jerry 2020-01-12  Jerry DeLisle      PR libfortran/90374     * io/format.c (parse_format_list): Zero width not allowed with     FMT_D.     * io/write_float.def (build_float_string): Include range of     higher exponent values that require w

Re: [PATCH] PR fortran/100602 - [11/12 Regression] Erroneous "pointer argument is not associated" runtime error

2021-05-26 Thread Jerry DeLisle
ntrinsic.c (gfc_conv_intrinsic_size): Use CLASS data attributes for CLASS arrays for generation of runtime error. gcc/testsuite/ChangeLog: PR fortran/100602 * gfortran.dg/pointer_check_14.f90: New test. Yes, OK for both, thanks Jerry

Re: [PATCH] PR fortran/100656 - ICE in gfc_conv_expr_present, at fortran/trans-expr.c:1934

2021-05-26 Thread Jerry DeLisle
* trans-array.c (gfc_conv_ss_startstride): Do not call check for presence of a dummy argument when a symbol actually refers to a non-dummy. gcc/testsuite/ChangeLog: PR fortran/100656 * gfortran.dg/bounds_check_22.f90: New test. Yes, OK on both. Thanks Jerry

Re: *PING* [PATCH] PR fortran/100274 - [9/10/11/12 Regression] ICE in gfc_conv_procedure_call, at fortran/trans-expr.c:6131

2021-05-04 Thread Jerry DeLisle
LGTM, OK for trunk and back ports. On 5/4/21 11:34 AM, Harald Anlauf via Fortran wrote: PING! --- Dear Fortranners, Gerhard found a case where the mismatch of actual and formal argument lead to an ICE instead of the relevant warning. Furthermore, the special case of character argument avoide

Re: [PATCH] Fortran: improve checks of NULL without MOLD as actual argument [PR104819]

2024-02-29 Thread Jerry D
ranch. Objections? Thanks, Harald Looks good to me. I think backport is OK as well. Jerry -

Re: [patch, libgfortran] Part 2: PR105456 Child I/O does not propage iostat

2024-03-04 Thread Jerry D
On 3/1/24 11:24 AM, rep.dot@gmail.com wrote: Hi Jerry and Steve, On 29 February 2024 19:28:19 CET, Jerry D wrote: On 2/29/24 10:13 AM, Steve Kargl wrote: On Thu, Feb 29, 2024 at 09:36:43AM -0800, Jerry D wrote: On 2/29/24 1:47 AM, Bernhard Reutner-Fischer wrote: And, just for my own

Re: [patch, libgfortran] Part 2: PR105456 Child I/O does not propage iostat

2024-03-05 Thread Jerry D
On 3/5/24 1:51 PM, Harald Anlauf wrote: Hi Jerry, on further thought, do we sanitize 'child_iomsg'? We pass it to snprintf as format. Wouldn't a strncpy be sufficient? Harald Just to be safe I will bump char message[IOMSG_LEN] to char message[IOMSG_LEN + 1] This is like

Re: [patch, libgfortran] Part 2: PR105456 Child I/O does not propage iostat

2024-03-06 Thread Jerry D
On 3/6/24 9:13 AM, Harald Anlauf wrote: Hi Jerry, can you please replace the user message in e.g. your new testcase pr105456-wf.f90 by say: piomsg="The users message containing % and %% and %s and other stuff" This behaves as expected with Intel, but dies horribly with gfortran

[patch, libgfortran] PR107031 - endfile truncates file at wrong position

2024-03-25 Thread Jerry D
Hi all, There has been a bit of discussio on which way to go on this. I took a look today and this trivial patch gives the behavior concluded on Fortran Discourse. See the bugzilla for all the relevant information. Regresion tested on x86-64. I will do the appropriate changelog. OK for trun

Re: [PATCH] Fortran: fix DATA and derived types with pointer components [PR114474]

2024-03-27 Thread Jerry D
this fix survives broader testing, I would like to backport. Thanks, Harald P.S.: while trying to extend coverage of conforming code, I had much fun also with other compilers (e.g. NAG panicking...) OK, for trunk and we will see how it survives for a bit before back port. Jerry -

[patch, libgfortran] PR114304 - [13/14 Regression] libgfortran I/O – bogus "Semicolon not allowed as separator with DECIMAL='point'"

2024-04-03 Thread Jerry D
incorrect separator as long as there is at least one space in front of it. New test case included. Regression tested on X86-64. OK for trunk? Backport to 13 after some time. Regards, Jerrycommit 7d1a958d6b099ea88b6c51649baf5dbd5e598909 Author: Jerry DeLisle Date: Wed Apr 3 18:07:30 2024

Re: [patch, libgfortran] PR114304 - [13/14 Regression] libgfortran I/O – bogus "Semicolon not allowed as separator with DECIMAL='point'"

2024-04-04 Thread Jerry D
On 4/4/24 2:31 AM, Tobias Burnus wrote: Hi Jerry, Jerry D wrote: The attached log entry and patch (git show) fixes this issue by adding logic to handle spaces in eat_separators. One or more spaces by themselves are a valid separator. So in this case we look at the character following the

Re: [patch, libgfortran] PR114304 - [13/14 Regression] libgfortran I/O – bogus "Semicolon not allowed as separator with DECIMAL='point'"

2024-04-04 Thread Jerry D
On 4/4/24 2:31 AM, Tobias Burnus wrote: Hi Jerry, --- snip --- The patch looks mostly like I would expect, except for decimal='point' and a ';' which is *not* preceded by a space. Thanks for working on it. Regarding the 'except' case: --- snip --- i.e. f

Re: [patch, libgfortran] PR114304 - [13/14 Regression] libgfortran I/O – bogus "Semicolon not allowed as separator with DECIMAL='point'"

2024-04-05 Thread Jerry D
On 4/4/24 2:41 PM, Tobias Burnus wrote: Hi Jerry, I think for the current testcases, I like the patch – the question is only what's about:   ',3' as input for 'comma'   (or '.3' as input for 'point') For 'point' – 0.3 is read an

Re: [patch, libgfortran] PR114304 - [13/14 Regression] libgfortran I/O – bogus "Semicolon not allowed as separator with DECIMAL='point'"

2024-04-05 Thread Jerry D
On 4/5/24 10:47 AM, Jerry D wrote: On 4/4/24 2:41 PM, Tobias Burnus wrote: Hi Jerry, I think for the current testcases, I like the patch – the question is only what's about:    ',3' as input for 'comma'   (or '.3' as input for 'point') For &

Re: [Patch] Fortran: List-directed read - accept again tab as alternative to space as separator [PR114304]

2024-04-08 Thread Jerry D
On 4/8/24 2:53 AM, Tobias Burnus wrote: Jerry D wrote: See attached updated patch. It turned rather quickly out that this patch – committed as r14-9822-g93adf88cc6744a – caused regressions. Namely, real-world code use tab(s) as separator instead of spaces. [For instance, PR114304 which

Re: [Patch, fortran] PR114535 - [13/14 regression] ICE with elemental finalizer

2024-04-08 Thread Jerry D
cc/fortran PR fortran/114535 * resolve.cc (resolve_symbol): Remove last chunk that checked for finalization of unreferenced symbols. gcc/testsuite/ PR fortran/114535 * gfortran.dg/pr114535d.f90: New test. * gfortran.dg/pr114535iv.f90: Additional source. Yes, OK Paul. Don't feel bad. It wasn't Tabs. LOL Jerry

Re: [patch, libgfortran] PR111022 ES0.0E0 format gave ES0.dE0 output with d too high.

2024-02-02 Thread Jerry D
On 1/30/24 12:36 PM, Harald Anlauf wrote: Hi Jerry, Am 30.01.24 um 19:15 schrieb Jerry D: The attached patch attempts to fix the handling of the EN0.0E0 and ES0.0E0 formatting by correctly calculating the number of digits needed for the exponents and building those exponents into the float

Re: [PATCH] Fortran: error recovery on arithmetic overflow on unary operations [PR113799]

2024-02-08 Thread Jerry D
patch looks OK. Thanks, Jerry

[patch, libgfortran] PR109358

2024-02-12 Thread Jerry D
The attached patch fixes this PR by properly adjusting some variables When using stream io. See log below. New test case included. Regression tested on x86_64. OK for trunk and backport? Regards, Jerry ChangeLog: libgfortran: Adjust bytes_left and pos for access="STREAM".

[patch, libgfortran] PR99210 X editing for reading file with encoding='utf-8'

2024-02-13 Thread Jerry DeLisle
The attached patch fixes the X editing. Fairly self explanatory. I created the patch a few years back. Regression tested on x86_64 and new test case. OK for trunk? Regards, Jerrydiff --git a/gcc/testsuite/gfortran.dg/pr99210.f90 b/gcc/testsuite/gfortran.dg/pr99210.f90 new file mode 100644 inde

[patch, fortran] Bug 105847 - namelist-object-name can be a renamed host associated entity

2024-02-14 Thread Jerry D
Pushed as simple and obvious. Regards, Jerry commit 8221201cc59870579b9dc451b173f94b8d8b0993 (HEAD -> master, origin/master, origin/HEAD) Author: Steve Kargl Date: Wed Feb 14 14:40:16 2024 -0800 Fortran: namelist-object-name renaming. PR fortran/105847 gcc/fort

Re: [PATCH] Fortran: deferred length of character variables shall not get lost [PR113911]

2024-02-16 Thread Jerry D
. Another temporarily disabled test was re-enabled. Regtested on x86_64-pc-linux-gnu. OK for mainline? Thanks, Harald Yes OK for mainline. Thanks, Jerry

[patch, libgfortran] PR107068 Run-time error when reading logical arrays with a namelist

2024-02-16 Thread Jerry D
The attached patch fixes this one. Se the ChangeLog below for explanation. OK for trunk? I think simple enough to backport to 13 as well. Regards, Jerry Author: Jerry DeLisle Date: Fri Feb 16 17:06:37 2024 -0800 libgfortran: Fix namelist read. PR libgfortran/107068

[patch, libgfortran] Bug 105473 - semicolon allowed when list-directed read integer with decimal='point'

2024-02-16 Thread Jerry D
Hello, I posted the attached patch in bugzilla some time ago. This includes a new test case. The patch adds additional checks in key places to catch eroneous use of semicolons Regression tested on x86_64, OK for trunk and later backport to 13? Jerrydiff --git a/gcc/testsuite/gfortran.dg/pr1

Re: [PATCH] Fix fortran/PR114024

2024-02-21 Thread Jerry D
Steve, I looked it over and looks reasonable. I will try to apply it next few days and test here. If OK, I will commit. Jerry

Re: [PATCH] Fix fortran/PR114024

2024-02-21 Thread Jerry D
On 2/21/24 12:28 PM, Harald Anlauf wrote: On 2/21/24 20:41, Jerry D wrote: On 2/21/24 10:30 AM, Steve Kargl wrote: I have attached a patch to PR114024, see https://gcc.gnu.org/pipermail/gcc-bugs/2024-February/854651.html The patch contains a new testcase and passes regression testing on

[patch, libgfortran] PR105456 Child I/O does not propage iostat

2024-02-22 Thread Jerry D
WRITE or unformatted READ/WRITE until I get some feedback on the approach. If this approach is OK I would like to commit and then do a separate patch for the cases I just mentioned. Feedback appreciated. Regression tested on x86_64. OK for trunk? Jerry Author: Jerry DeLisle Date: Thu Feb 22

Re: [patch, libgfortran] PR105456 Child I/O does not propage iostat

2024-02-25 Thread Jerry D
On 2/25/24 12:34 PM, Harald Anlauf wrote: Hi Jerry, On 2/22/24 20:11, Jerry D wrote: Hi all, The attached fix adds a check for an error condition from a UDDTIO procedure in the case where there is no actual underlying error, but the user defines an error by setting the iostat variable

Re: [PATCH] Fortran: do not evaluate polymorphic functions twice in assignment [PR114012]

2024-02-25 Thread Jerry D
. OK for mainline? And a backport to 13-branch after some delay? Thanks, Harald Yes, simple enough. OK. Thanks, Jerry

Re: [PATCH] Fortran testsuite: fix invalid Fortran in testcase

2024-02-27 Thread Jerry D
On 2/27/24 1:00 PM, Harald Anlauf wrote: Dear all, the attached patch fixes invalid Fortran in testcase gfortran.dg/pr101026.f, which might prohibit progress in fixing pr111781. (Note that the testcase was for a tree-optimizer issue, not the Fortran frontend.) OK for mainline? Will commit wit

[patch, libgfortran] Part 2: PR105456 Child I/O does not propage iostat

2024-02-28 Thread Jerry D
in the library. Regressions tested on x86_64. OK for trunk? Regards, Jerry commit 640991bd6b83df4197b2eaec63d1e0e695e48b75 Author: Jerry DeLisle Date: Wed Feb 28 20:51:06 2024 -0800 Fortran: Add user defined error messages for UDTIO. The defines IOMSG_LEN and MSGLEN were redundant

Re: [patch, libgfortran] Part 2: PR105456 Child I/O does not propage iostat

2024-02-29 Thread Jerry D
On 2/29/24 1:47 AM, Bernhard Reutner-Fischer wrote: On Wed, 28 Feb 2024 21:29:06 -0800 Jerry D wrote: The attached patch adds the error checks similar to the first patch previously committed. I noticed a redundancy in some defines MSGLEN and IOMSG_LEN so I consolidated this to one define in

Re: PR82943 - Suggested patch to fix

2024-01-20 Thread Jerry D
today if I can. Paul is unavailable for a few weeks. Harald can chime in. Do you have commit rights for gcc? Your efforts are appreciated. Regards, Jerry

Re: PR82943 - Suggested patch to fix

2024-01-20 Thread Jerry D
On 1/20/24 11:08 AM, Jerry D wrote: On 1/20/24 10:46 AM, Alexander Westbrooks wrote: Hello and Happy New Year! I wanted to follow up on this patch I made to address PR82943 for GFortran. Has anyone had a chance to review it? Thanks, Alexander Westbrooks Inserting myself in here just a

Re: PR82943 - Suggested patch to fix

2024-01-20 Thread Jerry D
On 1/20/24 12:08 PM, Harald Anlauf wrote: Am 20.01.24 um 20:08 schrieb Jerry D: On 1/20/24 10:46 AM, Alexander Westbrooks wrote: Hello and Happy New Year! I wanted to follow up on this patch I made to address PR82943 for GFortran. Has anyone had a chance to review it? Thanks, Alexander

[patch, libgfortran] PR111022 ES0.0E0 format gave ES0.dE0 output with d too high.

2024-01-30 Thread Jerry D
apologize for the clutter, but we might as well get rid of it now. Two existing test cases needed to be adjusted and I am adding one new test case to capture the changes in our testsuite. Regression tested on X86_64. OK for trunk? Do we need to backport this? Regards, Jerry Author: Jerry

[patch, fortran, committed] PR112873 F2023 degree trig functions

2023-12-14 Thread Jerry D
The following has been committed per discussion in the subject PR. commit 95b70545331764c85079a1d0e1e19b605bda1456 (HEAD -> master, origin/master, origin/HEAD) Author: Jerry DeLisle Date: Wed Dec 13 19:04:50 2023 -0800 fortran: Add degree based trig functions for F2023

[libgfortran, patch] PR113223 NAMELIST internal write missing leading blank character

2024-01-07 Thread Jerry D
Committed as simple and obvious. Initial patch thanks to Steve. When using git gcc-commit-mklog how does one add in the coauthor? The master branch has been updated by Jerry DeLisle : https://gcc.gnu.org/g:add995ec117d756e61d207041cd32f937c1a1cd9 commit r14-6986

Re: {Patch, fortran] PR112834 - Class array function selector causes chain of syntax and other spurious errors

2023-12-06 Thread Jerry D
then, if no fallout, 13 at your discretion. Regards, Jerry

Re: [PATCH] Fortran: fix ALLOCATE with SOURCE=, zero-length character [PR83865]

2024-06-03 Thread Jerry D
. Thanks, Harald OK and thanks for patch. Jerry

Re: [Ping, Fortran, Patch, PR85510, v2] Fix coarray token in associate not linking

2024-08-09 Thread Jerry Delisle
Ok and thanks. On Fri, Aug 9, 2024, 7:33 AM Andre Vehreschild wrote: > Ping! > > And the last ping in the series. I have a "bigger" patch in the queue and > want > the pending ones done beforehand. > > Regtested ok on x86_64-pc-linux-gnu / Fedora 39. Ok for mainline? > > - Andre > > On Mon, 22 J

Re: [Ping x 3, Fortran, Patch, coarray, PR84246, v2] Fix for [Coarray] ICE in conv_caf_send, at fortran/trans-intrinsic.c:1950

2024-08-19 Thread Jerry D
On 8/19/24 4:43 AM, Andre Vehreschild wrote: Hi all, another ping on this patch. Regtests ok on x86_64-pc-linux-gnu on updated master. Ok for mainline? Regards, Andre Looks OK to go ahead Andre. Thanks, Jerry On Fri, 9 Aug 2024 16:29:08 +0200 Andre Vehreschild wrote: Ping

Re: [Ping x2 , Fortran, Patch, PR77518, (coarray), v4] Fix ICE in sizeof(coarray)

2024-08-20 Thread Jerry D
On 8/20/24 5:35 AM, Andre Vehreschild wrote: Hi all, pinging this patch. Regtests ok on x86_64-pc-linux-gnu / Fedora 39. Ok for mainline? Regards, Andre Your approach looks reasonable so I think OK to push. Thanks, Jerry

Re: [pushed] readings: Drop FORTRAN 77 test suite at itl.nist.gov

2024-06-19 Thread Jerry D
On 6/18/24 10:20 AM, Steve Kargl wrote: On Tue, Jun 18, 2024 at 09:13:23AM +0200, Gerald Pfeifer wrote: The original subsite has disappeared and we couldn't find it elsewhere. https://github.com/gklimowicz/FCVS gklimowicz is a flang developer and member of J3. FWIW my copy of the tests st

[patch, libgfortran] PR105361 Incorrect end-of-file condition for derived-type I/O

2024-07-22 Thread Jerry D
. If I have some more time today I will try to come up with something. OK for mainline? Jerry commit e6fa9d84cf126630c9ea744aabec6d7715087310 (HEAD -> master) Author: Jerry DeLisle Date: Sun Jul 21 19:19:00 2024 -0700 Fortran: Suppress wrong End Of File error with user defined

ping Re: [patch, libgfortran] PR105361 Incorrect end-of-file condition for derived-type I/O

2024-07-24 Thread Jerry D
On 7/22/24 8:13 AM, Jerry D wrote: Hi all, The attached patch fixes this by avoiding looking for and avoiding the EOF condition in the parent READ after returning from the child IO process. I could not think of a simple test case yet since the problem occurred only when redirecting the

[Patch, libgfortran] PR105361 Followup fix to test case

2024-07-31 Thread Jerry D
I plan to push this soon to hopefully fix some test breakage on some architetures. It is simple and obvious. I did not get any feedback on this and I do not have access to the machines in question. Regression tested on linux-x86-64. Regards, Jerry commit

[Patch, Fortran] PR104626 ICE in gfc_format_decoder, at fortran/error.cc:1071

2024-08-02 Thread Jerry D
Hi all, Doing some catchup here. I plan to commit the following shortly. This is one of Steve's patches posted on bugzilla. I have created a new test case. Regression tested on linux x86-64. git show: commit 4d4549937b789afe4037c2f8f80dfc2285504a1e (HEAD -> master) Author: Steve Kargl Date

[Patch, Fortran] Bug 109105 - Error-prone format string building in resolve.cc

2024-08-06 Thread Jerry D
mentioned so I used the CHECK_INTERFACES name for the macro. Regression tested on linux-x86_64. No new test cases. OK for mainline? Backport? Regards, Jerry Author: Jerry DeLisle Date: Tue Aug 6 12:47:30 2024 -0700 Fortran: Eliminate error prone translations. PR fortran/109105

Re: [PATCH] Fortran: fix ICE in gfc_create_module_variable [PR100273]

2024-09-05 Thread Jerry D
d the possibility that that function need not appear at the top level of a module, but could be a contained function. Regtested on x86_64-pc-linux-gnu. OK for mainline? Thanks, Harald OK for mainline. Thanks. Jerry

Re: [patch, fortran] PR65996 [5/6 Regression] gfortran ICE with -dH

2016-01-21 Thread Jerry DeLisle
On 01/18/2016 11:01 AM, Jerry DeLisle wrote: > This patch follows the suggestion Jakub made in the PR and is very > straightforward. > The patch is simple enough. I will commit to trunk shortly if I hear from no one. Regards, Jerry > With the patch, an abort is given on actua

Re: [patch, fortran] Inline MATMUL(A,TRANSPOSE(B)), PR 66094

2016-01-23 Thread Jerry DeLisle
On 01/23/2016 04:26 AM, Thomas Koenig wrote: > Hi Toon, > >> However, today I *did* run the test harness with your modification: > > ... > > Thanks for the testing! > > So, what do people think? Is the patch OK for trunk? > > Regards > > Thomas > Yes, OK. Jerry

Re: [PATCH] Fix failure of gfortran.dg/backtrace_1.f90 on hppa*-*-hpux*

2016-01-23 Thread Jerry DeLisle
om >> show_backtrace(). This avoids a >> segmentation fault when threads are active. >> >> This fixes the failure of gfortran.dg/backtrace_1.f90 on hpux. >> >> Tested on hppa2.0w-hp-hpux11.11 and hppa64-hp-hpux11.11. >> >> Okay for trunk? >> OK, Jerry

[Patch, Fortran] PR69397 and PR6844 Internal Compiler Errors2

2016-01-24 Thread Jerry DeLisle
Ha all, The attached patch with new test cases fixes these by replacing gcc_assert and updating the error message depending on whether resolving an initialization expression or not. Regression tested on x86-64. OK for trunk? Jerry 2016-01-23 Jerry DeLisle PR fortran/69397

Re: [Patch, fortran, GCC-5/6, PR62536, v1] ICE (segfault) for invalid END BLOCK statement

2016-01-27 Thread Jerry DeLisle
analysis required namespace. > > Bootstrapped and regtested ok on x86_64-linux-gnu/F23. > > Ok for trunk and gcc-5-branch, respectively? > OK and thanks for fixes. Much appreciated? Jerry

[patch, fortran] PR50555 synonymous namelist/statement function dummy argument not allowed

2016-02-06 Thread Jerry DeLisle
Hi All, Attached patch adds diagnostics to catch this. Test case included. Regression tested on x86-64 OK for trunk? Regards, Jerry PS Want to get this one out of the way so I can do the new namelist regression with DELIM=NONE. 2016-02-06 Jerry DeLisle PR fortran/50555

[patch,libgfortran] Bug 69668 - [4.9/5/6 Regression] Error reading namelist opened with DELIM='NONE'

2016-02-09 Thread Jerry DeLisle
The attached patch reverts the guilty code. We were trying to honor delim=NONE on namelist reads which is invalid. Test cases updated. Regression tested on x86-64. OK for trunk and back port in about a week? Regards, Jerry 2016-02-09 Jerry DeLisle PR libgfortran/69668 * io

Re: [patch, fortran] PR56007 Remarkably bad error message with DO array=1,2

2016-02-09 Thread Jerry DeLisle
sage. > > Whoever wants to take it... > I will commit this under the simple rules as soon as I get the namelist regression fix committed on trunk. Thanks Harald Jerry

Re: [patch,libgfortran] Bug 69668 - [4.9/5/6 Regression] Error reading namelist opened with DELIM='NONE'

2016-02-11 Thread Jerry DeLisle
On 02/09/2016 06:45 PM, Jerry DeLisle wrote: > The attached patch reverts the guilty code. We were trying to honor delim=NONE > on namelist reads which is invalid. > > Test cases updated. Regression tested on x86-64. > > OK for trunk and back port in about a week? > No

Re: [patch,libgfortran] Bug 69668 - [4.9/5/6 Regression] Error reading namelist opened with DELIM='NONE'

2016-02-11 Thread Jerry DeLisle
On 02/11/2016 10:38 AM, Janne Blomqvist wrote: > On Wed, Feb 10, 2016 at 4:45 AM, Jerry DeLisle wrote: >> The attached patch reverts the guilty code. We were trying to honor >> delim=NONE >> on namelist reads which is invalid. >> >> Test cases updated. Regressi

[6 Regession] Usage of unitialized pointer io/list_read.c (

2016-02-15 Thread Jerry DeLisle
on is through making sure what we read in matches what we wrote out to the test scratch file Regression tested on x86_64-Linux. OK for trunk? any thoughts on back porting to 5 since it fixes a potentially bad pointer problem in push_char4? Regards, Jerry 2016-02-15 Jerry DeLisle PR l

Re: [6 Regession] Usage of unitialized pointer io/list_read.c (

2016-02-16 Thread Jerry DeLisle
On 02/16/2016 12:06 AM, Christophe Lyon wrote: > On 15 February 2016 at 23:16, Janne Blomqvist > wrote: >> On Mon, Feb 15, 2016 at 11:45 PM, Jerry DeLisle >> wrote: >>> The title of the PR should be "Mishandling of namelist comments" or >>> "

Re: [6 Regession] Usage of unitialized pointer io/list_read.c (

2016-02-16 Thread Jerry DeLisle
On 02/16/2016 05:17 AM, Dominique d'Humières wrote: > Hi Jerry, > >> Thanks for review. Committed to trunk r233436. > > The test gfortran.dg/read_bang4.f90 fails on x86_64-apple-darwin15: > > a.out(15552,0x7fff7b2e3000) malloc: *** error for object 0x7fb472804c00:

Re: [6 Regession] Usage of unitialized pointer io/list_read.c (

2016-02-16 Thread Jerry DeLisle
(15,*,iostat=ios) str1, str2 > close(15) > end program > > valgrind shows a lot of errors before and after the commit. > > Dominique > >> Le 16 févr. 2016 à 18:58, Jerry DeLisle a écrit : >> >> Up until now we have not had a test case like this for kind

Re: [6 Regession] Usage of unitialized pointer io/list_read.c (

2016-02-17 Thread Jerry DeLisle
On 02/16/2016 05:37 PM, Jerry DeLisle wrote: > See patch to fix this below. > Committed on trunk, r233500 after regression testing, -fsanitize=address testing, and valgrind testing. Jerry

[patch, libgfortran] PR69456 Namelist value with trailing sign is ignored without error

2016-02-22 Thread Jerry DeLisle
giving the correct item number (always zero) so I realized the item_count was not being incremented for each object. So fixed this as well. Regression tested on x86-64-Linux. One new test case and one modified. OK for trunk? Not strictly a regression, but simple enough. Jerry 2016-02-22 Jerry

[Fortran,4.9/5/6 Regression]ICE for Fortran files when specifying a file instead of an include directory

2016-02-22 Thread Jerry DeLisle
The ice on this one is actually in libcpp but I don't want to mess around in that area. Not sure why we were only doing a warning here. I plan to commit the following to trunk with a ChangeLog entry under the simple and obvious rule. Let me no if any objections. Jerry diff --git

[patch, fortran] PR69110 ICE with NEWUNIT

2016-02-24 Thread Jerry DeLisle
This patch from Steve on c.l.f Fixes the segfault from attempting a string compare where there is no string yet. Regression tested on x86-64. New test case. OK for trunk. Regards, Jerry 2016-02-24 Jerry DeLisle Steven G. Kargl PR fortran/69110 * io.c

Re: [patch, fortran] PR69110 ICE with NEWUNIT

2016-02-27 Thread Jerry DeLisle
On 02/27/2016 01:12 PM, Tom de Vries wrote: > On 25-02-16 01:54, Jerry DeLisle wrote: >> This patch from Steve on c.l.f >> >> Fixes the segfault from attempting a string compare where there is no string >> yet. >> >> Regression tested on x86-64. New test

Re: *ping* [patch, Fortran] Fix PR fortran/68147, temporaries for allocatable strings

2016-02-28 Thread Jerry DeLisle
;> * gfortran.dg/realloc_on_assign_26.f90: New test case. > > Fix spelling in ChangeLog of "...callback" and OK Jerry

Re: *ping* [patch, fortran] Two associate fixes in dump-parse-tree.c

2016-02-28 Thread Jerry DeLisle
t;> * dump-parse-tree.c (show_code_node): Print association >> list of a block if present. Handle EXEC_END_BLOCK. > > Yes, OK Jerry

Re: [Patch, fortran] PR69834 - Collision in derived type hashes

2016-03-03 Thread Jerry DeLisle
out and can be fixed at 6.2 or 6.3. It is the open nature of our software and the user feedback that makes this all work. (also we know Fortran is not release critical) I tested with my own OOP code which is an adaptation of Metcalf's linked anylist and it works fine. Thats the best I can do and it is fairly complex code. I can send it to you if you would like to have it in your test pile. Regards, Jerry

  1   2   3   4   5   6   7   8   >