Re: [pooma-dev] AltView* classes

freepooma-devel

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [pooma-dev] AltView* classes

From:	John H. Hall
Subject:	Re: [pooma-dev] AltView* classes
Date:	Tue, 2 Mar 2004 10:34:09 -0700

Dave, et al:

I have just had extensive discussion regarding the "Blanca is dead"issue. That is not quite the whole story. Management has decided toforce Blanca and Shavano to merge, but, it is up to Blanca and Shavanoto define what exactly that means. The head of ASCI at LANL is JamesPeery and he believes (to the best of my knowledge) that someamalgamation will win out.

Meanwhile, Don and Jean Marshall and I are still developing what weaffectionately refer to as "the Blanca Demo Project". This is funded(at a low level) through at least the end of this year, and around thattime, I believe the code will start really impressing some people. Weare having to develop a series of new POOMA engines for this project,some of which I hinted at a while back. The changes to fieldcomtemplated here won't really affect engines for the most part, so ifDave's happy then I am happy.

A special thanks to Richard for putting so much time into advancing thestatus of POOMA.

John Hall


On Mar 2, 2004, at 10:16 AM, Dave Nystrom wrote:

No problem. I think my neurons are less rusty because of my perverse
hobby:-). I've actually compiled the Blanca Demo stuff in the lastyear and
I try to keep up with evolution of Pooma 2 with the thought that I will
sometime get back to using it. John, Don and Jean have actually beendoing
some work on the source code base in the last year.
Also, the Blanca Demo project and it's progeny is the only Blancarelatedsource code base that to my knowledge ever used Pooma 2. Andsupposedly, the
Blanca project is dead and unlikely to use anything that we ever did.
--Dave Nystrom email: address@hidden
LANL X-2 phone: 505-667-7913 fax: 505-665-2227

James Crotinger writes:
OK - cool. Thanks for chipping in Dave. You're neurons are obviouslyless
rusty than mine. :)

 Jim
------------------------------------------------------------------------James A. Crotinger email:address@hiddenNumeriX, LLC phone: (505) 424-4477x104
2960 Rodeo Park Dr. W.
Santa Fe, NM 87505
-----Original Message-----
From: Dave Nystrom [mailto:address@hidden
Sent: Tuesday, March 02, 2004 10:00 AM
To: James Crotinger
Cc: 'Richard Guenther'; 'address@hidden'; 'JeffreyD.
Oldham'; John Hall; Dave Nystrom; Dave Nystrom
Subject: RE: [pooma-dev] AltView* classes
These AltView classes were put in by Jeffrey Oldham to fix a problemthat
I
was having trying to explicitly instantiate all the Pooma classes and
functions that Blanca code was using. Basically, the way the Pooma 2
Field
class was originally written, we were getting one of these "sv" enums
instantiated for every unique expression which we had in our sourcecodebase. I was unable to explicitly instantiate these "sv" enums theway thecode was originally written because of some issue which Arch Robisonof
Kuck
and Associates (now Intel) identified - basically the code in it's
original
form violated some corner case of the C++ standard according to Arch.
Jeffrey fixed this problem by adding the AltView class and then Iwas able
to
perform the explicit instantiations that I needed.
However, I believe Jeffrey or someone else has gone through the codeandreplaced the "sv" enums with some other construct which does nothave theexplicit instantiation difficulties that the "sv" enums had. So, Ithinkthat it is probably okay to remove the AltView classes and that thiswould
clean up the code considerably. I may revisit all this explicit
instantiation stuff sometime in the not to distant future on a hobbybasis
(weird hobby, huh) and if I encounter difficulties being able to
explicitly
instantiate anything, I could then raise it as an issue at that time.
In summary, the AltView classes were added by Jeffrey to fix anexplicitinstantiation problem that I think no longer exists and I am infavor of
now
removing them in the interest of cleaning up the code as much aspossible.
--
Dave Nystrom email: address@hidden
LANL X-2 phone: 505-667-7913 fax: 505-665-2227

James Crotinger writes:
Hi Richard,
This tickled a neuron the other day, but I couldn't recall thedetails.
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 impactonlink-time for Blanca. Unless these are hurting something else, Iwould
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 havebeen
easy
to determine via CVS.

 Jim
-----------------------------------------------------------------------
-
James A. Crotinger email:
address@hidden
NumeriX, LLC phone: (505) 424-4477
x104
2960 Rodeo Park Dr. W.
Santa Fe, NM 87505
-----Original Message-----
From: Richard Guenther [mailto:address@hidden
Sent: Tuesday, March 02, 2004 3:29 AM
To: address@hidden
Cc: Jeffrey D. Oldham
Subject: Re: [pooma-dev] AltView* classes

On Tue, 2 Mar 2004, Richard Guenther wrote:
On Fri, 27 Feb 2004, Richard Guenther wrote:
Hi!

Does anyone remember the exact reason for the existance of the
AltView*
classes in Array.h and Field.h? Removing them and fixing their
usage
to
use View* seems to work (full test with gcc 3.3 and 3.4 in
progress).
Testing completed with no new failures. Would it be ok to remove
those
AltView* classes?
Which would be...

Ok?

Richard.


2004Mar02 Richard Guenther <address@hidden>

 * src/Array/Array.h: remove all traces of AltView0 class.
 src/Field/Field.h: remove all traces of AltView1 and
 AltView1Implementation classes.

===== Array/Array.h 1.14 vs edited =====
--- 1.14/r2/src/Array/Array.h Tue Dec 16 15:19:57 2003
+++ edited/Array/Array.h Tue Mar 2 11:24:14 2004
@@ -649,8 +649,6 @@
 // an existing engine cannot be any kind of slice domain.
 // Also, bounds checking would make no sense because it would
 // reduce to contains(a.domain(), a.domain());
-//
-// Any changes to this class should also be made to AltView0.

 template<int Dim, class T, class EngineTag>
 struct View0<Array<Dim, T, EngineTag> >
@@ -695,41 +693,6 @@
 }
 };

-// AltView0 avoids an instantiation problem that arises when two
-// classes use each other. This class's definition should be
exactly
-// the same as View0 except omitting member functions.
-//
-// Do NOT explicitly instantiate this class.
-
-template<class ArrayTag>
-struct AltView0;
-
-template<int Dim, class T, class EngineTag>
-struct AltView0<Array<Dim, T, EngineTag> >
-{
- // Convenience typedef for the thing we're taking a view of.
-
- typedef Array<Dim, T, EngineTag> Subject_t;
-
- // Deduce domains for the output type.
- // At some point, we need to fix NewDomain1; until then, use
- // the temporary version from Array.h.
-
- typedef typename Subject_t::Engine_t Engine_t;
- typedef typename Subject_t::Domain_t Domain_t;
-
- // Deduce the template parameters for the output type.
-
- typedef typename NewEngine<Engine_t, Domain_t>::Type_t
NewEngine_t;
- enum { newDim = NewEngine_t::dimensions };
- typedef typename NewEngine_t::Tag_t NewEngineTag_t;
-
- // The output types.
-
- typedef Array<newDim, T, NewEngineTag_t> Type_t;
- typedef Type_t ReadType_t;
-};
-
 template<int Dim, class T, class EngineTag>
 struct View1<Array<Dim, T, EngineTag>, int>
 {
@@ -1864,7 +1827,7 @@
 /// A zero-argument version of operator(), which takes a view of
 /// array's domain, is also supplied.
 //@{
- typename AltView0<This_t>::ReadType_t
+ typename View0<This_t>::ReadType_t
 read() const
 {
 typedef View0<This_t> Ret_t;
@@ -1934,7 +1897,7 @@
 return Ret_t::makeRead(*this, s1, s2, s3, s4, s5, s6, s7);
 }

- typename AltView0<This_t>::Type_t
+ typename View0<This_t>::Type_t
 operator()() const
 {
 typedef View0<This_t> Ret_t;
===== Field/Field.h 1.14 vs edited =====
--- 1.14/r2/src/Field/Field.h Tue Dec 16 15:20:00 2003
+++ edited/Field/Field.h Tue Mar 2 11:25:22 2004
@@ -269,9 +269,6 @@
 * View1Implementation<Field, D, SV> specialization for indexing a
field
 * with a single domain. There is a single-valued version (SV ==
true)
 * and a multi-valued version (SV == false).
- *
- * Any changes to View1Implementation should also be made to
- * AltView1Implementation.
 */

 // Single-valued version. Handles scalars and Locs.
@@ -508,62 +505,7 @@


 /**
- * AltView1Implementation avoids an instantiation problem that
arises
when two
- * classes use each other. This class's definition should be
exactly
- * the same as View1Implementation except omitting memberfunctions.
- *
- * Do NOT explicitly instantiate this class.
- */
-
-// Single-valued version. Handles scalars and Locs.
-
-template<class Subject, class Domain, bool SV>
-struct AltView1Implementation;
-
-template<class Mesh, class T, class EngineTag, class Domain>
-struct AltView1Implementation<Field<Mesh, T, EngineTag>, Domain,
true>
-{
- // Convenience typedef for the thing we're taking a view of.
-
- typedef Field<Mesh, T, EngineTag> Subject_t;
-
- // The return types are pretty simple here.
-
- typedef typename Subject_t::Element_t ReadType_t;
- typedef typename Subject_t::ElementRef_t Type_t;
-};
-
-template<class Mesh, class T, class EngineTag, class Domain>
-struct AltView1Implementation<Field<Mesh, T, EngineTag>, Domain,
false>
-{
- // Convenience typedef for the thing we're taking a view of.
-
- typedef Field<Mesh, T, EngineTag> Subject_t;
-
- // Deduce domains for the output type.
-
- typedef typename Subject_t::Engine_t Engine_t;
- typedef typename NewEngine<Engine_t, Domain>::Type_tNewEngine_t;
- typedef typename NewEngine_t::Element_t NewT_t;
- typedef typename NewEngine_t::Tag_t NewEngineTag_t;
-
- // Deduce the new Mesh.
-
- typedef typename
- NewMeshTag<NewEngine_t::dimensions, Mesh, Domain>::Type_t
- NewMeshTag_t;
-
- // The output types.
-
- typedef Field<NewMeshTag_t, NewT_t, NewEngineTag_t> ReadType_t;
- typedef Field<NewMeshTag_t, NewT_t, NewEngineTag_t> Type_t;
-};
-
-
-/**
 * View1<Field, S1> specialization for indexing a field with a
single
domain.
- *
- * Any changes to View1 should also be made to AltView1.
 */

 template<class Mesh, class T, class EngineTag, class Sub1>
@@ -613,8 +555,6 @@

 /**
 * View1<Field, int> specialization for indexing a field with an
int.
- *
- * Any changes to View1 should also be made to AltView1.
 */

 template<class Mesh, class T, class EngineTag>
@@ -658,71 +598,6 @@


 /**
- * AltView1 avoids an instantiation problem that arises when two
- * classes use each other. This class's definition should be
exactly
- * the same as View1 except omitting member functions.
- *
- * Do NOT explicitly instantiate this class.
- */
-
-template<class FieldTag, class DomainTag>
-struct AltView1;
-
-template<class Mesh, class T, class EngineTag, class Sub1>
-struct AltView1<Field<Mesh, T, EngineTag>, Sub1>
-{
- // Convenience typedef for the thing we're taking a view of.
-
- typedef Field<Mesh, T, EngineTag> Subject_t;
-
- // Deduce domains for the output type.
- // At some point, we need to fix NewDomain1; until then, use
- // the temporary version from NewDomain.h.
-
- typedef typename Subject_t::Domain_t Domain_t;
- typedef TemporaryNewDomain1<Domain_t, Sub1> NewDomain_t;
- typedef typename NewDomain_t::SliceType_t SDomain_t;
-
- // Deduce appropriate version of implementation to dispatch to.
-
- enum { sv = DomainTraits<SDomain_t>::singleValued };
- typedef AltView1Implementation<Subject_t, SDomain_t, sv>
Dispatch_t;
-
- // The optimized domain combiner.
-
- typedef CombineDomainOpt<NewDomain_t, sv> Combine_t;
-
- // The return types.
-
- typedef typename Dispatch_t::ReadType_t ReadType_t;
- typedef typename Dispatch_t::Type_t Type_t;
-};
-
-
-/**
- * AltView1 avoids an instantiation problem that arises when two
- * classes use each other. This class's definition should be
exactly
- * the same as View1 except omitting member functions.
- *
- * Do NOT explicitly instantiate this class.
- */
-
-template<class Mesh, class T, class EngineTag>
-struct AltView1<Field<Mesh, T, EngineTag>, int>
-{
- // Convenience typedef for the thing we're taking a view of.
-
- typedef Field<Mesh, T, EngineTag> Subject_t;
-
- // The return types.
-
- typedef typename Subject_t::Element_t ReadType_t;
- typedef typename Subject_t::ElementRef_t Type_t;
-
-};
-
-
-/**
* View2<Field, S1, S2> specialization for indexing a field withtwo
 * domains.
 */
@@ -1455,14 +1330,14 @@
/// point fields the viewing domain is an actual centeringdomain.
 //@{

- inline typename AltView1<This_t, Domain_t>::ReadType_t
+ inline typename View1<This_t, Domain_t>::ReadType_t
 read() const
 {
 typedef View1<This_t, Domain_t> Ret_t;
 return Ret_t::makeRead(*this, physicalDomain());
 }

- inline typename AltView1<This_t, Domain_t>::ReadType_t
+ inline typename View1<This_t, Domain_t>::ReadType_t
 readAll() const
 {
 typedef View1<This_t, Domain_t> Ret_t;
@@ -1470,7 +1345,7 @@
 }

 template<class Sub1>
- inline typename AltView1<This_t, Sub1>::ReadType_t
+ inline typename View1<This_t, Sub1>::ReadType_t
 read(const Sub1 &s1) const
 {
 typedef View1<This_t, Sub1> Ret_t;
@@ -1493,14 +1368,14 @@
 return Ret_t::makeRead(*this, s1, s2, s3);
 }

- inline typename AltView1<This_t, Domain_t>::Type_t
+ inline typename View1<This_t, Domain_t>::Type_t
 operator()() const
 {
 typedef View1<This_t, Domain_t> Ret_t;
 return Ret_t::make(*this, physicalDomain());
 }

- inline typename AltView1<This_t, Domain_t>::Type_t
+ inline typename View1<This_t, Domain_t>::Type_t
 all() const
 {
 typedef View1<This_t, Domain_t> Ret_t;
@@ -1508,7 +1383,7 @@
 }

 template<class Sub1>
- inline typename AltView1<This_t, Sub1>::Type_t
+ inline typename View1<This_t, Sub1>::Type_t
 operator()(const Sub1 &s1) const
 {
 typedef View1<This_t, Sub1> Ret_t;
<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 3.2//EN">
<HTML>
<HEAD>
<META HTTP-EQUIV="Content-Type" CONTENT="text/html;charset=us-ascii">
<META NAME="Generator" CONTENT="MS Exchange Server version
5.5.2653.12">
<TITLE>RE: [pooma-dev] AltView* classes</TITLE>
</HEAD>
<BODY>

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 havingalink-time cost, both in space and time. I believe this refactoringwasdone 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. Unlessthese are
hurting something else, I would tend to leave them in. 
Does the CodeSourcery CVS repository have completeCVS
history? (i.e. did we copy the repository, or just the headversion?) This
should have been easy to determine via CVS. 
        <FONTSIZE=2>Jim

<FONTSIZE=2>-------------------------------------------------------
-----------------
 James A.
Crotinger                           email:    address@hidden
 NumeriX,
LLC                                 phone:  (505)424-
4477 x104
 2960 Rodeo Park Dr. W.
 Santa Fe, NM 87505
 &nbsp;


&gt; -----Original Message-----
 &gt; From: Richard Guenther [<A
HREF="mailto:address@hidden
tuebingen.de">mailto:address@hidden</A>]
 &gt; Sent: Tuesday, March 02, 2004 3:29 AM
 &gt; To: address@hidden
 &gt; Cc: Jeffrey D. Oldham
 > Subject: Re: [pooma-dev] AltView*classes
 &gt; 
 &gt; On Tue, 2 Mar 2004, Richard Guenther
wrote:
 &gt; 
 &gt; &gt; On Fri, 27 Feb 2004, Richard Guenther
wrote:
 &gt; &gt;
 &gt; &gt; &gt; Hi!
 &gt; &gt; &gt;
 > > > Does anyone remember the exactreason
for the existance of the
 &gt; AltView*
 > > > classes in Array.h andField.h? 
Removing them and fixing their usage
 &gt; to
 > > > use View* seems to work (full testwith
gcc 3.3 and 3.4 in progress).
 &gt; &gt;
 &gt; &gt; Testing completed with no new
failures.&nbsp; Would it be ok to remove those
 &gt; &gt; AltView* classes?
 &gt; 
 &gt; Which would be...
 &gt; 
 &gt; Ok?
 &gt; 
 &gt; Richard.
 &gt; 
 &gt; 
 &gt; 2004Mar02&nbsp; Richard Guenther
&lt;address@hidden&gt;
 &gt; 
 &gt; &nbsp;&nbsp;&nbsp;&nbsp;&nbsp; *
src/Array/Array.h: remove all traces of AltView0 class.
 >      src/Field/Field.h:
remove all traces of AltView1 and
 &gt; &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;
AltView1Implementation classes.
 &gt; 
 > ===== Array/Array.h 1.14 vs edited===== > --- 1.14/r2/src/Array/Array.h Tue Dec 1615:19:57
2003
 &gt; +++
edited/Array/Array.h&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; Tue Mar&nbsp; 2
11:24:14 2004
 &gt; @@ -649,8 +649,6 @@
 >  // an existing engine cannot be anykind of
slice domain.
 &gt;&nbsp; // Also, bounds checking would make no
sense because it would
 &gt;&nbsp; // reduce to contains(a.domain(),
a.domain());
 &gt; -//
 > -// Any changes to this class should also bemade
to AltView0.
 &gt; 
 &gt;&nbsp; template&lt;int Dim, class T, class
EngineTag&gt;
 &gt;&nbsp; struct View0&lt;Array&lt;Dim, T,
EngineTag&gt; &gt;
 &gt; @@ -695,41 +693,6 @@
 &gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; }
 &gt;&nbsp; };
 &gt; 
 > -// AltView0 avoids an instantiation problemthat
arises when two
 &gt; -// classes use each other.&nbsp; This class's
definition should be exactly
 &gt; -// the same as View0 except omitting member
functions.
 &gt; -//
 &gt; -// Do NOT explicitly instantiate this
class.
 &gt; -
 &gt; -template&lt;class ArrayTag&gt;
 &gt; -struct AltView0;
 &gt; -
 &gt; -template&lt;int Dim, class T, class
EngineTag&gt;
 &gt; -struct AltView0&lt;Array&lt;Dim, T,
EngineTag&gt; &gt;
 &gt; -{
 &gt; -&nbsp; // Convenience typedef for the thing
we're taking a view of.
 &gt; -
 &gt; -&nbsp; typedef Array&lt;Dim, T, EngineTag&gt;
Subject_t;
 &gt; -
 &gt; -&nbsp; // Deduce domains for the output
type.
 &gt; -&nbsp; // At some point, we need to fix
NewDomain1; until then, use
 &gt; -&nbsp; // the temporary version from
Array.h.
 &gt; -
 &gt; -&nbsp; typedef typename Subject_t::Engine_t
Engine_t;
 &gt; -&nbsp; typedef typename Subject_t::Domain_t
Domain_t;
 &gt; -
 > -  // Deduce the template parameters forthe
output type.
 &gt; -
 > -  typedef typenameNewEngine<Engine_t,
Domain_t&gt;::Type_t NewEngine_t;
 > -  enum { newDim =NewEngine_t::dimensions
};
 &gt; -&nbsp; typedef typename NewEngine_t::Tag_t
NewEngineTag_t;
 &gt; -
 &gt; -&nbsp; // The output types.
 &gt; -
 &gt; -&nbsp; typedef Array&lt;newDim, T,
NewEngineTag_t&gt; Type_t;
 &gt; -&nbsp; typedef Type_t ReadType_t;
 &gt; -};
 &gt; -
 &gt;&nbsp; template&lt;int Dim, class T, class
EngineTag&gt;
 &gt;&nbsp; struct View1&lt;Array&lt;Dim, T,
EngineTag&gt;, int&gt;
 &gt;&nbsp; {
 &gt; @@ -1864,7 +1827,7 @@
 >    /// A zero-argument versionof
operator(), which takes a view of
 &gt;&nbsp;&nbsp;&nbsp; /// array's domain, is also
supplied.
 &gt;&nbsp;&nbsp;&nbsp; //@{
 &gt; -&nbsp; typename
AltView0&lt;This_t&gt;::ReadType_t
 &gt; +&nbsp; typename
View0&lt;This_t&gt;::ReadType_t
 &gt;&nbsp;&nbsp;&nbsp; read() const
 &gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; {
 >       typedef
View0&lt;This_t&gt; Ret_t;
 &gt; @@ -1934,7 +1897,7 @@
 >       return
Ret_t::makeRead(*this, s1, s2, s3, s4, s5, s6, s7);
 &gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; }
 &gt; 
 &gt; -&nbsp; typename
AltView0&lt;This_t&gt;::Type_t
 &gt; +&nbsp; typename
View0&lt;This_t&gt;::Type_t
 &gt;&nbsp;&nbsp;&nbsp; operator()() const
 &gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; {
 >       typedef
View0&lt;This_t&gt; Ret_t;
 > ===== Field/Field.h 1.14 vs edited===== > --- 1.14/r2/src/Field/Field.h Tue Dec 1615:20:00
2003
 &gt; +++
edited/Field/Field.h&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; Tue Mar&nbsp; 2
11:25:22 2004
 &gt; @@ -269,9 +269,6 @@
 &gt;&nbsp;&nbsp; * View1Implementation&lt;Field, D,
SV&gt; specialization for indexing a field
 &gt;&nbsp;&nbsp; * with a single domain. There is a
single-valued version (SV == true)
 >   * and a multi-valued version (SV==
false).
 &gt; - *
 &gt; - * Any changes to View1Implementation should
also be made to
 &gt; - * AltView1Implementation.
 &gt;&nbsp;&nbsp; */
 &gt; 
 >  // Single-valued version. Handlesscalars
and Locs.
 &gt; @@ -508,62 +505,7 @@
 &gt; 
 &gt; 
 &gt;&nbsp; /**
 &gt; - * AltView1Implementation avoids an
instantiation problem that arises
 &gt; when two
 &gt; - * classes use each other.&nbsp; This class's
definition should be exactly
 &gt; - * the same as View1Implementation except
omitting member functions.
 &gt; - *
 &gt; - * Do NOT explicitly instantiate this
class.
 &gt; - */
 &gt; -
 &gt; -// Single-valued version. Handles scalars and
Locs.
 &gt; -
 &gt; -template&lt;class Subject, class Domain, bool
SV&gt;
 &gt; -struct AltView1Implementation;
 &gt; -
 &gt; -template&lt;class Mesh, class T, class
EngineTag, class Domain&gt;
 > -structAltView1Implementation<Field<Mesh,
T, EngineTag&gt;, Domain, true&gt;
 &gt; -{
 &gt; -&nbsp; // Convenience typedef for the thing
we're taking a view of.
 &gt; -
 > -  typedef Field<Mesh, T,EngineTag>
Subject_t;
 &gt; -
 &gt; -&nbsp; // The return types are pretty simple
here.
 &gt; -
 &gt; -&nbsp; typedef typename Subject_t::Element_t
ReadType_t;
 > -  typedef typenameSubject_t::ElementRef_t
Type_t;
 &gt; -};
 &gt; -
 &gt; -template&lt;class Mesh, class T, class
EngineTag, class Domain&gt;
 > -structAltView1Implementation<Field<Mesh,
T, EngineTag&gt;, Domain, false&gt;
 &gt; -{
 &gt; -&nbsp; // Convenience typedef for the thing
we're taking a view of.
 &gt; -
 > -  typedef Field<Mesh, T,EngineTag>
Subject_t;
 &gt; -
 &gt; -&nbsp; // Deduce domains for the output
type.
 &gt; -
 &gt; -&nbsp; typedef typename Subject_t::Engine_t
Engine_t;
 > -  typedef typenameNewEngine<Engine_t,
Domain&gt;::Type_t NewEngine_t;
 > -  typedef typenameNewEngine_t::Element_t
NewT_t;
 &gt; -&nbsp; typedef typename NewEngine_t::Tag_t
NewEngineTag_t;
 &gt; -
 &gt; -&nbsp; // Deduce the new Mesh.
 &gt; -
 &gt; -&nbsp; typedef typename
 &gt; -&nbsp;&nbsp;&nbsp;
NewMeshTag<NewEngine_t::dimensions, Mesh,Domain>::Type_t
 &gt; -&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;
NewMeshTag_t;
 &gt; -
 &gt; -&nbsp; // The output types.
 &gt; -
 &gt; -&nbsp; typedef Field&lt;NewMeshTag_t, NewT_t,
NewEngineTag_t&gt; ReadType_t;
 &gt; -&nbsp; typedef Field&lt;NewMeshTag_t, NewT_t,
NewEngineTag_t&gt; Type_t;
 &gt; -};
 &gt; -
 &gt; -
 &gt; -/**
 &gt;&nbsp;&nbsp; * View1&lt;Field, S1&gt;
specialization for indexing a field with a single
 &gt; domain.
 &gt; - *
 > - * Any changes to View1 should also be madeto
AltView1.
 &gt;&nbsp;&nbsp; */
 &gt; 
 &gt;&nbsp; template&lt;class Mesh, class T, class
EngineTag, class Sub1&gt;
 &gt; @@ -613,8 +555,6 @@
 &gt; 
 &gt;&nbsp; /**
 &gt;&nbsp;&nbsp; * View1&lt;Field, int&gt;
specialization for indexing a field with an int.
 &gt; - *
 > - * Any changes to View1 should also be madeto
AltView1.
 &gt;&nbsp;&nbsp; */
 &gt; 
 &gt;&nbsp; template&lt;class Mesh, class T, class
EngineTag&gt;
 &gt; @@ -658,71 +598,6 @@
 &gt; 
 &gt; 
 &gt;&nbsp; /**
 > - * AltView1 avoids an instantiation problemthat
arises when two
 &gt; - * classes use each other.&nbsp; This class's
definition should be exactly
 &gt; - * the same as View1 except omitting member
functions.
 &gt; - *
 &gt; - * Do NOT explicitly instantiate this
class.
 &gt; - */
 &gt; -
 &gt; -template&lt;class FieldTag, class
DomainTag&gt;
 &gt; -struct AltView1;
 &gt; -
 &gt; -template&lt;class Mesh, class T, class
EngineTag, class Sub1&gt;
 &gt; -struct AltView1&lt;Field&lt;Mesh, T,
EngineTag&gt;, Sub1&gt;
 &gt; -{
 &gt; -&nbsp; // Convenience typedef for the thing
we're taking a view of.
 &gt; -
 > -  typedef Field<Mesh, T,EngineTag>
Subject_t;
 &gt; -
 &gt; -&nbsp; // Deduce domains for the output
type.
 &gt; -&nbsp; // At some point, we need to fix
NewDomain1; until then, use
 &gt; -&nbsp; // the temporary version from
NewDomain.h.
 &gt; -
 &gt; -&nbsp; typedef typename Subject_t::Domain_t
Domain_t;
 > -  typedefTemporaryNewDomain1<Domain_t,
Sub1&gt; NewDomain_t;
 > -  typedef typenameNewDomain_t::SliceType_t
SDomain_t;
 &gt; -
 &gt; -&nbsp; // Deduce appropriate version of
implementation to dispatch to.
 &gt; -
 &gt; -&nbsp; enum { sv =
DomainTraits&lt;SDomain_t&gt;::singleValued };
 &gt; -&nbsp; typedef
AltView1Implementation<Subject_t, SDomain_t, sv>Dispatch_t;
 &gt; -
 > -  // The optimized domaincombiner.
 &gt; -
 > -  typedefCombineDomainOpt<NewDomain_t,
sv&gt; Combine_t;
 &gt; -
 &gt; -&nbsp; // The return types.
 &gt; -
 > -  typedef typenameDispatch_t::ReadType_t
ReadType_t;
 &gt; -&nbsp; typedef typename Dispatch_t::Type_t
Type_t;
 &gt; -};
 &gt; -
 &gt; -
 &gt; -/**
 > - * AltView1 avoids an instantiation problemthat
arises when two
 &gt; - * classes use each other.&nbsp; This class's
definition should be exactly
 &gt; - * the same as View1 except omitting member
functions.
 &gt; - *
 &gt; - * Do NOT explicitly instantiate this
class.
 &gt; - */
 &gt; -
 &gt; -template&lt;class Mesh, class T, class
EngineTag&gt;
 &gt; -struct AltView1&lt;Field&lt;Mesh, T,
EngineTag&gt;, int&gt;
 &gt; -{
 &gt; -&nbsp; // Convenience typedef for the thing
we're taking a view of.
 &gt; -
 > -  typedef Field<Mesh, T,EngineTag>
Subject_t;
 &gt; -
 &gt; -&nbsp; // The return types.
 &gt; -
 &gt; -&nbsp; typedef typename Subject_t::Element_t
ReadType_t;
 > -  typedef typenameSubject_t::ElementRef_t
Type_t;
 &gt; -
 &gt; -};
 &gt; -
 &gt; -
 &gt; -/**
 &gt;&nbsp;&nbsp; * View2&lt;Field, S1, S2&gt;
specialization for indexing a field with two
 &gt;&nbsp;&nbsp; * domains.
 &gt;&nbsp;&nbsp; */
 &gt; @@ -1455,14 +1330,14 @@
 &gt;&nbsp;&nbsp;&nbsp; /// point fields the viewing
domain is an actual centering domain.
 &gt;&nbsp;&nbsp;&nbsp; //@{
 &gt; 
 &gt; -&nbsp; inline typename AltView1&lt;This_t,
Domain_t&gt;::ReadType_t
 &gt; +&nbsp; inline typename View1&lt;This_t,
Domain_t&gt;::ReadType_t
 &gt;&nbsp;&nbsp;&nbsp; read() const
 &gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; {
 >       typedef
View1&lt;This_t, Domain_t&gt; Ret_t;
 >       return
Ret_t::makeRead(*this, physicalDomain());
 &gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; }
 &gt; 
 &gt; -&nbsp; inline typename AltView1&lt;This_t,
Domain_t&gt;::ReadType_t
 &gt; +&nbsp; inline typename View1&lt;This_t,
Domain_t&gt;::ReadType_t
 &gt;&nbsp;&nbsp;&nbsp; readAll() const
 &gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; {
 >       typedef
View1&lt;This_t, Domain_t&gt; Ret_t;
 &gt; @@ -1470,7 +1345,7 @@
 &gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; }
 &gt; 
 &gt;&nbsp;&nbsp;&nbsp; template&lt;class
Sub1&gt;
 &gt; -&nbsp; inline typename AltView1&lt;This_t,
Sub1&gt;::ReadType_t
 &gt; +&nbsp; inline typename View1&lt;This_t,
Sub1&gt;::ReadType_t
 &gt;&nbsp;&nbsp;&nbsp; read(const Sub1 &amp;s1)
const
 &gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; {
 >       typedef
View1&lt;This_t, Sub1&gt; Ret_t;
 &gt; @@ -1493,14 +1368,14 @@
 >       return
Ret_t::makeRead(*this, s1, s2, s3);
 &gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; }
 &gt; 
 &gt; -&nbsp; inline typename AltView1&lt;This_t,
Domain_t&gt;::Type_t
 &gt; +&nbsp; inline typename View1&lt;This_t,
Domain_t&gt;::Type_t
 &gt;&nbsp;&nbsp;&nbsp; operator()() const
 &gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; {
 >       typedef
View1&lt;This_t, Domain_t&gt; Ret_t;
 >       return
Ret_t::make(*this, physicalDomain());
 &gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; }
 &gt; 
 &gt; -&nbsp; inline typename AltView1&lt;This_t,
Domain_t&gt;::Type_t
 &gt; +&nbsp; inline typename View1&lt;This_t,
Domain_t&gt;::Type_t
 &gt;&nbsp;&nbsp;&nbsp; all() const
 &gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; {
 >       typedef
View1&lt;This_t, Domain_t&gt; Ret_t;
 &gt; @@ -1508,7 +1383,7 @@
 &gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; }
 &gt; 
 &gt;&nbsp;&nbsp;&nbsp; template&lt;class
Sub1&gt;
 &gt; -&nbsp; inline typename AltView1&lt;This_t,
Sub1&gt;::Type_t
 &gt; +&nbsp; inline typename View1&lt;This_t,
Sub1&gt;::Type_t
 >    operator()(const Sub1&s1)
const
 &gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; {
 >       typedef
View1&lt;This_t, Sub1&gt; Ret_t;


</BODY>
</HTML>
<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 3.2//EN">
<HTML>
<HEAD>
<META HTTP-EQUIV="Content-Type" CONTENT="text/html; charset=us-ascii">
<META NAME="Generator" CONTENT="MS Exchange Server version5.5.2653.12">
<TITLE>RE: [pooma-dev] AltView* classes</TITLE>
</HEAD>
<BODY>
OK - cool. Thanks for chipping in Dave. You'reneurons are obviously less rusty than mine. :)


&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; Jim

<FONTSIZE=2>------------------------------------------------------------------------ James A.Crotinger                           email:    address@hidden NumeriX,LLC                                phone:  (505) 424-4477 x104
 2960 Rodeo Park Dr. W.
 Santa Fe, NM 87505
 &nbsp;


&gt; -----Original Message-----
 > From: Dave Nystrom [<AHREF="mailto:address@hidden";>mailto:address@hidden</A>]
 &gt; Sent: Tuesday, March 02, 2004 10:00 AM
 &gt; To: James Crotinger
 > Cc: 'Richard Guenther';'address@hidden'; 'Jeffrey D. > Oldham'; John Hall; Dave Nystrom; DaveNystrom
 &gt; Subject: RE: [pooma-dev] AltView* classes
 &gt; 
 > These AltView classes were put in by JeffreyOldham to fix a problem that
 &gt; I
 > was having trying to explicitly instantiate allthe Pooma classes and > functions that Blanca code was using. Basically, the way the Pooma 2
 &gt; Field
 > class was originally written, we were gettingone of these "sv" enums > instantiated for every unique expression whichwe had in our source code > base.  I was unable to explicitlyinstantiate these "sv" enums the way the > code was originally written because of someissue which Arch Robison of
 &gt; Kuck
 > and Associates (now Intel) identified -basically the code in it's
 &gt; original
 > form violated some corner case of the C++standard according to Arch. > Jeffrey fixed this problem by adding theAltView class and then I was able
 &gt; to
 > perform the explicit instantiations that Ineeded.
 &gt; 
 > However, I believe Jeffrey or someone else hasgone through the code and > replaced the "sv" enums with someother construct which does not have the > explicit instantiation difficulties that the"sv" enums had.  So, I think > that it is probably okay to remove the AltViewclasses and that this would > clean up the code considerably.  I mayrevisit all this explicit > instantiation stuff sometime in the not todistant future on a hobby basis > (weird hobby, huh) and if I encounterdifficulties being able to
 &gt; explicitly
 > instantiate anything, I could then raise it asan issue at that time.
 &gt; 
 > In summary, the AltView classes were added byJeffrey to fix an explicit > instantiation problem that I think no longerexists and I am in favor of
 &gt; now
 > removing them in the interest of cleaning upthe code as much as possible.
 &gt; 
 &gt; --
 > Dave Nystrom                email: address@hidden > LANL X-2                    phone:505-667-7913     fax: 505-665-2227
 &gt; 
 &gt; James Crotinger writes:
 &gt;&nbsp; &gt; Hi Richard,
 &gt;&nbsp; &gt;
 >  > This tickled a neuron the other day,but I couldn't recall the details.
 &gt;&nbsp; &gt;
 >  > The AltView classes were put in toreduce link times and sizes for
 &gt; large
 >  > codes. The classes that have enumsend up having a link-time cost, both
 &gt; in
 >  > space and time. I believe thisrefactoring was done to reduce the >  > cardinality of classes having the"sv" enum. My recollection is that
 &gt; this,
 >  > and other similar"optimizations", had a pretty substantial impact on >  > link-time for Blanca. Unless theseare hurting something else, I would
 &gt; tend
 &gt;&nbsp; &gt; to leave them in.
 &gt;&nbsp; &gt;
 >  > Does the CodeSourcery CVS repositoryhave complete CVS history? (i.e.
 &gt; did we
 >  > copy the repository, or just thehead version?) This should have been
 &gt; easy
 &gt;&nbsp; &gt; to determine via CVS.
 &gt;&nbsp; &gt;
 &gt;&nbsp; &gt; &nbsp;&nbsp; Jim
 &gt;&nbsp; &gt;
 >  >-----------------------------------------------------------------------
 &gt; -
 >  > James A.Crotinger                           email:
 &gt; address@hidden
 >  > NumeriX,LLC                                phone:  (505) 424-4477
 &gt; x104
 &gt;&nbsp; &gt; 2960 Rodeo Park Dr. W.
 &gt;&nbsp; &gt; Santa Fe, NM 87505
 &gt;&nbsp; &gt;
 &gt;&nbsp; &gt;
 >  > > -----OriginalMessage----- >  > > From: Richard Guenther [<AHREF="mailto:address@hidden";>mailto:address@hidden</A>] >  > > Sent: Tuesday, March 02, 20043:29 AM >  > > To:address@hidden
 &gt;&nbsp; &gt; &gt; Cc: Jeffrey D. Oldham
 >  > > Subject: Re: [pooma-dev]AltView* classes
 &gt;&nbsp; &gt; &gt;
 >  > > On Tue, 2 Mar 2004, RichardGuenther wrote:
 &gt;&nbsp; &gt; &gt;
 >  > > > On Fri, 27 Feb 2004,Richard Guenther wrote:
 &gt;&nbsp; &gt; &gt; &gt;
 &gt;&nbsp; &gt; &gt; &gt; &gt; Hi!
 &gt;&nbsp; &gt; &gt; &gt; &gt;
 >  > > > > Does anyone rememberthe exact reason for the existance of the
 &gt;&nbsp; &gt; &gt; AltView*
 >  > > > > classes in Array.hand Field.h?  Removing them and fixing their
 &gt; usage
 &gt;&nbsp; &gt; &gt; to
 >  > > > > use View* seems towork (full test with gcc 3.3 and 3.4 in
 &gt; progress).
 &gt;&nbsp; &gt; &gt; &gt;
 >  > > > Testing completed with nonew failures.  Would it be ok to remove
 &gt; those
 &gt;&nbsp; &gt; &gt; &gt; AltView* classes?
 &gt;&nbsp; &gt; &gt;
 &gt;&nbsp; &gt; &gt; Which would be...
 &gt;&nbsp; &gt; &gt;
 &gt;&nbsp; &gt; &gt; Ok?
 &gt;&nbsp; &gt; &gt;
 &gt;&nbsp; &gt; &gt; Richard.
 &gt;&nbsp; &gt; &gt;
 &gt;&nbsp; &gt; &gt;
 >  > > 2004Mar02  RichardGuenther <address@hidden>
 &gt;&nbsp; &gt; &gt;
 >  > >  * src/Array/Array.h:remove all traces of AltView0 class. >  > >  src/Field/Field.h: removeall traces of AltView1 and >  > >  AltView1Implementationclasses.
 &gt;&nbsp; &gt; &gt;
 >  > > ===== Array/Array.h 1.14 vsedited ===== >  > > ---1.14/r2/src/Array/Array.h    Tue Dec 16 15:19:572003 >  > > +++ edited/Array/Array.h TueMar  2 11:24:14 2004
 &gt;&nbsp; &gt; &gt; @@ -649,8 +649,6 @@
 >  > >  // an existing enginecannot be any kind of slice domain. >  > >  // Also, bounds checkingwould make no sense because it would >  > >  // reduce tocontains(a.domain(), a.domain());
 &gt;&nbsp; &gt; &gt; -//
 >  > > -// Any changes to this classshould also be made to AltView0.
 &gt;&nbsp; &gt; &gt;
 >  > >  template<int Dim,class T, class EngineTag> >  > >  structView0<Array<Dim, T, EngineTag> >
 &gt;&nbsp; &gt; &gt; @@ -695,41 +693,6 @@
 >  > >     }
 &gt;&nbsp; &gt; &gt;&nbsp; };
 &gt;&nbsp; &gt; &gt;
 >  > > -// AltView0 avoids aninstantiation problem that arises when two >  > > -// classes use eachother.  This class's definition should be
 &gt; exactly
 >  > > -// the same as View0 exceptomitting member functions.
 &gt;&nbsp; &gt; &gt; -//
 >  > > -// Do NOT explicitlyinstantiate this class.
 &gt;&nbsp; &gt; &gt; -
 >  > > -template<classArrayTag>
 &gt;&nbsp; &gt; &gt; -struct AltView0;
 &gt;&nbsp; &gt; &gt; -
 >  > > -template<int Dim, class T,class EngineTag> >  > > -structAltView0<Array<Dim, T, EngineTag> >
 &gt;&nbsp; &gt; &gt; -{
 >  > > -  // Convenience typedeffor the thing we're taking a view of.
 &gt;&nbsp; &gt; &gt; -
 >  > > -  typedef Array<Dim,T, EngineTag> Subject_t;
 &gt;&nbsp; &gt; &gt; -
 >  > > -  // Deduce domains forthe output type. >  > > -  // At some point, weneed to fix NewDomain1; until then, use >  > > -  // the temporaryversion from Array.h.
 &gt;&nbsp; &gt; &gt; -
 >  > > -  typedef typenameSubject_t::Engine_t Engine_t; >  > > -  typedef typenameSubject_t::Domain_t Domain_t;
 &gt;&nbsp; &gt; &gt; -
 >  > > -  // Deduce the templateparameters for the output type.
 &gt;&nbsp; &gt; &gt; -
 >  > > -  typedef typenameNewEngine<Engine_t, Domain_t>::Type_t
 &gt; NewEngine_t;
 >  > > -  enum { newDim =NewEngine_t::dimensions }; >  > > -  typedef typenameNewEngine_t::Tag_t NewEngineTag_t;
 &gt;&nbsp; &gt; &gt; -
 >  > > -  // The outputtypes.
 &gt;&nbsp; &gt; &gt; -
 >  > > -  typedefArray<newDim, T, NewEngineTag_t> Type_t; >  > > -  typedef Type_tReadType_t;
 &gt;&nbsp; &gt; &gt; -};
 &gt;&nbsp; &gt; &gt; -
 >  > >  template<int Dim,class T, class EngineTag> >  > >  structView1<Array<Dim, T, EngineTag>, int>
 &gt;&nbsp; &gt; &gt;&nbsp; {
 &gt;&nbsp; &gt; &gt; @@ -1864,7 +1827,7 @@
 >  > >    /// Azero-argument version of operator(), which takes a view of >  > >    /// array'sdomain, is also supplied.
 &gt;&nbsp; &gt; &gt;&nbsp;&nbsp;&nbsp; //@{
 >  > > -  typenameAltView0<This_t>::ReadType_t >  > > +  typenameView0<This_t>::ReadType_t >  > >    read()const >  > >     { >  >>        typedefView0<This_t> Ret_t;
 &gt;&nbsp; &gt; &gt; @@ -1934,7 +1897,7 @@
 >  >>        returnRet_t::makeRead(*this, s1, s2, s3, s4, s5, s6, s7); >  > >     }
 &gt;&nbsp; &gt; &gt;
 >  > > -  typenameAltView0<This_t>::Type_t >  > > +  typenameView0<This_t>::Type_t >  > >    operator()()const >  > >     { >  >>        typedefView0<This_t> Ret_t; >  > > ===== Field/Field.h 1.14 vsedited ===== >  > > ---1.14/r2/src/Field/Field.h    Tue Dec 16 15:20:002003 >  > > +++ edited/Field/Field.h TueMar  2 11:25:22 2004
 &gt;&nbsp; &gt; &gt; @@ -269,9 +269,6 @@
 >  > >   *View1Implementation<Field, D, SV> specialization for indexinga
 &gt; field
 >  > >   * with a singledomain. There is a single-valued version (SV ==
 &gt; true)
 >  > >   * and amulti-valued version (SV == false).
 &gt;&nbsp; &gt; &gt; - *
 >  > > - * Any changes toView1Implementation should also be made to >  > > - *AltView1Implementation.
 &gt;&nbsp; &gt; &gt;&nbsp;&nbsp; */
 &gt;&nbsp; &gt; &gt;
 >  > >  // Single-valued version.Handles scalars and Locs.
 &gt;&nbsp; &gt; &gt; @@ -508,62 +505,7 @@
 &gt;&nbsp; &gt; &gt;
 &gt;&nbsp; &gt; &gt;
 &gt;&nbsp; &gt; &gt;&nbsp; /**
 >  > > - * AltView1Implementationavoids an instantiation problem that
 &gt; arises
 &gt;&nbsp; &gt; &gt; when two
 >  > > - * classes use eachother.  This class's definition should be
 &gt; exactly
 >  > > - * the same asView1Implementation except omitting member functions.
 &gt;&nbsp; &gt; &gt; - *
 >  > > - * Do NOT explicitlyinstantiate this class.
 &gt;&nbsp; &gt; &gt; - */
 &gt;&nbsp; &gt; &gt; -
 >  > > -// Single-valued version.Handles scalars and Locs.
 &gt;&nbsp; &gt; &gt; -
 >  > > -template<class Subject,class Domain, bool SV> >  > > -structAltView1Implementation;
 &gt;&nbsp; &gt; &gt; -
 >  > > -template<class Mesh, classT, class EngineTag, class Domain> >  > > -structAltView1Implementation<Field<Mesh, T, EngineTag>,Domain,
 &gt; true&gt;
 &gt;&nbsp; &gt; &gt; -{
 >  > > -  // Convenience typedeffor the thing we're taking a view of.
 &gt;&nbsp; &gt; &gt; -
 >  > > -  typedef Field<Mesh,T, EngineTag> Subject_t;
 &gt;&nbsp; &gt; &gt; -
 >  > > -  // The return types arepretty simple here.
 &gt;&nbsp; &gt; &gt; -
 >  > > -  typedef typenameSubject_t::Element_t ReadType_t; >  > > -  typedef typenameSubject_t::ElementRef_t Type_t;
 &gt;&nbsp; &gt; &gt; -};
 &gt;&nbsp; &gt; &gt; -
 >  > > -template<class Mesh, classT, class EngineTag, class Domain> >  > > -structAltView1Implementation<Field<Mesh, T, EngineTag>,Domain,
 &gt; false&gt;
 &gt;&nbsp; &gt; &gt; -{
 >  > > -  // Convenience typedeffor the thing we're taking a view of.
 &gt;&nbsp; &gt; &gt; -
 >  > > -  typedef Field<Mesh,T, EngineTag> Subject_t;
 &gt;&nbsp; &gt; &gt; -
 >  > > -  // Deduce domains forthe output type.
 &gt;&nbsp; &gt; &gt; -
 >  > > -  typedef typenameSubject_t::Engine_t Engine_t; >  > > -  typedef typenameNewEngine<Engine_t, Domain>::Type_t NewEngine_t; >  > > -  typedef typenameNewEngine_t::Element_t NewT_t; >  > > -  typedef typenameNewEngine_t::Tag_t NewEngineTag_t;
 &gt;&nbsp; &gt; &gt; -
 >  > > -  // Deduce the newMesh.
 &gt;&nbsp; &gt; &gt; -
 &gt;&nbsp; &gt; &gt; -&nbsp; typedef typename
 >  > > -   NewMeshTag<NewEngine_t::dimensions, Mesh,Domain>::Type_t >  > > -     NewMeshTag_t;
 &gt;&nbsp; &gt; &gt; -
 >  > > -  // The outputtypes.
 &gt;&nbsp; &gt; &gt; -
 >  > > -  typedefField<NewMeshTag_t, NewT_t, NewEngineTag_t> ReadType_t; >  > > -  typedefField<NewMeshTag_t, NewT_t, NewEngineTag_t> Type_t;
 &gt;&nbsp; &gt; &gt; -};
 &gt;&nbsp; &gt; &gt; -
 &gt;&nbsp; &gt; &gt; -
 &gt;&nbsp; &gt; &gt; -/**
 >  > >   * View1<Field,S1> specialization for indexing a field with a
 &gt; single
 &gt;&nbsp; &gt; &gt; domain.
 &gt;&nbsp; &gt; &gt; - *
 >  > > - * Any changes to View1 shouldalso be made to AltView1.
 &gt;&nbsp; &gt; &gt;&nbsp;&nbsp; */
 &gt;&nbsp; &gt; &gt;
 >  > >  template<class Mesh,class T, class EngineTag, class Sub1>
 &gt;&nbsp; &gt; &gt; @@ -613,8 +555,6 @@
 &gt;&nbsp; &gt; &gt;
 &gt;&nbsp; &gt; &gt;&nbsp; /**
 >  > >   * View1<Field,int> specialization for indexing a field with an
 &gt; int.
 &gt;&nbsp; &gt; &gt; - *
 >  > > - * Any changes to View1 shouldalso be made to AltView1.
 &gt;&nbsp; &gt; &gt;&nbsp;&nbsp; */
 &gt;&nbsp; &gt; &gt;
 >  > >  template<class Mesh,class T, class EngineTag>
 &gt;&nbsp; &gt; &gt; @@ -658,71 +598,6 @@
 &gt;&nbsp; &gt; &gt;
 &gt;&nbsp; &gt; &gt;
 &gt;&nbsp; &gt; &gt;&nbsp; /**
 >  > > - * AltView1 avoids aninstantiation problem that arises when two >  > > - * classes use eachother.  This class's definition should be
 &gt; exactly
 >  > > - * the same as View1 exceptomitting member functions.
 &gt;&nbsp; &gt; &gt; - *
 >  > > - * Do NOT explicitlyinstantiate this class.
 &gt;&nbsp; &gt; &gt; - */
 &gt;&nbsp; &gt; &gt; -
 >  > > -template<class FieldTag,class DomainTag>
 &gt;&nbsp; &gt; &gt; -struct AltView1;
 &gt;&nbsp; &gt; &gt; -
 >  > > -template<class Mesh, classT, class EngineTag, class Sub1> >  > > -structAltView1<Field<Mesh, T, EngineTag>, Sub1>
 &gt;&nbsp; &gt; &gt; -{
 >  > > -  // Convenience typedeffor the thing we're taking a view of.
 &gt;&nbsp; &gt; &gt; -
 >  > > -  typedef Field<Mesh,T, EngineTag> Subject_t;
 &gt;&nbsp; &gt; &gt; -
 >  > > -  // Deduce domains forthe output type. >  > > -  // At some point, weneed to fix NewDomain1; until then, use >  > > -  // the temporaryversion from NewDomain.h.
 &gt;&nbsp; &gt; &gt; -
 >  > > -  typedef typenameSubject_t::Domain_t Domain_t; >  > > -  typedefTemporaryNewDomain1<Domain_t, Sub1> NewDomain_t; >  > > -  typedef typenameNewDomain_t::SliceType_t SDomain_t;
 &gt;&nbsp; &gt; &gt; -
 >  > > -  // Deduce appropriateversion of implementation to dispatch to.
 &gt;&nbsp; &gt; &gt; -
 >  > > -  enum { sv =DomainTraits<SDomain_t>::singleValued }; >  > > -  typedefAltView1Implementation<Subject_t, SDomain_t, sv>
 &gt; Dispatch_t;
 &gt;&nbsp; &gt; &gt; -
 >  > > -  // The optimized domaincombiner.
 &gt;&nbsp; &gt; &gt; -
 >  > > -  typedefCombineDomainOpt<NewDomain_t, sv> Combine_t;
 &gt;&nbsp; &gt; &gt; -
 >  > > -  // The returntypes.
 &gt;&nbsp; &gt; &gt; -
 >  > > -  typedef typenameDispatch_t::ReadType_t ReadType_t; >  > > -  typedef typenameDispatch_t::Type_t Type_t;
 &gt;&nbsp; &gt; &gt; -};
 &gt;&nbsp; &gt; &gt; -
 &gt;&nbsp; &gt; &gt; -
 &gt;&nbsp; &gt; &gt; -/**
 >  > > - * AltView1 avoids aninstantiation problem that arises when two >  > > - * classes use eachother.  This class's definition should be
 &gt; exactly
 >  > > - * the same as View1 exceptomitting member functions.
 &gt;&nbsp; &gt; &gt; - *
 >  > > - * Do NOT explicitlyinstantiate this class.
 &gt;&nbsp; &gt; &gt; - */
 &gt;&nbsp; &gt; &gt; -
 >  > > -template<class Mesh, classT, class EngineTag> >  > > -structAltView1<Field<Mesh, T, EngineTag>, int>
 &gt;&nbsp; &gt; &gt; -{
 >  > > -  // Convenience typedeffor the thing we're taking a view of.
 &gt;&nbsp; &gt; &gt; -
 >  > > -  typedef Field<Mesh,T, EngineTag> Subject_t;
 &gt;&nbsp; &gt; &gt; -
 >  > > -  // The returntypes.
 &gt;&nbsp; &gt; &gt; -
 >  > > -  typedef typenameSubject_t::Element_t ReadType_t; >  > > -  typedef typenameSubject_t::ElementRef_t Type_t;
 &gt;&nbsp; &gt; &gt; -
 &gt;&nbsp; &gt; &gt; -};
 &gt;&nbsp; &gt; &gt; -
 &gt;&nbsp; &gt; &gt; -
 &gt;&nbsp; &gt; &gt; -/**
 >  > >   * View2<Field,S1, S2> specialization for indexing a field with two
 &gt;&nbsp; &gt; &gt;&nbsp;&nbsp; * domains.
 &gt;&nbsp; &gt; &gt;&nbsp;&nbsp; */
 &gt;&nbsp; &gt; &gt; @@ -1455,14 +1330,14 @@
 >  > >    /// pointfields the viewing domain is an actual centering domain.
 &gt;&nbsp; &gt; &gt;&nbsp;&nbsp;&nbsp; //@{
 &gt;&nbsp; &gt; &gt;
 >  > > -  inline typenameAltView1<This_t, Domain_t>::ReadType_t >  > > +  inline typenameView1<This_t, Domain_t>::ReadType_t >  > >    read()const >  > >     { >  >>        typedefView1<This_t, Domain_t> Ret_t; >  >>        returnRet_t::makeRead(*this, physicalDomain()); >  > >     }
 &gt;&nbsp; &gt; &gt;
 >  > > -  inline typenameAltView1<This_t, Domain_t>::ReadType_t >  > > +  inline typenameView1<This_t, Domain_t>::ReadType_t >  > >    readAll()const >  > >     { >  >>        typedefView1<This_t, Domain_t> Ret_t;
 &gt;&nbsp; &gt; &gt; @@ -1470,7 +1345,7 @@
 >  > >     }
 &gt;&nbsp; &gt; &gt;
 >  > >   template<class Sub1> >  > > -  inline typenameAltView1<This_t, Sub1>::ReadType_t >  > > +  inline typenameView1<This_t, Sub1>::ReadType_t >  > >    read(constSub1 &s1) const >  > >     { >  >>        typedefView1<This_t, Sub1> Ret_t;
 &gt;&nbsp; &gt; &gt; @@ -1493,14 +1368,14 @@
 >  >>        returnRet_t::makeRead(*this, s1, s2, s3); >  > >     }
 &gt;&nbsp; &gt; &gt;
 >  > > -  inline typenameAltView1<This_t, Domain_t>::Type_t >  > > +  inline typenameView1<This_t, Domain_t>::Type_t >  > >    operator()()const >  > >     { >  >>        typedefView1<This_t, Domain_t> Ret_t; >  >>        returnRet_t::make(*this, physicalDomain()); >  > >     }
 &gt;&nbsp; &gt; &gt;
 >  > > -  inline typenameAltView1<This_t, Domain_t>::Type_t >  > > +  inline typenameView1<This_t, Domain_t>::Type_t >  > >    all()const >  > >     { >  >>        typedefView1<This_t, Domain_t> Ret_t;
 &gt;&nbsp; &gt; &gt; @@ -1508,7 +1383,7 @@
 >  > >     }
 &gt;&nbsp; &gt; &gt;
 >  > >   template<class Sub1> >  > > -  inline typenameAltView1<This_t, Sub1>::Type_t >  > > +  inline typenameView1<This_t, Sub1>::Type_t >  > >   operator()(const Sub1 &s1) const >  > >     { >  >>        typedefView1<This_t, Sub1> Ret_t; >  > <!DOCTYPE HTML PUBLIC"-//W3C//DTD HTML 3.2//EN">
 &gt;&nbsp; &gt; &lt;HTML&gt;
 &gt;&nbsp; &gt; &lt;HEAD&gt;
 >  > <METAHTTP-EQUIV="Content-Type" CONTENT="text/html;charset=us-ascii"> >  > <META NAME="Generator"CONTENT="MS Exchange Server version
 &gt; 5.5.2653.12&quot;&gt;
 >  > <TITLE>RE: [pooma-dev]AltView* classes</TITLE>
 &gt;&nbsp; &gt; &lt;/HEAD&gt;
 &gt;&nbsp; &gt; &lt;BODY&gt;
 &gt;&nbsp; &gt;
 >  > HiRichard,
 &gt;&nbsp; &gt; &lt;/P&gt;
 &gt;&nbsp; &gt;
 >  > Thistickled a neuron the other day, but I couldn't
 &gt; recall the details.&lt;/FONT&gt;
 &gt;&nbsp; &gt; &lt;/P&gt;
 &gt;&nbsp; &gt;
 >  > TheAltView classes were put in to reduce link times > and sizes for large codes. The classes thathave enums end up having a > link-time cost, both in space and time. Ibelieve this refactoring was > done to reduce the cardinality of classeshaving the &quot;sv&quot; enum. > My recollection is that this, and other similar&quot;optimizations&quot;, > had a pretty substantial impact on link-timefor Blanca. Unless these are > hurting something else, I would tend to leavethem in. 
 &gt;&nbsp; &gt;
 >  > Does theCodeSourcery CVS repository have complete CVS > history? (i.e. did we copy the repository, orjust the head version?) This > should have been easy to determine via CVS.
 &gt;&nbsp; &gt;
 >  >&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; Jim
 &gt;&nbsp; &gt; &lt;/P&gt;
 &gt;&nbsp; &gt;
 >  > <FONTSIZE=2>-------------------------------------------------------
 &gt; -----------------&lt;/FONT&gt;
 >  > JamesA. >Crotinger&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp >;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;& > nbsp;&nbsp;&nbsp;email:&nbsp;&nbsp;&nbsp;&nbsp;address@hidden >  > <FONTSIZE=2>NumeriX, >LLC&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp >;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;& >nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; phone:&nbsp; (505) 424-
 &gt; 4477 x104&lt;/FONT&gt;
 >  > 2960Rodeo Park Dr. W. >  > SantaFe, NM 87505 >  > <FONTSIZE=2>&nbsp;
 &gt;&nbsp; &gt; &lt;/P&gt;
 &gt;&nbsp; &gt;
 >  > &gt;-----Original Message----- >  > <FONTSIZE=2>&gt; From: Richard Guenther [<A > HREF="<AHREF="mailto:address@hidden";>mailto:address@hidden</A> > tuebingen.de"><AHREF="mailto:address@hidden";>mailto:address@hidden</A></A>] >  > <FONTSIZE=2>&gt; Sent: Tuesday, March 02, 2004 3:29AM >  > <FONTSIZE=2>&gt; To:address@hidden >  > <FONTSIZE=2>&gt; Cc: Jeffrey D. Oldham >  > <FONTSIZE=2>&gt; Subject: Re: [pooma-dev] AltView*classes >  > <FONTSIZE=2>&gt; >  > <FONTSIZE=2>&gt; On Tue, 2 Mar 2004, Richard Guenther
 &gt; wrote:&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; >  > <FONTSIZE=2>&gt; &gt; On Fri, 27 Feb 2004, RichardGuenther
 &gt; wrote:&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; &gt; >  > <FONTSIZE=2>&gt; &gt; &gt; Hi! >  > <FONTSIZE=2>&gt; &gt; &gt; >  > <FONTSIZE=2>&gt; &gt; &gt; Does anyone remember the exactreason
 &gt; for the existance of the&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; AltView* >  > <FONTSIZE=2>&gt; &gt; &gt; classes in Array.h andField.h?&nbsp; > Removing them and fixing theirusage >  > <FONTSIZE=2>&gt; to >  > <FONTSIZE=2>&gt; &gt; &gt; use View* seems to work (fulltest with > gcc 3.3 and 3.4 inprogress). >  > <FONTSIZE=2>&gt; &gt; >  > <FONTSIZE=2>&gt; &gt; Testing completed with no new > failures.&nbsp; Would it be ok to removethose >  > <FONTSIZE=2>&gt; &gt; AltView* classes? >  > <FONTSIZE=2>&gt; >  > <FONTSIZE=2>&gt; Which would be... >  > <FONTSIZE=2>&gt; >  > <FONTSIZE=2>&gt; Ok? >  > <FONTSIZE=2>&gt; >  > <FONTSIZE=2>&gt; Richard. >  > <FONTSIZE=2>&gt; >  > <FONTSIZE=2>&gt; >  > <FONTSIZE=2>&gt; 2004Mar02&nbsp; Richard Guenther >&lt;address@hidden&gt; >  > <FONTSIZE=2>&gt; >  > <FONTSIZE=2>&gt; &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;* > src/Array/Array.h: remove all traces ofAltView0 class. >  > <FONTSIZE=2>&gt; &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;src/Field/Field.h: > remove all traces of AltView1and >  > <FONTSIZE=2>&gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; > AltView1Implementationclasses. >  > <FONTSIZE=2>&gt; >  > <FONTSIZE=2>&gt; ===== Array/Array.h 1.14 vs edited===== >  > <FONTSIZE=2>&gt; --- 1.14/r2/src/Array/Array.h Tue Dec 1615:19:57
 &gt; 2003&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; +++ >edited/Array/Array.h&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; Tue Mar&nbsp; 2
 &gt; 11:24:14 2004&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; @@ -649,8 +649,6 @@ >  > <FONTSIZE=2>&gt;&nbsp; // an existing engine cannot be any kindof
 &gt; slice domain.&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt;&nbsp; // Also, bounds checking would makeno
 &gt; sense because it would&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt;&nbsp; // reduce to contains(a.domain(),
 &gt; a.domain());&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; -// >  > <FONTSIZE=2>&gt; -// Any changes to this class should also bemade
 &gt; to AltView0.&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; >  > <FONTSIZE=2>&gt;&nbsp; template&lt;int Dim, class T,class
 &gt; EngineTag&amp;gt;&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt;&nbsp; struct View0&lt;Array&lt;Dim,T,
 &gt; EngineTag&amp;gt; &amp;gt;&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; @@ -695,41 +693,6 @@ >  > <FONTSIZE=2>&gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;} >  > <FONTSIZE=2>&gt;&nbsp; }; >  > <FONTSIZE=2>&gt; >  > <FONTSIZE=2>&gt; -// AltView0 avoids an instantiation problemthat
 &gt; arises when two&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; -// classes use each other.&nbsp; Thisclass's
 &gt; definition should be exactly&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; -// the same as View0 except omittingmember
 &gt; functions.&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; -// >  > <FONTSIZE=2>&gt; -// Do NOT explicitly instantiate this
 &gt; class.&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; - >  > <FONTSIZE=2>&gt; -template&lt;classArrayTag&gt; >  > <FONTSIZE=2>&gt; -struct AltView0; >  > <FONTSIZE=2>&gt; - >  > <FONTSIZE=2>&gt; -template&lt;int Dim, class T, class
 &gt; EngineTag&amp;gt;&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; -struct AltView0&lt;Array&lt;Dim,T,
 &gt; EngineTag&amp;gt; &amp;gt;&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; -{ >  > <FONTSIZE=2>&gt; -&nbsp; // Convenience typedef for thething
 &gt; we're taking a view of.&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; - >  > <FONTSIZE=2>&gt; -&nbsp; typedef Array&lt;Dim, T,EngineTag&gt;
 &gt; Subject_t;&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; - >  > <FONTSIZE=2>&gt; -&nbsp; // Deduce domains for theoutput
 &gt; type.&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; -&nbsp; // At some point, we need tofix
 &gt; NewDomain1; until then, use&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; -&nbsp; // the temporary version from
 &gt; Array.h.&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; - >  > <FONTSIZE=2>&gt; -&nbsp; typedef typenameSubject_t::Engine_t
 &gt; Engine_t;&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; -&nbsp; typedef typenameSubject_t::Domain_t
 &gt; Domain_t;&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; - >  > <FONTSIZE=2>&gt; -&nbsp; // Deduce the template parameters forthe
 &gt; output type.&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; - >  > <FONTSIZE=2>&gt; -&nbsp; typedef typenameNewEngine&lt;Engine_t, > Domain_t&gt;::Type_tNewEngine_t; >  > <FONTSIZE=2>&gt; -&nbsp; enum { newDim =NewEngine_t::dimensions
 &gt; };&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; -&nbsp; typedef typenameNewEngine_t::Tag_t
 &gt; NewEngineTag_t;&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; - >  > <FONTSIZE=2>&gt; -&nbsp; // The outputtypes. >  > <FONTSIZE=2>&gt; - >  > <FONTSIZE=2>&gt; -&nbsp; typedef Array&lt;newDim, T, > NewEngineTag_t&gt;Type_t; >  > <FONTSIZE=2>&gt; -&nbsp; typedef Type_tReadType_t; >  > <FONTSIZE=2>&gt; -}; >  > <FONTSIZE=2>&gt; - >  > <FONTSIZE=2>&gt;&nbsp; template&lt;int Dim, class T,class
 &gt; EngineTag&amp;gt;&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt;&nbsp; struct View1&lt;Array&lt;Dim,T, > EngineTag&gt;,int&gt; >  > <FONTSIZE=2>&gt;&nbsp; { >  > <FONTSIZE=2>&gt; @@ -1864,7 +1827,7 @@ >  > <FONTSIZE=2>&gt;&nbsp;&nbsp;&nbsp; /// A zero-argumentversion of > operator(), which takes a viewof >  > <FONTSIZE=2>&gt;&nbsp;&nbsp;&nbsp; /// array's domain,is also
 &gt; supplied.&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt;&nbsp;&nbsp;&nbsp;//@{ >  > <FONTSIZE=2>&gt; -&nbsp; typename >AltView0&lt;This_t&gt;::ReadType_t >  > <FONTSIZE=2>&gt; +&nbsp; typename >View0&lt;This_t&gt;::ReadType_t >  > <FONTSIZE=2>&gt;&nbsp;&nbsp;&nbsp; read()const >  > <FONTSIZE=2>&gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;{ >  > <FONTSIZE=2>&gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; typedef > View0&lt;This_t&gt;Ret_t; >  > <FONTSIZE=2>&gt; @@ -1934,7 +1897,7 @@ >  > <FONTSIZE=2>&gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; return > Ret_t::makeRead(*this, s1, s2, s3, s4, s5, s6,s7); >  > <FONTSIZE=2>&gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;} >  > <FONTSIZE=2>&gt; >  > <FONTSIZE=2>&gt; -&nbsp; typename >AltView0&lt;This_t&gt;::Type_t >  > <FONTSIZE=2>&gt; +&nbsp; typename >View0&lt;This_t&gt;::Type_t >  > <FONTSIZE=2>&gt;&nbsp;&nbsp;&nbsp; operator()()const >  > <FONTSIZE=2>&gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;{ >  > <FONTSIZE=2>&gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; typedef > View0&lt;This_t&gt;Ret_t; >  > <FONTSIZE=2>&gt; ===== Field/Field.h 1.14 vs edited===== >  > <FONTSIZE=2>&gt; --- 1.14/r2/src/Field/Field.h Tue Dec 1615:20:00
 &gt; 2003&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; +++ >edited/Field/Field.h&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; Tue Mar&nbsp; 2
 &gt; 11:25:22 2004&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; @@ -269,9 +269,6 @@ >  > <FONTSIZE=2>&gt;&nbsp;&nbsp; *View1Implementation&lt;Field, D, > SV&gt; specialization for indexing afield >  > <FONTSIZE=2>&gt;&nbsp;&nbsp; * with a single domain. Thereis a > single-valued version (SV ==true) >  > <FONTSIZE=2>&gt;&nbsp;&nbsp; * and a multi-valued version(SV ==
 &gt; false).&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; - * >  > <FONTSIZE=2>&gt; - * Any changes to View1Implementationshould
 &gt; also be made to&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; - * AltView1Implementation. >  > <FONTSIZE=2>&gt;&nbsp;&nbsp; */ >  > <FONTSIZE=2>&gt; >  > <FONTSIZE=2>&gt;&nbsp; // Single-valued version. Handlesscalars
 &gt; and Locs.&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; @@ -508,62 +505,7 @@ >  > <FONTSIZE=2>&gt; >  > <FONTSIZE=2>&gt; >  > <FONTSIZE=2>&gt;&nbsp; /** >  > <FONTSIZE=2>&gt; - * AltView1Implementation avoids an > instantiation problem thatarises >  > <FONTSIZE=2>&gt; when two >  > <FONTSIZE=2>&gt; - * classes use each other.&nbsp; Thisclass's
 &gt; definition should be exactly&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; - * the same as View1Implementation except
 &gt; omitting member functions.&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; - * >  > <FONTSIZE=2>&gt; - * Do NOT explicitly instantiate this
 &gt; class.&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; - */ >  > <FONTSIZE=2>&gt; - >  > <FONTSIZE=2>&gt; -// Single-valued version. Handles scalarsand
 &gt; Locs.&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; - >  > <FONTSIZE=2>&gt; -template&lt;class Subject, class Domain,bool
 &gt; SV&amp;gt;&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; -structAltView1Implementation; >  > <FONTSIZE=2>&gt; - >  > <FONTSIZE=2>&gt; -template&lt;class Mesh, class T, class > EngineTag, classDomain&gt; >  > <FONTSIZE=2>&gt; -structAltView1Implementation&lt;Field&lt;Mesh, > T, EngineTag&gt;, Domain,true&gt; >  > <FONTSIZE=2>&gt; -{ >  > <FONTSIZE=2>&gt; -&nbsp; // Convenience typedef for thething
 &gt; we're taking a view of.&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; - >  > <FONTSIZE=2>&gt; -&nbsp; typedef Field&lt;Mesh, T,EngineTag&gt;
 &gt; Subject_t;&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; - >  > <FONTSIZE=2>&gt; -&nbsp; // The return types are prettysimple
 &gt; here.&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; - >  > <FONTSIZE=2>&gt; -&nbsp; typedef typenameSubject_t::Element_t
 &gt; ReadType_t;&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; -&nbsp; typedef typenameSubject_t::ElementRef_t
 &gt; Type_t;&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; -}; >  > <FONTSIZE=2>&gt; - >  > <FONTSIZE=2>&gt; -template&lt;class Mesh, class T, class > EngineTag, classDomain&gt; >  > <FONTSIZE=2>&gt; -structAltView1Implementation&lt;Field&lt;Mesh, > T, EngineTag&gt;, Domain,false&gt; >  > <FONTSIZE=2>&gt; -{ >  > <FONTSIZE=2>&gt; -&nbsp; // Convenience typedef for thething
 &gt; we're taking a view of.&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; - >  > <FONTSIZE=2>&gt; -&nbsp; typedef Field&lt;Mesh, T,EngineTag&gt;
 &gt; Subject_t;&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; - >  > <FONTSIZE=2>&gt; -&nbsp; // Deduce domains for theoutput
 &gt; type.&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; - >  > <FONTSIZE=2>&gt; -&nbsp; typedef typenameSubject_t::Engine_t
 &gt; Engine_t;&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; -&nbsp; typedef typenameNewEngine&lt;Engine_t, > Domain&gt;::Type_tNewEngine_t; >  > <FONTSIZE=2>&gt; -&nbsp; typedef typenameNewEngine_t::Element_t
 &gt; NewT_t;&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; -&nbsp; typedef typenameNewEngine_t::Tag_t
 &gt; NewEngineTag_t;&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; - >  > <FONTSIZE=2>&gt; -&nbsp; // Deduce the newMesh. >  > <FONTSIZE=2>&gt; - >  > <FONTSIZE=2>&gt; -&nbsp; typedef typename >  > <FONTSIZE=2>&gt; -&nbsp;&nbsp;&nbsp; > NewMeshTag&lt;NewEngine_t::dimensions,Mesh, Domain&gt;::Type_t >  > <FONTSIZE=2>&gt;-&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;
 &gt; NewMeshTag_t;&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; - >  > <FONTSIZE=2>&gt; -&nbsp; // The outputtypes. >  > <FONTSIZE=2>&gt; - >  > <FONTSIZE=2>&gt; -&nbsp; typedef Field&lt;NewMeshTag_t,NewT_t, > NewEngineTag_t&gt;ReadType_t; >  > <FONTSIZE=2>&gt; -&nbsp; typedef Field&lt;NewMeshTag_t,NewT_t, > NewEngineTag_t&gt;Type_t; >  > <FONTSIZE=2>&gt; -}; >  > <FONTSIZE=2>&gt; - >  > <FONTSIZE=2>&gt; - >  > <FONTSIZE=2>&gt; -/** >  > <FONTSIZE=2>&gt;&nbsp;&nbsp; * View1&lt;Field,S1&gt; > specialization for indexing a field with asingle >  > <FONTSIZE=2>&gt; domain. >  > <FONTSIZE=2>&gt; - * >  > <FONTSIZE=2>&gt; - * Any changes to View1 should also be madeto
 &gt; AltView1.&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt;&nbsp;&nbsp; */ >  > <FONTSIZE=2>&gt; >  > <FONTSIZE=2>&gt;&nbsp; template&lt;class Mesh, class T,class > EngineTag, classSub1&gt; >  > <FONTSIZE=2>&gt; @@ -613,8 +555,6 @@ >  > <FONTSIZE=2>&gt; >  > <FONTSIZE=2>&gt;&nbsp; /** >  > <FONTSIZE=2>&gt;&nbsp;&nbsp; * View1&lt;Field,int&gt; > specialization for indexing a field with anint. >  > <FONTSIZE=2>&gt; - * >  > <FONTSIZE=2>&gt; - * Any changes to View1 should also be madeto
 &gt; AltView1.&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt;&nbsp;&nbsp; */ >  > <FONTSIZE=2>&gt; >  > <FONTSIZE=2>&gt;&nbsp; template&lt;class Mesh, class T,class
 &gt; EngineTag&amp;gt;&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; @@ -658,71 +598,6 @@ >  > <FONTSIZE=2>&gt; >  > <FONTSIZE=2>&gt; >  > <FONTSIZE=2>&gt;&nbsp; /** >  > <FONTSIZE=2>&gt; - * AltView1 avoids an instantiation problemthat
 &gt; arises when two&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; - * classes use each other.&nbsp; Thisclass's
 &gt; definition should be exactly&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; - * the same as View1 except omittingmember
 &gt; functions.&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; - * >  > <FONTSIZE=2>&gt; - * Do NOT explicitly instantiate this
 &gt; class.&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; - */ >  > <FONTSIZE=2>&gt; - >  > <FONTSIZE=2>&gt; -template&lt;class FieldTag, class
 &gt; DomainTag&amp;gt;&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; -struct AltView1; >  > <FONTSIZE=2>&gt; - >  > <FONTSIZE=2>&gt; -template&lt;class Mesh, class T, class > EngineTag, classSub1&gt; >  > <FONTSIZE=2>&gt; -struct AltView1&lt;Field&lt;Mesh,T, > EngineTag&gt;,Sub1&gt; >  > <FONTSIZE=2>&gt; -{ >  > <FONTSIZE=2>&gt; -&nbsp; // Convenience typedef for thething
 &gt; we're taking a view of.&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; - >  > <FONTSIZE=2>&gt; -&nbsp; typedef Field&lt;Mesh, T,EngineTag&gt;
 &gt; Subject_t;&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; - >  > <FONTSIZE=2>&gt; -&nbsp; // Deduce domains for theoutput
 &gt; type.&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; -&nbsp; // At some point, we need tofix
 &gt; NewDomain1; until then, use&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; -&nbsp; // the temporary version from
 &gt; NewDomain.h.&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; - >  > <FONTSIZE=2>&gt; -&nbsp; typedef typenameSubject_t::Domain_t
 &gt; Domain_t;&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; -&nbsp; typedefTemporaryNewDomain1&lt;Domain_t,
 &gt; Sub1&amp;gt; NewDomain_t;&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; -&nbsp; typedef typenameNewDomain_t::SliceType_t
 &gt; SDomain_t;&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; - >  > <FONTSIZE=2>&gt; -&nbsp; // Deduce appropriate versionof > implementation to dispatchto. >  > <FONTSIZE=2>&gt; - >  > <FONTSIZE=2>&gt; -&nbsp; enum { sv = >DomainTraits&lt;SDomain_t&gt;::singleValued}; >  > <FONTSIZE=2>&gt; -&nbsp; typedef > AltView1Implementation&lt;Subject_t,SDomain_t, sv&gt; Dispatch_t; >  > <FONTSIZE=2>&gt; - >  > <FONTSIZE=2>&gt; -&nbsp; // The optimized domaincombiner. >  > <FONTSIZE=2>&gt; - >  > <FONTSIZE=2>&gt; -&nbsp; typedefCombineDomainOpt&lt;NewDomain_t,
 &gt; sv&amp;gt; Combine_t;&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; - >  > <FONTSIZE=2>&gt; -&nbsp; // The returntypes. >  > <FONTSIZE=2>&gt; - >  > <FONTSIZE=2>&gt; -&nbsp; typedef typenameDispatch_t::ReadType_t
 &gt; ReadType_t;&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; -&nbsp; typedef typenameDispatch_t::Type_t
 &gt; Type_t;&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; -}; >  > <FONTSIZE=2>&gt; - >  > <FONTSIZE=2>&gt; - >  > <FONTSIZE=2>&gt; -/** >  > <FONTSIZE=2>&gt; - * AltView1 avoids an instantiation problemthat
 &gt; arises when two&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; - * classes use each other.&nbsp; Thisclass's
 &gt; definition should be exactly&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; - * the same as View1 except omittingmember
 &gt; functions.&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; - * >  > <FONTSIZE=2>&gt; - * Do NOT explicitly instantiate this
 &gt; class.&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; - */ >  > <FONTSIZE=2>&gt; - >  > <FONTSIZE=2>&gt; -template&lt;class Mesh, class T, class
 &gt; EngineTag&amp;gt;&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; -struct AltView1&lt;Field&lt;Mesh,T, > EngineTag&gt;,int&gt; >  > <FONTSIZE=2>&gt; -{ >  > <FONTSIZE=2>&gt; -&nbsp; // Convenience typedef for thething
 &gt; we're taking a view of.&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; - >  > <FONTSIZE=2>&gt; -&nbsp; typedef Field&lt;Mesh, T,EngineTag&gt;
 &gt; Subject_t;&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; - >  > <FONTSIZE=2>&gt; -&nbsp; // The returntypes. >  > <FONTSIZE=2>&gt; - >  > <FONTSIZE=2>&gt; -&nbsp; typedef typenameSubject_t::Element_t
 &gt; ReadType_t;&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; -&nbsp; typedef typenameSubject_t::ElementRef_t
 &gt; Type_t;&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; - >  > <FONTSIZE=2>&gt; -}; >  > <FONTSIZE=2>&gt; - >  > <FONTSIZE=2>&gt; - >  > <FONTSIZE=2>&gt; -/** >  > <FONTSIZE=2>&gt;&nbsp;&nbsp; * View2&lt;Field, S1,S2&gt; > specialization for indexing a field withtwo >  > <FONTSIZE=2>&gt;&nbsp;&nbsp; * domains. >  > <FONTSIZE=2>&gt;&nbsp;&nbsp; */ >  > <FONTSIZE=2>&gt; @@ -1455,14 +1330,14 @@ >  > <FONTSIZE=2>&gt;&nbsp;&nbsp;&nbsp; /// point fields theviewing > domain is an actual centeringdomain. >  > <FONTSIZE=2>&gt;&nbsp;&nbsp;&nbsp;//@{ >  > <FONTSIZE=2>&gt; >  > <FONTSIZE=2>&gt; -&nbsp; inline typenameAltView1&lt;This_t,
 &gt; Domain_t&amp;gt;::ReadType_t&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; +&nbsp; inline typenameView1&lt;This_t,
 &gt; Domain_t&amp;gt;::ReadType_t&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt;&nbsp;&nbsp;&nbsp; read()const >  > <FONTSIZE=2>&gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;{ >  > <FONTSIZE=2>&gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; typedef > View1&lt;This_t, Domain_t&gt;Ret_t; >  > <FONTSIZE=2>&gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; return > Ret_t::makeRead(*this,physicalDomain()); >  > <FONTSIZE=2>&gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;} >  > <FONTSIZE=2>&gt; >  > <FONTSIZE=2>&gt; -&nbsp; inline typenameAltView1&lt;This_t,
 &gt; Domain_t&amp;gt;::ReadType_t&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; +&nbsp; inline typenameView1&lt;This_t,
 &gt; Domain_t&amp;gt;::ReadType_t&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt;&nbsp;&nbsp;&nbsp; readAll()const >  > <FONTSIZE=2>&gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;{ >  > <FONTSIZE=2>&gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; typedef > View1&lt;This_t, Domain_t&gt;Ret_t; >  > <FONTSIZE=2>&gt; @@ -1470,7 +1345,7 @@ >  > <FONTSIZE=2>&gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;} >  > <FONTSIZE=2>&gt; >  > <FONTSIZE=2>&gt;&nbsp;&nbsp;&nbsp;template&lt;class
 &gt; Sub1&amp;gt;&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; -&nbsp; inline typenameAltView1&lt;This_t,
 &gt; Sub1&amp;gt;::ReadType_t&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; +&nbsp; inline typenameView1&lt;This_t,
 &gt; Sub1&amp;gt;::ReadType_t&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt;&nbsp;&nbsp;&nbsp; read(const Sub1&amp;s1)
 &gt; const&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;{ >  > <FONTSIZE=2>&gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; typedef > View1&lt;This_t, Sub1&gt;Ret_t; >  > <FONTSIZE=2>&gt; @@ -1493,14 +1368,14 @@ >  > <FONTSIZE=2>&gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; return > Ret_t::makeRead(*this, s1, s2,s3); >  > <FONTSIZE=2>&gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;} >  > <FONTSIZE=2>&gt; >  > <FONTSIZE=2>&gt; -&nbsp; inline typenameAltView1&lt;This_t,
 &gt; Domain_t&amp;gt;::Type_t&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; +&nbsp; inline typenameView1&lt;This_t,
 &gt; Domain_t&amp;gt;::Type_t&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt;&nbsp;&nbsp;&nbsp; operator()()const >  > <FONTSIZE=2>&gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;{ >  > <FONTSIZE=2>&gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; typedef > View1&lt;This_t, Domain_t&gt;Ret_t; >  > <FONTSIZE=2>&gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; return > Ret_t::make(*this,physicalDomain()); >  > <FONTSIZE=2>&gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;} >  > <FONTSIZE=2>&gt; >  > <FONTSIZE=2>&gt; -&nbsp; inline typenameAltView1&lt;This_t,
 &gt; Domain_t&amp;gt;::Type_t&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; +&nbsp; inline typenameView1&lt;This_t,
 &gt; Domain_t&amp;gt;::Type_t&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt;&nbsp;&nbsp;&nbsp; all()const >  > <FONTSIZE=2>&gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;{ >  > <FONTSIZE=2>&gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; typedef > View1&lt;This_t, Domain_t&gt;Ret_t; >  > <FONTSIZE=2>&gt; @@ -1508,7 +1383,7 @@ >  > <FONTSIZE=2>&gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;} >  > <FONTSIZE=2>&gt; >  > <FONTSIZE=2>&gt;&nbsp;&nbsp;&nbsp;template&lt;class
 &gt; Sub1&amp;gt;&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; -&nbsp; inline typenameAltView1&lt;This_t,
 &gt; Sub1&amp;gt;::Type_t&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt; +&nbsp; inline typenameView1&lt;This_t,
 &gt; Sub1&amp;gt;::Type_t&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt;&nbsp;&nbsp;&nbsp; operator()(constSub1 &amp;s1)
 &gt; const&lt;/FONT&gt;
 >  > <FONTSIZE=2>&gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;{ >  > <FONTSIZE=2>&gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; typedef > View1&lt;This_t, Sub1&gt;Ret_t;
 &gt;&nbsp; &gt; &lt;/P&gt;
 &gt;&nbsp; &gt;
 &gt;&nbsp; &gt; &lt;/BODY&gt;
 &gt;&nbsp; &gt; &lt;/HTML&gt;


</BODY>
</HTML>

[Prev in Thread]

Current Thread

[Next in Thread]

Re: [pooma-dev] AltView* classes, (continued)
- RE: [pooma-dev] AltView* classes, James Crotinger, 2004/03/02
- RE: [pooma-dev] AltView* classes, James Crotinger, 2004/03/02
 - RE: [pooma-dev] AltView* classes, Dave Nystrom, 2004/03/02
 - Re: [pooma-dev] AltView* classes, Richard Guenther, 2004/03/02
 - Releasing Blanca Demo, John H. Hall, 2004/03/02
 - Re: [pooma-dev] AltView* classes, Dave Nystrom, 2004/03/02
 - Re: [pooma-dev] AltView* classes, Dave Nystrom, 2004/03/02
 - Re: [pooma-dev] AltView* classes, John H. Hall <=

Prev by Date: Re: [pooma-dev] AltView* classes
Next by Date: Releasing Blanca Demo
Previous by thread: Re: [pooma-dev] AltView* classes
Next by thread: [PATCH] Add missing #includes
Index(es):
- Date
- Thread