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

--- Comment #15 from Christopher Albert <albert at tugraz dot at> ---
(In reply to kargls from comment #14)
> (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

Nice! I removed us both now and kept only the PR, since this is such a trivial
test case and anyway one sees git history.

Reply via email to