On 03/16/2012 02:33 PM, Zdenek Dvorak wrote: >>>> Yes, that could work. ?Though it might end up being incredibly ugly ;) >>> >>> In the code? Should really only change a few lines in the patch I would >>> have thought. I'll get back to you when I have a new version - thanks >>> for the tip. >> >> No, in the GIMPLE IL. >> >> Richard. > > And I can just imagine how happy would other optimizations be about such a > change. > Are we speaking about something that you would like to get into mainline? > All of this looks like a lot of hacks to deal with a rather weird target.
The final piece - using &TARGET_MEM_REF rather than POINTER_PLUS_EXPR to avoid overflow issues - is clearly a bit of a hack. But otherwise, keeping around the proper types rather than casting everything to unsigned int is IMO much less ugly and hackish than what we have now. Certainly the gimple IL is much less cluttered after the patch than before. Bernd
