Relay-Version: version B 2.10 5/3/83; site utzoo.UUCP
Posting-Version: version B 2.10.2 9/18/84; site brl-tgr.ARPA
Path: utzoo!watmath!clyde!burl!ulysses!allegra!mit-eddie!godot!harvard!seismo!brl-tgr!gwyn
From: gwyn@brl-tgr.ARPA (Doug Gwyn <gwyn>)
Newsgroups: net.bugs.usg
Subject: Re: more on PCC ICON type bug
Message-ID: <9351@brl-tgr.ARPA>
Date: Wed, 20-Mar-85 00:06:49 EST
Article-I.D.: brl-tgr.9351
Posted: Wed Mar 20 00:06:49 1985
Date-Received: Thu, 21-Mar-85 03:12:31 EST
References: <9328@brl-tgr.ARPA>
Distribution: net
Organization: Ballistic Research Lab
Lines: 19

Oops, the bug fix I posted for the ICON size problem in the System V PCC
was too drastic:  In local2.c, in the acon() routine, the printf should
only shrink the tn.lval when it is indeed an ICON!  The bug fix should
read:

/*	@(#)local2.c	1.5	*/
...
acon( p ) register NODE *p; { /* print out a constant */
	extern CONSZ	shrink();	/* DAG -- added for assembler glitch fix */

	if( p->in.name[0] == '\0' ){
		/* DAG -- keep assembler happy with size of integer constants */
		printf( CONFMT, p->in.op==ICON ? shrink( p ) : p->tn.lval );
		}
...

I wish whoever wrote the compiler (Steve?) had inserted more comments
explaining what in the world is going on inside it.  It is no fun trying
to fix bugs like this..