--- Comment #14 from burnus at gcc dot gnu dot org 2007-12-16 20:24 ---
Subject: Bug 34246
Author: burnus
Date: Sun Dec 16 20:24:32 2007
New Revision: 130991
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=130991
Log:
2007-12-16 Tobias Burnus [EMAIL PROTECTED]
PR
--- Comment #15 from burnus at gcc dot gnu dot org 2007-12-16 20:37 ---
FIXED on the trunk (4.3.0). Please open a new bug if there are new/remaining
problems.
The Bind(C) ENTRY part of this PR is now tracked as PR 34500.
--
burnus at gcc dot gnu dot org changed:
What
--- Comment #12 from dominiq at lps dot ens dot fr 2007-12-10 16:11 ---
The patch in comment #11, fix the problem for ppc Darwin9 without regression
for ppc/Intel Darwin9.
As noted, the test in comment #6 gives an ICE (g95 gives an error 'Duplicate
BIND attribute specified at (1)')
--- Comment #13 from burnus at gcc dot gnu dot org 2007-12-10 18:36 ---
As noted, the test in comment #6 gives an ICE
The ENTRY fix needs to go into build_entry_thunks:
if (thunk_sym-attr.function)
{
if (gfc_return_by_reference (ns-proc_name))
{
--- Comment #10 from danglin at gcc dot gnu dot org 2007-12-09 17:47
---
Also fails on hppa64-hp-hpux11.11.
--
danglin at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #11 from burnus at gcc dot gnu dot org 2007-12-09 23:11 ---
Patch, which fixes the test-suite failure (arrays were replaced by scalars):
http://gcc.gnu.org/ml/gcc-patches/2007-12/msg00424.html
Still left to do: ENTRY functions, see comment 6.
--
burnus at gcc dot gnu
--- Comment #9 from burnus at gcc dot gnu dot org 2007-12-03 23:15 ---
For scalar character vs. array, see also PR 32732, which added
gfc_conv_scalar_char_value(). However, I failed to get it working properly :-(
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=34246
--- Comment #8 from burnus at gcc dot gnu dot org 2007-11-30 16:07 ---
powerpc64-unknown-linux-gnu shows the same pattern as powerpc-apple-darwin9:
anything linked to endianness?
Well, x86-32 and x86-64 should have the same endianness.
--
burnus at gcc dot gnu dot org changed:
--- Comment #1 from burnus at gcc dot gnu dot org 2007-11-27 12:19 ---
Works on x86-64 with -m64, but fails with -m32.
--
burnus at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #2 from burnus at gcc dot gnu dot org 2007-11-27 12:38 ---
Valgrind finds:
==26293== Use of uninitialised value of size 4
==26293==at 0x80485D5: returnA (bind_c_usage_16.f03:14)
==26293== Bad permissions for mapped region at address 0x8048B10
==26293==at 0x80485C1:
--- Comment #3 from dominiq at lps dot ens dot fr 2007-11-27 20:49 ---
Works for me on Intel Darwin9, but fails with default (-m32) on ppc Darwin9.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=34246
--- Comment #4 from dominiq at lps dot ens dot fr 2007-11-27 21:39 ---
Works for me on Intel Darwin9, but fails with default (-m32) on ppc Darwin9.
More precisely, works with both -m32 and -m64 on Intel Darwin9, but fails with
both on ppc Darwin9.
--
--- Comment #5 from dominiq at lps dot ens dot fr 2007-11-27 23:00 ---
On Intel Darwin9 (working) -fdump-tree-gimple shows without -m64:
foo ()
{
character(kind=1) __result_foo[1:1];
__result_foo[1]{lb: 1 sz: 1} = 66;
D.831 = __result_foo;
return D.831;
}
bar (x)
{
--- Comment #7 from dominiq at lps dot ens dot fr 2007-11-27 23:27 ---
From http://gcc.gnu.org/ml/gcc-testresults/2007-11/msg01459.html,
powerpc64-unknown-linux-gnu shows the same pattern as powerpc-apple-darwin9:
anything linked to endianness?
--
--- Comment #6 from burnus at gcc dot gnu dot org 2007-11-27 23:12 ---
character(kind=1) __result_foo[1:1];
retval = __result_foo;
As Andrew pointed out, we return an array and not a scalar.
TODO:
a) Check that we expect a scalar to be returned in gfc_conv_function_call.
b)
15 matches
Mail list logo
