Fix the bug in the variable analysis: When we define nested tasks, the task liveness...
[IRC.git] / Robust / src / Analysis / OoOJava / OoOJavaAnalysis.java
index d5e42ebeffab8ad2607ce9bfce6f4c8ff328c5a3..4a5283d57b4a1135d48e4184ff089b5d8089b28a 100644 (file)
@@ -7,16 +7,17 @@ import Analysis.*;
 import Analysis.CallGraph.*;
 import Analysis.Disjoint.*;
 import Analysis.Pointer.*;
+import Util.*;
 import IR.*;
 import IR.Flat.*;
 
-
 public class OoOJavaAnalysis {
 
   // data from the compiler
   private State state;
   private TypeUtil typeUtil;
   private CallGraph callGraph;
+  private Liveness liveness;
   private RBlockRelationAnalysis rblockRel;
   private HeapAnalysis disjointAnalysisTaints;
   private DisjointAnalysis disjointAnalysisReach;
@@ -62,29 +63,29 @@ public class OoOJavaAnalysis {
     return codePlans.keySet();
   }
 
-  public ContextTaskNames getContextTaskNames( FlatMethod fm ) {
-    ContextTaskNames out = fn2contextTaskNames.get( fm );
-    if( out == null ) {
+  public ContextTaskNames getContextTaskNames(FlatMethod fm) {
+    ContextTaskNames out = fn2contextTaskNames.get(fm);
+    if (out == null) {
       out = new ContextTaskNames();
     }
     return out;
   }
 
-  public ContextTaskNames getContextTaskNames( FlatSESEEnterNode fsen ) {
-    ContextTaskNames out = fn2contextTaskNames.get( fsen );
-    if( out == null ) {
+  public ContextTaskNames getContextTaskNames(FlatSESEEnterNode fsen) {
+    ContextTaskNames out = fn2contextTaskNames.get(fsen);
+    if (out == null) {
       out = new ContextTaskNames();
     }
     return out;
   }
 
-  public FlatMethod getContainingFlatMethod( FlatNode fn ) {
-    FlatMethod fm = fn2fm.get( fn );
+  public FlatMethod getContainingFlatMethod(FlatNode fn) {
+    FlatMethod fm = fn2fm.get(fn);
     assert fm != null;
     return fm;
   }
 
-  public HeapAnalysis getDisjointAnalysis() {
+  public HeapAnalysis getHeapAnalysis() {
     return disjointAnalysisTaints;
   }
 
@@ -92,34 +93,31 @@ public class OoOJavaAnalysis {
     return buildStateMachines;
   }
 
-  public OoOJavaAnalysis( State            state, 
-                          TypeUtil         typeUtil, 
-                          CallGraph        callGraph, 
-                          Liveness         liveness,
-                          ArrayReferencees arrayReferencees ) {
-
-    double timeStartAnalysis = (double) System.nanoTime();
+  public OoOJavaAnalysis(State state, TypeUtil typeUtil, CallGraph callGraph, Liveness liveness,
+                         ArrayReferencees arrayReferencees) {
 
+    State.logEvent("Starting OoOJavaAnalysis");
     this.state = state;
     this.typeUtil = typeUtil;
     this.callGraph = callGraph;
+    this.liveness = liveness;
     this.maxSESEage = state.OOO_MAXSESEAGE;
 
-    livenessGlobalView     = new Hashtable<FlatNode, Set<TempDescriptor>>();
-    livenessVirtualReads   = new Hashtable<FlatNode, Set<TempDescriptor>>();
-    variableResults        = new Hashtable<FlatNode, VarSrcTokTable>();
-    notAvailableResults    = new Hashtable<FlatNode, Set<TempDescriptor>>();
-    codePlans              = new Hashtable<FlatNode, CodePlan>();
-    wdvNodesToSpliceIn     = new Hashtable<FlatEdge, FlatWriteDynamicVarNode>();
-    notAvailableIntoSESE   = new Hashtable<FlatSESEEnterNode, Set<TempDescriptor>>();
-    sese2conflictGraph     = new Hashtable<FlatNode, ConflictGraph>();
+    livenessGlobalView = new Hashtable<FlatNode, Set<TempDescriptor>>();
+    livenessVirtualReads = new Hashtable<FlatNode, Set<TempDescriptor>>();
+    variableResults = new Hashtable<FlatNode, VarSrcTokTable>();
+    notAvailableResults = new Hashtable<FlatNode, Set<TempDescriptor>>();
+    codePlans = new Hashtable<FlatNode, CodePlan>();
+    wdvNodesToSpliceIn = new Hashtable<FlatEdge, FlatWriteDynamicVarNode>();
+    notAvailableIntoSESE = new Hashtable<FlatSESEEnterNode, Set<TempDescriptor>>();
+    sese2conflictGraph = new Hashtable<FlatNode, ConflictGraph>();
     conflictGraph2SESELock = new Hashtable<ConflictGraph, HashSet<SESELock>>();
-    fn2contextTaskNames    = new Hashtable<FlatNode, ContextTaskNames>();
-    fn2fm                  = new Hashtable<FlatNode, FlatMethod>();
+    fn2contextTaskNames = new Hashtable<FlatNode, ContextTaskNames>();
+    fn2fm = new Hashtable<FlatNode, FlatMethod>();
 
     // state machines support heap examiners with
     // state transitions to improve precision
-    if( state.RCR ) {
+    if (state.RCR) {
       buildStateMachines = new BuildStateMachines();
     }
 
@@ -137,93 +135,99 @@ public class OoOJavaAnalysis {
     Iterator<MethodDescriptor> methItr = descriptorsToAnalyze.iterator();
     while (methItr.hasNext()) {
       Descriptor d = methItr.next();
-      FlatMethod fm = state.getMethodFlat( d );
-      buildFlatNodeToFlatMethod( fm );
+      FlatMethod fm = state.getMethodFlat(d);
+      buildFlatNodeToFlatMethod(fm);
     }
+    State.logEvent("OoOJavaAnalysis Oth pass completed");
 
     // 1st pass, find basic rblock relations & potential stall sites
     rblockRel = new RBlockRelationAnalysis(state, typeUtil, callGraph);
     VarSrcTokTable.rblockRel = rblockRel;
 
+    State.logEvent("OoOJavaAnalysis 1st pass completed");
+
     // 2nd pass, liveness, in-set out-set (no virtual reads yet!)
-    methItr = descriptorsToAnalyze.iterator();
-    while (methItr.hasNext()) {
-      Descriptor d = methItr.next();
-      FlatMethod fm = state.getMethodFlat(d);
+    // run liveness analysis for each sese blocks
+    Iterator<FlatSESEEnterNode> seseItr = rblockRel.getAllSESEs().iterator();
+    while (seseItr.hasNext()) {
+      FlatSESEEnterNode sese = seseItr.next();
+      livenessAnalysisBackward(sese);
+    }  
 
-      // note we can't use the general liveness analysis already in
-      // the compiler because this analysis is task-aware
-      livenessAnalysisBackward(fm);
-    }
+    State.logEvent("OoOJavaAnalysis 2nd pass completed");
 
     // 3rd pass, variable analysis
-    methItr = descriptorsToAnalyze.iterator();
+    methItr = rblockRel.getMethodsWithSESEs().iterator();
     while (methItr.hasNext()) {
       Descriptor d = methItr.next();
       FlatMethod fm = state.getMethodFlat(d);
-
       // starting from roots do a forward, fixed-point
       // variable analysis for refinement and stalls
       variableAnalysisForward(fm);
     }
 
+    State.logEvent("OoOJavaAnalysis 3rd pass completed");
+
     // 4th pass, compute liveness contribution from
     // virtual reads discovered in variable pass
-    methItr = descriptorsToAnalyze.iterator();
-    while (methItr.hasNext()) {
-      Descriptor d = methItr.next();
-      FlatMethod fm = state.getMethodFlat(d);
-      livenessAnalysisBackward(fm);
+    seseItr = rblockRel.getLocalRootSESEs().iterator();
+    while (seseItr.hasNext()) {
+      FlatSESEEnterNode sese = seseItr.next();
+      livenessAnalysisBackward(sese);
     }
 
+    State.logEvent("OoOJavaAnalysis 4th pass completed");
+
     // 5th pass, use disjointness with NO FLAGGED REGIONS
     // to compute taints and effects
     if (state.POINTER) {
-      disjointAnalysisTaints = new Pointer(state, typeUtil, callGraph, rblockRel, liveness);
-      ((Pointer)disjointAnalysisTaints).doAnalysis();
-    } else
       disjointAnalysisTaints =
-        new DisjointAnalysis(state, typeUtil, callGraph, liveness, arrayReferencees, null, 
-                             rblockRel, buildStateMachines,
-                             true ); // suppress output--this is an intermediate pass
+        new Pointer(state, typeUtil, callGraph, rblockRel, liveness, buildStateMachines);
+      ((Pointer) disjointAnalysisTaints).doAnalysis();
+    } else {
+      disjointAnalysisTaints =
+        new DisjointAnalysis(state, typeUtil, callGraph, liveness, arrayReferencees, null,
+                             rblockRel, buildStateMachines, 
+                             !state.OOODEBUG // only print out in OoOJava debug mode
+                             );
+    }
+    State.logEvent("OoOJavaAnalysis 5th pass completed");
 
     // 6th pass, not available analysis FOR VARIABLES!
-    methItr = descriptorsToAnalyze.iterator();
+    methItr = rblockRel.getMethodsWithSESEs().iterator();
     while (methItr.hasNext()) {
       Descriptor d = methItr.next();
       FlatMethod fm = state.getMethodFlat(d);
-
       // compute what is not available at every program
       // point, in a forward fixed-point pass
       notAvailableForward(fm);
     }
-
+    State.logEvent("OoOJavaAnalysis 6th pass completed");
     // 7th pass, start conflict graphs where a parent's graph has a
     // node for possibly conflicting children and its own stall sites
     startConflictGraphs();
-    
+    State.logEvent("OoOJavaAnalysis 7th pass completed");
     // 8th pass, calculate all possible conflicts without using
     // reachability info and identify set of FlatNew that next
     // disjoint reach. analysis should flag
     Set<FlatNew> sitesToFlag = new HashSet<FlatNew>();
     calculateConflicts(sitesToFlag, false);
-
+    State.logEvent("OoOJavaAnalysis 8th pass completed");
     if (!state.RCR) {
       // 9th pass, ask disjoint analysis to compute reachability
       // for objects that may cause heap conflicts so the most
       // efficient method to deal with conflict can be computed
-      // later      
+      // later
       disjointAnalysisReach =
-        new DisjointAnalysis(state, typeUtil, callGraph, liveness, 
-                             arrayReferencees, sitesToFlag,
-                            null, // don't do effects analysis again!
+        new DisjointAnalysis(state, typeUtil, callGraph, liveness, arrayReferencees, sitesToFlag,
+                             null, // don't do effects analysis again!
                              null, // no BuildStateMachines needed
-                             false // don't suppress progress output
-                            );
-
+                             !state.OOODEBUG // only print out in OoOJava debug mode
+                             );
+      State.logEvent("OoOJavaAnalysis 9th pass completed");
       // 10th pass, calculate conflicts with reachability info
       calculateConflicts(null, true);
-
+      State.logEvent("OoOJavaAnalysis 10th pass completed");
     } else {
       // in RCR/DFJ we want to do some extra processing on the
       // state machines before they get handed off to code gen,
@@ -231,11 +235,13 @@ public class OoOJavaAnalysis {
       // to identify heap examiners that are weakly connected, so
       // accomplish both at the same time
       pruneMachinesAndFindWeaklyConnectedExaminers();
+      State.logEvent("OoOJavaAnalysis RCR pruneMachines pass completed");
     }
 
-    // 11th pass, compiling memory Qs!  The name "lock" is a legacy
+    // 11th pass, compiling memory Qs! The name "lock" is a legacy
     // term for the heap dependence queue, or memQ as the runtime calls it
     synthesizeLocks();
+    State.logEvent("OoOJavaAnalysis 11th pass completed");
 
     // 12th pass, compute a plan for code injections
     methItr = descriptorsToAnalyze.iterator();
@@ -245,74 +251,70 @@ public class OoOJavaAnalysis {
       codePlansForward(fm);
     }
 
+    State.logEvent("OoOJavaAnalysis 12th pass completed");
     // 13th pass,
     // splice new IR nodes into graph after all
     // analysis passes are complete
     Iterator spliceItr = wdvNodesToSpliceIn.entrySet().iterator();
     while (spliceItr.hasNext()) {
-      Map.Entry me = (Map.Entry) spliceItr.next();
+      Map.Entry me = (Map.Entry)spliceItr.next();
       FlatWriteDynamicVarNode fwdvn = (FlatWriteDynamicVarNode) me.getValue();
       fwdvn.spliceIntoIR();
     }
-
+    State.logEvent("OoOJavaAnalysis 13th pass completed");
 
     if (state.OOODEBUG) {
       try {
         writeReports("");
-       disjointAnalysisTaints.getEffectsAnalysis().writeEffects("effects.txt");
+        disjointAnalysisTaints.getEffectsAnalysis().writeEffects("effects.txt");
         writeConflictGraph();
-      } catch (IOException e) {}
+      } catch (IOException e) {
+      }
     }
-    
+    State.logEvent("OoOJavaAnalysis completed");
   }
 
-
-  private void buildFlatNodeToFlatMethod( FlatMethod fm ) {
+  private void buildFlatNodeToFlatMethod(FlatMethod fm) {
     Set<FlatNode> flatNodesToVisit = new HashSet<FlatNode>();
-    flatNodesToVisit.add( fm );
+    flatNodesToVisit.add(fm);
 
     Set<FlatNode> flatNodesVisited = new HashSet<FlatNode>();
 
-    while( !flatNodesToVisit.isEmpty() ) {
+    while (!flatNodesToVisit.isEmpty()) {
       FlatNode fn = (FlatNode) flatNodesToVisit.iterator().next();
-      flatNodesToVisit.remove( fn );
-      flatNodesVisited.add( fn );
+      flatNodesToVisit.remove(fn);
+      flatNodesVisited.add(fn);
 
-      fn2fm.put( fn, fm );
+      fn2fm.put(fn, fm);
 
-      for( int i = 0; i < fn.numNext(); i++ ) {
-        FlatNode nn = fn.getNext( i );
-        if( !flatNodesVisited.contains( nn ) ) {
-          flatNodesToVisit.add( nn );
+      for (int i = 0; i < fn.numNext(); i++) {
+        FlatNode nn = fn.getNext(i);
+        if (!flatNodesVisited.contains(nn)) {
+          flatNodesToVisit.add(nn);
         }
       }
     }
   }
 
-
-    // debug routine
-    /*
-     * Iterator iter = sese2conflictGraph.entrySet().iterator(); while
-     * (iter.hasNext()) { Entry e = (Entry) iter.next(); FlatNode fn =
-     * (FlatNode) e.getKey(); ConflictGraph conflictGraph = (ConflictGraph)
-     * e.getValue();
-     * System.out.println("---------------------------------------");
-     * System.out.println("CONFLICT GRAPH for " + fn); Set<String> keySet =
-     * conflictGraph.id2cn.keySet(); for (Iterator iterator = keySet.iterator();
-     * iterator.hasNext();) { String key = (String) iterator.next();
-     * ConflictNode node = conflictGraph.id2cn.get(key);
-     * System.out.println("key=" + key + " \n" + node.toStringAllEffects()); } }
-     */
-
-
-  
+  // debug routine
+  /*
+   * Iterator iter = sese2conflictGraph.entrySet().iterator(); while
+   * (iter.hasNext()) { Entry e = (Entry) iter.next(); FlatNode fn = (FlatNode)
+   * e.getKey(); ConflictGraph conflictGraph = (ConflictGraph) e.getValue();
+   * System.out.println("---------------------------------------");
+   * System.out.println("CONFLICT GRAPH for " + fn); Set<String> keySet =
+   * conflictGraph.id2cn.keySet(); for (Iterator iterator = keySet.iterator();
+   * iterator.hasNext();) { String key = (String) iterator.next(); ConflictNode
+   * node = conflictGraph.id2cn.get(key); System.out.println("key=" + key +
+   * " \n" + node.toStringAllEffects()); } }
+   */
 
   private void writeFile(Set<FlatNew> sitesToFlag) {
 
     try {
       BufferedWriter bw = new BufferedWriter(new FileWriter("sitesToFlag.txt"));
 
-      for (Iterator iterator = sitesToFlag.iterator(); iterator.hasNext();) {
+      for (Iterator iterator = sitesToFlag.iterator(); iterator.hasNext(); ) {
         FlatNew fn = (FlatNew) iterator.next();
         bw.write(fn + "\n");
       }
@@ -324,96 +326,113 @@ public class OoOJavaAnalysis {
   }
 
 
-  private void livenessAnalysisBackward(FlatMethod fm) {
+  private void livenessAnalysisBackward(FlatSESEEnterNode fsen) {
+
+    // each task maintains a local liveness result
+    Hashtable<FlatNode, Set<TempDescriptor>> livenessLocalView =
+        new Hashtable<FlatNode, Set<TempDescriptor>>();
 
-    // flow backward across nodes to compute liveness, and
-    // take special care with sese enter/exit nodes that
-    // alter this from normal liveness analysis
+    // flow backward across nodes to compute liveness.
+    // it contributes liveness results to the global view
+    // only if the current flat node belongs directly to the currently analyzing
+    // sese.
     Set<FlatNode> flatNodesToVisit = new HashSet<FlatNode>();
-    flatNodesToVisit.add( fm.getFlatExit() );
+    flatNodesToVisit.add(fsen.getFlatExit());
 
-    while( !flatNodesToVisit.isEmpty() ) {
+    while (!flatNodesToVisit.isEmpty()) {
       FlatNode fn = (FlatNode) flatNodesToVisit.iterator().next();
-      flatNodesToVisit.remove( fn );
+      flatNodesToVisit.remove(fn);
 
-      Set<TempDescriptor> prev = livenessGlobalView.get( fn );
+      Set<TempDescriptor> prev = livenessLocalView.get(fn);
 
       // merge sets from control flow joins
       Set<TempDescriptor> livein = new HashSet<TempDescriptor>();
       for (int i = 0; i < fn.numNext(); i++) {
-        FlatNode nn = fn.getNext( i );
-        Set<TempDescriptor> s = livenessGlobalView.get( nn );
-        if( s != null ) {
-          livein.addAll( s );
+        FlatNode nn = fn.getNext(i);
+        Set<TempDescriptor> s = livenessLocalView.get(nn);
+        if (s != null) {
+          livein.addAll(s);
         }
       }
-      
-      Set<TempDescriptor> curr = liveness_nodeActions( fn, livein );
+
+      Set<TempDescriptor> curr = liveness_nodeActions(fsen, fn, livein);
 
       // if a new result, schedule backward nodes for analysis
-      if( !curr.equals( prev ) ) {
-        livenessGlobalView.put( fn, curr );
+      if (!curr.equals(prev)) {
+
+        if (fn != fsen) {
+          livenessLocalView.put(fn, curr);
+
+          if (rblockRel.getLocalInnerRBlock(fn).equals(fsen)) {
+            // the current flat node belongs to the currently analyzing sese
+            // store its liveness in the gloval view
+            livenessGlobalView.put(fn, curr);
+          }
 
-        for( int i = 0; i < fn.numPrev(); i++ ) {
-          FlatNode nn = fn.getPrev( i );
-          flatNodesToVisit.add( nn );
+          for (int i = 0; i < fn.numPrev(); i++) {
+            FlatNode nn = fn.getPrev(i);
+            flatNodesToVisit.add(nn);
+          }
         }
       }
     }
   }
 
-  private Set<TempDescriptor> liveness_nodeActions( FlatNode            fn, 
-                                                    Set<TempDescriptor> liveIn
-                                                    ) {
-    switch( fn.kind() ) {
+  private Set<TempDescriptor> liveness_nodeActions(FlatSESEEnterNode currSESE, FlatNode fn,
+      Set<TempDescriptor> liveIn) {
+
+    switch (fn.kind()) {
 
     case FKind.FlatSESEEnterNode: {
       // add whatever is live-in at a task enter to that
       // task's in-var set
-      FlatSESEEnterNode fsen = (FlatSESEEnterNode)fn;
-      if( liveIn != null ) {
-        fsen.addInVarSet( liveIn );
+      FlatSESEEnterNode fsen = (FlatSESEEnterNode) fn;
+      if (currSESE.equals(fsen)) {
+        if (liveIn != null) {
+          fsen.addInVarSet(liveIn);
+        }
+        // no break, should also execute default actions
       }
-      // no break, should also execute default actions
     }
 
     default: {
       // handle effects of statement in reverse, writes then reads
       TempDescriptor[] writeTemps = fn.writesTemps();
-      for( int i = 0; i < writeTemps.length; ++i ) {
-        liveIn.remove( writeTemps[i] );
+      for (int i = 0; i < writeTemps.length; ++i) {
+        liveIn.remove(writeTemps[i]);
 
         // if we are analyzing code declared directly in a task,
-        FlatSESEEnterNode fsen = rblockRel.getLocalInnerRBlock( fn );
-        if( fsen != null ) {
+        FlatSESEEnterNode fsen = rblockRel.getLocalInnerRBlock(fn);
+        if (fsen != null) {
           // check to see if we are writing to variables that will
           // be live-out at the task's exit (and therefore should
           // go in the task's out-var set)
           FlatSESEExitNode fsexn = fsen.getFlatExit();
-          Set<TempDescriptor> livetemps = livenessGlobalView.get( fsexn );
-          if( livetemps != null && livetemps.contains( writeTemps[i] ) ) {
-            fsen.addOutVar( writeTemps[i] );
-          }          
+          // note: liveness analysis can have corresponding decisions
+          Set<TempDescriptor> livetemps = liveness.getLiveInTemps(fsen.getfmEnclosing(), fsexn);
+          if (livetemps != null && livetemps.contains(writeTemps[i])) {
+            fsen.addOutVar(writeTemps[i]);
+          }
         }
       }
 
       TempDescriptor[] readTemps = fn.readsTemps();
-      for( int i = 0; i < readTemps.length; ++i ) {
-        liveIn.add( readTemps[i] );
+      for (int i = 0; i < readTemps.length; ++i) {
+        liveIn.add(readTemps[i]);
       }
 
-      Set<TempDescriptor> virtualReadTemps = livenessVirtualReads.get( fn );
-      if( virtualReadTemps != null ) {
-        liveIn.addAll( virtualReadTemps );
-      }      
-    } break;
+      Set<TempDescriptor> virtualReadTemps = livenessVirtualReads.get(fn);
+      if (virtualReadTemps != null) {
+        liveIn.addAll(virtualReadTemps);
+      }
+    }
+      break;
 
     } // end switch
 
     return liveIn;
   }
 
-
   private void variableAnalysisForward(FlatMethod fm) {
 
     Set<FlatNode> flatNodesToVisit = new HashSet<FlatNode>();
@@ -433,11 +452,11 @@ public class OoOJavaAnalysis {
         curr.merge(incoming);
       }
 
-      FlatSESEEnterNode currentSESE = rblockRel.getLocalInnerRBlock( fn );
-      if( currentSESE == null ) {
+      FlatSESEEnterNode currentSESE = rblockRel.getLocalInnerRBlock(fn);
+      if (currentSESE == null) {
         currentSESE = rblockRel.getCallerProxySESE();
       }
-      
+
       variable_nodeActions(fn, curr, currentSESE);
 
       // if a new result, schedule forward nodes for analysis
@@ -452,12 +471,10 @@ public class OoOJavaAnalysis {
     }
   }
 
-  private void variable_nodeActions(FlatNode          fn, 
-                                    VarSrcTokTable    vstTable,
+  private void variable_nodeActions(FlatNode fn, VarSrcTokTable vstTable,
                                     FlatSESEEnterNode currentSESE) {
     switch (fn.kind()) {
 
-
     case FKind.FlatSESEEnterNode: {
       FlatSESEEnterNode fsen = (FlatSESEEnterNode) fn;
       // ignore currently executing SESE, at this point
@@ -465,8 +482,8 @@ public class OoOJavaAnalysis {
       // the current SESE
       vstTable.age(fsen);
       vstTable.assertConsistency();
-    } break;
-
+    }
+    break;
 
     case FKind.FlatSESEExitNode: {
       FlatSESEExitNode fsexn = (FlatSESEExitNode) fn;
@@ -482,10 +499,11 @@ public class OoOJavaAnalysis {
       // written by an SESE and should be added to the in-set
       // anything virtually read by this SESE should be pruned
       // of parent or sibling sources
+      // Set<TempDescriptor> liveVars = liveness.getLiveInTemps(fsen.getfmEnclosing(), fn);
       Set<TempDescriptor> liveVars = livenessGlobalView.get(fn);
+
       Set<TempDescriptor> fsenVirtReads =
-        vstTable.calcVirtReadsAndPruneParentAndSiblingTokens(fsen, 
-                                                             liveVars);
+        vstTable.calcVirtReadsAndPruneParentAndSiblingTokens(fsen, liveVars);
 
       Set<TempDescriptor> fsenVirtReadsOld = livenessVirtualReads.get(fn);
       if (fsenVirtReadsOld != null) {
@@ -493,6 +511,9 @@ public class OoOJavaAnalysis {
       }
       livenessVirtualReads.put(fn, fsenVirtReads);
 
+      // virtual reads are forced in-vars!
+      fsen.addInVarSet(fsenVirtReads);
+
       // then all child out-set tokens are guaranteed
       // to be filled in, so clobber those entries with
       // the latest, clean sources
@@ -506,8 +527,8 @@ public class OoOJavaAnalysis {
         vstTable.add(vst);
       }
       vstTable.assertConsistency();
-    } break;
-
+    }
+    break;
 
     case FKind.FlatOpNode: {
       FlatOpNode fon = (FlatOpNode) fn;
@@ -529,34 +550,26 @@ public class OoOJavaAnalysis {
 
           // when we do x = y for variables, just copy over from a child,
           // there are two cases:
-          //  1. if the current task is the caller proxy, any local root is a child
-          boolean case1 = 
-            currentSESE.getIsCallerProxySESE() &&
-            rblockRel.getLocalRootSESEs().contains( vst.getSESE() );
-
-          //  2. if the child task is a locally-defined child of the current task
-          boolean case2 = currentSESE.getLocalChildren().contains( vst.getSESE() );
-            
-          if( case1 || case2 ) {
+          // 1. if the current task is the caller proxy, any local root is a
+          // child
+          boolean case1 =
+            currentSESE.getIsCallerProxySESE()
+            && rblockRel.getLocalRootSESEs().contains(vst.getSESE());
+
+          // 2. if the child task is a locally-defined child of the current task
+          boolean case2 = currentSESE.getLocalChildren().contains(vst.getSESE());
+
+          if (case1 || case2) {
             // if the source comes from a child, copy it over
-            forAddition.add( new VariableSourceToken( ts, 
-                                                      vst.getSESE(), 
-                                                      vst.getAge(), 
-                                                      vst.getAddrVar()
-                                                      )
-                             );
+            forAddition.add(new VariableSourceToken(ts, vst.getSESE(), vst.getAge(), vst
+                                                    .getAddrVar()));
           } else {
             // otherwise, stamp it as us as the source
-            forAddition.add( new VariableSourceToken( ts, 
-                                                      currentSESE, 
-                                                      new Integer( 0 ), 
-                                                      lhs
-                                                      )
-                             );
+            forAddition.add(new VariableSourceToken(ts, currentSESE, new Integer(0), lhs));
           }
         }
 
-        vstTable.addAll( forAddition );
+        vstTable.addAll(forAddition);
 
         // only break if this is an ASSIGN op node,
         // otherwise fall through to default case
@@ -565,40 +578,35 @@ public class OoOJavaAnalysis {
       }
     }
 
-      // note that FlatOpNode's that aren't ASSIGN
-      // fall through to this default case
+    // note that FlatOpNode's that aren't ASSIGN
+    // fall through to this default case
     default: {
       TempDescriptor[] writeTemps = fn.writesTemps();
-      if( writeTemps.length > 0 ) {
+      if (writeTemps.length > 0) {
 
         // for now, when writeTemps > 1, make sure
         // its a call node, programmer enforce only
         // doing stuff like calling a print routine
-        if( writeTemps.length > 1 ) {
+        if (writeTemps.length > 1) {
           assert fn.kind() == FKind.FlatCall || fn.kind() == FKind.FlatMethod;
           break;
         }
 
-        vstTable.remove( writeTemps[0] );
+        vstTable.remove(writeTemps[0]);
 
         HashSet<TempDescriptor> ts = new HashSet<TempDescriptor>();
-        ts.add( writeTemps[0] );
-
-        vstTable.add( new VariableSourceToken( ts,
-                                               currentSESE, 
-                                               new Integer( 0 ), 
-                                               writeTemps[0]
-                                               )
-                      );
+        ts.add(writeTemps[0]);
+
+        vstTable.add(new VariableSourceToken(ts, currentSESE, new Integer(0), writeTemps[0]));
       }
 
       vstTable.assertConsistency();
-    } break;
+    }
+    break;
 
     } // end switch
   }
 
-
   private void notAvailableForward(FlatMethod fm) {
 
     Set<FlatNode> flatNodesToVisit = new HashSet<FlatNode>();
@@ -619,8 +627,8 @@ public class OoOJavaAnalysis {
         }
       }
 
-      FlatSESEEnterNode currentSESE = rblockRel.getLocalInnerRBlock( fn );
-      if( currentSESE == null ) {
+      FlatSESEEnterNode currentSESE = rblockRel.getLocalInnerRBlock(fn);
+      if (currentSESE == null) {
         currentSESE = rblockRel.getCallerProxySESE();
       }
 
@@ -638,10 +646,8 @@ public class OoOJavaAnalysis {
     }
   }
 
-  private void notAvailable_nodeActions(FlatNode            fn, 
-                                        Set<TempDescriptor> notAvailSet,
-                                        FlatSESEEnterNode   currentSESE
-                                        ) {
+  private void notAvailable_nodeActions(FlatNode fn, Set<TempDescriptor> notAvailSet,
+                                        FlatSESEEnterNode currentSESE) {
 
     // any temps that are removed from the not available set
     // at this node should be marked in this node's code plan
@@ -662,7 +668,8 @@ public class OoOJavaAnalysis {
       notAvailableIntoSESE.put(fsen, notAvailCopy);
 
       notAvailSet.clear();
-    } break;
+    }
+    break;
 
     case FKind.FlatSESEExitNode: {
       FlatSESEExitNode fsexn = (FlatSESEExitNode) fn;
@@ -673,11 +680,13 @@ public class OoOJavaAnalysis {
       Set<TempDescriptor> notAvailIn = notAvailableIntoSESE.get(fsen);
       assert notAvailIn != null;
       notAvailSet.addAll(notAvailIn);
-    } break;
+    }
+    break;
 
     case FKind.FlatMethod: {
       notAvailSet.clear();
-    } break;
+    }
+    break;
 
     case FKind.FlatOpNode: {
       FlatOpNode fon = (FlatOpNode) fn;
@@ -699,8 +708,8 @@ public class OoOJavaAnalysis {
       }
     }
 
-      // note that FlatOpNode's that aren't ASSIGN
-      // fall through to this default case
+    // note that FlatOpNode's that aren't ASSIGN
+    // fall through to this default case
     default: {
       TempDescriptor[] writeTemps = fn.writesTemps();
       for (int i = 0; i < writeTemps.length; i++) {
@@ -724,7 +733,7 @@ public class OoOJavaAnalysis {
           VariableSourceToken vst = vstIfStatic.vst;
 
           Iterator<VariableSourceToken> availItr =
-              vstTable.get(vst.getSESE(), vst.getAge()).iterator();
+            vstTable.get(vst.getSESE(), vst.getAge()).iterator();
 
           // look through things that are also available from same source
           while (availItr.hasNext()) {
@@ -738,7 +747,7 @@ public class OoOJavaAnalysis {
               // only comes from one statically known source, mark it available
               VSTWrapper vstIfStaticNotUsed = new VSTWrapper();
               Integer srcTypeAlso =
-                  vstTable.getRefVarSrcType(refVarAlso, currentSESE, vstIfStaticNotUsed);
+                vstTable.getRefVarSrcType(refVarAlso, currentSESE, vstIfStaticNotUsed);
               if (srcTypeAlso.equals(VarSrcTokTable.SrcType_STATIC)) {
                 notAvailSet.remove(refVarAlso);
               }
@@ -746,12 +755,12 @@ public class OoOJavaAnalysis {
           }
         }
       }
-    } break;
+    }
+    break;
 
     } // end switch
   }
 
-
   private void codePlansForward(FlatMethod fm) {
 
     // start from flat method top, visit every node in
@@ -788,14 +797,12 @@ public class OoOJavaAnalysis {
 
       Set<TempDescriptor> dotSTlive = livenessGlobalView.get(fn);
 
-      FlatSESEEnterNode currentSESE = rblockRel.getLocalInnerRBlock( fn );
-      if( currentSESE == null ) {
+      FlatSESEEnterNode currentSESE = rblockRel.getLocalInnerRBlock(fn);
+      if (currentSESE == null) {
         currentSESE = rblockRel.getCallerProxySESE();
       }
 
-      codePlans_nodeActions(fm, fn, 
-                            dotSTlive, dotSTtable, dotSTnotAvailSet, 
-                            currentSESE);
+      codePlans_nodeActions(fm, fn, dotSTtable, dotSTnotAvailSet, currentSESE);
 
       for (int i = 0; i < fn.numNext(); i++) {
         FlatNode nn = fn.getNext(i);
@@ -806,13 +813,9 @@ public class OoOJavaAnalysis {
       }
     }
   }
-      
-  private void codePlans_nodeActions(FlatMethod fm,
-                                     FlatNode fn,
-                                     Set<TempDescriptor> liveSetIn,
-                                     VarSrcTokTable vstTableIn, 
-                                     Set<TempDescriptor> notAvailSetIn, 
-                                     FlatSESEEnterNode currentSESE) {
+
+  private void codePlans_nodeActions(FlatMethod fm, FlatNode fn, VarSrcTokTable vstTableIn,
+                                     Set<TempDescriptor> notAvailSetIn, FlatSESEEnterNode currentSESE) {
 
     CodePlan plan = new CodePlan(currentSESE);
 
@@ -834,11 +837,11 @@ public class OoOJavaAnalysis {
         // in order to classify in-vars correctly, pass
         // the parent SESE in--at other FlatNode types just
         // use the currentSESE
-        FlatSESEEnterNode parent = rblockRel.getLocalInnerRBlock( fn );
-        if( parent == null ) {
+        FlatSESEEnterNode parent = rblockRel.getLocalInnerRBlock(fn);
+        if (parent == null) {
           parent = rblockRel.getCallerProxySESE();
         }
-                
+
         VSTWrapper vstIfStatic = new VSTWrapper();
         Integer srcType = vstTableIn.getRefVarSrcType(inVar, parent, vstIfStatic);
 
@@ -846,28 +849,57 @@ public class OoOJavaAnalysis {
         // variable and the child needs space in its SESE record
         if (srcType.equals(VarSrcTokTable.SrcType_DYNAMIC)) {
           fsen.addDynamicInVar(inVar);
-          addDynamicVar( fsen, fm, inVar );
+          addDynamicVar(parent, fm, inVar);
 
         } else if (srcType.equals(VarSrcTokTable.SrcType_STATIC)) {
           fsen.addStaticInVar(inVar);
           VariableSourceToken vst = vstIfStatic.vst;
           fsen.putStaticInVar2src(inVar, vst);
           fsen.addStaticInVarSrc(new SESEandAgePair(vst.getSESE(), vst.getAge()));
+
         } else {
           assert srcType.equals(VarSrcTokTable.SrcType_READY);
           fsen.addReadyInVar(inVar);
         }
       }
-    } break;
+    }
+    break;
 
     case FKind.FlatSESEExitNode: {
       FlatSESEExitNode fsexn = (FlatSESEExitNode) fn;
-      //TODO! Shouldn't there be a code plan for task exit
-      // where the exiting task calculates whether its own
-      // siblings need variables from its children, so the
-      // exiter should copy those variables into its own out-set
-      // and make the available?
-    } break;
+
+      // Classify the sources of out-set variables so code
+      // gen can acquire them from children if necessary
+      // before this task exits
+      FlatSESEEnterNode exiter = fsexn.getFlatEnter();
+
+      Iterator<TempDescriptor> outVarItr = exiter.getOutVarSet().iterator();
+      while (outVarItr.hasNext()) {
+        TempDescriptor outVar = outVarItr.next();
+
+        VSTWrapper vstIfStatic = new VSTWrapper();
+        Integer srcType = vstTableIn.getRefVarSrcType(outVar, exiter, vstIfStatic);
+
+        if (srcType.equals(VarSrcTokTable.SrcType_DYNAMIC)) {
+          // if the out-var is dynamic, put it in the set of dyn out vars
+          // so exiting code gen knows to look for the value, but also put
+          // it in the set of dynamic vars the exiter must track!
+          exiter.addDynamicOutVar(outVar);
+          addDynamicVar(exiter, fm, outVar);
+
+        } else if (srcType.equals(VarSrcTokTable.SrcType_STATIC)) {
+          exiter.addStaticOutVar(outVar);
+          VariableSourceToken vst = vstIfStatic.vst;
+          exiter.putStaticOutVar2src(outVar, vst);
+          exiter.addStaticOutVarSrc(new SESEandAgePair(vst.getSESE(), vst.getAge()));
+
+        } else {
+          assert srcType.equals(VarSrcTokTable.SrcType_READY);
+          exiter.addReadyOutVar(outVar);
+        }
+      }
+    }
+    break;
 
     case FKind.FlatOpNode: {
       FlatOpNode fon = (FlatOpNode) fn;
@@ -887,9 +919,9 @@ public class OoOJavaAnalysis {
         if (rhsSrcType.equals(VarSrcTokTable.SrcType_DYNAMIC)) {
           // if rhs is dynamic going in, lhs will definitely be dynamic
           // going out of this node, so track that here
-          plan.addDynAssign( lhs, rhs );
-          addDynamicVar( currentSESE, fm, lhs );
-          addDynamicVar( currentSESE, fm, rhs );
+          plan.addDynAssign(lhs, rhs);
+          addDynamicVar(currentSESE, fm, lhs);
+          addDynamicVar(currentSESE, fm, rhs);
 
         } else if (lhsSrcType.equals(VarSrcTokTable.SrcType_DYNAMIC)) {
           // otherwise, if the lhs is dynamic, but the rhs is not, we
@@ -903,14 +935,15 @@ public class OoOJavaAnalysis {
       }
     }
 
-      // note that FlatOpNode's that aren't ASSIGN
-      // fall through to this default case
+    // note that FlatOpNode's that aren't ASSIGN
+    // fall through to this default case
     default: {
 
       // a node with no live set has nothing to stall for
-      if (liveSetIn == null) {
-        break;
-      }
+      // note: no reason to check here, remove this....
+      // if (liveSetIn == null) {
+      // break;
+      // }
 
       TempDescriptor[] readarray = fn.readsTemps();
       for (int i = 0; i < readarray.length; i++) {
@@ -931,8 +964,8 @@ public class OoOJavaAnalysis {
           // come from, so dynamically we must keep track
           // along various control paths, and therefore when we stall,
           // just stall for the exact thing we need and move on
-          plan.addDynamicStall( readtmp );
-          addDynamicVar( currentSESE, fm, readtmp );
+          plan.addDynamicStall(readtmp);
+          addDynamicVar(currentSESE, fm, readtmp);
 
         } else if (srcType.equals(VarSrcTokTable.SrcType_STATIC)) {
           // 2) Single token/age pair: Stall for token/age pair, and copy
@@ -942,7 +975,7 @@ public class OoOJavaAnalysis {
           VariableSourceToken vst = vstIfStatic.vst;
 
           Iterator<VariableSourceToken> availItr =
-              vstTableIn.get(vst.getSESE(), vst.getAge()).iterator();
+            vstTableIn.get(vst.getSESE(), vst.getAge()).iterator();
 
           while (availItr.hasNext()) {
             VariableSourceToken vstAlsoAvail = availItr.next();
@@ -951,9 +984,12 @@ public class OoOJavaAnalysis {
             Set<TempDescriptor> copySet = new HashSet<TempDescriptor>();
 
             Iterator<TempDescriptor> refVarItr = vstAlsoAvail.getRefVars().iterator();
+
             while (refVarItr.hasNext()) {
               TempDescriptor refVar = refVarItr.next();
-              if (liveSetIn.contains(refVar)) {
+              // note: this should just use normal liveness in...only want to
+              // copy live variables...
+              if (liveness.getLiveInTemps(fm, fn).contains(refVar)) {
                 copySet.add(refVar);
               }
             }
@@ -974,7 +1010,7 @@ public class OoOJavaAnalysis {
 
       }
     }
-      break;
+    break;
 
     } // end switch
 
@@ -990,16 +1026,16 @@ public class OoOJavaAnalysis {
       // the caller proxy generates useful analysis facts, but we
       // never need to generate another name for it in code (it is
       // ALWAYS the task executing the local method context)
-      if( vst.getSESE().getIsCallerProxySESE() ) {
+      if (vst.getSESE().getIsCallerProxySESE()) {
         continue;
       }
 
-      SESEandAgePair sap = new SESEandAgePair( vst.getSESE(), vst.getAge() );
-      sap.getSESE().mustTrackAtLeastAge( sap.getAge() );
+      SESEandAgePair sap = new SESEandAgePair(vst.getSESE(), vst.getAge());
+      sap.getSESE().mustTrackAtLeastAge(sap.getAge());
 
       FlatSESEEnterNode sese = currentSESE;
-      while( sese != null ) {
-        addNeededStaticName( sese, fm, sap );      
+      while (sese != null) {
+        addNeededStaticName(sese, fm, sap);
         sese = sese.getLocalParent();
       }
     }
@@ -1010,18 +1046,45 @@ public class OoOJavaAnalysis {
     // vst)
     // but go to a dynamic source at next-node-*dot*, create a new IR graph
     // node on that edge to track the sources dynamically
+    // NOTE: for this calculation use the currentSESE variable, except when the
+    // FlatNode fn is an Exit--in that case currentSESE is for the exiting task,
+    // be we want to consider that the parent is tracking a variable coming out
+    // of the exiting task
+    FlatSESEEnterNode fsenDoingTracking;
+    if (fn instanceof FlatSESEExitNode) {
+      fsenDoingTracking = currentSESE.getLocalParent();
+
+      if (fsenDoingTracking == null) {
+        // if there is no local parent, there are one of two cases
+        // 1) the current task is main, in which case this FlatNode
+        // is the main's exit, and doesn't need to do any of the
+        // following dynamic tracking
+        // 2) the current task is defined in a method, so use the
+        // caller proxy in the variable source calcs below
+        if (currentSESE.equals(rblockRel.getMainSESE())) {
+          return;
+        } else {
+          fsenDoingTracking = rblockRel.getCallerProxySESE();
+        }
+      }
+    } else {
+      fsenDoingTracking = currentSESE;
+    }
+
     VarSrcTokTable thisVstTable = variableResults.get(fn);
     for (int i = 0; i < fn.numNext(); i++) {
       FlatNode nn = fn.getNext(i);
       VarSrcTokTable nextVstTable = variableResults.get(nn);
-      Set<TempDescriptor> nextLiveIn = livenessGlobalView.get(nn);
+      // note: using the result of liveness analysis regardless of task
+      // structures
+      Set<TempDescriptor> nextLiveIn = liveness.getLiveInTemps(fm, nn);
 
       // the table can be null if it is one of the few IR nodes
       // completely outside of the root SESE scope
       if (nextVstTable != null && nextLiveIn != null) {
 
         Hashtable<TempDescriptor, VSTWrapper> readyOrStatic2dynamicSet =
-            thisVstTable.getReadyOrStatic2DynamicSet(nextVstTable, nextLiveIn, currentSESE);
+          thisVstTable.getReadyOrStatic2DynamicSet(nextVstTable, nextLiveIn, fsenDoingTracking);
 
         if (!readyOrStatic2dynamicSet.isEmpty()) {
 
@@ -1031,7 +1094,8 @@ public class OoOJavaAnalysis {
           FlatWriteDynamicVarNode fwdvn = wdvNodesToSpliceIn.get(fe);
 
           if (fwdvn == null) {
-            fwdvn = new FlatWriteDynamicVarNode(fn, nn, readyOrStatic2dynamicSet, currentSESE);
+            fwdvn =
+              new FlatWriteDynamicVarNode(fn, nn, readyOrStatic2dynamicSet, fsenDoingTracking);
             wdvNodesToSpliceIn.put(fe, fwdvn);
           } else {
             fwdvn.addMoreVar2Src(readyOrStatic2dynamicSet);
@@ -1041,12 +1105,13 @@ public class OoOJavaAnalysis {
     }
   }
 
-  private void addDynamicVar( FlatSESEEnterNode fsen, 
-                              FlatMethod        fm, 
-                              TempDescriptor    var ) {
+  private void addDynamicVar(FlatSESEEnterNode fsen, FlatMethod fm, TempDescriptor var) {
     FlatNode fnContext;
 
-    if( fsen.getIsCallerProxySESE() ) {
+    // note: dynamic variable declarations are always located in the flat method
+    // that encloses task block
+    // there is no need to set fnContext to fsen
+    if (fsen.getIsCallerProxySESE()) {
       // attach the dynamic variable to track to
       // the flat method, so it can be declared at entry
       fnContext = fm;
@@ -1054,22 +1119,22 @@ public class OoOJavaAnalysis {
       // otherwise the code context is a task body
       fnContext = fsen;
     }
+    // fnContext=fm;
 
-    ContextTaskNames ctn = fn2contextTaskNames.get( fnContext );
-    if( ctn == null ) {
+    ContextTaskNames ctn = fn2contextTaskNames.get(fnContext);
+    if (ctn == null) {
       ctn = new ContextTaskNames();
     }
 
-    ctn.addDynamicVar( var );
-    fn2contextTaskNames.put( fnContext, ctn );
+    ctn.addDynamicVar(var);
+    fn2contextTaskNames.put(fnContext, ctn);
+
   }
 
-  private void addNeededStaticName( FlatSESEEnterNode fsen, 
-                                    FlatMethod        fm, 
-                                    SESEandAgePair    sap ) {
+  private void addNeededStaticName(FlatSESEEnterNode fsen, FlatMethod fm, SESEandAgePair sap) {
     FlatNode fnContext;
 
-    if( fsen.getIsCallerProxySESE() ) {
+    if (fsen.getIsCallerProxySESE()) {
       // attach the dynamic variable to track to
       // the flat method, so it can be declared at entry
       fnContext = fm;
@@ -1078,116 +1143,111 @@ public class OoOJavaAnalysis {
       fnContext = fsen;
     }
 
-    ContextTaskNames ctn = fn2contextTaskNames.get( fnContext );
-    if( ctn == null ) {
+    ContextTaskNames ctn = fn2contextTaskNames.get(fnContext);
+    if (ctn == null) {
       ctn = new ContextTaskNames();
     }
 
-    ctn.addNeededStaticName( sap );
+    ctn.addNeededStaticName(sap);
 
-    fn2contextTaskNames.put( fnContext, ctn );
+    fn2contextTaskNames.put(fnContext, ctn);
   }
 
-
   private void startConflictGraphs() {
 
     // first, for each task, consider whether it has any children, and if
     // effects analysis says they should be a conflict node in the that
     // parent's conflict graph
     Set<FlatSESEEnterNode> allSESEs = rblockRel.getAllSESEs();
-    forIterator iterator = allSESEs.iterator(); iterator.hasNext(); ) {
+    for (Iterator iterator = allSESEs.iterator(); iterator.hasNext(); ) {
 
       FlatSESEEnterNode parent = (FlatSESEEnterNode) iterator.next();
-      if( parent.getIsLeafSESE() ) {
+      if (parent.getIsLeafSESE()) {
         continue;
       }
 
       EffectsAnalysis effectsAnalysis = disjointAnalysisTaints.getEffectsAnalysis();
-      ConflictGraph conflictGraph = sese2conflictGraph.get( parent );
+      ConflictGraph conflictGraph = sese2conflictGraph.get(parent);
       assert conflictGraph == null;
-      conflictGraph = new ConflictGraph( state );
+      conflictGraph = new ConflictGraph(state);
 
       Set<FlatSESEEnterNode> children = parent.getChildren();
-      forIterator iterator2 = children.iterator(); iterator2.hasNext(); ) {
+      for (Iterator iterator2 = children.iterator(); iterator2.hasNext(); ) {
         FlatSESEEnterNode child = (FlatSESEEnterNode) iterator2.next();
-        Hashtable<Taint, Set<Effect>> taint2Effects = effectsAnalysis.get( child );
-        conflictGraph.addLiveIn( taint2Effects );
+        Hashtable<Taint, Set<Effect>> taint2Effects = effectsAnalysis.get(child);
+        conflictGraph.addLiveIn(taint2Effects);
       }
 
-      sese2conflictGraph.put( parent, conflictGraph );
+      sese2conflictGraph.put(parent, conflictGraph);
     }
 
     // then traverse all methods looking for potential stall sites, and
     // add those stall sites as nodes in any task's conflict graph that
     // might be executing at the point of the stall site
     Iterator<MethodDescriptor> descItr = descriptorsToAnalyze.iterator();
-    while( descItr.hasNext() ) {
+    while (descItr.hasNext()) {
       MethodDescriptor md = descItr.next();
-      FlatMethod       fm = state.getMethodFlat( md );
-      if( fm != null ) {
-        addStallSitesToConflictGraphs( fm );
+      FlatMethod fm = state.getMethodFlat(md);
+      if (fm != null) {
+        addStallSitesToConflictGraphs(fm);
       }
-    }    
+    }
   }
 
-  private void addStallSitesToConflictGraphs( FlatMethod fm ) {
+  private void addStallSitesToConflictGraphs(FlatMethod fm) {
 
     Set<FlatNode> flatNodesToVisit = new HashSet<FlatNode>();
-    flatNodesToVisit.add( fm );
+    flatNodesToVisit.add(fm);
 
     Set<FlatNode> visited = new HashSet<FlatNode>();
 
-    while( !flatNodesToVisit.isEmpty() ) {
+    while (!flatNodesToVisit.isEmpty()) {
       FlatNode fn = (FlatNode) flatNodesToVisit.iterator().next();
-      flatNodesToVisit.remove( fn );
-      visited.add( fn );
+      flatNodesToVisit.remove(fn);
+      visited.add(fn);
 
-      Set<FlatSESEEnterNode> currentSESEs = 
-        rblockRel.getPossibleExecutingRBlocks( fn );
+      Set<FlatSESEEnterNode> currentSESEs = rblockRel.getPossibleExecutingRBlocks(fn);
 
-      conflictGraph_nodeAction( fn, currentSESEs );
+      conflictGraph_nodeAction(fn, currentSESEs, fm.getMethod().getClassDesc());
 
       // schedule forward nodes for analysis
-      for( int i = 0; i < fn.numNext(); i++ ) {
-        FlatNode nn = fn.getNext( i );
-        if( !visited.contains( nn ) ) {
-          flatNodesToVisit.add( nn );
+      for (int i = 0; i < fn.numNext(); i++) {
+        FlatNode nn = fn.getNext(i);
+        if (!visited.contains(nn)) {
+          flatNodesToVisit.add(nn);
         }
       }
     }
   }
 
-  private void conflictGraph_nodeAction( FlatNode fn, 
-                                         Set<FlatSESEEnterNode> currentSESEs
-                                         ) {
+  private void conflictGraph_nodeAction(FlatNode fn, Set<FlatSESEEnterNode> currentSESEs,
+                                        ClassDescriptor cd) {
 
     EffectsAnalysis effectsAnalysis = disjointAnalysisTaints.getEffectsAnalysis();
 
-    Hashtable<Taint, Set<Effect>> taint2Effects = effectsAnalysis.get( fn );
-
+    Hashtable<Taint, Set<Effect>> taint2Effects = effectsAnalysis.get(fn);
 
     // repeat the process of adding a stall site to a conflict graph
     // for each task that might be executing at a possible stall site
     Iterator<FlatSESEEnterNode> seseItr = currentSESEs.iterator();
-    while( seseItr.hasNext() ) {
+    while (seseItr.hasNext()) {
       FlatSESEEnterNode currentSESE = seseItr.next();
 
-      ConflictGraph conflictGraph = sese2conflictGraph.get( currentSESE );
-      if( conflictGraph == null ) {
+      ConflictGraph conflictGraph = sese2conflictGraph.get(currentSESE);
+      if (conflictGraph == null) {
         assert currentSESE.getIsLeafSESE();
         continue;
       }
 
       TempDescriptor lhs;
       TempDescriptor rhs;
-   
-      switch( fn.kind() ) {
 
+      switch (fn.kind()) {
 
       case FKind.FlatFieldNode:
       case FKind.FlatElementNode: {
 
-        if( fn instanceof FlatFieldNode ) {
+        if (fn instanceof FlatFieldNode) {
           FlatFieldNode ffn = (FlatFieldNode) fn;
           rhs = ffn.getSrc();
         } else {
@@ -1195,14 +1255,14 @@ public class OoOJavaAnalysis {
           rhs = fen.getSrc();
         }
 
-        conflictGraph.addStallSite( taint2Effects, rhs );
-      } break;
-
+        conflictGraph.addStallSite(taint2Effects, rhs, cd);
+      }
+      break;
 
       case FKind.FlatSetFieldNode:
       case FKind.FlatSetElementNode: {
 
-        if( fn instanceof FlatSetFieldNode ) {
+        if (fn instanceof FlatSetFieldNode) {
           FlatSetFieldNode fsfn = (FlatSetFieldNode) fn;
           lhs = fsfn.getDst();
           rhs = fsfn.getSrc();
@@ -1212,29 +1272,28 @@ public class OoOJavaAnalysis {
           rhs = fsen.getSrc();
         }
 
-        conflictGraph.addStallSite( taint2Effects, rhs );
-        conflictGraph.addStallSite( taint2Effects, lhs );
-      } break;
-
+        conflictGraph.addStallSite(taint2Effects, rhs, cd);
+        conflictGraph.addStallSite(taint2Effects, lhs, cd);
+      }
+      break;
 
       case FKind.FlatCall: {
         FlatCall fc = (FlatCall) fn;
         lhs = fc.getThis();
 
-        conflictGraph.addStallSite( taint2Effects, lhs );
-      } break;
+        conflictGraph.addStallSite(taint2Effects, lhs, cd);
+      }
+      break;
 
       }
-      
-      if( conflictGraph.id2cn.size() > 0 ) {
-        sese2conflictGraph.put( currentSESE, conflictGraph );
+
+      if (conflictGraph.id2cn.size() > 0) {
+        sese2conflictGraph.put(currentSESE, conflictGraph);
       }
     }
   }
 
-
-  private void calculateConflicts( Set<FlatNew> sitesToFlag, 
-                                   boolean      useReachInfo ) {
+  private void calculateConflicts(Set<FlatNew> sitesToFlag, boolean useReachInfo) {
 
     // decide fine-grain edge or coarse-grain edge among all vertexes by
     // pair-wise comparison
@@ -1254,7 +1313,6 @@ public class OoOJavaAnalysis {
     }
   }
 
-
   private void writeConflictGraph() {
     Enumeration<FlatNode> keyEnum = sese2conflictGraph.keys();
     while (keyEnum.hasMoreElements()) {
@@ -1271,50 +1329,23 @@ public class OoOJavaAnalysis {
     }
   }
 
-
   // the traversal for pruning state machines and finding
   // machines that are weakly connected BOTH consider conflicting
   // effects between heap roots, so it is smart to compute all of
   // this together
   public void pruneMachinesAndFindWeaklyConnectedExaminers() {
-
-    /*
-    // TODO, calcualte the set of taints that lead to conflicts (for which
-    // traversers must be built...)
-
-    EffectsAnalysis effectsAnalysis = disjointAnalysisTaints.getEffectsAnalysis();
-
-    // visit every conflict graph once, so iterate through the
-    // the non-leaf tasks to find them all
-    Set<FlatSESEEnterNode> allSESEs = rblockRel.getAllSESEs();
-    for( Iterator allItr = allSESEs.iterator(); allItr.hasNext(); ) {
-      
-      FlatSESEEnterNode parent = (FlatSESEEnterNode) allItr.next();
-      if( parent.getIsLeafSESE() ) {
-        continue;
-      }
-      
-      ConflictGraph conflictGraph = sese2conflictGraph.get( parent );
-      assert conflictGraph != null;
-      
-      // from the conflict graph we want to extract all conflicting effects
-      // and use them to identify (1) weakly connected heap examiners and
-      // (2) states/examiner nodes with a conflicting effect that will later
-      // support the examiner pruning process
-      Hashtable<Taint, Set<Effect>> conflicts = conflictGraph.getConflictEffectSet( fsen ) );
-      
-    }
-    */
+    ProcessStateMachines psm = new ProcessStateMachines(buildStateMachines, rblockRel);
+    psm.doProcess();
+    buildStateMachines.writeStateMachines("pruned");
   }
 
-
-
   private void synthesizeLocks() {
     // for every conflict graph, generate a set of memory queues
     // (called SESELock in this code!) to cover the graph
     Set<Map.Entry<FlatNode, ConflictGraph>> graphEntrySet = sese2conflictGraph.entrySet();
-    for (Iterator iterator = graphEntrySet.iterator(); iterator.hasNext();) {
-      Map.Entry<FlatNode, ConflictGraph> graphEntry = (Map.Entry<FlatNode, ConflictGraph>) iterator.next();
+    for (Iterator iterator = graphEntrySet.iterator(); iterator.hasNext(); ) {
+      Map.Entry<FlatNode, ConflictGraph> graphEntry =
+        (Map.Entry<FlatNode, ConflictGraph>)iterator.next();
       FlatNode sese = graphEntry.getKey();
       ConflictGraph conflictGraph = graphEntry.getValue();
       calculateCovering(conflictGraph);
@@ -1328,7 +1359,7 @@ public class OoOJavaAnalysis {
     HashSet<SESELock> lockSet = new HashSet<SESELock>();
 
     Set<ConflictEdge> tempCover = conflictGraph.getEdgeSet();
-    for (Iterator iterator = tempCover.iterator(); iterator.hasNext();) {
+    for (Iterator iterator = tempCover.iterator(); iterator.hasNext(); ) {
       ConflictEdge conflictEdge = (ConflictEdge) iterator.next();
       if (conflictEdge.isCoarseEdge()) {
         coarseToCover.add(conflictEdge);
@@ -1352,7 +1383,7 @@ public class OoOJavaAnalysis {
 
         changed = false;
 
-        for (Iterator iterator = fineToCover.iterator(); iterator.hasNext();) {
+        for (Iterator iterator = fineToCover.iterator(); iterator.hasNext(); ) {
 
           int type;
           ConflictEdge edge = (ConflictEdge) iterator.next();
@@ -1397,8 +1428,8 @@ public class OoOJavaAnalysis {
             changed = true;
             seseLock.addConflictEdge(edge);
             fineToCover.remove(edge);
-            break;// exit iterator loop
-          }// end of initial setup
+            break; // exit iterator loop
+          } // end of initial setup
 
           ConflictNode newNode;
           if ((newNode = seseLock.getNewNodeConnectedWithGroup(edge)) != null) {
@@ -1433,7 +1464,7 @@ public class OoOJavaAnalysis {
 
             seseLock.addEdge(edge);
             Set<ConflictEdge> edgeSet = newNode.getEdgeSet();
-            for (Iterator iterator2 = edgeSet.iterator(); iterator2.hasNext();) {
+            for (Iterator iterator2 = edgeSet.iterator(); iterator2.hasNext(); ) {
               ConflictEdge conflictEdge = (ConflictEdge) iterator2.next();
 
               // mark all fine edges between new node and nodes in the group as
@@ -1454,16 +1485,16 @@ public class OoOJavaAnalysis {
 
             }
 
-            break;// exit iterator loop
+            break; // exit iterator loop
           }
         }
 
       } while (changed);
-      HashSet<ConflictEdge> notCovered=new HashSet<ConflictEdge>();
+      HashSet<ConflictEdge> notCovered = new HashSet<ConflictEdge>();
       do { // coarse
         changed = false;
         int type;
-        for (Iterator iterator = coarseToCover.iterator(); iterator.hasNext();) {
+        for (Iterator iterator = coarseToCover.iterator(); iterator.hasNext(); ) {
 
           ConflictEdge edge = (ConflictEdge) iterator.next();
           if (seseLock.getConflictNodeSet().size() == 0) {
@@ -1474,18 +1505,18 @@ public class OoOJavaAnalysis {
                 // and it is not parent
                 type = ConflictNode.SCC;
               } else {
-                if(state.RCR){
+                if (state.RCR) {
                   type = ConflictNode.PARENT_COARSE;
-                }else{
+                } else {
                   type = ConflictNode.PARENT_WRITE;
                 }
               }
               seseLock.addConflictNode(edge.getVertexU(), type);
             } else {
               if (edge.getVertexU().isStallSiteNode()) {
-                if(state.RCR){
+                if (state.RCR) {
                   type = ConflictNode.PARENT_COARSE;
-                }else{
+                } else {
                   if (edge.getVertexU().getWriteEffectSet().isEmpty()) {
                     type = ConflictNode.PARENT_READ;
                   } else {
@@ -1503,18 +1534,18 @@ public class OoOJavaAnalysis {
                 // and it is not parent
                 type = ConflictNode.SCC;
               } else {
-                if(state.RCR){
+                if (state.RCR) {
                   type = ConflictNode.PARENT_COARSE;
-                }else{
+                } else {
                   type = ConflictNode.PARENT_WRITE;
                 }
               }
               seseLock.addConflictNode(edge.getVertexV(), type);
             } else {
               if (edge.getVertexV().isStallSiteNode()) {
-                if(state.RCR){
+                if (state.RCR) {
                   type = ConflictNode.PARENT_COARSE;
-                }else{
+                } else {
                   if (edge.getVertexV().getWriteEffectSet().isEmpty()) {
                     type = ConflictNode.PARENT_READ;
                   } else {
@@ -1529,15 +1560,15 @@ public class OoOJavaAnalysis {
             changed = true;
             coarseToCover.remove(edge);
             seseLock.addConflictEdge(edge);
-            break;// exit iterator loop
-          }// end of initial setup
+            break; // exit iterator loop
+          } // end of initial setup
 
           ConflictNode newNode;
           if ((newNode = seseLock.getNewNodeConnectedWithGroup(edge)) != null) {
             // new node has a coarse-grained edge to all fine-read, fine-write,
             // parent
             changed = true;
-            
+
             if (newNode.isInVarNode() && (!seseLock.hasSelfCoarseEdge(newNode))
                 && seseLock.hasCoarseEdgeWithParentCoarse(newNode)) {
               // this case can't be covered by this queue
@@ -1551,7 +1582,7 @@ public class OoOJavaAnalysis {
               coarseToCover.remove(edge);
               break;
             }
-            
+
             if (seseLock.hasSelfCoarseEdge(newNode)) {
               // SCC
               if (newNode.isStallSiteNode()) {
@@ -1571,7 +1602,7 @@ public class OoOJavaAnalysis {
 
             seseLock.addEdge(edge);
             Set<ConflictEdge> edgeSet = newNode.getEdgeSet();
-            for (Iterator iterator2 = edgeSet.iterator(); iterator2.hasNext();) {
+            for (Iterator iterator2 = edgeSet.iterator(); iterator2.hasNext(); ) {
               ConflictEdge conflictEdge = (ConflictEdge) iterator2.next();
               // mark all coarse edges between new node and nodes in the group
               // as covered
@@ -1590,7 +1621,7 @@ public class OoOJavaAnalysis {
               }
 
             }
-            break;// exit iterator loop
+            break; // exit iterator loop
           }
 
         }
@@ -1628,11 +1659,10 @@ public class OoOJavaAnalysis {
     return rblockRel.getCallerProxySESE();
   }
 
-  public Set<FlatSESEEnterNode> getPossibleExecutingRBlocks( FlatNode fn ) {
-    return rblockRel.getPossibleExecutingRBlocks( fn );
+  public Set<FlatSESEEnterNode> getPossibleExecutingRBlocks(FlatNode fn) {
+    return rblockRel.getPossibleExecutingRBlocks(fn);
   }
 
-
   public void writeReports(String timeReport) throws java.io.IOException {
 
     BufferedWriter bw = new BufferedWriter(new FileWriter("ooojReport_summary.txt"));
@@ -1648,17 +1678,18 @@ public class OoOJavaAnalysis {
       MethodDescriptor md = methItr.next();
       FlatMethod fm = state.getMethodFlat(md);
       if (fm != null) {
-        bw = new BufferedWriter(new FileWriter("ooojReport_" + 
-                                               md.getClassMethodName() +
-                                               md.getSafeMethodDescriptor() + 
-                                               ".txt"));
+        bw =
+          new BufferedWriter(new FileWriter("ooojReport_" + md.getClassMethodName()
+                                            + md.getSafeMethodDescriptor() + ".txt"));
         bw.write("OoOJava Results for " + md + "\n-------------------\n");
 
-        bw.write("Dynamic vars to manage:\n  " + getContextTaskNames( fm ).getDynamicVarSet() );
+        bw.write("Dynamic vars to manage:\n  " + getContextTaskNames(fm).getDynamicVarSet());
 
-        bw.write("\n\nLive-In, Root View\n------------------\n" + fm.printMethod(livenessGlobalView));
+        bw.write("\n\nLive-In, Root View\n------------------\n"
+                 + fm.printMethod(livenessGlobalView));
         bw.write("\n\nVariable Results-Out\n----------------\n" + fm.printMethod(variableResults));
-        bw.write("\n\nNot Available Results-Out\n---------------------\n" + fm.printMethod(notAvailableResults));
+        bw.write("\n\nNot Available Results-Out\n---------------------\n"
+                 + fm.printMethod(notAvailableResults));
         bw.write("\n\nCode Plans\n----------\n" + fm.printMethod(codePlans));
         bw.close();
       }
@@ -1670,14 +1701,12 @@ public class OoOJavaAnalysis {
     Iterator<FlatSESEEnterNode> rootItr = rblockRel.getLocalRootSESEs().iterator();
     while (rootItr.hasNext()) {
       FlatSESEEnterNode root = rootItr.next();
-      printSESEHierarchyTree(bw, root, 0);      
+      printSESEHierarchyTree(bw, root, 0);
     }
   }
 
-  private void printSESEHierarchyTree(BufferedWriter    bw, 
-                                      FlatSESEEnterNode fsen, 
-                                      int               depth
-                                      ) throws java.io.IOException {
+  private void printSESEHierarchyTree(BufferedWriter bw, FlatSESEEnterNode fsen, int depth)
+  throws java.io.IOException {
     for (int i = 0; i < depth; ++i) {
       bw.write("  ");
     }
@@ -1693,11 +1722,11 @@ public class OoOJavaAnalysis {
   private void printSESEInfo(BufferedWriter bw) throws java.io.IOException {
     bw.write("\nSESE info\n-------------\n");
     Iterator<FlatSESEEnterNode> fsenItr = rblockRel.getAllSESEs().iterator();
-    while( fsenItr.hasNext() ) {
+    while (fsenItr.hasNext()) {
       FlatSESEEnterNode fsen = fsenItr.next();
 
       bw.write("SESE " + fsen.getPrettyIdentifier());
-      if( fsen.getIsLeafSESE() ) {
+      if (fsen.getIsLeafSESE()) {
         bw.write(" (leaf)");
       }
       bw.write(" {\n");
@@ -1717,9 +1746,22 @@ public class OoOJavaAnalysis {
         }
       }
 
-      bw.write("   Dynamic vars to manage: " + getContextTaskNames( fsen ).getDynamicVarSet() + "\n");
+      bw.write("   Dynamic vars to manage: " + getContextTaskNames(fsen).getDynamicVarSet() + "\n");
 
       bw.write("  out-set: " + fsen.getOutVarSet() + "\n");
+      tItr = fsen.getOutVarSet().iterator();
+      while (tItr.hasNext()) {
+        TempDescriptor outVar = tItr.next();
+        if (fsen.getReadyOutVarSet().contains(outVar)) {
+          bw.write("    (ready)  " + outVar + "\n");
+        }
+        if (fsen.getStaticOutVarSet().contains(outVar)) {
+          bw.write("    (static) " + outVar + " from " + fsen.getStaticOutVarSrc(outVar) + "\n");
+        }
+        if (fsen.getDynamicOutVarSet().contains(outVar)) {
+          bw.write("    (dynamic)" + outVar + "\n");
+        }
+      }
 
       bw.write("  local parent:   " + fsen.getLocalParent() + "\n");
       bw.write("  local children: " + fsen.getLocalChildren() + "\n");