From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 19672 invoked by alias); 23 Feb 2002 21:06:01 -0000 Mailing-List: contact gcc-prs-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Archive: List-Post: List-Help: Sender: gcc-prs-owner@gcc.gnu.org Received: (qmail 19641 invoked by uid 71); 23 Feb 2002 21:06:01 -0000 Date: Sat, 23 Feb 2002 14:12:00 -0000 Message-ID: <20020223210601.19639.qmail@sources.redhat.com> To: ro@gcc.gnu.org Cc: gcc-prs@gcc.gnu.org, From: "Richard B. Kreckel" Subject: Re: target/5505: Doubts about a patch for OSF Reply-To: "Richard B. Kreckel" X-SW-Source: 2002-02/txt/msg00610.txt.bz2 List-Id: The following reply was made to PR target/5505; it has been noted by GNATS. From: "Richard B. Kreckel" To: Rainer Orth Cc: gcc-bugs@gcc.gnu.org, gcc-gnats@gcc.gnu.org Subject: Re: target/5505: Doubts about a patch for OSF Date: Sat, 23 Feb 2002 22:03:28 +0100 (CET) This message is in MIME format. The first part should be readable text, while the remaining parts are likely unreadable without MIME-aware tools. Send mail to mime@docserver.cac.washington.edu for more info. --1569097344-122519837-1014498208=:18949 Content-Type: TEXT/PLAIN; charset=US-ASCII Hi, On Fri, 22 Feb 2002, Rainer Orth wrote: > Richard B. Kreckel writes: > > > Rainer, you probably know by now that this whole issue has nothing to do > > at all with CLN? It seems like your patch breaks anything when using > > most likely, yes. > > > -fno-exceptions. Attached is a trial program that only uses doubles and > > Only in combination with -O2, it seems. > > > some STL containers. (Sorry for not boiling it down from 100 to 10 lines, > > I have unfortunately very little time right now.) > > Please try to do so: my knowledge of C++ is close to zero, so a minimal > testcase would help enormously. Sure, I will, next week. Sorry, again I am only for a couple of minutes in my office... > > Can you confirm this? Otherwise we are back with the theory that my boxen > > are broken / patched wrongly... > > Not yet: I get a SEGV even if compiling without any special options: > > > g++ -o minor minor.cc > > ./minor > permanent of 6x6-matrix > c==4Segmentation fault > > The crash happens in minor.c, l. 80: > > 80 Pkey[j] = Pkey[j-1]+1; > > j = 0 at this point, so Pkey[j-1] is out of bounds. The crash happens when > compiling with Compaq cxx as well. Sorry, silly me. I've attached an old case and that index out of bound has been fixed in the real sources ages ago. However, the crash I see seems to have nothing to do with this line! Modifying the condition such that j==0 does not happen still results in a crashing program when compiled with -O2 -fno-exceptions and a working program when compiled with -O2 alone. There is an example modified accordingly attached to this email and this time I have made sure it also runs with -lefence and stuff. If you have some time, it would be nice if you could confirm the dependence on -fno-exceptions before I start stripping that down. Thanks! Regards -richy. -- Richard B. Kreckel --1569097344-122519837-1014498208=:18949 Content-Type: TEXT/x-c++src; name="minor.cc" Content-Transfer-Encoding: BASE64 Content-ID: Content-Description: minor.cc Content-Disposition: attachment; filename="minor.cc" I2luY2x1ZGUgPGNtYXRoPg0KI2luY2x1ZGUgPGlvc3RyZWFtPg0KI2luY2x1 ZGUgPHZlY3Rvcj4NCiNpbmNsdWRlIDxtYXA+DQojaW5jbHVkZSA8YWxnb3Jp dGhtPg0KdXNpbmcgbmFtZXNwYWNlIHN0ZDsNCg0KaW50IHNpemUgPSA3Ow0K DQp2b2lkIGluaXQodmVjdG9yPGRvdWJsZT4gJiB2KQ0Kew0KICAgIHYuY2xl YXIoKTsNCiAgICBmb3IgKGludCBpPTA7IGk8c2l6ZSpzaXplOyArK2kpDQog ICAgICAgIHYucHVzaF9iYWNrKCgxMC4wKnJhbmQoKSkvKFJBTkRfTUFYKzEu MCktNS4wKTsNCn0NCg0KZG91YmxlIGRldF9taW5vcihjb25zdCB2ZWN0b3I8 ZG91YmxlPiAmdikNCnsNCiAgICAvLyBmb3Igc21hbGwgbWF0cmljZXMgdGhl IGFsZ29yaXRobSBkb2VzIG5vdCBtYWtlIHNlbnNlOg0KICAgIGlmIChzaXpl PT0xKQ0KICAgICAgICByZXR1cm4gdlswXTsNCiAgICBpZiAoc2l6ZT09MikN CiAgICAgICAgcmV0dXJuIHZbMF0qdlszXS12WzJdKnZbMV07DQogICAgaWYg KHNpemU9PTMpDQogICAgICAgIHJldHVybiAoKHZbNF0qdls4XS12WzVdKnZb N10pKnZbMF0tDQogICAgICAgICAgICAgICAgKHZbMV0qdls4XS12WzJdKnZb N10pKnZbM10rDQogICAgICAgICAgICAgICAgKHZbMV0qdls1XS12WzRdKnZb Ml0pKnZbNl0pOw0KDQogICAgLy8gd2Ugc3RvcmUgb3VyIHN1Ym1pbm9ycyBp biB0aGVzZSBjb250YWluZXJzDQogICAgdHlwZWRlZiBtYXA8dmVjdG9yPHVu c2lnbmVkPixkb3VibGU+IFJtYXA7DQogICAgdHlwZWRlZiBtYXA8dmVjdG9y PHVuc2lnbmVkPixkb3VibGU+Ojp2YWx1ZV90eXBlIFJtYXBfdmFsdWU7DQog ICAgUm1hcCBBLCBCOw0KICAgIGRvdWJsZSBkZXQgPSAwLjA7DQogICAgdmVj dG9yPHVuc2lnbmVkPiBQa2V5OyAgICAvLyBVbmlxdWUgZmxpcHBlciBjb3Vu dGVyIGZvciB0aGUgcGFydGl0aW9uDQogICAgUGtleS5yZXNlcnZlKHNpemUp Ow0KICAgIHZlY3Rvcjx1bnNpZ25lZD4gTWtleTsgICAgLy8ga2V5IGZvciBt aW5vciBkZXRlcm1pbmFudCAoYSBwYXJ0aXRpb24gb2YgUGtleSkNCiAgICBN a2V5LnJlc2VydmUoc2l6ZS0xKTsNCiAgICAvLyBpbml0aWFsaXplIEEgd2l0 aCBsYXN0IGNvbHVtbjoNCiAgICBmb3IgKHVuc2lnbmVkIHI9MDsgcjxzaXpl OyArK3IpIHsNCiAgICAgICAgUGtleS5lcmFzZShQa2V5LmJlZ2luKCksUGtl eS5lbmQoKSk7DQogICAgICAgIFBrZXkucHVzaF9iYWNrKHIpOw0KICAgICAg ICBBLmluc2VydChSbWFwX3ZhbHVlKFBrZXksdltzaXplKnIrc2l6ZS0xXSkp Ow0KICAgIH0NCiAgICAvLyBjbG9nIDw8ICJsb29wOiAiIDw8IGVuZGw7DQog ICAgZm9yIChpbnQgYz1zaXplLTI7IGM+PTA7IC0tYykgew0KICAgICAgY2xv ZyA8PCAiYz09IiA8PCBjICA8PCBmbHVzaDsNCiAgICAgICAgUGtleS5lcmFz ZShQa2V5LmJlZ2luKCksUGtleS5lbmQoKSk7ICAvLyBkb24ndCBjaGFuZ2Ug Y2FwYWNpdHkNCiAgICAgICAgTWtleS5lcmFzZShNa2V5LmJlZ2luKCksTWtl eS5lbmQoKSk7DQogICAgICAgIGZvciAodW5zaWduZWQgaT0wOyBpPHNpemUt YzsgKytpKQ0KICAgICAgICAgICAgUGtleS5wdXNoX2JhY2soaSk7DQogICAg ICAgIHVuc2lnbmVkIGZjID0gMDsgIC8vIGNvbnRyb2xzIGxvZ2ljIGZvciBv dXIgc3RyYW5nZSBmbGlwcGVyIGNvdW50ZXINCiAgICAgICAgZG8gew0KICAg ICAgICAgICAgZGV0ID0gMC4wOw0KICAgICAgICAgICAgZm9yICh1bnNpZ25l ZCByPTA7IHI8c2l6ZS1jOyArK3IpIHsNCiAgICAgICAgICAgICAgICAvLyBt YXliZSB0aGVyZSBpcyBub3RoaW5nIHRvIGRvPw0KICAgICAgICAgICAgICAg IGlmICh2W1BrZXlbcl0qc2l6ZStjXT09MC4wKQ0KICAgICAgICAgICAgICAg ICAgICBjb250aW51ZTsNCiAgICAgICAgICAgICAgICAvLyBjcmVhdGUgdGhl IHNvcnRlZCBrZXkgZm9yIGFsbCBwb3NzaWJsZSBtaW5vcnMNCiAgICAgICAg ICAgICAgICBNa2V5LmVyYXNlKE1rZXkuYmVnaW4oKSxNa2V5LmVuZCgpKTsN CiAgICAgICAgICAgICAgICBmb3IgKHVuc2lnbmVkIGk9MDsgaTxzaXplLWM7 ICsraSkNCiAgICAgICAgICAgICAgICAgICAgaWYgKGkhPXIpDQogICAgICAg ICAgICAgICAgICAgICAgICBNa2V5LnB1c2hfYmFjayhQa2V5W2ldKTsNCiAg ICAgICAgICAgICAgICAvLyBmZXRjaCB0aGUgbWlub3JzIGFuZCBjb21wdXRl IHRoZSBuZXcgZGV0ZXJtaW5hbnQNCiAgICAgICAgICAgICAgICBpZiAociUy KQ0KICAgICAgICAgICAgICAgICAgICBkZXQgLT0gdltQa2V5W3JdKnNpemUr Y10qQVtNa2V5XTsNCiAgICAgICAgICAgICAgICBlbHNlDQogICAgICAgICAg ICAgICAgICAgIGRldCArPSB2W1BrZXlbcl0qc2l6ZStjXSpBW01rZXldOw0K ICAgICAgICAgICAgfQ0KICAgICAgICAgICAgLy8gU3RvcmUgdGhlIG5ldyBk ZXRlcm1pbmFudCBhdCBpdHMgcGxhY2UgaW4gQjoNCiAgICAgICAgICAgIGlm IChkZXQhPTAuMCkNCiAgICAgICAgICAgICAgICBCLmluc2VydChSbWFwX3Zh bHVlKFBrZXksZGV0KSk7DQogICAgICAgICAgICAvLyBpbmNyZW1lbnQgb3Vy IHN0cmFuZ2UgZmxpcHBlciBjb3VudGVyDQogICAgICAgICAgICBmb3IgKGZj PXNpemUtYzsgZmM+MDsgLS1mYykgew0KICAgICAgICAgICAgICAgICsrUGtl eVtmYy0xXTsNCiAgICAgICAgICAgICAgICBpZiAoUGtleVtmYy0xXTxmYytj KQ0KICAgICAgICAgICAgICAgICAgICBicmVhazsNCiAgICAgICAgICAgIH0N CiAgICAgICAgICAgIGlmIChmYzxzaXplLWMgJiYgZmM+MCkNCiAgICAgICAg ICAgICAgICBmb3IgKHVuc2lnbmVkIGo9ZmM7IGo8c2l6ZS1jOyArK2opDQog ICAgICAgICAgICAgICAgICAgIFBrZXlbal0gPSBQa2V5W2otMV0rMTsNCiAg ICAgICAgfSB3aGlsZShmYyk7DQogICAgICAgIC8vIGNoYW5nZSB0aGUgcm9s ZSBvZiBBIGFuZCBCOg0KICAgICAgICBBID0gQjsNCiAgICAgICAgQi5jbGVh cigpOw0KCWNsb2cgPDwgZW5kbDsNCiAgICB9DQoNCiAgICByZXR1cm4gZGV0 Ow0KfQ0KDQppbnQgbWFpbih2b2lkKQ0Kew0KICAgIHNyYW5kKCh1bnNpZ25l ZCl0aW1lKE5VTEwpKTsNCiAgICBkb3VibGUgZDA7DQogICAgdmVjdG9yPGRv dWJsZT4gbTsNCiAgICBmb3IgKHNpemU9Njsgc2l6ZTwxMTsgKytzaXplKSB7 DQogICAgICAgIGNvdXQgPDwgImRldGVybWluYW50IG9mICIgPDwgc2l6ZSA8 PCAieCIgPDwgc2l6ZSA8PCAiLW1hdHJpeCIgPDwgZW5kbDsNCiAgICAgICAg Zm9yIChpbnQgaT0wOyBpPDIwOyArK2kpIHsNCiAgICAgICAgICAgIGluaXQo bSk7DQogICAgICAgICAgICBkMCA9IGRldF9taW5vcihtKTsNCiAgICAgICAg ICAgIGNvdXQgPDwgZDAgPDwgZW5kbDsNCiAgICAgICAgfQ0KICAgIH0NCn0N Cg== --1569097344-122519837-1014498208=:18949--