changes.
[IRC.git] / Robust / src / Analysis / SSJava / LocationInference.java
index 4d87afb743ed2b532ec7e6db72e009947fc9a06b..d28772bb818b1eefdf14f4237788cdda80aeb438 100644 (file)
@@ -46,6 +46,7 @@ import IR.Tree.SubBlockNode;
 import IR.Tree.SwitchStatementNode;
 import IR.Tree.TertiaryNode;
 import IR.Tree.TreeNode;
+import Util.Pair;
 
 public class LocationInference {
 
@@ -72,13 +73,13 @@ public class LocationInference {
   // invoked by the method descriptor
   private Map<MethodDescriptor, Set<MethodInvokeNode>> mapMethodDescriptorToMethodInvokeNodeSet;
 
-  private Map<MethodInvokeNode, Map<Integer, NTuple<Descriptor>>> mapMethodInvokeNodeToArgIdxMap;
+  private Map<MethodInvokeNode, Map<Integer, NodeTupleSet>> mapMethodInvokeNodeToArgIdxMap;
 
   private Map<MethodDescriptor, MethodLocationInfo> mapMethodDescToMethodLocationInfo;
 
   private Map<ClassDescriptor, LocationInfo> mapClassToLocationInfo;
 
-  private Map<MethodDescriptor, Set<MethodDescriptor>> mapMethodDescToPossibleMethodDescSet;
+  private Map<MethodDescriptor, Set<MethodDescriptor>> mapMethodToCalleeSet;
 
   public static final String GLOBALLOC = "GLOBALLOC";
 
@@ -88,6 +89,8 @@ public class LocationInference {
 
   public static final Descriptor TOPDESC = new NameDescriptor(TOPLOC);
 
+  LocationInfo curMethodInfo;
+
   boolean debug = true;
 
   public LocationInference(SSJavaAnalysis ssjava, State state) {
@@ -102,10 +105,9 @@ public class LocationInference {
     this.mapMethodDescriptorToMethodInvokeNodeSet =
         new HashMap<MethodDescriptor, Set<MethodInvokeNode>>();
     this.mapMethodInvokeNodeToArgIdxMap =
-        new HashMap<MethodInvokeNode, Map<Integer, NTuple<Descriptor>>>();
+        new HashMap<MethodInvokeNode, Map<Integer, NodeTupleSet>>();
     this.mapMethodDescToMethodLocationInfo = new HashMap<MethodDescriptor, MethodLocationInfo>();
-    this.mapMethodDescToPossibleMethodDescSet =
-        new HashMap<MethodDescriptor, Set<MethodDescriptor>>();
+    this.mapMethodToCalleeSet = new HashMap<MethodDescriptor, Set<MethodDescriptor>>();
     this.mapClassToLocationInfo = new HashMap<ClassDescriptor, LocationInfo>();
   }
 
@@ -272,7 +274,7 @@ public class LocationInference {
     // dependency in the call graph
     methodDescriptorsToVisitStack.clear();
 
-    descriptorListToAnalyze.removeFirst();
+    // descriptorListToAnalyze.removeFirst();
 
     Set<MethodDescriptor> methodDescriptorToVistSet = new HashSet<MethodDescriptor>();
     methodDescriptorToVistSet.addAll(descriptorListToAnalyze);
@@ -291,11 +293,16 @@ public class LocationInference {
           new SSJavaLattice<String>(SSJavaAnalysis.TOP, SSJavaAnalysis.BOTTOM);
 
       MethodLocationInfo methodInfo = new MethodLocationInfo(md);
+      curMethodInfo = methodInfo;
 
       System.out.println();
       System.out.println("SSJAVA: Inferencing the lattice from " + md);
 
-      analyzeMethodLattice(md, methodLattice, methodInfo);
+      try {
+        analyzeMethodLattice(md, methodLattice, methodInfo);
+      } catch (CyclicFlowException e) {
+        throw new Error("Fail to generate the method lattice for " + md);
+      }
 
       SSJavaLattice<String> prevMethodLattice = getMethodLattice(md);
       MethodLocationInfo prevMethodInfo = getMethodLocationInfo(md);
@@ -395,9 +402,9 @@ public class LocationInference {
         if (i != k) {
           CompositeLocation locB1 = list1.get(k);
           CompositeLocation locB2 = list2.get(k);
-          boolean r1 = isGreaterThan(locA1, locB1);
+          boolean r1 = isGreaterThan(getLattice(md1), locA1, locB1);
 
-          boolean r2 = isGreaterThan(locA2, locB2);
+          boolean r2 = isGreaterThan(getLattice(md1), locA2, locB2);
 
           if (r1 != r2) {
             throw new Error("The method " + md1 + " is not consistent with the method " + md2
@@ -421,18 +428,22 @@ public class LocationInference {
   }
 
   private void analyzeMethodLattice(MethodDescriptor md, SSJavaLattice<String> methodLattice,
-      MethodLocationInfo methodInfo) {
+      MethodLocationInfo methodInfo) throws CyclicFlowException {
 
     // first take a look at method invocation nodes to newly added relations
     // from the callee
-    analyzeLatticeMethodInvocationNode(md);
+    analyzeLatticeMethodInvocationNode(md, methodLattice, methodInfo);
 
-    // set the this location
-    String thisLocSymbol = md.getThis().getSymbol();
-    methodInfo.setThisLocName(thisLocSymbol);
+    if (!md.isStatic()) {
+      // set the this location
+      String thisLocSymbol = md.getThis().getSymbol();
+      methodInfo.setThisLocName(thisLocSymbol);
+    }
 
     // set the global location
     methodInfo.setGlobalLocName(LocationInference.GLOBALLOC);
+    methodInfo.mapDescriptorToLocation(GLOBALDESC, new CompositeLocation(
+        new Location(md, GLOBALLOC)));
 
     // visit each node of method flow graph
     FlowGraph fg = getFlowGraph(md);
@@ -458,9 +469,17 @@ public class LocationInference {
               && srcNodeTuple.get(0).equals(dstNodeTuple.get(0))) {
 
             // value flows between fields
-            VarDescriptor varDesc = (VarDescriptor) srcNodeTuple.get(0);
-            ClassDescriptor varClassDesc = varDesc.getType().getClassDesc();
-            extractRelationFromFieldFlows(varClassDesc, srcNode, dstNode, 1);
+            Descriptor desc = srcNodeTuple.get(0);
+            ClassDescriptor classDesc;
+
+            if (desc.equals(GLOBALDESC)) {
+              classDesc = md.getClassDesc();
+            } else {
+              VarDescriptor varDesc = (VarDescriptor) srcNodeTuple.get(0);
+              classDesc = varDesc.getType().getClassDesc();
+            }
+
+            extractRelationFromFieldFlows(classDesc, srcNode, dstNode, 1);
 
           } else if (srcNodeTuple.size() == 1 || dstNodeTuple.size() == 1) {
             // for the method lattice, we need to look at the first element of
@@ -517,52 +536,29 @@ public class LocationInference {
     }
 
     // calculate a return location
+    // the return location type is lower than all parameters
     if (!md.getReturnType().isVoid()) {
-      Set<FlowNode> returnNodeSet = fg.getReturnNodeSet();
-      Set<String> returnVarSymbolSet = new HashSet<String>();
-
-      for (Iterator iterator = returnNodeSet.iterator(); iterator.hasNext();) {
-        FlowNode rtrNode = (FlowNode) iterator.next();
-        String localSymbol =
-            methodInfo.getInferLocation(rtrNode.getDescTuple().get(0)).get(0).getLocIdentifier();
-        returnVarSymbolSet.add(localSymbol);
-      }
 
-      String returnGLB = methodLattice.getGLB(returnVarSymbolSet);
-      if (returnGLB.equals(SSJavaAnalysis.BOTTOM)) {
-        // need to insert a new location in-between the bottom and all
-        // locations
-        // that is directly connected to the bottom
-        String returnNewLocationSymbol = "Loc" + (SSJavaLattice.seed++);
-        methodLattice.insertNewLocationAtOneLevelHigher(returnGLB, returnNewLocationSymbol);
-        methodInfo.setReturnLocName(returnNewLocationSymbol);
-      } else {
-        methodInfo.setReturnLocName(returnGLB);
-      }
-    }
+      String returnLocSymbol = "RETURNLOC";
 
-  }
-
-  private CompositeLocation getHighestLocation(Collection<CompositeLocation> locSet) {
-
-    Iterator<CompositeLocation> locIter = locSet.iterator();
-
-    CompositeLocation highest = locIter.next();
-
-    for (; locIter.hasNext();) {
-      CompositeLocation loc = (CompositeLocation) locIter.next();
-      if (isGreaterThan(loc, highest)) {
-        highest = loc;
+      for (Iterator iterator = keySet.iterator(); iterator.hasNext();) {
+        Integer paramIdx = (Integer) iterator.next();
+        CompositeLocation inferLoc = mapParamToLoc.get(paramIdx);
+        String paramLocLocalSymbol = inferLoc.get(0).getLocIdentifier();
+        if (!methodLattice.isGreaterThan(paramLocLocalSymbol, returnLocSymbol)) {
+          addRelationHigherToLower(methodLattice, methodInfo, paramLocLocalSymbol, returnLocSymbol);
+        }
       }
     }
 
-    return highest;
-
   }
 
-  private boolean isGreaterThan(CompositeLocation comp1, CompositeLocation comp2) {
+  private boolean isGreaterThan(SSJavaLattice<String> methodLattice, CompositeLocation comp1,
+      CompositeLocation comp2) {
 
-    for (int idx = 0; idx < comp1.getSize(); idx++) {
+    int size = comp1.getSize() >= comp2.getSize() ? comp2.getSize() : comp1.getSize();
+
+    for (int idx = 0; idx < size; idx++) {
       Location loc1 = comp1.get(idx);
       Location loc2 = comp2.get(idx);
 
@@ -576,9 +572,15 @@ public class LocationInference {
       String symbol1 = loc1.getLocIdentifier();
       String symbol2 = loc2.getLocIdentifier();
 
+      SSJavaLattice<String> lattice;
+      if (idx == 0) {
+        lattice = methodLattice;
+      } else {
+        lattice = getLattice(desc1);
+      }
       if (symbol1.equals(symbol2)) {
         continue;
-      } else if (getLattice(desc1).isGreaterThan(symbol1, symbol2)) {
+      } else if (lattice.isGreaterThan(symbol1, symbol2)) {
         return true;
       } else {
         return false;
@@ -590,7 +592,7 @@ public class LocationInference {
   }
 
   private void recursiveAddRelationToLattice(int idx, MethodDescriptor md,
-      CompositeLocation srcInferLoc, CompositeLocation dstInferLoc) {
+      CompositeLocation srcInferLoc, CompositeLocation dstInferLoc) throws CyclicFlowException {
 
     String srcLocSymbol = srcInferLoc.get(idx).getLocIdentifier();
     String dstLocSymbol = dstInferLoc.get(idx).getLocIdentifier();
@@ -608,7 +610,9 @@ public class LocationInference {
 
   }
 
-  private void analyzeLatticeMethodInvocationNode(MethodDescriptor mdCaller) {
+  private void analyzeLatticeMethodInvocationNode(MethodDescriptor mdCaller,
+      SSJavaLattice<String> methodLattice, MethodLocationInfo methodInfo)
+      throws CyclicFlowException {
 
     // the transformation for a call site propagates all relations between
     // parameters from the callee
@@ -617,6 +621,7 @@ public class LocationInference {
 
     Set<MethodInvokeNode> setMethodInvokeNode =
         mapMethodDescriptorToMethodInvokeNodeSet.get(mdCaller);
+
     if (setMethodInvokeNode != null) {
 
       for (Iterator iterator = setMethodInvokeNode.iterator(); iterator.hasNext();) {
@@ -626,13 +631,15 @@ public class LocationInference {
         if (mdCallee.isStatic()) {
           setPossibleCallees.add(mdCallee);
         } else {
-          setPossibleCallees.addAll(ssjava.getCallGraph().getMethods(mdCallee));
+          Set<MethodDescriptor> calleeSet = ssjava.getCallGraph().getMethods(mdCallee);
+          // removes method descriptors that are not invoked by the caller
+          calleeSet.retainAll(mapMethodToCalleeSet.get(mdCaller));
+          setPossibleCallees.addAll(calleeSet);
         }
 
-        System.out.println("mdCaller=" + mdCaller + " setPossibleCallees=" + setPossibleCallees);
         for (Iterator iterator2 = setPossibleCallees.iterator(); iterator2.hasNext();) {
           MethodDescriptor possibleMdCallee = (MethodDescriptor) iterator2.next();
-          propagateRelationToCaller(min, mdCaller, possibleMdCallee);
+          propagateRelationToCaller(min, mdCaller, possibleMdCallee, methodLattice, methodInfo);
         }
 
       }
@@ -641,56 +648,138 @@ public class LocationInference {
   }
 
   private void propagateRelationToCaller(MethodInvokeNode min, MethodDescriptor mdCaller,
-      MethodDescriptor possibleMdCallee) {
+      MethodDescriptor possibleMdCallee, SSJavaLattice<String> methodLattice,
+      MethodLocationInfo methodInfo) throws CyclicFlowException {
 
     SSJavaLattice<String> calleeLattice = getMethodLattice(possibleMdCallee);
-
+    MethodLocationInfo calleeLocInfo = getMethodLocationInfo(possibleMdCallee);
     FlowGraph calleeFlowGraph = getFlowGraph(possibleMdCallee);
 
-    System.out.println("calleeFlowGraph=" + calleeFlowGraph + " of " + possibleMdCallee);
-    // find parameter node
-    Set<FlowNode> paramNodeSet = calleeFlowGraph.getParameterNodeSet();
-
-    for (Iterator iterator = paramNodeSet.iterator(); iterator.hasNext();) {
-      FlowNode paramFlowNode1 = (FlowNode) iterator.next();
-
-      for (Iterator iterator2 = paramNodeSet.iterator(); iterator2.hasNext();) {
-        FlowNode paramFlowNode2 = (FlowNode) iterator2.next();
-
-        String paramSymbol1 = getSymbol(0, paramFlowNode1);
-        String paramSymbol2 = getSymbol(0, paramFlowNode2);
-        // if two parameters have a relation, we need to propagate this relation
-        // to the caller
-        if (!(paramSymbol1.equals(paramSymbol2))
-            && calleeLattice.isComparable(paramSymbol1, paramSymbol2)) {
-          int higherLocIdxCallee;
-          int lowerLocIdxCallee;
-          if (calleeLattice.isGreaterThan(paramSymbol1, paramSymbol2)) {
-            higherLocIdxCallee = calleeFlowGraph.getParamIdx(paramFlowNode1.getDescTuple());
-            lowerLocIdxCallee = calleeFlowGraph.getParamIdx(paramFlowNode2.getDescTuple());
-          } else {
-            higherLocIdxCallee = calleeFlowGraph.getParamIdx(paramFlowNode2.getDescTuple());
-            lowerLocIdxCallee = calleeFlowGraph.getParamIdx(paramFlowNode1.getDescTuple());
-          }
+    int numParam = calleeLocInfo.getNumParam();
+    for (int i = 0; i < numParam; i++) {
+      CompositeLocation param1 = calleeLocInfo.getParamCompositeLocation(i);
+      for (int k = 0; k < numParam; k++) {
+        if (i != k) {
+          CompositeLocation param2 = calleeLocInfo.getParamCompositeLocation(k);
+          if (isGreaterThan(getLattice(possibleMdCallee), param1, param2)) {
+            NodeTupleSet argDescTupleSet1 = getNodeTupleSetByArgIdx(min, i);
+            NodeTupleSet argDescTupleSet2 = getNodeTupleSetByArgIdx(min, k);
 
-          NTuple<Descriptor> higherArg = getArgTupleByArgIdx(min, higherLocIdxCallee);
-          NTuple<Descriptor> lowerArg = getArgTupleByArgIdx(min, lowerLocIdxCallee);
+            // the callee has the relation in which param1 is higher than param2
+            // therefore, the caller has to have the relation in which arg1 is
+            // higher than arg2
 
-          if (higherArg != null && lowerArg != null) {
-            // if the argument has the TOP location, getArgTupleByArgIdx returns
-            // null
-            addFlowGraphEdge(mdCaller, higherArg, lowerArg);
-          }
+            for (Iterator<NTuple<Descriptor>> iterator = argDescTupleSet1.iterator(); iterator
+                .hasNext();) {
+              NTuple<Descriptor> argDescTuple1 = iterator.next();
+
+              for (Iterator<NTuple<Descriptor>> iterator2 = argDescTupleSet2.iterator(); iterator2
+                  .hasNext();) {
+                NTuple<Descriptor> argDescTuple2 = iterator2.next();
+
+                // retreive inferred location by the local var descriptor
+
+                NTuple<Location> tuple1 = getFlowGraph(mdCaller).getLocationTuple(argDescTuple1);
+                NTuple<Location> tuple2 = getFlowGraph(mdCaller).getLocationTuple(argDescTuple2);
 
+                // CompositeLocation higherInferLoc =
+                // methodInfo.getInferLocation(argTuple1.get(0));
+                // CompositeLocation lowerInferLoc =
+                // methodInfo.getInferLocation(argTuple2.get(0));
+
+                CompositeLocation inferLoc1 = generateInferredCompositeLocation(methodInfo, tuple1);
+                CompositeLocation inferLoc2 = generateInferredCompositeLocation(methodInfo, tuple2);
+
+                // addRelation(methodLattice, methodInfo, inferLoc1, inferLoc2);
+
+                addFlowGraphEdge(mdCaller, argDescTuple1, argDescTuple2);
+
+              }
+
+            }
+
+          }
         }
+      }
+    }
+
+  }
+
+  private CompositeLocation generateInferredCompositeLocation(MethodLocationInfo methodInfo,
+      NTuple<Location> tuple) {
+
+    // System.out.println("@@@@@generateInferredCompositeLocation=" + tuple);
+    // System.out.println("generateInferredCompositeLocation=" + tuple + "   0="
+    // + tuple.get(0).getLocDescriptor());
+    // first, retrieve inferred location by the local var descriptor
+    CompositeLocation inferLoc = new CompositeLocation();
+
+    CompositeLocation localVarInferLoc =
+        methodInfo.getInferLocation(tuple.get(0).getLocDescriptor());
+
+    localVarInferLoc.get(0).setLocDescriptor(tuple.get(0).getLocDescriptor());
+
+    for (int i = 0; i < localVarInferLoc.getSize(); i++) {
+      inferLoc.addLocation(localVarInferLoc.get(i));
+    }
+    // System.out.println("@@@@@localVarInferLoc=" + localVarInferLoc);
+
+    for (int i = 1; i < tuple.size(); i++) {
+      Location cur = tuple.get(i);
+      Descriptor enclosingDesc = cur.getDescriptor();
+      Descriptor curDesc = cur.getLocDescriptor();
 
+      Location inferLocElement;
+      if (curDesc == null) {
+        // in this case, we have a newly generated location.
+        // System.out.println("!!! generated location=" +
+        // cur.getLocIdentifier());
+        inferLocElement = new Location(enclosingDesc, cur.getLocIdentifier());
+      } else {
+        String fieldLocSymbol =
+            getLocationInfo(enclosingDesc).getInferLocation(curDesc).get(0).getLocIdentifier();
+        inferLocElement = new Location(enclosingDesc, fieldLocSymbol);
+        inferLocElement.setLocDescriptor(curDesc);
       }
 
+      inferLoc.addLocation(inferLocElement);
+
     }
+    // System.out.println("@@@@@inferLoc=" + inferLoc);
+    return inferLoc;
+  }
+
+  private void addRelation(SSJavaLattice<String> methodLattice, MethodLocationInfo methodInfo,
+      CompositeLocation srcInferLoc, CompositeLocation dstInferLoc) throws CyclicFlowException {
+
+    System.out.println("addRelation --- srcInferLoc=" + srcInferLoc + "  dstInferLoc="
+        + dstInferLoc);
+    String srcLocalLocSymbol = srcInferLoc.get(0).getLocIdentifier();
+    String dstLocalLocSymbol = dstInferLoc.get(0).getLocIdentifier();
+
+    if (srcInferLoc.getSize() == 1 && dstInferLoc.getSize() == 1) {
+      // add a new relation to the local lattice
+      addRelationHigherToLower(methodLattice, methodInfo, srcLocalLocSymbol, dstLocalLocSymbol);
+    } else if (srcInferLoc.getSize() > 1 && dstInferLoc.getSize() > 1) {
+      // both src and dst have assigned to a composite location
+
+      if (!srcLocalLocSymbol.equals(dstLocalLocSymbol)) {
+        addRelationHigherToLower(methodLattice, methodInfo, srcLocalLocSymbol, dstLocalLocSymbol);
+      } else {
+        recursivelyAddRelation(1, srcInferLoc, dstInferLoc);
+      }
+    } else {
+      // either src or dst has assigned to a composite location
+      if (!srcLocalLocSymbol.equals(dstLocalLocSymbol)) {
+        addRelationHigherToLower(methodLattice, methodInfo, srcLocalLocSymbol, dstLocalLocSymbol);
+      }
+    }
+
+    System.out.println();
 
   }
 
-  private LocationInfo getLocationInfo(Descriptor d) {
+  public LocationInfo getLocationInfo(Descriptor d) {
     if (d instanceof MethodDescriptor) {
       return getMethodLocationInfo((MethodDescriptor) d);
     } else {
@@ -719,113 +808,68 @@ public class LocationInference {
   }
 
   private void addRelationToLattice(MethodDescriptor md, SSJavaLattice<String> methodLattice,
-      MethodLocationInfo methodInfo, FlowNode srcNode, FlowNode dstNode) {
+      MethodLocationInfo methodInfo, FlowNode srcNode, FlowNode dstNode) throws CyclicFlowException {
 
     System.out.println();
     System.out.println("### addRelationToLattice src=" + srcNode + " dst=" + dstNode);
 
     // add a new binary relation of dstNode < srcNode
     FlowGraph flowGraph = getFlowGraph(md);
-
-    Descriptor srcDesc = getDescriptor(0, srcNode);
-    Descriptor dstDesc = getDescriptor(0, dstNode);
-
-    // boolean isAssignedCompositeLocation = false;
-    // if (!methodInfo.getInferLocation(srcDesc).get(0).getLocIdentifier()
-    // .equals(methodInfo.getThisLocName())) {
-    // isAssignedCompositeLocation =
-    calculateCompositeLocation(flowGraph, methodLattice, methodInfo, srcNode);
-    // }
-
-    String srcSymbol = methodInfo.getInferLocation(srcDesc).get(0).getLocIdentifier();
-    String dstSymbol = methodInfo.getInferLocation(dstDesc).get(0).getLocIdentifier();
-
-    // if (srcNode.isParameter()) {
-    // int paramIdx = flowGraph.getParamIdx(srcNode.getDescTuple());
-    // methodInfo.addParameter(srcSymbol, srcDesc, paramIdx);
-    // }
-    //
-    // if (dstNode.isParameter()) {
-    // int paramIdx = flowGraph.getParamIdx(dstNode.getDescTuple());
-    // methodInfo.addParameter(dstSymbol, dstDesc, paramIdx);
-    // }
-
-    CompositeLocation srcInferLoc = methodInfo.getInferLocation(srcDesc);
-    CompositeLocation dstInferLoc = methodInfo.getInferLocation(dstDesc);
-
-    String srcLocalLocSymbol = srcInferLoc.get(0).getLocIdentifier();
-    String dstLocalLocSymbol = dstInferLoc.get(0).getLocIdentifier();
-
-    if (srcInferLoc.getSize() == 1 && dstInferLoc.getSize() == 1) {
-      // add a new relation to the local lattice
-      addRelationHigherToLower(methodLattice, methodInfo, srcLocalLocSymbol, dstLocalLocSymbol);
-    } else if (srcInferLoc.getSize() > 1 && dstInferLoc.getSize() > 1) {
-      // both src and dst have assigned to a composite location
-      recursivelyAddRelation(1, srcInferLoc, dstInferLoc);
-    } else {
-      // either src or dst has assigned to a composite location
-      if (!srcLocalLocSymbol.equals(dstLocalLocSymbol)) {
-        addRelationHigherToLower(methodLattice, methodInfo, srcLocalLocSymbol, dstLocalLocSymbol);
+    try {
+      System.out.println("***** src composite case::");
+      calculateCompositeLocation(flowGraph, methodLattice, methodInfo, srcNode);
+
+      CompositeLocation srcInferLoc =
+          generateInferredCompositeLocation(methodInfo, flowGraph.getLocationTuple(srcNode));
+      CompositeLocation dstInferLoc =
+          generateInferredCompositeLocation(methodInfo, flowGraph.getLocationTuple(dstNode));
+
+      addRelation(methodLattice, methodInfo, srcInferLoc, dstInferLoc);
+    } catch (CyclicFlowException e) {
+      // there is a cyclic value flow... try to calculate a composite location
+      // for the destination node
+      System.out.println("***** dst composite case::");
+      calculateCompositeLocation(flowGraph, methodLattice, methodInfo, dstNode);
+      CompositeLocation srcInferLoc =
+          generateInferredCompositeLocation(methodInfo, flowGraph.getLocationTuple(srcNode));
+      CompositeLocation dstInferLoc =
+          generateInferredCompositeLocation(methodInfo, flowGraph.getLocationTuple(dstNode));
+      try {
+        addRelation(methodLattice, methodInfo, srcInferLoc, dstInferLoc);
+      } catch (CyclicFlowException e1) {
+        throw new Error("Failed to merge cyclic value flows into a shared location.");
       }
     }
-    // if (!isAssignedCompositeLocation) {
-    // // source does not have a composite location
-    //
-    // NTuple<Location> srcTuple = flowGraph.getLocationTuple(srcNode);
-    // NTuple<Location> dstTuple = flowGraph.getLocationTuple(dstNode);
-    //
-    // recursivelyAddCompositeRelation(md, flowGraph, methodInfo, srcNode,
-    // dstNode, srcDesc, dstDesc);
-    //
-    // // if (!srcSymbol.equals(dstSymbol)) {
-    // // // add a local relation
-    // // if (!methodLattice.isGreaterThan(srcSymbol, dstSymbol)) {
-    // // // if the lattice does not have this relation, add it
-    // // addRelationHigherToLower(methodLattice, methodInfo, srcSymbol,
-    // // dstSymbol);
-    // // // methodLattice.addRelationHigherToLower(srcSymbol, dstSymbol);
-    // // }
-    // // } else {
-    // // // if src and dst have the same local location...
-    // // recursivelyAddCompositeRelation(md, flowGraph, methodInfo, srcNode,
-    // // dstNode, srcDesc,
-    // // dstDesc);
-    // // }
-    //
-    // } else {
-    // // source variable has a composite location
-    // if (methodInfo.getInferLocation(dstDesc).getSize() == 1) {
-    // if (!srcSymbol.equals(dstSymbol)) {
-    // addRelationHigherToLower(methodLattice, methodInfo, srcSymbol,
-    // dstSymbol);
-    // }
-    // }
-    //
-    // }
 
   }
 
   private void recursivelyAddRelation(int idx, CompositeLocation srcInferLoc,
-      CompositeLocation dstInferLoc) {
+      CompositeLocation dstInferLoc) throws CyclicFlowException {
 
     String srcLocSymbol = srcInferLoc.get(idx).getLocIdentifier();
     String dstLocSymbol = dstInferLoc.get(idx).getLocIdentifier();
 
+    Descriptor parentDesc = srcInferLoc.get(idx).getDescriptor();
+
     if (srcLocSymbol.equals(dstLocSymbol)) {
-      recursivelyAddRelation(idx + 1, srcInferLoc, dstInferLoc);
+      // check if it is the case of shared location
+      if (srcInferLoc.getSize() == (idx + 1) && dstInferLoc.getSize() == (idx + 1)) {
+        Location inferLocElement = srcInferLoc.get(idx);
+        System.out.println("SET SHARED LOCATION=" + inferLocElement);
+        getLattice(inferLocElement.getDescriptor())
+            .addSharedLoc(inferLocElement.getLocIdentifier());
+      } else if (srcInferLoc.getSize() > (idx + 1) && dstInferLoc.getSize() > (idx + 1)) {
+        recursivelyAddRelation(idx + 1, srcInferLoc, dstInferLoc);
+      }
     } else {
-
-      Descriptor parentDesc = srcInferLoc.get(idx).getDescriptor();
-
       addRelationHigherToLower(getLattice(parentDesc), getLocationInfo(parentDesc), srcLocSymbol,
           dstLocSymbol);
     }
-
   }
 
   private void recursivelyAddCompositeRelation(MethodDescriptor md, FlowGraph flowGraph,
       MethodLocationInfo methodInfo, FlowNode srcNode, FlowNode dstNode, Descriptor srcDesc,
-      Descriptor dstDesc) {
+      Descriptor dstDesc) throws CyclicFlowException {
 
     CompositeLocation inferSrcLoc;
     CompositeLocation inferDstLoc = methodInfo.getInferLocation(dstDesc);
@@ -869,10 +913,15 @@ public class LocationInference {
   }
 
   private boolean calculateCompositeLocation(FlowGraph flowGraph,
-      SSJavaLattice<String> methodLattice, MethodLocationInfo methodInfo, FlowNode flowNode) {
+      SSJavaLattice<String> methodLattice, MethodLocationInfo methodInfo, FlowNode flowNode)
+      throws CyclicFlowException {
 
     Descriptor localVarDesc = flowNode.getDescTuple().get(0);
 
+    if (localVarDesc.equals(methodInfo.getMethodDesc())) {
+      return false;
+    }
+
     Set<FlowNode> inNodeSet = flowGraph.getIncomingFlowNodeSet(flowNode);
     Set<FlowNode> reachableNodeSet = flowGraph.getReachableFlowNodeSet(flowNode);
 
@@ -882,19 +931,27 @@ public class LocationInference {
     Set<FlowNode> localInNodeSet = new HashSet<FlowNode>();
     Set<FlowNode> localOutNodeSet = new HashSet<FlowNode>();
 
+    CompositeLocation flowNodeInferLoc =
+        generateInferredCompositeLocation(methodInfo, flowGraph.getLocationTuple(flowNode));
+
     List<NTuple<Location>> prefixList = new ArrayList<NTuple<Location>>();
 
     for (Iterator iterator = inNodeSet.iterator(); iterator.hasNext();) {
       FlowNode inNode = (FlowNode) iterator.next();
-      NTuple<Location> inTuple = flowGraph.getLocationTuple(inNode);
+      NTuple<Location> inNodeTuple = flowGraph.getLocationTuple(inNode);
+
+      CompositeLocation inNodeInferredLoc =
+          generateInferredCompositeLocation(methodInfo, inNodeTuple);
 
-      if (inTuple.size() > 1) {
-        for (int i = 1; i < inTuple.size(); i++) {
-          NTuple<Location> prefix = inTuple.subList(0, i);
+      NTuple<Location> inNodeInferredLocTuple = inNodeInferredLoc.getTuple();
+
+      if (inNodeTuple.size() > 1) {
+        for (int i = 1; i < inNodeInferredLocTuple.size(); i++) {
+          NTuple<Location> prefix = inNodeInferredLocTuple.subList(0, i);
           if (!prefixList.contains(prefix)) {
             prefixList.add(prefix);
           }
-          addPrefixMapping(mapPrefixToIncomingLocTupleSet, prefix, inTuple);
+          addPrefixMapping(mapPrefixToIncomingLocTupleSet, prefix, inNodeInferredLocTuple);
         }
       } else {
         localInNodeSet.add(inNode);
@@ -930,10 +987,18 @@ public class LocationInference {
       for (Iterator iterator2 = reachableNodeSet.iterator(); iterator2.hasNext();) {
         FlowNode reachableNode = (FlowNode) iterator2.next();
         NTuple<Location> reachLocTuple = flowGraph.getLocationTuple(reachableNode);
-        if (reachLocTuple.startsWith(curPrefix)) {
+        CompositeLocation reachLocInferLoc =
+            generateInferredCompositeLocation(methodInfo, reachLocTuple);
+        if (reachLocInferLoc.getTuple().startsWith(curPrefix)) {
           reachableCommonPrefixSet.add(reachLocTuple);
         }
+      }
 
+      // check if the lattice has the relation in which higher prefix is
+      // actually lower than the current node
+      CompositeLocation prefixInferLoc = generateInferredCompositeLocation(methodInfo, curPrefix);
+      if (isGreaterThan(methodLattice, flowNodeInferLoc, prefixInferLoc)) {
+        reachableCommonPrefixSet.add(curPrefix);
       }
 
       if (!reachableCommonPrefixSet.isEmpty()) {
@@ -955,113 +1020,139 @@ public class LocationInference {
         SSJavaLattice<String> lattice = getLattice(desc);
         LocationInfo locInfo = getLocationInfo(desc);
 
-        // CompositeLocation inferLocation =
-        // methodInfo.getInferLocation(flowNode);
         CompositeLocation inferLocation = methodInfo.getInferLocation(localVarDesc);
+        CompositeLocation newInferLocation = new CompositeLocation();
 
-        String newlyInsertedLocName;
-        if (inferLocation.getSize() == 1) {
-          // need to replace the old local location with a new composite
-          // location
-
-          String oldMethodLocationSymbol = inferLocation.get(0).getLocIdentifier();
+        if (inferLocation.getTuple().startsWith(curPrefix)) {
+          // the same infer location is already existed. no need to do
+          // anything
+          return true;
+        } else {
+          // assign a new composite location
 
+          // String oldMethodLocationSymbol =
+          // inferLocation.get(0).getLocIdentifier();
           String newLocSymbol = "Loc" + (SSJavaLattice.seed++);
-          inferLocation = new CompositeLocation();
           for (int locIdx = 0; locIdx < curPrefix.size(); locIdx++) {
-            inferLocation.addLocation(curPrefix.get(locIdx));
+            newInferLocation.addLocation(curPrefix.get(locIdx));
           }
           Location fieldLoc = new Location(desc, newLocSymbol);
-          inferLocation.addLocation(fieldLoc);
+          newInferLocation.addLocation(fieldLoc);
 
-          methodInfo.mapDescriptorToLocation(localVarDesc, inferLocation);
-          methodInfo.removeMaplocalVarToLocSet(localVarDesc);
-
-          String newMethodLocationSymbol = curPrefix.get(0).getLocIdentifier();
-
-          replaceOldLocWithNewLoc(methodLattice, oldMethodLocationSymbol, newMethodLocationSymbol);
+          if (flowNode.getDescTuple().size() == 1) {
+            // maps local variable to location types of the common prefix
+            methodInfo.mapDescriptorToLocation(localVarDesc, newInferLocation.clone());
+          }
 
-        } else {
+          // methodInfo.mapDescriptorToLocation(localVarDesc, newInferLocation);
+          addMapLocSymbolToInferredLocation(methodInfo.getMethodDesc(), localVarDesc,
+              newInferLocation);
+          methodInfo.removeMaplocalVarToLocSet(localVarDesc);
 
-          String localLocName = methodInfo.getInferLocation(localVarDesc).get(0).getLocIdentifier();
-          return true;
+          // add the field/var descriptor to the set of the location symbol
+          int flowNodeTupleSize = flowNode.getDescTuple().size();
+          Descriptor lastFlowNodeDesc = flowNode.getDescTuple().get(flowNodeTupleSize - 1);
+          int inferLocSize = newInferLocation.getSize();
+          Location lastLoc = newInferLocation.get(inferLocSize - 1);
+          Descriptor enclosingDesc = lastLoc.getDescriptor();
+          getLocationInfo(enclosingDesc).addMapLocSymbolToDescSet(lastLoc.getLocIdentifier(),
+              lastFlowNodeDesc);
+
+          // clean up the previous location
+          // Location prevInferLocElement =
+          // inferLocation.get(inferLocation.getSize() - 1);
+          // Descriptor prevEnclosingDesc = prevInferLocElement.getDescriptor();
+          //
+          // SSJavaLattice<String> targetLattice;
+          // LocationInfo targetInfo;
+          // if (prevEnclosingDesc.equals(methodInfo.getMethodDesc())) {
+          // targetLattice = methodLattice;
+          // targetInfo = methodInfo;
+          // } else {
+          // targetLattice = getLattice(prevInferLocElement.getDescriptor());
+          // targetInfo = getLocationInfo(prevInferLocElement.getDescriptor());
+          // }
+          //
+          // Set<Pair<Descriptor, Descriptor>> associstedDescSet =
+          // targetInfo.getRelatedInferLocSet(prevInferLocElement.getLocIdentifier());
+          //
+          // if (associstedDescSet.size() == 1) {
+          // targetLattice.remove(prevInferLocElement.getLocIdentifier());
+          // } else {
+          // associstedDescSet.remove(lastFlowNodeDesc);
+          // }
 
         }
 
-        newlyInsertedLocName = inferLocation.get(inferLocation.getSize() - 1).getLocIdentifier();
+        System.out.println("ASSIGN NEW COMPOSITE LOCATION =" + newInferLocation + "    to "
+            + flowNode);
+
+        String newlyInsertedLocName =
+            newInferLocation.get(newInferLocation.getSize() - 1).getLocIdentifier();
 
+        System.out.println("-- add in-flow");
         for (Iterator iterator = incomingCommonPrefixSet.iterator(); iterator.hasNext();) {
           NTuple<Location> tuple = (NTuple<Location>) iterator.next();
-
+          System.out.println("--in-flow tuple=" + tuple);
           Location loc = tuple.get(idx);
           String higher = locInfo.getFieldInferLocation(loc.getLocDescriptor()).getLocIdentifier();
-          System.out.println("--");
-          System.out.println("add in-flow relation:");
           addRelationHigherToLower(lattice, locInfo, higher, newlyInsertedLocName);
         }
-        System.out.println("end of add-inflow relation");
 
+        System.out.println("-- add local in-flow");
         for (Iterator iterator = localInNodeSet.iterator(); iterator.hasNext();) {
           FlowNode localNode = (FlowNode) iterator.next();
-          Descriptor localInVarDesc = localNode.getDescTuple().get(0);
-          CompositeLocation inNodeInferLoc = methodInfo.getInferLocation(localInVarDesc);
+
+          if (localNode.equals(flowNode)) {
+            continue;
+          }
+
+          CompositeLocation inNodeInferLoc =
+              generateInferredCompositeLocation(methodInfo, flowGraph.getLocationTuple(localNode));
 
           if (isCompositeLocation(inNodeInferLoc)) {
             // need to make sure that newLocSymbol is lower than the infernode
             // location in the field lattice
-
-            if (inNodeInferLoc.getTuple().startsWith(curPrefix)
-                && inNodeInferLoc.getSize() == (curPrefix.size() + 1)) {
-              String higher = inNodeInferLoc.get(inNodeInferLoc.getSize() - 1).getLocIdentifier();
-              if (!higher.equals(newlyInsertedLocName)) {
-                System.out.println("add localInNodeSet relation:");
-                addRelationHigherToLower(lattice, locInfo, higher, newlyInsertedLocName);
-              }
-            } else {
-              throw new Error("Failed to generate a composite location.");
-            }
+            System.out.println("----srcNode=" + localNode + "  dstNode=" + flowNode);
+            addRelationToLattice(methodInfo.getMethodDesc(), methodLattice, methodInfo, localNode,
+                flowNode);
 
           }
+
         }
 
+        System.out.println("-- add out flow");
         for (Iterator iterator = reachableCommonPrefixSet.iterator(); iterator.hasNext();) {
           NTuple<Location> tuple = (NTuple<Location>) iterator.next();
-          Location loc = tuple.get(idx);
-          String lower = locInfo.getFieldInferLocation(loc.getLocDescriptor()).getLocIdentifier();
-          // lattice.addRelationHigherToLower(newlyInsertedLocName, lower);
-          System.out.println("add out-flow relation:");
-          addRelationHigherToLower(lattice, locInfo, newlyInsertedLocName, lower);
+          if (tuple.size() > idx) {
+            Location loc = tuple.get(idx);
+            String lower = locInfo.getFieldInferLocation(loc.getLocDescriptor()).getLocIdentifier();
+            addRelationHigherToLower(lattice, locInfo, newlyInsertedLocName, lower);
+          }
         }
-        System.out.println("end of add out-flow relation");
 
+        System.out.println("-- add local out flow");
         for (Iterator iterator = localOutNodeSet.iterator(); iterator.hasNext();) {
           FlowNode localOutNode = (FlowNode) iterator.next();
 
-          Descriptor localOutDesc = localOutNode.getDescTuple().get(0);
-          // String localOutNodeSymbol =
-          // localOutNode.getDescTuple().get(0).getSymbol();
-          CompositeLocation outNodeInferLoc = methodInfo.getInferLocation(localOutDesc);
+          if (localOutNode.equals(flowNode)) {
+            continue;
+          }
+
+          CompositeLocation outNodeInferLoc =
+              generateInferredCompositeLocation(methodInfo,
+                  flowGraph.getLocationTuple(localOutNode));
 
-          // System.out
-          // .println("localOutNode=" + localOutNode + " outNodeInferLoc=" +
-          // outNodeInferLoc);
           if (isCompositeLocation(outNodeInferLoc)) {
             // need to make sure that newLocSymbol is higher than the infernode
             // location
+            System.out.println("--- srcNode=" + flowNode + "  dstNode=" + localOutNode);
+            addRelationToLattice(methodInfo.getMethodDesc(), methodLattice, methodInfo, flowNode,
+                localOutNode);
 
-            if (outNodeInferLoc.getTuple().startsWith(curPrefix)
-                && outNodeInferLoc.getSize() == (curPrefix.size() + 1)) {
-
-              String lower = outNodeInferLoc.get(outNodeInferLoc.getSize() - 1).getLocIdentifier();
-              System.out.println("add outNodeInferLoc relation:");
-
-              addRelationHigherToLower(lattice, locInfo, newlyInsertedLocName, lower);
-
-            } else {
-              throw new Error("Failed to generate a composite location.");
-            }
           }
         }
+        System.out.println("-- end of add local out flow");
 
         return true;
       }
@@ -1072,6 +1163,15 @@ public class LocationInference {
 
   }
 
+  private void addMapLocSymbolToInferredLocation(MethodDescriptor md, Descriptor localVar,
+      CompositeLocation inferLoc) {
+
+    Location locElement = inferLoc.get((inferLoc.getSize() - 1));
+    Descriptor enclosingDesc = locElement.getDescriptor();
+    LocationInfo locInfo = getLocationInfo(enclosingDesc);
+    locInfo.addMapLocSymbolToRelatedInferLoc(locElement.getLocIdentifier(), md, localVar);
+  }
+
   private boolean isCompositeLocation(CompositeLocation cl) {
     return cl.getSize() > 1;
   }
@@ -1097,15 +1197,14 @@ public class LocationInference {
   }
 
   private void addRelationHigherToLower(SSJavaLattice<String> lattice, LocationInfo locInfo,
-      String higher, String lower) {
+      String higher, String lower) throws CyclicFlowException {
 
+    System.out.println("---addRelationHigherToLower " + higher + " -> " + lower
+        + " to the lattice of " + locInfo.getDescIdentifier());
     // if (higher.equals(lower) && lattice.isSharedLoc(higher)) {
     // return;
     // }
-
     Set<String> cycleElementSet = lattice.getPossibleCycleElements(higher, lower);
-    System.out.println("#Check cycle=" + lower + " < " + higher);
-    System.out.println("#cycleElementSet=" + cycleElementSet);
 
     boolean hasNonPrimitiveElement = false;
     for (Iterator iterator = cycleElementSet.iterator(); iterator.hasNext();) {
@@ -1119,19 +1218,52 @@ public class LocationInference {
     }
 
     if (hasNonPrimitiveElement) {
+      System.out.println("#Check cycle= " + lower + " < " + higher + "     cycleElementSet="
+          + cycleElementSet);
       // if there is non-primitive element in the cycle, no way to merge cyclic
       // elements into the shared location
-      throw new Error("Failed to merge cyclic value flows into a shared location.");
+      throw new CyclicFlowException();
     }
 
     if (cycleElementSet.size() > 0) {
+
       String newSharedLoc = "SharedLoc" + (SSJavaLattice.seed++);
 
+      System.out.println("---ASSIGN NEW SHARED LOC=" + newSharedLoc + "   to  " + cycleElementSet);
       lattice.mergeIntoSharedLocation(cycleElementSet, newSharedLoc);
 
       for (Iterator iterator = cycleElementSet.iterator(); iterator.hasNext();) {
         String oldLocSymbol = (String) iterator.next();
-        locInfo.mergeMapping(oldLocSymbol, newSharedLoc);
+
+        Set<Pair<Descriptor, Descriptor>> inferLocSet = locInfo.getRelatedInferLocSet(oldLocSymbol);
+        System.out.println("---update related locations=" + inferLocSet);
+        for (Iterator iterator2 = inferLocSet.iterator(); iterator2.hasNext();) {
+          Pair<Descriptor, Descriptor> pair = (Pair<Descriptor, Descriptor>) iterator2.next();
+          Descriptor enclosingDesc = pair.getFirst();
+          Descriptor desc = pair.getSecond();
+
+          CompositeLocation inferLoc;
+          if (curMethodInfo.md.equals(enclosingDesc)) {
+            inferLoc = curMethodInfo.getInferLocation(desc);
+          } else {
+            inferLoc = getLocationInfo(enclosingDesc).getInferLocation(desc);
+          }
+
+          Location locElement = inferLoc.get(inferLoc.getSize() - 1);
+
+          locElement.setLocIdentifier(newSharedLoc);
+          locInfo.addMapLocSymbolToRelatedInferLoc(newSharedLoc, enclosingDesc, desc);
+
+          if (curMethodInfo.md.equals(enclosingDesc)) {
+            inferLoc = curMethodInfo.getInferLocation(desc);
+          } else {
+            inferLoc = getLocationInfo(enclosingDesc).getInferLocation(desc);
+          }
+          System.out.println("---New Infer Loc=" + inferLoc);
+
+        }
+        locInfo.removeRelatedInferLocSet(oldLocSymbol, newSharedLoc);
+
       }
 
       lattice.addSharedLoc(newSharedLoc);
@@ -1198,7 +1330,7 @@ public class LocationInference {
   }
 
   private void extractRelationFromFieldFlows(ClassDescriptor cd, FlowNode srcNode,
-      FlowNode dstNode, int idx) {
+      FlowNode dstNode, int idx) throws CyclicFlowException {
 
     if (srcNode.getDescTuple().get(idx).equals(dstNode.getDescTuple().get(idx))
         && srcNode.getDescTuple().size() > (idx + 1) && dstNode.getDescTuple().size() > (idx + 1)) {
@@ -1244,16 +1376,20 @@ public class LocationInference {
   public void constructFlowGraph() {
 
     setupToAnalyze();
-    
-    Set<MethodDescriptor> visited=new HashSet<MethodDescriptor>();
+
+    Set<MethodDescriptor> visited = new HashSet<MethodDescriptor>();
+    Set<MethodDescriptor> reachableCallee = new HashSet<MethodDescriptor>();
 
     while (!toAnalyzeIsEmpty()) {
       ClassDescriptor cd = toAnalyzeNext();
 
       setupToAnalazeMethod(cd);
+      toanalyzeMethodList.removeAll(visited);
+
       while (!toAnalyzeMethodIsEmpty()) {
         MethodDescriptor md = toAnalyzeMethodNext();
-//        if (ssjava.needTobeAnnotated(md)) {
+        if ((!visited.contains(md))
+            && (ssjava.needTobeAnnotated(md) || reachableCallee.contains(md))) {
           if (state.SSJAVADEBUG) {
             System.out.println();
             System.out.println("SSJAVA: Constructing a flow graph: " + md);
@@ -1266,17 +1402,23 @@ public class LocationInference {
           } else {
             setPossibleCallees.addAll(ssjava.getCallGraph().getMethods(md));
           }
-          
-          Set<MethodDescriptor> calleeSet=ssjava.getCallGraph().getCalleeSet(md);
-          
+
+          Set<MethodDescriptor> calleeSet = ssjava.getCallGraph().getCalleeSet(md);
+          Set<MethodDescriptor> needToAnalyzeCalleeSet = new HashSet<MethodDescriptor>();
+
           for (Iterator iterator = calleeSet.iterator(); iterator.hasNext();) {
             MethodDescriptor calleemd = (MethodDescriptor) iterator.next();
-            if((!ssjava.isSSJavaUtil(calleemd.getClassDesc())) && (! visited.contains(calleemd))){
-              toanalyzeMethodList.add(calleemd);
+            if ((!ssjava.isTrustMethod(calleemd))
+                && (!ssjava.isSSJavaUtil(calleemd.getClassDesc()))) {
+              if (!visited.contains(calleemd)) {
+                toanalyzeMethodList.add(calleemd);
+              }
+              reachableCallee.add(calleemd);
+              needToAnalyzeCalleeSet.add(calleemd);
             }
           }
 
-          mapMethodDescToPossibleMethodDescSet.put(md, setPossibleCallees);
+          mapMethodToCalleeSet.put(md, needToAnalyzeCalleeSet);
 
           // creates a mapping from a parameter descriptor to its index
           Map<Descriptor, Integer> mapParamDescToIdx = new HashMap<Descriptor, Integer>();
@@ -1291,10 +1433,10 @@ public class LocationInference {
 
           visited.add(md);
           analyzeMethodBody(cd, md);
-          
+
         }
       }
-//    }
+    }
 
     _debug_printGraph();
   }
@@ -1488,21 +1630,26 @@ public class LocationInference {
     switch (en.kind()) {
 
     case Kind.AssignmentNode:
-      analyzeFlowAssignmentNode(md, nametable, (AssignmentNode) en, base, implicitFlowTupleSet);
+      analyzeFlowAssignmentNode(md, nametable, (AssignmentNode) en, nodeSet, base,
+          implicitFlowTupleSet);
       break;
 
     case Kind.FieldAccessNode:
       flowTuple =
           analyzeFlowFieldAccessNode(md, nametable, (FieldAccessNode) en, nodeSet, base,
-              implicitFlowTupleSet);
-      nodeSet.addTuple(flowTuple);
+              implicitFlowTupleSet, isLHS);
+      if (flowTuple != null) {
+        nodeSet.addTuple(flowTuple);
+      }
       return flowTuple;
 
     case Kind.NameNode:
       NodeTupleSet nameNodeSet = new NodeTupleSet();
       flowTuple =
           analyzeFlowNameNode(md, nametable, (NameNode) en, nameNodeSet, base, implicitFlowTupleSet);
-      nodeSet.addTuple(flowTuple);
+      if (flowTuple != null) {
+        nodeSet.addTuple(flowTuple);
+      }
       return flowTuple;
 
     case Kind.OpNode:
@@ -1530,9 +1677,8 @@ public class LocationInference {
       break;
 
     case Kind.CastNode:
-      analyzeFlowCastNode(md, nametable, (CastNode) en, implicitFlowTupleSet);
+      analyzeFlowCastNode(md, nametable, (CastNode) en, nodeSet, base, implicitFlowTupleSet);
       break;
-
     // case Kind.InstanceOfNode:
     // checkInstanceOfNode(md, nametable, (InstanceOfNode) en, td);
     // return null;
@@ -1556,10 +1702,10 @@ public class LocationInference {
   }
 
   private void analyzeFlowCastNode(MethodDescriptor md, SymbolTable nametable, CastNode cn,
-      NodeTupleSet implicitFlowTupleSet) {
+      NodeTupleSet nodeSet, NTuple<Descriptor> base, NodeTupleSet implicitFlowTupleSet) {
 
-    NodeTupleSet nodeTupleSet = new NodeTupleSet();
-    analyzeFlowExpressionNode(md, nametable, cn.getExpression(), nodeTupleSet, false);
+    analyzeFlowExpressionNode(md, nametable, cn.getExpression(), nodeSet, base,
+        implicitFlowTupleSet, false);
 
   }
 
@@ -1656,7 +1802,7 @@ public class LocationInference {
       // checkCallerArgumentLocationConstraints(md, nametable, min,
       // baseLocation, constraint);
 
-      if (min.getMethod().getReturnType()!=null && !min.getMethod().getReturnType().isVoid()) {
+      if (min.getMethod().getReturnType() != null && !min.getMethod().getReturnType().isVoid()) {
         // If method has a return value, compute the highest possible return
         // location in the caller's perspective
         // CompositeLocation ceilingLoc =
@@ -1670,17 +1816,17 @@ public class LocationInference {
 
   }
 
-  private NTuple<Descriptor> getArgTupleByArgIdx(MethodInvokeNode min, int idx) {
+  private NodeTupleSet getNodeTupleSetByArgIdx(MethodInvokeNode min, int idx) {
     return mapMethodInvokeNodeToArgIdxMap.get(min).get(new Integer(idx));
   }
 
-  private void addArgIdxMap(MethodInvokeNode min, int idx, NTuple<Descriptor> argTuple) {
-    Map<Integer, NTuple<Descriptor>> mapIdxToArgTuple = mapMethodInvokeNodeToArgIdxMap.get(min);
-    if (mapIdxToArgTuple == null) {
-      mapIdxToArgTuple = new HashMap<Integer, NTuple<Descriptor>>();
-      mapMethodInvokeNodeToArgIdxMap.put(min, mapIdxToArgTuple);
+  private void addArgIdxMap(MethodInvokeNode min, int idx, NodeTupleSet tupleSet) {
+    Map<Integer, NodeTupleSet> mapIdxToTupleSet = mapMethodInvokeNodeToArgIdxMap.get(min);
+    if (mapIdxToTupleSet == null) {
+      mapIdxToTupleSet = new HashMap<Integer, NodeTupleSet>();
+      mapMethodInvokeNodeToArgIdxMap.put(min, mapIdxToTupleSet);
     }
-    mapIdxToArgTuple.put(new Integer(idx), argTuple);
+    mapIdxToTupleSet.put(new Integer(idx), tupleSet);
   }
 
   private void analyzeFlowMethodParameters(MethodDescriptor callermd, SymbolTable nametable,
@@ -1688,15 +1834,24 @@ public class LocationInference {
 
     if (min.numArgs() > 0) {
 
-      int offset = min.getMethod().isStatic() ? 0 : 1;
+      int offset;
+      if (min.getMethod().isStatic()) {
+        offset = 0;
+      } else {
+        offset = 1;
+        NTuple<Descriptor> thisArgTuple = new NTuple<Descriptor>();
+        thisArgTuple.add(callermd.getThis());
+        NodeTupleSet argTupleSet = new NodeTupleSet();
+        argTupleSet.addTuple(thisArgTuple);
+        addArgIdxMap(min, 0, argTupleSet);
+      }
 
       for (int i = 0; i < min.numArgs(); i++) {
         ExpressionNode en = min.getArg(i);
-        NTuple<Descriptor> argTuple =
-            analyzeFlowExpressionNode(callermd, nametable, en, new NodeTupleSet(), false);
-
+        NodeTupleSet argTupleSet = new NodeTupleSet();
+        analyzeFlowExpressionNode(callermd, nametable, en, argTupleSet, false);
         // if argument is liternal node, argTuple is set to NULL.
-        addArgIdxMap(min, i + offset, argTuple);
+        addArgIdxMap(min, i + offset, argTupleSet);
       }
 
     }
@@ -1732,7 +1887,6 @@ public class LocationInference {
       nodeSet.addTupleSet(expNodeTupleSet);
       nodeSet.addTupleSet(idxNodeTupleSet);
     }
-
   }
 
   private void analyzeCreateObjectNode(MethodDescriptor md, SymbolTable nametable,
@@ -1798,6 +1952,7 @@ public class LocationInference {
     default:
       throw new Error(op.toString());
     }
+
   }
 
   private NTuple<Descriptor> analyzeFlowNameNode(MethodDescriptor md, SymbolTable nametable,
@@ -1810,8 +1965,13 @@ public class LocationInference {
     NameDescriptor nd = nn.getName();
 
     if (nd.getBase() != null) {
-      analyzeFlowExpressionNode(md, nametable, nn.getExpression(), nodeSet, base,
-          implicitFlowTupleSet, false);
+      base =
+          analyzeFlowExpressionNode(md, nametable, nn.getExpression(), nodeSet, base,
+              implicitFlowTupleSet, false);
+      if (base == null) {
+        // base node has the top location
+        return base;
+      }
     } else {
       String varname = nd.toString();
       if (varname.equals("this")) {
@@ -1830,10 +1990,9 @@ public class LocationInference {
         FieldDescriptor fd = (FieldDescriptor) d;
         if (fd.isStatic()) {
           if (fd.isFinal()) {
-            // if it is 'static final', assign the default TOP LOCATION
-            // DESCRIPTOR
-            base.add(TOPDESC);
-            return base;
+            // if it is 'static final', no need to have flow node for the TOP
+            // location
+            return null;
           } else {
             // if 'static', assign the default GLOBAL LOCATION to the first
             // element of the tuple
@@ -1879,7 +2038,7 @@ public class LocationInference {
 
   private NTuple<Descriptor> analyzeFlowFieldAccessNode(MethodDescriptor md, SymbolTable nametable,
       FieldAccessNode fan, NodeTupleSet nodeSet, NTuple<Descriptor> base,
-      NodeTupleSet implicitFlowTupleSet) {
+      NodeTupleSet implicitFlowTupleSet, boolean isLHS) {
 
     ExpressionNode left = fan.getExpression();
     TypeDescriptor ltd = left.getType();
@@ -1894,31 +2053,38 @@ public class LocationInference {
     if (ltd.isClassNameRef() || (varName != null && varName.equals("this"))) {
       // using a class name directly or access using this
       if (fd.isStatic() && fd.isFinal()) {
-        // loc.addLocation(Location.createTopLocation(md));
-        // return loc;
+        return null;
       }
     }
 
     if (left instanceof ArrayAccessNode) {
+
       ArrayAccessNode aan = (ArrayAccessNode) left;
       left = aan.getExpression();
+      analyzeFlowExpressionNode(md, nametable, aan.getIndex(), nodeSet, base, implicitFlowTupleSet,
+          isLHS);
     }
     // fanNodeSet
     base =
-        analyzeFlowExpressionNode(md, nametable, left, nodeSet, base, implicitFlowTupleSet, false);
+        analyzeFlowExpressionNode(md, nametable, left, nodeSet, base, implicitFlowTupleSet, isLHS);
+    if (base == null) {
+      // in this case, field is TOP location
+      return null;
+    } else {
 
-    if (!left.getType().isPrimitive()) {
+      if (!left.getType().isPrimitive()) {
+
+        if (!fd.getSymbol().equals("length")) {
+          // array.length access, just have the location of the array
+          base.add(fd);
+        }
 
-      if (fd.getSymbol().equals("length")) {
-        // array.length access, just have the location of the array
-      } else {
-        base.add(fd);
       }
 
-    }
+      getFlowGraph(md).createNewFlowNode(base);
+      return base;
 
-    getFlowGraph(md).createNewFlowNode(base);
-    return base;
+    }
 
   }
 
@@ -1929,7 +2095,8 @@ public class LocationInference {
   }
 
   private void analyzeFlowAssignmentNode(MethodDescriptor md, SymbolTable nametable,
-      AssignmentNode an, NTuple<Descriptor> base, NodeTupleSet implicitFlowTupleSet) {
+      AssignmentNode an, NodeTupleSet nodeSet, NTuple<Descriptor> base,
+      NodeTupleSet implicitFlowTupleSet) {
 
     NodeTupleSet nodeSetRHS = new NodeTupleSet();
     NodeTupleSet nodeSetLHS = new NodeTupleSet();
@@ -1950,6 +2117,12 @@ public class LocationInference {
       analyzeFlowExpressionNode(md, nametable, an.getSrc(), nodeSetRHS, null, implicitFlowTupleSet,
           false);
 
+      // System.out.println("-analyzeFlowAssignmentNode=" + an.printNode(0));
+      // System.out.println("-nodeSetLHS=" + nodeSetLHS);
+      // System.out.println("-nodeSetRHS=" + nodeSetRHS);
+      // System.out.println("-implicitFlowTupleSet=" + implicitFlowTupleSet);
+      // System.out.println("-");
+
       if (an.getOperation().getOp() >= 2 && an.getOperation().getOp() <= 12) {
         // if assignment contains OP+EQ operator, creates edges from LHS to LHS
         for (Iterator<NTuple<Descriptor>> iter = nodeSetLHS.iterator(); iter.hasNext();) {
@@ -1986,8 +2159,20 @@ public class LocationInference {
         addFlowGraphEdge(md, tuple, tuple);
       }
 
+      // creates edges from implicitFlowTupleSet to LHS
+      for (Iterator<NTuple<Descriptor>> iter = implicitFlowTupleSet.iterator(); iter.hasNext();) {
+        NTuple<Descriptor> fromTuple = iter.next();
+        for (Iterator<NTuple<Descriptor>> iter2 = nodeSetLHS.iterator(); iter2.hasNext();) {
+          NTuple<Descriptor> toTuple = iter2.next();
+          addFlowGraphEdge(md, fromTuple, toTuple);
+        }
+      }
+
     }
 
+    if (nodeSet != null) {
+      nodeSet.addTupleSet(nodeSetLHS);
+    }
   }
 
   public FlowGraph getFlowGraph(MethodDescriptor md) {
@@ -2019,3 +2204,7 @@ public class LocationInference {
   }
 
 }
+
+class CyclicFlowException extends Exception {
+
+}