[pooma-dev] Re: [PATCH] Speed up guard update.

Jeffrey D. Oldham oldham at codesourcery.com
Fri Jan 16 17:11:48 UTC 2004


Richard Guenther wrote:
> On Thu, 15 Jan 2004, Jeffrey D. Oldham wrote:
> 
> 
>>Richard Guenther wrote:
>>
>>>Hi!
>>>
>>>This is a refined (aka shorter) patch which unifies the tracking of
>>>up-to-date faces and the special optimized copy for MPI.
>>>
>>>Tested on serial ia32 linux with gcc3.4 with no regression.
>>>
>>>Ok?
>>
>>Yes, assuming the user interface did not change.  It looks like
>>GCFillInfo's interface changed but existing code will still run because
>>a parameter with a default argument was added.
> 
> 
> It depends on where you draw the line between "user" interface and
> internal interface.  I'd consider GCFillInfo's internal interface, only
> the fillGuards(), setDirty(), etc. methods from the MultiPatchEngine I'd
> consider "user" interface - and these ones will still work with old code.
> I also changed the various intersectors to take an additional parameter - old code will
> break if it used this interface (but I consider this not user interface
> either).

Great!  I interpret this a "no change to the user interface".  Would you 
please commit the patch?

-- 
Jeffrey D. Oldham
oldham at codesourcery.com




More information about the pooma-dev mailing list