Re: [9fans] remember when C compiler used to produce working code? i don't!

[David Presotto <presotto@closedmind.org>]

[-- Attachment #1: Type: text/plain, Size: 174 bytes --]

Looks like a problem in cc/com.c in tcomo().  I'm going to be away
for a week and will take a crack at it when I get back.  If someone
else wants to try in the mean time...

[-- Attachment #2: Type: message/rfc822, Size: 5303 bytes --]

[-- Attachment #2.1.1: Type: text/plain, Size: 180 bytes --]

It does work if you replace:

	        elasticity *= ratio;

with

	        elasticity = elasticity * ratio;

Clearly we're analyzing wrong when there's only an assignment.

[-- Attachment #2.1.2: Type: message/rfc822, Size: 3015 bytes --]

From: andrey mirtchovski <mirtchov@cpsc.ucalgary.ca>
To: 9fans@cse.psu.edu
Subject: [9fans] remember when C compiler used to produce working code? i don't!
Date: Fri, 30 Jan 2004 15:05:43 -0700
Message-ID: <d45fb131231fd5a3515e0f7518a6568a@plan9.ucalgary.ca>

I got bitten by this one pretty hard today, took me the better part of
the day to find out...  The 'elasticity *= ratio' code comes from the
XScreensaver distribution and was written sometime in 1997, so
presumably this has been working since then (and braking when compiled
with 8c?).

In Plan 9 the following code produces 800/0:

	plan9-2% cat t.c
	#include <u.h>
	#include <libc.h>

	main()
	{
		long elasticity = 8000L;
		double ratio = 0.1;

		/* this is fine */
		elasticity = (long)(((double)elasticity)*ratio);
		print("%ld\n", elasticity);

		/* this breaks with 8c */
		elasticity *= ratio;
		print("%ld\n", elasticity);

	}
	plan9-2% 8c t.c; 8l t.8
	plan9-2% 8.out
	800
	0
	plan9-2%


Its (almost) identical lunix counterpart, when compiled with gcc 3.2,
produces 800/80:

	mirtchov@fbsd$ cat t.c
	main()
	{
	        long elasticity = 8000L;
	        double ratio = 0.1;

		/* this is fine */
	        elasticity = (long)(((float)elasticity)*ratio);
	        printf("%ld\n", elasticity);

		/* this if fine with gcc 3.2 */
	        elasticity *= ratio;
	        printf("%ld\n", elasticity);

	}
	mirtchov@fbsd$ gcc t.c
	mirtchov@fbsd$ ./a.out
	800
	80
	mirtchov@fbsd$


Not looking to assign any blame, just letting you know...

andrey