[M3commit] CVS Update: cm3

[Tony Hosking]

That should be handled by placing an appropriate cast in the gcc tree.

On Nov 23, 2010, at 1:45 PM, Jay K wrote:

> 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/720c4d5b/attachment-0002.html>