[M3commit] CVS Update: cm3

[Jay K]

Try building m3-sys/m3tests/p2/p240.
There is an error in the configure enable-checking about the types being mismatched.

 - Jay


> From: hosking at cs.purdue.edu
> Date: Tue, 23 Nov 2010 09:48:19 -0500
> To: jkrell at elego.de
> CC: m3commit at elegosoft.com
> Subject: Re: [M3commit] CVS Update: cm3
> 
> 
> On Nov 23, 2010, at 10:49 AM, Jay Krell wrote:
> 
> > CVSROOT:	/usr/cvs
> > Changes by:	jkrell at birch.	10/11/23 10:49:57
> > 
> > Modified files:
> > 	cm3/m3-sys/m3cc/gcc/gcc/m3cg/: parse.c 
> > 
> > Log message:
> > 	Go with the obvious less heavy handed fix of only avoiding
> > 	fold for integer division.
> 
> I don't understand what the issue is with folding on integer division.  Please explain.
> 
> > 	This is enough for test p240 and the unoptimized code is definitely better.
> > 	NOT fully tested, but really probably works.
> > 	It is probably worth digging into the frontend to see if there is a difference
> > 	in the types. It is probably also worth digging in the backend, like,
> > 	we do put in the casts consistently, why are they only not effective
> > 	for divide? I realize that a little bit of analysis goes a long way
> > 	on many divide cases: "divide tends to produce small results", sort of.
> > 	On the other hand, it isn't worth too much effort.
> > 	Divide is the slowest least used integer operation.
> 
 		 	   		  
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://m3lists.elegosoft.com/pipermail/m3commit/attachments/20101123/b4a87bad/attachment-0002.html>