https://gcc.gnu.org/bugzilla/show_bug.cgi?id=121628

--- Comment #14 from kargls at comcast dot net ---
(In reply to Christopher Albert from comment #13)
> Created attachment 62580 [details]
> Updated patch
> 
> Updated the patch including correct formatting and further changes.
> Actually, the original patch had a severe issue as it produced trampolines
> on executable stack. This is now also fixed.

I tested the new patch on x86_64-*-freebsd.  It applied
cleanly and passed regression testing.  I've only done
a quick scan of the patch.  In the updated array_memcpy_2.f90
test, you don't need to include me; but, if you do please
use [email protected].  It might be appropriate to also include
the PR number, something like

! PR fortran/122114
!   Steve Kargl <[email protected]> - identified bogus original test
!   Christopher Albert <[email protected]> - fixed test with proper allocation

Reply via email to