Re: target/6569: sparc-sun-solaris2.7 C testsuite regression in compile/20011119-2.c

[Franz Sirl <Franz.Sirl-kernel@lauterbach.com>]

The following reply was made to PR target/6569; it has been noted by GNATS.

From: Franz Sirl <Franz.Sirl-kernel@lauterbach.com>
To: Mark Mitchell <mark@codesourcery.com>
Cc: "Kaveh R. Ghazi" <ghazi@caip.rutgers.edu>,
 "davem@gcc.gnu.org" <davem@gcc.gnu.org>,
 "davem@redhat.com" <davem@redhat.com>,
 "gcc-bugs@gcc.gnu.org" <gcc-bugs@gcc.gnu.org>,
 "gcc-gnats@gcc.gnu.org" <gcc-gnats@gcc.gnu.org>,
 "jakub@redhat.com" <jakub@redhat.com>,
 "nobody@gcc.gnu.org" <nobody@gcc.gnu.org>,
 "rth@redhat.com" <rth@redhat.com>
Subject: Re: target/6569: sparc-sun-solaris2.7 C testsuite regression 
  in compile/20011119-2.c
Date: Tue, 07 May 2002 17:35:02 +0200

 --=====================_26563516==_
 Content-Type: text/plain; charset="us-ascii"; format=flowed
 
 At 16:40 07.05.2002, Mark Mitchell wrote:
 
 
 >--On Tuesday, May 07, 2002 03:14:49 PM +0200 Franz Sirl 
 ><Franz.Sirl-kernel@lauterbach.com> wrote:
 >
 >>At 14:45 07.05.2002, Kaveh R. Ghazi wrote:
 >>>  > From: Mark Mitchell <mark@codesourcery.com>
 >>>  >
 >>>  > >> Can you try the attached patch? It seems to work for me, but the
 >>>  > >> bootstrap hasn't completed yet. While I was at it, I improved the
 >>>  > >> placing of the warning messages. I'm still a bit confused, cause it
 >>>  > >> sometimes seems olddecl and newdecl appeared to be swapped
 >>>  > >> compared to their sourcefile ordering.
 >>>  > >
 >>>  > > FYI, bootstrap+regtest on x86-linux-gnu completed successfully.
 >>>  >
 >>>  > OK; once you have confirmation of the SPARC results go ahead and
 >>>  > check it in.
 >>>
 >>>Unfortunately, the patch did not solve the problem listed in the PR.
 >>>I still get the same 'as' errors from compile/20011119-2.c
 >>>
 >>>compile/20011119-2.c:3: warning: weak declaration of `foo' after first
 >>>use  results in unspecified behavior
 >>>/usr/ccs/bin/as: "/var/tmp//ccQOIg1d.s", line 51: error: invalid operand
 >>>--------------------------------------------------^^^^^
 >>
 >>Yeah, I managed to get access to a solaris-2.8 machine, even though I
 >>wasn't able to bootstrap with Solaris as/ld (see below), I was able to
 >>reproduce the failure. Re-checking on x86-linux-gnu revealed that it even
 >>doesn't fix the problem there, so I must have mixed something up
 >>yesterday. Frankly, I'm a bit at a loss here, cause I've tried several
 >>strategies yesterday and they either didn't fix the testcase or caused
 >>some of the weak tests to fail. Especially I tried to use the TREE_USED
 >>flag of the WEAK_DECLS TREE_LIST to mark when a weak already had been
 >>assembled, but it didn't work out :-(. I seem to misunderstand something
 >>about how the tree structures are handled.
 >
 >I'll look into this problem.
 
 FYI, this is the patch I'm currently playing with, it would fix 20011119-2, 
 but these fail:
 
 FAIL: gcc.dg/weak-3.c scan-assembler weak[^     ]*[     ]ffoo1b
 FAIL: gcc.dg/weak-3.c scan-assembler weak[^     ]*[     ]ffoo1c
 FAIL: gcc.dg/weak-3.c scan-assembler weak[^     ]*[     ]ffoo1e
 FAIL: gcc.dg/weak-5.c scan-assembler weak[^     ]*[     ]vfoo1b
 FAIL: gcc.dg/weak-5.c scan-assembler weak[^     ]*[     ]vfoo1c
 
 I don't understand what I'm doing wrong :-(.
 
 Franz.
 
 --=====================_26563516==_
 Content-Type: application/octet-stream; name="gcc-weaksym-9x1.patch"
 Content-Transfer-Encoding: base64
 Content-Disposition: attachment; filename="gcc-weaksym-9x1.patch"
 
 SW5kZXg6IGdjYy92YXJhc20uYwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09ClJDUyBmaWxlOiAvY3Zzcm9vdC9nY2MvZ2Nj
 L2djYy92YXJhc20uYyx2CnJldHJpZXZpbmcgcmV2aXNpb24gMS4yNTAuMi45CmRpZmYgLXUgLXAg
 LXIxLjI1MC4yLjkgdmFyYXNtLmMKLS0tIGdjYy92YXJhc20uYwkxIE1heSAyMDAyIDE4OjAzOjM0
 IC0wMDAwCTEuMjUwLjIuOQorKysgZ2NjL3ZhcmFzbS5jCTcgTWF5IDIwMDIgMTU6Mjc6MTIgLTAw
 MDAKQEAgLTE2Niw2ICsxNjYsNyBAQCBzdGF0aWMgdW5zaWduZWQgSE9TVF9XSURFX0lOVCBhcnJh
 eV9zaXplCiBzdGF0aWMgdW5zaWduZWQgbWluX2FsaWduCQlQQVJBTVMgKCh1bnNpZ25lZCwgdW5z
 aWduZWQpKTsKIHN0YXRpYyB2b2lkIG91dHB1dF9jb25zdHJ1Y3RvcgkJUEFSQU1TICgodHJlZSwg
 SE9TVF9XSURFX0lOVCwKIAkJCQkJCSB1bnNpZ25lZCBpbnQpKTsKK3N0YXRpYyB2b2lkIGRlY2xh
 cmVfd2Vha18xCQlQQVJBTVMgKCh0cmVlLCBpbnQpKTsKIHN0YXRpYyB2b2lkIGdsb2JhbGl6ZV9k
 ZWNsCQlQQVJBTVMgKCh0cmVlKSk7CiBzdGF0aWMgaW50IGluX25hbWVkX2VudHJ5X2VxCQlQQVJB
 TVMgKChjb25zdCBQVFIsIGNvbnN0IFBUUikpOwogc3RhdGljIGhhc2h2YWxfdCBpbl9uYW1lZF9l
 bnRyeV9oYXNoCVBBUkFNUyAoKGNvbnN0IFBUUikpOwpAQCAtNTAxMCwxNiArNTAxMSwxOCBAQCBt
 ZXJnZV93ZWFrIChuZXdkZWNsLCBvbGRkZWNsKQogCSAgfHwgISBUUkVFX1NUQVRJQyAoZGVjbCkp
 CiAgICAgICAmJiBUUkVFX1VTRUQgKGRlY2wpCiAgICAgICAmJiBUUkVFX1NZTUJPTF9SRUZFUkVO
 Q0VEIChERUNMX0FTU0VNQkxFUl9OQU1FIChkZWNsKSkpCi0gICAgd2FybmluZ193aXRoX2RlY2wg
 KGRlY2wsICJ3ZWFrIGRlY2xhcmF0aW9uIG9mIGAlcycgYWZ0ZXIgZmlyc3QgdXNlIHJlc3VsdHMg
 aW4gdW5zcGVjaWZpZWQgYmVoYXZpb3IiKTsKKyAgICB3YXJuaW5nX3dpdGhfZGVjbCAobmV3ZGVj
 bCwgIndlYWsgZGVjbGFyYXRpb24gb2YgYCVzJyBhZnRlciBmaXJzdCB1c2UgcmVzdWx0cyBpbiB1
 bnNwZWNpZmllZCBiZWhhdmlvciIpOwogCi0gIGRlY2xhcmVfd2VhayAoZGVjbCk7CisgIGRlY2xh
 cmVfd2Vha18xIChkZWNsLCBERUNMX1dFQUsgKG5ld2RlY2wpKTsKIH0KIAotLyogRGVjbGFyZSBE
 RUNMIHRvIGJlIGEgd2VhayBzeW1ib2wuICAqLworLyogRGVjbGFyZSBERUNMIHRvIGJlIGEgd2Vh
 ayBzeW1ib2wsIHB1dCBpdCBvbiB0aGUgV0VBS19ERUNMUyBsaXN0CisgICBpZiBORVcgaXMgc2V0
 LiAgKi8KIAotdm9pZAotZGVjbGFyZV93ZWFrIChkZWNsKQorc3RhdGljIHZvaWQKK2RlY2xhcmVf
 d2Vha18xIChkZWNsLCBuZXcpCiAgICAgIHRyZWUgZGVjbDsKKyAgICAgaW50IG5ldzsKIHsKICAg
 aWYgKCEgVFJFRV9QVUJMSUMgKGRlY2wpKQogICAgIGVycm9yX3dpdGhfZGVjbCAoZGVjbCwgIndl
 YWsgZGVjbGFyYXRpb24gb2YgYCVzJyBtdXN0IGJlIHB1YmxpYyIpOwpAQCAtNTAyNyw4ICs1MDMw
 LDE2IEBAIGRlY2xhcmVfd2VhayAoZGVjbCkKICAgICBlcnJvcl93aXRoX2RlY2wgKGRlY2wsICJ3
 ZWFrIGRlY2xhcmF0aW9uIG9mIGAlcycgbXVzdCBwcmVjZWRlIGRlZmluaXRpb24iKTsKICAgZWxz
 ZSBpZiAoU1VQUE9SVFNfV0VBSykKICAgICB7Ci0gICAgICBpZiAoISBERUNMX1dFQUsgKGRlY2wp
 KQotCXdlYWtfZGVjbHMgPSB0cmVlX2NvbnMgKE5VTEwsIGRlY2wsIHdlYWtfZGVjbHMpOworICAg
 ICAgdHJlZSAqcCwgdDsKKyAgICAgIGZvciAocCA9ICZ3ZWFrX2RlY2xzOyAodCA9ICpwKTsgcCA9
 ICZUUkVFX0NIQUlOICh0KSkKKwlpZiAoREVDTF9BU1NFTUJMRVJfTkFNRSAoZGVjbCkgPT0gREVD
 TF9BU1NFTUJMRVJfTkFNRSAoVFJFRV9WQUxVRSAodCkpKQorCSAgYnJlYWs7CisKKyAgICAgIGlm
 ICghIHQpCisJeworCSAgd2Vha19kZWNscyA9IHRyZWVfY29ucyAoTlVMTCwgZGVjbCwgd2Vha19k
 ZWNscyk7CisJICBUUkVFX1VTRUQgKHdlYWtfZGVjbHMpID0gMDsKKwl9CiAgICAgfQogICBlbHNl
 CiAgICAgd2FybmluZ193aXRoX2RlY2wgKGRlY2wsICJ3ZWFrIGRlY2xhcmF0aW9uIG9mIGAlcycg
 bm90IHN1cHBvcnRlZCIpOwpAQCAtNTA0Miw2ICs1MDUzLDE1IEBAIGRlY2xhcmVfd2VhayAoZGVj
 bCkKICAgICBTWU1CT0xfUkVGX1dFQUsgKFhFWFAgKERFQ0xfUlRMIChkZWNsKSwgMCkpID0gMTsK
 IH0KIAorLyogRGVjbGFyZSBERUNMIHRvIGJlIGEgd2VhayBzeW1ib2wuICAqLworCit2b2lkCitk
 ZWNsYXJlX3dlYWsgKGRlY2wpCisgICAgIHRyZWUgZGVjbDsKK3sKKyAgZGVjbGFyZV93ZWFrXzEg
 KGRlY2wsIDEpOworfQorCiAvKiBFbWl0IGFueSBwZW5kaW5nIHdlYWsgZGVjbGFyYXRpb25zLiAg
 Ki8KIAogdm9pZApAQCAtNTA1NCw3ICs1MDc0LDcgQEAgd2Vha19maW5pc2ggKCkKICAgICAgIHRy
 ZWUgZGVjbCA9IFRSRUVfVkFMVUUgKHQpOwogICAgICAgY29uc3QgY2hhciAqbmFtZSA9IElERU5U
 SUZJRVJfUE9JTlRFUiAoREVDTF9BU1NFTUJMRVJfTkFNRSAoZGVjbCkpOwogCi0gICAgICBpZiAo
 ISBUUkVFX1VTRUQgKGRlY2wpKQorICAgICAgaWYgKFRSRUVfVVNFRCAodCkgfHwgISBUUkVFX1VT
 RUQgKGRlY2wpKQogCWNvbnRpbnVlOwogCiAjaWZkZWYgQVNNX1dFQUtFTl9ERUNMCkBAIC01MDk2
 LDkgKzUxMTYsOCBAQCBnbG9iYWxpemVfZGVjbCAoZGVjbCkKICAgICAgIGZvciAocCA9ICZ3ZWFr
 X2RlY2xzOyAodCA9ICpwKSA7ICkKIAl7CiAJICBpZiAoREVDTF9BU1NFTUJMRVJfTkFNRSAoZGVj
 bCkgPT0gREVDTF9BU1NFTUJMRVJfTkFNRSAoVFJFRV9WQUxVRSAodCkpKQotCSAgICAqcCA9IFRS
 RUVfQ0hBSU4gKHQpOwotCSAgZWxzZQotCSAgICBwID0gJlRSRUVfQ0hBSU4gKHQpOworCSAgICBU
 UkVFX1VTRUQgKHQpID0gMTsKKwkgIHAgPSAmVFJFRV9DSEFJTiAodCk7CiAJfQogICAgICAgcmV0
 dXJuOwogICAgIH0KQEAgLTUxODYsNyArNTIwNSw3IEBAIG1ha2VfZGVjbF9vbmVfb25seSAoZGVj
 bCkKICAgICAgIERFQ0xfT05FX09OTFkgKGRlY2wpID0gMTsKICAgICB9CiAgIGVsc2UgaWYgKFNV
 UFBPUlRTX1dFQUspCi0gICAgREVDTF9XRUFLIChkZWNsKSA9IDE7CisgICAgZGVjbGFyZV93ZWFr
 IChkZWNsKTsKICAgZWxzZQogICAgIGFib3J0ICgpOwogfQo=
 --=====================_26563516==_--