https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87689
Bug 87689 depends on bug 33097, which changed state.
Bug 33097 Summary: Function decl trees without proper argument list
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=33097
What|Removed |Added
---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87689
Bill Schmidt changed:
What|Removed |Added
CC||wschmidt at gcc dot gnu.org
--- Comment #
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87689
Thomas Koenig changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87689
--- Comment #28 from Thomas Koenig ---
Author: tkoenig
Date: Sun Mar 3 09:20:09 2019
New Revision: 269350
URL: https://gcc.gnu.org/viewcvs?rev=269350&root=gcc&view=rev
Log:
2019-03-03 Thomas Koenig
PR fortran/87689
Backport from tru
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87689
--- Comment #27 from Thomas Koenig ---
Author: tkoenig
Date: Sun Mar 3 09:04:23 2019
New Revision: 269349
URL: https://gcc.gnu.org/viewcvs?rev=269349&root=gcc&view=rev
Log:
2019-03-03 Thomas Koenig
PR fortran/87689
Backport
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87689
--- Comment #26 from Thomas Koenig ---
Fixed on gcc 9 so far. I will backport this to the other
open branches, but only after the release of the next
version of gcc 8.3.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87689
--- Comment #25 from Thomas Koenig ---
Author: tkoenig
Date: Mon Feb 18 18:28:58 2019
New Revision: 268992
URL: https://gcc.gnu.org/viewcvs?rev=268992&root=gcc&view=rev
Log:
2019-02-18 Thomas Koenig
PR fortran/87689
* trans-decl.c (g
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87689
Thomas Koenig changed:
What|Removed |Added
Keywords||patch
URL|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87689
--- Comment #23 from Thomas Koenig ---
(In reply to Thomas Koenig from comment #22)
> FAIL: gfortran.dg/lto/20091028-1 f_lto_20091028-1_0.o-f_lto_20091028-1_1.o
> link, -O0 -flto -flto-partition=none -fuse-linker-plugin
These test cases are inv
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87689
--- Comment #22 from Thomas Koenig ---
Created attachment 45742
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=45742&action=edit
Patch that may work, but causes some regressions
The regressions are:
FAIL: gfortran.dg/lto/20091028-1 f_lto_
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87689
Thomas Koenig changed:
What|Removed |Added
Status|NEW |ASSIGNED
Assignee|unassigned a
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87689
--- Comment #20 from Alan Modra ---
That is what I thought too at first, and why I though that making calls with
hidden args to be varargs was not going to be a problem. However, calling
build_varargs_function_type_vec with an empty typelist doe
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87689
--- Comment #19 from Thomas Koenig ---
This is worse than I thought.
It seems that, if we compile
subroutine foo(a)
real :: a
end subroutine foo
program main
real :: x
call foo(x)
end program main
we call build_function_type_vec (type,
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87689
--- Comment #18 from rguenther at suse dot de ---
On February 13, 2019 6:56:15 AM GMT+01:00, amodra at gmail dot com
wrote:
>https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87689
>
>--- Comment #17 from Alan Modra ---
>> On platforms where varargs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87689
--- Comment #17 from Alan Modra ---
> On platforms where varargs have a different calling
> signature from normal functions, this would be an ABI change.
True, but in C terms, gfortran is currently doing this:
void f (char *res, int reslen);
..
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87689
--- Comment #16 from Thomas Koenig ---
(In reply to Alan Modra from comment #12)
> A little more sophisticated.
>
> * fortran/trans-types.c (gfc_get_function_type): Use a varargs decl
> unless we have args other than hidden ones.
Wi
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87689
--- Comment #15 from Alan Modra ---
I've regression tested the patch on powerpc64le-linux, and of course verified
that it fixes the testcase. I'm also fairly certain the patch intent is
correct in the narrow sense of it correcting gfc_get_functi
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87689
--- Comment #14 from rguenther at suse dot de ---
On February 12, 2019 9:40:46 AM GMT+01:00, "tkoenig at gcc dot gnu.org"
wrote:
>https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87689
>
>Thomas Koenig changed:
>
> What|Removed
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87689
Thomas Koenig changed:
What|Removed |Added
CC||rguenth at gcc dot gnu.org
--- Comment #
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87689
--- Comment #12 from Alan Modra ---
A little more sophisticated.
* fortran/trans-types.c (gfc_get_function_type): Use a varargs decl
unless we have args other than hidden ones.
--- a/gcc/fortran/trans-types.c
+++ b/gcc/fortran/t
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87689
Alan Modra changed:
What|Removed |Added
Summary|Memory corruption on Power |PowerPC64 ELFv2 function
21 matches
Mail list logo