Re: [pooma-dev] AltView* classes

[Jeffrey D. Oldham]

James Crotinger wrote:
> Hi Richard,
>
> This tickled a neuron the other day, but I couldn't recall the details.
>
> The AltView classes were put in to reduce link times and sizes for large 
> codes. The classes that have enums end up having a link-time cost, both 
> in space and time. I believe this refactoring was done to reduce the 
> cardinality of classes having the "sv" enum. My recollection is that 
> this, and other similar "optimizations", had a pretty substantial impact 
> on link-time for Blanca. Unless these are hurting something else, I 
> would tend to leave them in.
>
> Does the CodeSourcery CVS repository have complete CVS history? (i.e. 
> did we copy the repository, or just the head version?) This should have 
> been easy to determine via CVS.

The POOMA CVS repository history goes back to at least 1998 March.  For 
example, see the log for src/Array/Array.h.  This log gives an entry for 
AltView0 and AltView1Implementation:

revision 1.141
date: 2001/05/25 17:42:48;  author: oldham;  state: Exp;  lines: +47 -6
branches:  1.141.4;
2001-05-25  Jeffrey D. Oldham <address@hidden>
        * Array/Array.h (View0): Modify initial comment to indicate
        changes should also be made to AltView0.
        (AltView0): New class to avoid explicit instantiation problems.
        (ComponentView): Modify initial comment to indicate class should
        not be explicitly instantiated.
        (Array::read()): Change return type to AltView0.
        (Array::operator()()): Likewise.

        * NewField/Field.h (View1Implementation): Modify initial comment
        to indicate changes should also be made to AltView1Implementation.
        (AltView1Implementation): New class to avoid explicit
        instantiation problems.
        (View1): Modify initial comment to indicate changes should also be
        made to AltView1.
        (AltView1): New class to avoid explicit instantiation problems.
        (Field::read()): Change return type to AltView1.
        (Field::readAll()): Likewise.
        (Field::read(s1)): Likewise.
        (Field::operator()()): Likewise.
        (Field::all()): Likewise.
        (Field::operator()(s1)): Likewise.

Perhaps this will jog Jim's memory more.  I do not remember much about this.
--
Jeffrey D. Oldham
address@hidden