changes: fixes the case that a shared location appears in the middle of a composite...
[IRC.git] / Robust / src / Analysis / SSJava / LocationInference.java
index 03de37f3baf5fab1d28a709b087d53da8ab803e4..f320a260f851ffcc2cd4a185b111e920825a6713 100644 (file)
@@ -98,6 +98,8 @@ public class LocationInference {
 
   private Map<MethodInvokeNode, NTuple<Descriptor>> mapMethodInvokeNodeToBaseTuple;
 
+  private Map<MethodInvokeNode, Set<NTuple<Location>>> mapMethodInvokeNodeToPCLocTupleSet;
+
   private Map<MethodDescriptor, MethodLocationInfo> mapMethodDescToMethodLocationInfo;
 
   private Map<ClassDescriptor, LocationInfo> mapClassToLocationInfo;
@@ -120,28 +122,38 @@ public class LocationInference {
 
   public static final String GLOBALLOC = "GLOBALLOC";
 
-  public static final String TOPLOC = "TOPLOC";
-
   public static final String INTERLOC = "INTERLOC";
 
-  public static final String PCLOC = "PCLOC";
+  public static final String PCLOC = "_PCLOC_";
 
-  public static final String RLOC = "RLOC";
+  public static final String RLOC = "_RLOC_";
 
   public static final Descriptor GLOBALDESC = new NameDescriptor(GLOBALLOC);
 
-  public static final Descriptor TOPDESC = new NameDescriptor(TOPLOC);
+  public static final Descriptor TOPDESC = new NameDescriptor(SSJavaAnalysis.TOP);
+
+  public static final Descriptor BOTTOMDESC = new NameDescriptor(SSJavaAnalysis.BOTTOM);
 
   public static final Descriptor RETURNLOC = new NameDescriptor(RLOC);
 
+  public static final Descriptor LITERALDESC = new NameDescriptor("LITERAL");
+
+  public static final HNode TOPHNODE = new HNode(TOPDESC);
+
+  public static final HNode BOTTOMHNODE = new HNode(BOTTOMDESC);
+
   public static String newline = System.getProperty("line.separator");
 
   LocationInfo curMethodInfo;
 
+  private boolean hasChanges = false;
+
   boolean debug = true;
 
   public static int locSeed = 0;
 
+  private Stack<String> arrayAccessNodeStack;
+
   public LocationInference(SSJavaAnalysis ssjava, State state) {
     this.ssjava = ssjava;
     this.state = state;
@@ -180,6 +192,11 @@ public class LocationInference {
     this.mapMethodInvokeNodeToMapCallerArgToCalleeArg =
         new HashMap<MethodInvokeNode, Map<NTuple<Descriptor>, NTuple<Descriptor>>>();
 
+    this.mapMethodInvokeNodeToPCLocTupleSet =
+        new HashMap<MethodInvokeNode, Set<NTuple<Location>>>();
+
+    this.arrayAccessNodeStack = new Stack<String>();
+
   }
 
   public void setupToAnalyze() {
@@ -223,10 +240,25 @@ public class LocationInference {
 
   public void inference() {
 
+    ssjava.init();
+
     // construct value flow graph
     constructFlowGraph();
 
-    assignCompositeLocation();
+    constructGlobalFlowGraph();
+    // _debug_writeFlowGraph();
+    // System.exit(0);
+
+    do {
+      assignCompositeLocation();
+      updateFlowGraph();
+      calculateExtraLocations();
+      hasChanges = false;
+      addAdditionalOrderingConstraints();
+      System.out.println("&&&&&&&&&&&&&&&&&&&&&&has changes=" + hasChanges);
+    } while (hasChanges);
+
+    _debug_writeFlowGraph();
 
     // System.exit(0);
 
@@ -234,9 +266,6 @@ public class LocationInference {
 
     debug_writeHierarchyDotFiles();
 
-    // calculate RETURNLOC,PCLOC
-    calculateExtraLocations();
-
     simplifyHierarchyGraph();
 
     debug_writeSimpleHierarchyDotFiles();
@@ -263,6 +292,21 @@ public class LocationInference {
 
   }
 
+  private void updateFlowGraph() {
+
+    LinkedList<MethodDescriptor> methodDescList =
+        (LinkedList<MethodDescriptor>) toanalyze_methodDescList.clone();
+
+    while (!methodDescList.isEmpty()) {
+      MethodDescriptor md = methodDescList.removeLast();
+      if (state.SSJAVADEBUG) {
+        System.out.println();
+        System.out.println("SSJAVA: Updating a flow graph: " + md);
+        propagateFlowsFromCalleesWithNoCompositeLocation(md);
+      }
+    }
+  }
+
   public Map<NTuple<Descriptor>, NTuple<Descriptor>> getMapCallerArgToCalleeParam(
       MethodInvokeNode min) {
 
@@ -288,7 +332,12 @@ public class LocationInference {
     System.out.println("\nSSJAVA: Translate composite location assignments to flow graphs:");
     MethodDescriptor methodEventLoopDesc = ssjava.getMethodContainingSSJavaLoop();
     translateCompositeLocationAssignmentToFlowGraph(methodEventLoopDesc);
-    _debug_printGraph();
+  }
+
+  private void addAdditionalOrderingConstraints() {
+    System.out.println("\nSSJAVA: Add addtional ordering constriants:");
+    MethodDescriptor methodEventLoopDesc = ssjava.getMethodContainingSSJavaLoop();
+    addAddtionalOrderingConstraints(methodEventLoopDesc);
   }
 
   private void updateCompositeLocationAssignments() {
@@ -316,6 +365,7 @@ public class LocationInference {
           System.out.println("---updatedCompLoc1=" + updatedCompLoc);
         } else {
           NTuple<Descriptor> descTuple = node.getDescTuple();
+          System.out.println("update desc=" + descTuple);
           CompositeLocation compLoc = convertToCompositeLocation(md, descTuple);
           compLoc = updateCompositeLocation(compLoc);
           node.setCompositeLocation(compLoc);
@@ -350,10 +400,9 @@ public class LocationInference {
       String locName;
       if (!enclosingDesc.equals(GLOBALDESC)) {
         LocationSummary locSummary = getLocationSummary(enclosingDesc);
-        HierarchyGraph hierarchyGraph = getSimpleHierarchyGraph(enclosingDesc);
-        if (hierarchyGraph != null) {
-
-          HNode curNode = hierarchyGraph.getCurrentHNode(nodeIdentifier);
+        HierarchyGraph scGraph = getSkeletonCombinationHierarchyGraph(enclosingDesc);
+        if (scGraph != null) {
+          HNode curNode = scGraph.getCurrentHNode(nodeIdentifier);
           if (curNode != null) {
             nodeIdentifier = curNode.getName();
           }
@@ -371,12 +420,16 @@ public class LocationInference {
 
   private void translateCompositeLocationAssignmentToFlowGraph(MethodDescriptor mdCaller) {
 
+    System.out.println("\n\n###translateCompositeLocationAssignmentToFlowGraph mdCaller="
+        + mdCaller);
+
     // First, assign a composite location to a node in the flow graph
     GlobalFlowGraph callerGlobalFlowGraph = getSubGlobalFlowGraph(mdCaller);
 
     FlowGraph callerFlowGraph = getFlowGraph(mdCaller);
     Map<Location, CompositeLocation> callerMapLocToCompLoc =
         callerGlobalFlowGraph.getMapLocationToInferCompositeLocation();
+
     Set<Location> methodLocSet = callerMapLocToCompLoc.keySet();
     for (Iterator iterator = methodLocSet.iterator(); iterator.hasNext();) {
       Location methodLoc = (Location) iterator.next();
@@ -394,14 +447,258 @@ public class LocationInference {
       // need to translate a composite location that is started with the base
       // tuple of 'min'.
       translateMapLocationToInferCompositeLocationToCalleeGraph(callerGlobalFlowGraph, min);
-      calleeSet.add(min.getMethod());
+      MethodDescriptor mdCallee = min.getMethod();
+      calleeSet.add(mdCallee);
+
+      FlowGraph calleeFlowGraph = getFlowGraph(mdCallee);
+
+      NTuple<Descriptor> methodInvokeBaseDescTuple = mapMethodInvokeNodeToBaseTuple.get(min);
+      NTuple<Location> methodInvokeBaseLocTuple = null;
+      if (methodInvokeBaseDescTuple != null) {
+        methodInvokeBaseLocTuple = translateToLocTuple(mdCaller, methodInvokeBaseDescTuple);
+      }
+
+      // ////////////////
+      // ////////////////
+
+      // If the location of an argument has a composite location
+      // need to assign a proper composite location to the corresponding callee parameter
+      // System.out.println("---translate arg composite location to callee param. min="
+      // + min.printNode(0));
+      Map<Integer, NTuple<Descriptor>> mapIdxToArgTuple = mapMethodInvokeNodeToArgIdxMap.get(min);
+      Set<Integer> idxSet = mapIdxToArgTuple.keySet();
+      for (Iterator iterator2 = idxSet.iterator(); iterator2.hasNext();) {
+        Integer idx = (Integer) iterator2.next();
+
+        if (idx == 0 && !min.getMethod().isStatic()) {
+          continue;
+        }
+
+        NTuple<Descriptor> argTuple = mapIdxToArgTuple.get(idx);
+        if (argTuple.size() > 0) {
+          // check if an arg tuple has been already assigned to a composite location
+          NTuple<Location> argLocTuple = translateToLocTuple(mdCaller, argTuple);
+          Location argLocalLoc = argLocTuple.get(0);
+
+          // if (!isPrimitiveType(argTuple)) {
+          if (callerMapLocToCompLoc.containsKey(argLocalLoc)) {
+
+            CompositeLocation argLocalCompositeLocation = callerMapLocToCompLoc.get(argLocalLoc);
+            CompositeLocation argCompLoc = argLocalCompositeLocation.clone();
+            for (int i = 1; i < argLocTuple.size(); i++) {
+              argCompLoc.addLocation(argLocTuple.get(i));
+            }
+
+            FlowNode calleeParamFlowNode = calleeFlowGraph.getParamFlowNode(idx);
+
+            System.out
+                .println("----- argLocTuple=" + argLocTuple + "  argLocalLoc=+" + argLocalLoc);
+            System.out.println("-------need to translate argCompLoc=" + argCompLoc
+                + " with baseTuple=" + methodInvokeBaseLocTuple + "   calleeParamLocTuple="
+                + calleeParamFlowNode);
+
+            CompositeLocation paramCompLoc = translateArgCompLocToParamCompLoc(min, argCompLoc);
+            calleeParamFlowNode.setCompositeLocation(paramCompLoc);
+
+            // if (baseLocTuple != null && callerCompLoc.getTuple().startsWith(baseLocTuple)) {
+            //
+            // FlowNode calleeParamFlowNode = calleeFlowGraph.getParamFlowNode(idx);
+            // NTuple<Descriptor> calleeParamDescTuple = calleeParamFlowNode.getDescTuple();
+            // NTuple<Location> calleeParamLocTuple
+            // =###translateCompositeLocationAssignmentToFlowGraph mdCaller=public static void
+            // huffcodetab.huffman_decoder(int htIdx, int x, BitReserve br)
+
+            // translateToLocTuple(mdCallee, calleeParamDescTuple);
+            //
+            // System.out.println("---need to translate callerCompLoc=" + callerCompLoc
+            // + " with baseTuple=" + baseLocTuple + "   calleeParamLocTuple="
+            // + calleeParamLocTuple);
+            //
+            // CompositeLocation newCalleeCompLoc =
+            // translateCompositeLocationToCallee(callerCompLoc, baseLocTuple, mdCallee);
+            //
+            // calleeGlobalGraph.addMapLocationToInferCompositeLocation(calleeParamLocTuple.get(0),
+            // newCalleeCompLoc);
+            //
+            // System.out.println("---callee loc=" + calleeParamLocTuple.get(0)
+            // + "  newCalleeCompLoc=" + newCalleeCompLoc);
+            //
+            // // System.out.println("###need to assign composite location to=" +
+            // // calleeParamDescTuple
+            // // + " with baseTuple=" + baseLocTuple);
+            // }
+
+          }
+        }
+      }
     }
+    // ////////////////
+    // ////////////////
 
     for (Iterator iterator = calleeSet.iterator(); iterator.hasNext();) {
       MethodDescriptor callee = (MethodDescriptor) iterator.next();
       translateCompositeLocationAssignmentToFlowGraph(callee);
     }
 
+    for (Iterator iterator = minSet.iterator(); iterator.hasNext();) {
+      MethodInvokeNode min = (MethodInvokeNode) iterator.next();
+      // add an additional ordering constraint
+      // if the first element of a parameter composite location matches 'this' reference,
+      // the corresponding argument in the caller is required to be higher than the translated
+      // parameter location in the caller lattice
+      // TODO
+      // addOrderingConstraintFromCompLocParamToArg(mdCaller, min);
+    }
+
+  }
+
+  private CompositeLocation translateArgCompLocToParamCompLoc(MethodInvokeNode min,
+      CompositeLocation argCompLoc) {
+
+    System.out.println("--------translateArgCompLocToParamCompLoc argCompLoc=" + argCompLoc);
+    MethodDescriptor mdCallee = min.getMethod();
+    FlowGraph calleeFlowGraph = getFlowGraph(mdCallee);
+
+    NTuple<Location> argLocTuple = argCompLoc.getTuple();
+    Location argLocalLoc = argLocTuple.get(0);
+
+    Map<Integer, NTuple<Descriptor>> mapIdxToArgTuple = mapMethodInvokeNodeToArgIdxMap.get(min);
+    Set<Integer> idxSet = mapIdxToArgTuple.keySet();
+    for (Iterator iterator2 = idxSet.iterator(); iterator2.hasNext();) {
+      Integer idx = (Integer) iterator2.next();
+
+      if (idx == 0 && !min.getMethod().isStatic()) {
+        continue;
+      }
+
+      NTuple<Descriptor> argTuple = mapIdxToArgTuple.get(idx);
+      if (argTuple.size() > 0 && argTuple.get(0).equals(argLocalLoc.getLocDescriptor())) {
+        // it matches with the current argument composite location
+        // so what is the corresponding parameter local descriptor?
+        FlowNode paramNode = calleeFlowGraph.getParamFlowNode(idx);
+        System.out.println("----------found paramNode=" + paramNode);
+        NTuple<Descriptor> paramDescTuple = paramNode.getCurrentDescTuple();
+
+        NTuple<Location> newParamTupleFromArgTuple = translateToLocTuple(mdCallee, paramDescTuple);
+        for (int i = 1; i < argLocTuple.size(); i++) {
+          newParamTupleFromArgTuple.add(argLocTuple.get(i));
+        }
+
+        System.out.println("-----------newParamTuple=" + newParamTupleFromArgTuple);
+        return new CompositeLocation(newParamTupleFromArgTuple);
+
+      }
+    }
+    return null;
+  }
+
+  private void addAddtionalOrderingConstraints(MethodDescriptor mdCaller) {
+
+    // First, assign a composite location to a node in the flow graph
+    GlobalFlowGraph callerGlobalFlowGraph = getSubGlobalFlowGraph(mdCaller);
+
+    FlowGraph callerFlowGraph = getFlowGraph(mdCaller);
+    Map<Location, CompositeLocation> callerMapLocToCompLoc =
+        callerGlobalFlowGraph.getMapLocationToInferCompositeLocation();
+    Set<Location> methodLocSet = callerMapLocToCompLoc.keySet();
+
+    Set<MethodInvokeNode> minSet = mapMethodDescriptorToMethodInvokeNodeSet.get(mdCaller);
+
+    Set<MethodDescriptor> calleeSet = new HashSet<MethodDescriptor>();
+    for (Iterator iterator = minSet.iterator(); iterator.hasNext();) {
+      MethodInvokeNode min = (MethodInvokeNode) iterator.next();
+      MethodDescriptor mdCallee = min.getMethod();
+      calleeSet.add(mdCallee);
+
+      //
+      // add an additional ordering constraint
+      // if the first element of a parameter composite location matches 'this' reference,
+      // the corresponding argument in the caller is required to be higher than the translated
+      // parameter location in the caller lattice
+      // TODO
+      // addOrderingConstraintFromCompLocParamToArg(mdCaller, min);
+
+      //
+      // update return flow nodes in the caller
+      CompositeLocation returnLoc = getMethodSummary(mdCallee).getRETURNLoc();
+
+      System.out.println("### min=" + min.printNode(0) + "  returnLoc=" + returnLoc);
+      if (returnLoc != null && returnLoc.get(0).getLocDescriptor().equals(mdCallee.getThis())
+          && returnLoc.getSize() > 1) {
+        System.out.println("###RETURN COMP LOC=" + returnLoc);
+        NTuple<Location> returnLocTuple = returnLoc.getTuple();
+        NTuple<Descriptor> baseTuple = mapMethodInvokeNodeToBaseTuple.get(min);
+        NTuple<Descriptor> newReturnTuple = baseTuple.clone();
+        for (int i = 1; i < returnLocTuple.size(); i++) {
+          newReturnTuple.add(returnLocTuple.get(i).getLocDescriptor());
+        }
+        System.out.println("###NEW RETURN TUPLE FOR CALLER=" + newReturnTuple);
+        callerFlowGraph.getFlowReturnNode(min).setNewTuple(newReturnTuple);
+      }
+
+    }
+
+    for (Iterator iterator = calleeSet.iterator(); iterator.hasNext();) {
+      MethodDescriptor callee = (MethodDescriptor) iterator.next();
+      addAddtionalOrderingConstraints(callee);
+    }
+
+  }
+
+  private void addOrderingConstraintFromCompLocParamToArg(MethodDescriptor mdCaller,
+      MethodInvokeNode min) {
+    System.out.println("-addOrderingConstraintFromCompLocParamToArg=" + min.printNode(0));
+
+    GlobalFlowGraph globalGraph = getSubGlobalFlowGraph(ssjava.getMethodContainingSSJavaLoop());
+
+    Set<NTuple<Location>> pcLocTupleSet = getPCLocTupleSet(min);
+
+    MethodDescriptor mdCallee = min.getMethod();
+
+    FlowGraph calleeFlowGraph = getFlowGraph(mdCallee);
+    for (int idx = 0; idx < calleeFlowGraph.getNumParameters(); idx++) {
+      FlowNode paramNode = calleeFlowGraph.getParamFlowNode(idx);
+      NTuple<Location> globalParamLocTuple =
+          translateToLocTuple(mdCallee, paramNode.getDescTuple());
+      translateToLocTuple(mdCallee, paramNode.getDescTuple());
+      CompositeLocation compLoc = paramNode.getCompositeLocation();
+      System.out.println("---paramNode=" + paramNode + "    compLoc=" + compLoc);
+      if (compLoc != null) {
+        NTuple<Descriptor> argTuple = getNodeTupleByArgIdx(min, idx);
+        NTuple<Location> globalArgLocTuple = translateToLocTuple(mdCaller, argTuple);
+
+        if (!isLiteralValueLocTuple(globalArgLocTuple)
+            && !isLiteralValueLocTuple(globalParamLocTuple)) {
+          if (!globalGraph.hasValueFlowEdge(globalArgLocTuple, globalParamLocTuple)) {
+            System.out.println("----- add global flow globalArgLocTuple=" + globalArgLocTuple
+                + "-> globalParamLocTuple=" + globalParamLocTuple);
+            hasChanges = true;
+            globalGraph.addValueFlowEdge(globalArgLocTuple, globalParamLocTuple);
+          }
+        }
+
+        for (Iterator iterator = pcLocTupleSet.iterator(); iterator.hasNext();) {
+          NTuple<Location> pcLocTuple = (NTuple<Location>) iterator.next();
+
+          if (!isLiteralValueLocTuple(pcLocTuple) && !isLiteralValueLocTuple(globalParamLocTuple)) {
+            if (!globalGraph.hasValueFlowEdge(pcLocTuple, globalParamLocTuple)) {
+              System.out
+                  .println("----- add global flow PCLOC="
+                      + pcLocTuple
+                      + "-> globalParamLocTu!globalArgLocTuple.get(0).getLocDescriptor().equals(LITERALDESC)ple="
+                      + globalParamLocTuple);
+              hasChanges = true;
+              globalGraph.addValueFlowEdge(pcLocTuple, globalParamLocTuple);
+            }
+          }
+
+        }
+      }
+    }
+  }
+
+  private boolean isLiteralValueLocTuple(NTuple<Location> locTuple) {
+    return locTuple.get(0).getLocDescriptor().equals(LITERALDESC);
   }
 
   public void assignCompositeLocationToFlowGraph(FlowGraph flowGraph, Location loc,
@@ -411,7 +708,8 @@ public class LocationInference {
     Set<FlowNode> nodeSet = flowGraph.getNodeSet();
     for (Iterator iterator = nodeSet.iterator(); iterator.hasNext();) {
       FlowNode node = (FlowNode) iterator.next();
-      if (node.getDescTuple().startsWith(localDesc)) {
+      if (node.getDescTuple().startsWith(localDesc)
+          && !node.getDescTuple().get(0).equals(LITERALDESC)) {
         // need to assign the inferred composite location to this node
         CompositeLocation newCompLoc = generateCompositeLocation(node.getDescTuple(), inferCompLoc);
         node.setCompositeLocation(newCompLoc);
@@ -458,6 +756,8 @@ public class LocationInference {
     Map<Location, CompositeLocation> callerMapLocToCompLoc =
         callerGraph.getMapLocationToInferCompositeLocation();
 
+    Map<Integer, NTuple<Descriptor>> mapIdxToArgTuple = mapMethodInvokeNodeToArgIdxMap.get(min);
+
     FlowGraph calleeFlowGraph = getFlowGraph(mdCallee);
     GlobalFlowGraph calleeGlobalGraph = getSubGlobalFlowGraph(mdCallee);
 
@@ -466,8 +766,11 @@ public class LocationInference {
       baseLocTuple = translateToLocTuple(mdCaller, mapMethodInvokeNodeToBaseTuple.get(min));
     }
 
-    // System.out.println("\n-translate caller infer composite loc to callee=" + mdCallee
-    // + " baseLocTuple=" + baseLocTuple);
+    System.out.println("\n-#translate caller=" + mdCaller + " infer composite loc to callee="
+        + mdCallee + " baseLocTuple=" + baseLocTuple);
+    // System.out.println("-mapIdxToArgTuple=" + mapIdxToArgTuple);
+    System.out.println("-callerMapLocToCompLoc=" + callerMapLocToCompLoc);
+
     Set<Location> keySet = callerMapLocToCompLoc.keySet();
     for (Iterator iterator = keySet.iterator(); iterator.hasNext();) {
       Location key = (Location) iterator.next();
@@ -477,13 +780,16 @@ public class LocationInference {
 
         CompositeLocation newCalleeCompLoc;
         if (baseLocTuple != null && callerCompLoc.getTuple().startsWith(baseLocTuple)) {
-          // System.out.println("---need to translate callerCompLoc=" + callerCompLoc
-          // + " with baseTuple=" + baseLocTuple);
+          System.out.println("-----need to translate callerCompLoc=" + callerCompLoc
+              + " with baseTuple=" + baseLocTuple);
           newCalleeCompLoc =
               translateCompositeLocationToCallee(callerCompLoc, baseLocTuple, mdCallee);
 
           calleeGlobalGraph.addMapLocationToInferCompositeLocation(key, newCalleeCompLoc);
-          // System.out.println("---callee loc=" + key + "  newCalleeCompLoc=" + newCalleeCompLoc);
+          System.out.println("---key=" + key + "  callerCompLoc=" + callerCompLoc
+              + "  newCalleeCompLoc=" + newCalleeCompLoc);
+          System.out.println("-----baseLoctuple=" + baseLocTuple);
+          System.out.println("-----caller=" + mdCaller + "    callee=" + mdCallee);
         } else {
           // check if it is the global access
           Location compLocFirstElement = callerCompLoc.getTuple().get(0);
@@ -498,6 +804,33 @@ public class LocationInference {
               newCalleeCompLoc.addLocation(callerCompLoc.get(i));
             }
             calleeGlobalGraph.addMapLocationToInferCompositeLocation(key, newCalleeCompLoc);
+            System.out.println("---key=" + key + "  callerCompLoc=" + callerCompLoc
+                + "  newCalleeCompLoc=" + newCalleeCompLoc);
+            System.out.println("-----caller=" + mdCaller + "    callee=" + mdCallee);
+
+          } else {
+            int paramIdx = getParamIdx(callerCompLoc, mapIdxToArgTuple);
+            if (paramIdx == -1) {
+              continue;
+            }
+            NTuple<Descriptor> argTuple = mapIdxToArgTuple.get(paramIdx);
+
+            FlowNode paramFlowNode = calleeFlowGraph.getParamFlowNode(paramIdx);
+            System.out.println("-----paramIdx=" + paramIdx + "  paramFlowNode=" + paramFlowNode);
+            NTuple<Location> paramLocTuple =
+                translateToLocTuple(mdCallee, paramFlowNode.getDescTuple());
+            newCalleeCompLoc = new CompositeLocation();
+            for (int i = 0; i < paramLocTuple.size(); i++) {
+              newCalleeCompLoc.addLocation(paramLocTuple.get(i));
+            }
+            for (int i = argTuple.size(); i < callerCompLoc.getSize(); i++) {
+              newCalleeCompLoc.addLocation(callerCompLoc.get(i));
+            }
+            calleeGlobalGraph.addMapLocationToInferCompositeLocation(key, newCalleeCompLoc);
+            System.out.println("---key=" + key + "  callerCompLoc=" + callerCompLoc
+                + "  newCalleeCompLoc=" + newCalleeCompLoc);
+            System.out.println("------argTuple=" + argTuple);
+            System.out.println("-----caller=" + mdCaller + "    callee=" + mdCallee);
 
           }
 
@@ -511,9 +844,6 @@ public class LocationInference {
 
     // If the location of an argument has a composite location
     // need to assign a proper composite location to the corresponding callee parameter
-    // System.out.println("---translate arg composite location to callee param. min="
-    // + min.printNode(0));
-    Map<Integer, NTuple<Descriptor>> mapIdxToArgTuple = mapMethodInvokeNodeToArgIdxMap.get(min);
     Set<Integer> idxSet = mapIdxToArgTuple.keySet();
     for (Iterator iterator = idxSet.iterator(); iterator.hasNext();) {
       Integer idx = (Integer) iterator.next();
@@ -543,9 +873,9 @@ public class LocationInference {
             NTuple<Location> calleeParamLocTuple =
                 translateToLocTuple(mdCallee, calleeParamDescTuple);
 
-            System.out.println("---need to translate callerCompLoc=" + callerCompLoc
-                + " with baseTuple=" + baseLocTuple + "   calleeParamLocTuple="
-                + calleeParamLocTuple);
+            // System.out.println("---need to translate callerCompLoc=" + callerCompLoc
+            // + " with baseTuple=" + baseLocTuple + "   calleeParamLocTuple="
+            // + calleeParamLocTuple);
 
             CompositeLocation newCalleeCompLoc =
                 translateCompositeLocationToCallee(callerCompLoc, baseLocTuple, mdCallee);
@@ -553,11 +883,9 @@ public class LocationInference {
             calleeGlobalGraph.addMapLocationToInferCompositeLocation(calleeParamLocTuple.get(0),
                 newCalleeCompLoc);
 
-            System.out.println("---callee loc=" + calleeParamLocTuple.get(0)
-                + "  newCalleeCompLoc=" + newCalleeCompLoc);
+            // System.out.println("---callee loc=" + calleeParamLocTuple.get(0)
+            // + "  newCalleeCompLoc=" + newCalleeCompLoc);
 
-            // System.out.println("###need to assign composite location to=" + calleeParamDescTuple
-            // + " with baseTuple=" + baseLocTuple);
           }
 
         }
@@ -567,6 +895,25 @@ public class LocationInference {
 
   }
 
+  private int getParamIdx(CompositeLocation compLoc,
+      Map<Integer, NTuple<Descriptor>> mapIdxToArgTuple) {
+
+    // if the composite location is started with the argument descriptor
+    // return the argument's index. o.t. return -1
+
+    Set<Integer> keySet = mapIdxToArgTuple.keySet();
+    for (Iterator iterator = keySet.iterator(); iterator.hasNext();) {
+      Integer key = (Integer) iterator.next();
+      NTuple<Descriptor> argTuple = mapIdxToArgTuple.get(key);
+      if (argTuple.size() > 0 && translateToDescTuple(compLoc.getTuple()).startsWith(argTuple)) {
+        System.out.println("compLoc.getTuple=" + compLoc + " is started with " + argTuple);
+        return key.intValue();
+      }
+    }
+
+    return -1;
+  }
+
   private boolean isPrimitiveType(NTuple<Descriptor> argTuple) {
 
     Descriptor lastDesc = argTuple.get(argTuple.size() - 1);
@@ -575,9 +922,11 @@ public class LocationInference {
       return ((FieldDescriptor) lastDesc).getType().isPrimitive();
     } else if (lastDesc instanceof VarDescriptor) {
       return ((VarDescriptor) lastDesc).getType().isPrimitive();
+    } else if (lastDesc instanceof InterDescriptor) {
+      return true;
     }
 
-    return true;
+    return false;
   }
 
   private CompositeLocation translateCompositeLocationToCallee(CompositeLocation callerCompLoc,
@@ -628,8 +977,7 @@ public class LocationInference {
           globalFlowGraph.getReachableNodeSetByPrefix(node.getLocTuple().get(0));
       // Set<GlobalFlowNode> reachNodeSet = globalFlowGraph.getReachableNodeSetFrom(node);
 
-      // System.out.println("node=" + node + "    prefixList=" + prefixList + "   reachableNodeSet="
-      // + reachableNodeSet);
+      // System.out.println("node=" + node + "    prefixList=" + prefixList);
 
       for (int i = 0; i < prefixList.size(); i++) {
         NTuple<Location> curPrefix = prefixList.get(i);
@@ -641,6 +989,7 @@ public class LocationInference {
             reachableCommonPrefixSet.add(reachNode.getLocTuple());
           }
         }
+        // System.out.println("reachableCommonPrefixSet=" + reachableCommonPrefixSet);
 
         if (!reachableCommonPrefixSet.isEmpty()) {
 
@@ -650,6 +999,11 @@ public class LocationInference {
           MethodDescriptor nodePrefixLocFirstElementMethodDesc =
               (MethodDescriptor) prefixLoc.getDescriptor();
 
+          // System.out.println("curPrefixFirstElementMethodDesc=" +
+          // curPrefixFirstElementMethodDesc);
+          // System.out.println("nodePrefixLocFirstElementMethodDesc="
+          // + nodePrefixLocFirstElementMethodDesc);
+
           if (curPrefixFirstElementMethodDesc.equals(nodePrefixLocFirstElementMethodDesc)
               || isTransitivelyCalledFrom(nodePrefixLocFirstElementMethodDesc,
                   curPrefixFirstElementMethodDesc)) {
@@ -675,7 +1029,8 @@ public class LocationInference {
 
             CompositeLocation newCompLoc = generateCompositeLocation(curPrefix);
             System.out.println("NEED TO ASSIGN COMP LOC TO " + node + " with prefix=" + curPrefix);
-            System.out.println("- newCompLoc=" + newCompLoc);
+            System.out.println("-targetLocalLoc=" + targetLocalLoc + "   - newCompLoc="
+                + newCompLoc);
             globalFlowGraph.addMapLocationToInferCompositeLocation(targetLocalLoc, newCompLoc);
             // }
 
@@ -709,18 +1064,17 @@ public class LocationInference {
 
     Set<GlobalFlowNode> incomingNodeSetPrefix =
         graph.getIncomingNodeSetByPrefix(node.getLocTuple().get(0));
-    // System.out.println("incomingNodeSetPrefix=" + incomingNodeSetPrefix);
-    //
-    // Set<GlobalFlowNode> reachableNodeSetPrefix =
-    // graph.getReachableNodeSetByPrefix(node.getLocTuple().get(0));
-    // System.out.println("reachableNodeSetPrefix=" + reachableNodeSetPrefix);
+    System.out.println("incomingNodeSetPrefix=" + incomingNodeSetPrefix);
+
+    Set<GlobalFlowNode> reachableNodeSetPrefix =
+        graph.getReachableNodeSetByPrefix(node.getLocTuple().get(0));
+    System.out.println("reachableNodeSetPrefix=" + reachableNodeSetPrefix);
 
     List<NTuple<Location>> prefixList = new ArrayList<NTuple<Location>>();
 
     for (Iterator iterator = incomingNodeSetPrefix.iterator(); iterator.hasNext();) {
       GlobalFlowNode inNode = (GlobalFlowNode) iterator.next();
       NTuple<Location> inNodeTuple = inNode.getLocTuple();
-
       for (int i = 1; i < inNodeTuple.size(); i++) {
         NTuple<Location> prefix = inNodeTuple.subList(0, i);
         if (!prefixList.contains(prefix)) {
@@ -749,14 +1103,15 @@ public class LocationInference {
     ClassDescriptor cd = md.getClassDesc();
 
     int idx = 0;
-
     Set<NTuple<Location>> toberemoved = new HashSet<NTuple<Location>>();
-    for (int i = 0; i < prefixList.size(); i++) {
-      NTuple<Location> prefixLocTuple = prefixList.get(i);
-      if (!containsClassDesc(cd, prefixLocTuple)) {
-        toberemoved.add(prefixLocTuple);
-      }
-    }
+    // for (int i = 0; i < prefixList.size(); i++) {
+    // NTuple<Location> prefixLocTuple = prefixList.get(i);
+    // if (!containsClassDesc(cd, prefixLocTuple)) {
+    // toberemoved.add(prefixLocTuple);
+    // }
+    // }
+
+    // System.out.println("method class=" + cd + "  toberemoved=" + toberemoved);
 
     prefixList.removeAll(toberemoved);
 
@@ -810,7 +1165,7 @@ public class LocationInference {
 
     MethodDescriptor md = flowGraph.getMethodDescriptor();
 
-    GlobalFlowGraph globalGraph = new GlobalFlowGraph(md);
+    GlobalFlowGraph globalGraph = getSubGlobalFlowGraph(md);
 
     // Set<FlowNode> nodeSet = flowGraph.getNodeSet();
     Set<FlowEdge> edgeSet = flowGraph.getEdgeSet();
@@ -915,8 +1270,8 @@ public class LocationInference {
     FlowGraph calleeFlowGraph = getFlowGraph(possibleMdCallee);
     Map<Integer, NTuple<Descriptor>> mapIdxToArg = mapMethodInvokeNodeToArgIdxMap.get(min);
 
-    System.out.println("-----mapMethodInvokeNodeToArgIdxMap.get(min)="
-        + mapMethodInvokeNodeToArgIdxMap.get(min));
+    // System.out.println("-----mapMethodInvokeNodeToArgIdxMap.get(min)="
+    // + mapMethodInvokeNodeToArgIdxMap.get(min));
     Set<Integer> keySet = mapIdxToArg.keySet();
     for (Iterator iterator = keySet.iterator(); iterator.hasNext();) {
       Integer idx = (Integer) iterator.next();
@@ -929,6 +1284,8 @@ public class LocationInference {
       }
     }
 
+    addValueFlowBetweenParametersToCaller(min, mdCaller, possibleMdCallee);
+
     NTuple<Descriptor> baseTuple = mapMethodInvokeNodeToBaseTuple.get(min);
     GlobalFlowGraph calleeSubGlobalGraph = getSubGlobalFlowGraph(possibleMdCallee);
     Set<GlobalFlowNode> calleeNodeSet = calleeSubGlobalGraph.getNodeSet();
@@ -937,307 +1294,191 @@ public class LocationInference {
       addValueFlowFromCalleeNode(min, mdCaller, possibleMdCallee, calleeNode);
     }
 
-    // int numParam = calleeFlowGraph.getNumParameters();
-    // for (int idx = 0; idx < numParam; idx++) {
-    //
-    // FlowNode paramNode = calleeFlowGraph.getParamFlowNode(idx);
-    //
-    // NTuple<Location> paramLocTuple =
-    // translateToLocTuple(possibleMdCallee, paramNode.getCurrentDescTuple());
-    //
-    // GlobalFlowNode globalParamNode =
-    // calleeSubGlobalGraph.getFlowNode(paramLocTuple);
-    //
-    // NTuple<Descriptor> argTuple =
-    // mapMethodInvokeNodeToArgIdxMap.get(min).get(idx);
-    //
-    // NTuple<Location> argLocTuple = translateToLocTuple(mdCaller, argTuple);
-    //
-    // System.out.println("argTupleSet=" + argLocTuple + "   param=" +
-    // paramLocTuple);
-    // // here, it adds all value flows reachable from the paramNode in the
-    // callee's flow graph
-    //
-    // addValueFlowsFromCalleeParam(mdCaller, argLocTuple, baseLocTuple,
-    // possibleMdCallee,
-    // globalParamNode);
-    // }
-    //
-    // // TODO
-    // // FlowGraph callerSubGlobalGraph = getSubGlobalFlowGraph(mdCaller);
-    // // FlowGraph calleeSubGlobalGraph =
-    // getSubGlobalFlowGraph(possibleMdCallee);
-    // //
-    // // int numParam = calleeSubGlobalGraph.getNumParameters();
-    // // for (int idx = 0; idx < numParam; idx++) {
-    // // FlowNode paramNode = calleeSubGlobalGraph.getParamFlowNode(idx);
-    // // NTuple<Descriptor> argTuple =
-    // mapMethodInvokeNodeToArgIdxMap.get(min).get(idx);
-    // // System.out.println("argTupleSet=" + argTuple + "   param=" +
-    // paramNode);
-    // // // here, it adds all value flows reachable from the paramNode in the
-    // callee's flow graph
-    // // addValueFlowsFromCalleeParam(min, calleeSubGlobalGraph, paramNode,
-    // callerSubGlobalGraph,
-    // // argTuple, baseTuple);
-    // // }
-
   }
 
-  private void addValueFlowFromCalleeNode(MethodInvokeNode min, MethodDescriptor mdCaller,
-      MethodDescriptor mdCallee, GlobalFlowNode calleeSrcNode) {
+  private void addValueFlowBetweenParametersToCaller(MethodInvokeNode min,
+      MethodDescriptor mdCaller, MethodDescriptor mdCallee) {
+
+    System.out.println("***addValueFlowBetweenParametersToCaller from mdCallee=" + mdCallee);
+
+    Set<NTuple<Location>> PCLocTupleSet = mapMethodInvokeNodeToPCLocTupleSet.get(min);
+    System.out.println("-PCLocTupleSet=" + PCLocTupleSet);
 
     GlobalFlowGraph calleeSubGlobalGraph = getSubGlobalFlowGraph(mdCallee);
     GlobalFlowGraph callerSubGlobalGraph = getSubGlobalFlowGraph(mdCaller);
 
-    NTuple<Location> callerSrcNodeLocTuple =
-        translateToCallerLocTuple(min, mdCallee, mdCaller, calleeSrcNode.getLocTuple());
-
-    if (callerSrcNodeLocTuple != null) {
-      Set<GlobalFlowNode> outNodeSet = calleeSubGlobalGraph.getOutNodeSet(calleeSrcNode);
+    // if the parameter A reaches to the parameter B
+    // then, add an edge the argument A -> the argument B to the global flow graph
+    FlowGraph calleeFlowGraph = getFlowGraph(mdCallee);
+    FlowGraph callerFlowGraph = getFlowGraph(mdCaller);
+    int numParam = calleeFlowGraph.getNumParameters();
 
-      for (Iterator iterator = outNodeSet.iterator(); iterator.hasNext();) {
-        GlobalFlowNode outNode = (GlobalFlowNode) iterator.next();
-        NTuple<Location> callerDstNodeLocTuple =
-            translateToCallerLocTuple(min, mdCallee, mdCaller, outNode.getLocTuple());
-        if (callerDstNodeLocTuple != null) {
-          callerSubGlobalGraph.addValueFlowEdge(callerSrcNodeLocTuple, callerDstNodeLocTuple);
-        }
-      }
-    }
+    for (int i = 0; i < numParam; i++) {
+      for (int k = 0; k < numParam; k++) {
 
-  }
+        if (i != k) {
 
-  private NTuple<Location> translateToCallerLocTuple(MethodInvokeNode min,
-      MethodDescriptor mdCallee, MethodDescriptor mdCaller, NTuple<Location> nodeLocTuple) {
-    // this method will return the same nodeLocTuple if the corresponding argument is literal
-    // value.
+          System.out.println("i=" + i + " k=" + k);
 
-    FlowGraph calleeFlowGraph = getFlowGraph(mdCallee);
+          FlowNode paramNode1 = calleeFlowGraph.getParamFlowNode(i);
+          FlowNode paramNode2 = calleeFlowGraph.getParamFlowNode(k);
 
-    NTuple<Descriptor> nodeDescTuple = translateToDescTuple(nodeLocTuple);
-    if (calleeFlowGraph.isParameter(nodeDescTuple)) {
-      int paramIdx = calleeFlowGraph.getParamIdx(nodeDescTuple);
-      NTuple<Descriptor> argDescTuple = mapMethodInvokeNodeToArgIdxMap.get(min).get(paramIdx);
+          NTuple<Descriptor> arg1Tuple = getNodeTupleByArgIdx(min, i);
+          NTuple<Descriptor> arg2Tuple = getNodeTupleByArgIdx(min, k);
 
-      if (isPrimitive(nodeLocTuple.get(0).getLocDescriptor())) {
-        // the type of argument is primitive.
-        return nodeLocTuple.clone();
-      }
-      NTuple<Location> argLocTuple = translateToLocTuple(mdCaller, argDescTuple);
+          NTuple<Descriptor> paramDescTuple1 = paramNode1.getCurrentDescTuple();
+          NTuple<Descriptor> paramDescTuple2 = paramNode2.getCurrentDescTuple();
 
-      NTuple<Location> callerLocTuple = new NTuple<Location>();
+          if (paramDescTuple1.get(0).equals(paramDescTuple2.get(0))) {
+            // if two parameters share the same prefix
+            // it already has been assigned to a composite location
+            // so we don't need to add an additional ordering relation caused by these two
+            // paramters.
+            continue;
+          }
 
-      callerLocTuple.addAll(argLocTuple);
-      for (int i = 1; i < nodeLocTuple.size(); i++) {
-        callerLocTuple.add(nodeLocTuple.get(i));
-      }
-      return callerLocTuple;
-    } else {
-      return nodeLocTuple.clone();
-    }
+          NTuple<Location> paramLocTuple1 = translateToLocTuple(mdCallee, paramDescTuple1);
+          NTuple<Location> paramLocTuple2 = translateToLocTuple(mdCallee, paramDescTuple2);
 
-  }
+          // check if the callee propagates an ordering constraints through
+          // parameters
 
-  public static boolean isPrimitive(Descriptor desc) {
+          // Set<FlowNode> localReachSet = calleeFlowGraph.getLocalReachFlowNodeSetFrom(paramNode1);
 
-    if (desc instanceof FieldDescriptor) {
-      return ((FieldDescriptor) desc).getType().isPrimitive();
-    } else if (desc instanceof VarDescriptor) {
-      return ((VarDescriptor) desc).getType().isPrimitive();
-    } else if (desc instanceof InterDescriptor) {
-      return true;
-    }
+          Set<GlobalFlowNode> reachToParam1Set =
+              calleeSubGlobalGraph.getIncomingNodeSetByPrefix(paramLocTuple1.get(0));
 
-    return false;
-  }
+          // System.out.println("-- localReachSet from param1=" + localReachSet);
 
-  private NTuple<Descriptor> translateToDescTuple(NTuple<Location> locTuple) {
+          GlobalFlowNode globalFlowNodeParam1 = calleeSubGlobalGraph.getFlowNode(paramLocTuple1);
+          GlobalFlowNode globalFlowNodeParam2 = calleeSubGlobalGraph.getFlowNode(paramLocTuple2);
 
-    NTuple<Descriptor> descTuple = new NTuple<Descriptor>();
-    for (int i = 0; i < locTuple.size(); i++) {
-      descTuple.add(locTuple.get(i).getLocDescriptor());
-    }
-    return descTuple;
+          System.out.println("-param1CurTuple=" + paramDescTuple1 + " param2CurTuple="
+              + paramDescTuple2);
 
-  }
+          System.out.println("arg1Tuple=" + arg1Tuple + "   arg2Tuple=" + arg2Tuple);
+          // System.out.println("-reachToParam1Set=" + reachToParam1Set);
 
-  private void addValueFlowsFromCalleeParam(MethodDescriptor mdCaller,
-      NTuple<Location> argLocTuple, NTuple<Location> baseLocTuple, MethodDescriptor mdCallee,
-      GlobalFlowNode globalParamNode) {
+          if (arg1Tuple.size() > 0 && arg2Tuple.size() > 0
+              && reachToParam1Set.contains(globalFlowNodeParam2)) {
+            // need to propagate an ordering relation s.t. arg1 is higher
+            // than arg2
+            System.out.println("---param1=" + paramNode1 + " is higher than param2=" + paramNode2);
 
-    Set<GlobalFlowNode> visited = new HashSet<GlobalFlowNode>();
-    visited.add(globalParamNode);
-    recurAddValueFlowsFromCalleeParam(mdCaller, argLocTuple, baseLocTuple, mdCallee,
-        globalParamNode);
+            NTuple<Location> callerSrcNodeLocTuple =
+                translateToCallerLocTuple(min, mdCallee, mdCaller, paramLocTuple1);
 
-  }
+            NTuple<Location> callerDstNodeLocTuple =
+                translateToCallerLocTuple(min, mdCallee, mdCaller, paramLocTuple2);
 
-  private void recurAddValueFlowsFromCalleeParam(MethodDescriptor mdCaller,
-      NTuple<Location> argLocTuple, NTuple<Location> baseLocTuple, MethodDescriptor mdCallee,
-      GlobalFlowNode calleeCurNode) {
+            System.out.println("---callerSrcNodeLocTuple=" + callerSrcNodeLocTuple);
+            System.out.println("---callerDstNodeLocTuple=" + callerDstNodeLocTuple);
 
-    // FlowGraph calleeFlowGraph = getFlowGraph(mdCallee);
-    // GlobalFlowGraph calleeSubGlobalGraph = getSubGlobalFlowGraph(mdCallee);
-    //
-    // NTuple<Location> curNodeLocTuple = calleeCurNode.getLocTuple();
-    // NTuple<Descriptor> curNodeDescTuple = calleeCurNode.getDescTuple();
-    // if (calleeFlowGraph.isParameter(curNodeDescTuple)) {
-    // curNodeLocTuple = translateToCaller(argLocTuple, curNodeLocTuple);
-    // }
-    //
-    // Set<GlobalFlowNode> outNodeSet =
-    // calleeSubGlobalGraph.getOutNodeSet(calleeCurNode);
-    // for (Iterator iterator = outNodeSet.iterator(); iterator.hasNext();) {
-    // GlobalFlowNode outNode = (GlobalFlowNode) iterator.next();
-    //
-    // NTuple<Location> curNodeLocTuple = calleeCurNode.getLocTuple();
-    // NTuple<Descriptor> curNodeDescTuple = calleeCurNode.getDescTuple();
-    // if (calleeFlowGraph.isParameter(curNodeDescTuple)) {
-    // curNodeLocTuple = translateToCaller(argLocTuple, curNodeLocTuple);
-    // }
-    //
-    // outNode.getDescTuple();
-    //
-    // if (calleeFlowGraph.is)
-    //
-    // if (calleeSubGlobalGraph.isParameter(srcDescTuple)) {
-    // // destination node is started with 'parameter'
-    // // need to translate it in terms of the caller's a node
-    // srcDescTuple =
-    // translateToCaller(min, calleeSubGlobalGraph.getParamIdx(srcDescTuple),
-    // srcDescTuple);
-    // }
-    //
-    // }
-    //
-    // Set<FlowEdge> edgeSet =
-    // calleeSubGlobalGraph.getOutEdgeSetStartingFrom(calleeSrcNode);
-    // for (Iterator iterator = edgeSet.iterator(); iterator.hasNext();) {
-    // FlowEdge flowEdge = (FlowEdge) iterator.next();
-    //
-    // NTuple<Descriptor> srcDescTuple = flowEdge.getInitTuple();
-    // NTuple<Descriptor> dstDescTuple = flowEdge.getEndTuple();
-    //
-    // FlowNode dstNode = calleeSubGlobalGraph.getFlowNode(dstDescTuple);
-    //
-    // if (calleeSubGlobalGraph.isParameter(srcDescTuple)) {
-    // // destination node is started with 'parameter'
-    // // need to translate it in terms of the caller's a node
-    // srcDescTuple =
-    // translateToCaller(min, calleeSubGlobalGraph.getParamIdx(srcDescTuple),
-    // srcDescTuple);
-    // }
-    //
-    // if (calleeSubGlobalGraph.isParameter(dstDescTuple)) {
-    // // destination node is started with 'parameter'
-    // // need to translate it in terms of the caller's a node
-    // dstDescTuple =
-    // translateToCaller(min, calleeSubGlobalGraph.getParamIdx(dstDescTuple),
-    // dstDescTuple);
-    // }
-    //
-    // callerSubGlobalGraph.addValueFlowEdge(srcDescTuple, dstDescTuple);
-    //
-    // if (!visited.contains(dstNode)) {
-    // visited.add(dstNode);
-    // recurAddValueFlowsFromCalleeParam(min, calleeSubGlobalGraph, dstNode,
-    // callerSubGlobalGraph,
-    // dstDescTuple, visited, baseTuple);
-    // }
-    //
-    // }
+            System.out.println("-----add global value flow :" + callerSrcNodeLocTuple + "->"
+                + callerDstNodeLocTuple);
+            callerSubGlobalGraph.addValueFlowEdge(callerSrcNodeLocTuple, callerDstNodeLocTuple);
+            for (Iterator iterator = PCLocTupleSet.iterator(); iterator.hasNext();) {
+              NTuple<Location> pcLocTuple = (NTuple<Location>) iterator.next();
+              System.out.println("-----add global value flow PC :" + pcLocTuple + "->"
+                  + callerSrcNodeLocTuple);
+              callerSubGlobalGraph.addValueFlowEdge(pcLocTuple, callerSrcNodeLocTuple);
+            }
 
-  }
+            // add a new flow between the corresponding arguments.
+            callerFlowGraph.addValueFlowEdge(arg1Tuple, arg2Tuple);
+            System.out.println("arg1=" + arg1Tuple + "   arg2=" + arg2Tuple);
 
-  private NTuple<Location> translateToCaller(NTuple<Location> argLocTuple,
-      NTuple<Location> curNodeLocTuple) {
+            // System.out
+            // .println("-arg1Tuple=" + arg1Tuple + " is higher than arg2Tuple=" + arg2Tuple);
 
-    NTuple<Location> callerLocTuple = new NTuple<Location>();
+          }
 
-    callerLocTuple.addAll(argLocTuple);
-    for (int i = 1; i < curNodeLocTuple.size(); i++) {
-      callerLocTuple.add(curNodeLocTuple.get(i));
+          System.out.println();
+        }
+      }
     }
 
-    return callerLocTuple;
   }
 
-  private void recurAddValueFlowsFromCalleeParam(MethodInvokeNode min,
-      FlowGraph calleeSubGlobalGraph, FlowNode calleeSrcNode, FlowGraph callerSubGlobalGraph,
-      NTuple<Descriptor> callerSrcTuple, Set<FlowNode> visited, NTuple<Descriptor> baseTuple) {
+  private void addValueFlowFromCalleeNode(MethodInvokeNode min, MethodDescriptor mdCaller,
+      MethodDescriptor mdCallee, GlobalFlowNode calleeSrcNode) {
 
-    MethodDescriptor mdCallee = calleeSubGlobalGraph.getMethodDescriptor();
+    GlobalFlowGraph calleeSubGlobalGraph = getSubGlobalFlowGraph(mdCallee);
+    GlobalFlowGraph callerSubGlobalGraph = getSubGlobalFlowGraph(mdCaller);
 
-    // Set<FlowEdge> edgeSet =
-    // calleeSubGlobalGraph.getOutEdgeSet(calleeSrcNode);
-    Set<FlowEdge> edgeSet = calleeSubGlobalGraph.getOutEdgeSetStartingFrom(calleeSrcNode);
-    for (Iterator iterator = edgeSet.iterator(); iterator.hasNext();) {
-      FlowEdge flowEdge = (FlowEdge) iterator.next();
+    System.out.println("$addValueFlowFromCalleeNode calleeSrcNode=" + calleeSrcNode);
 
-      NTuple<Descriptor> srcDescTuple = flowEdge.getInitTuple();
-      NTuple<Descriptor> dstDescTuple = flowEdge.getEndTuple();
+    NTuple<Location> callerSrcNodeLocTuple =
+        translateToCallerLocTuple(min, mdCallee, mdCaller, calleeSrcNode.getLocTuple());
+    System.out.println("---callerSrcNodeLocTuple=" + callerSrcNodeLocTuple);
 
-      FlowNode dstNode = calleeSubGlobalGraph.getFlowNode(dstDescTuple);
+    if (callerSrcNodeLocTuple != null && callerSrcNodeLocTuple.size() > 0) {
+      Set<GlobalFlowNode> outNodeSet = calleeSubGlobalGraph.getOutNodeSet(calleeSrcNode);
 
-      if (calleeSubGlobalGraph.isParameter(srcDescTuple)) {
-        // destination node is started with 'parameter'
-        // need to translate it in terms of the caller's a node
-        srcDescTuple =
-            translateToCaller(min, calleeSubGlobalGraph.getParamIdx(srcDescTuple), srcDescTuple);
+      for (Iterator iterator = outNodeSet.iterator(); iterator.hasNext();) {
+        GlobalFlowNode outNode = (GlobalFlowNode) iterator.next();
+        NTuple<Location> callerDstNodeLocTuple =
+            translateToCallerLocTuple(min, mdCallee, mdCaller, outNode.getLocTuple());
+        if (callerDstNodeLocTuple != null) {
+          callerSubGlobalGraph.addValueFlowEdge(callerSrcNodeLocTuple, callerDstNodeLocTuple);
+        }
       }
+    }
+
+  }
+
+  private NTuple<Location> translateToCallerLocTuple(MethodInvokeNode min,
+      MethodDescriptor mdCallee, MethodDescriptor mdCaller, NTuple<Location> nodeLocTuple) {
+    // this method will return the same nodeLocTuple if the corresponding argument is literal
+    // value.
+
+    FlowGraph calleeFlowGraph = getFlowGraph(mdCallee);
+
+    NTuple<Descriptor> nodeDescTuple = translateToDescTuple(nodeLocTuple);
+    if (calleeFlowGraph.isParameter(nodeDescTuple)) {
+      int paramIdx = calleeFlowGraph.getParamIdx(nodeDescTuple);
+      NTuple<Descriptor> argDescTuple = mapMethodInvokeNodeToArgIdxMap.get(min).get(paramIdx);
 
-      if (calleeSubGlobalGraph.isParameter(dstDescTuple)) {
-        // destination node is started with 'parameter'
-        // need to translate it in terms of the caller's a node
-        dstDescTuple =
-            translateToCaller(min, calleeSubGlobalGraph.getParamIdx(dstDescTuple), dstDescTuple);
-      }
+      // if (isPrimitive(nodeLocTuple.get(0).getLocDescriptor())) {
+      // // the type of argument is primitive.
+      // return nodeLocTuple.clone();
+      // }
+      NTuple<Location> argLocTuple = translateToLocTuple(mdCaller, argDescTuple);
 
-      callerSubGlobalGraph.addValueFlowEdge(srcDescTuple, dstDescTuple);
+      NTuple<Location> callerLocTuple = new NTuple<Location>();
 
-      if (!visited.contains(dstNode)) {
-        visited.add(dstNode);
-        recurAddValueFlowsFromCalleeParam(min, calleeSubGlobalGraph, dstNode, callerSubGlobalGraph,
-            dstDescTuple, visited, baseTuple);
+      callerLocTuple.addAll(argLocTuple);
+      for (int i = 1; i < nodeLocTuple.size(); i++) {
+        callerLocTuple.add(nodeLocTuple.get(i));
       }
-
+      return callerLocTuple;
+    } else {
+      return nodeLocTuple.clone();
     }
 
   }
 
-  private NTuple<Descriptor> translateToCaller(MethodInvokeNode min, int paramIdx,
-      NTuple<Descriptor> srcDescTuple) {
-
-    NTuple<Descriptor> callerTuple = new NTuple<Descriptor>();
-
-    NTuple<Descriptor> argTuple = mapMethodInvokeNodeToArgIdxMap.get(min).get(paramIdx);
-
-    for (int i = 0; i < argTuple.size(); i++) {
-      callerTuple.add(argTuple.get(i));
-    }
+  public static boolean isPrimitive(Descriptor desc) {
 
-    for (int i = 1; i < srcDescTuple.size(); i++) {
-      callerTuple.add(srcDescTuple.get(i));
+    if (desc instanceof FieldDescriptor) {
+      return ((FieldDescriptor) desc).getType().isPrimitive();
+    } else if (desc instanceof VarDescriptor) {
+      return ((VarDescriptor) desc).getType().isPrimitive();
+    } else if (desc instanceof InterDescriptor) {
+      return true;
     }
 
-    return callerTuple;
+    return false;
   }
 
-  private NTuple<Descriptor> traslateToCalleeParamTupleToCallerArgTuple(
-      NTuple<Descriptor> calleeInitTuple, NTuple<Descriptor> callerSrcTuple) {
-
-    NTuple<Descriptor> callerInitTuple = new NTuple<Descriptor>();
-
-    for (int i = 0; i < callerSrcTuple.size(); i++) {
-      callerInitTuple.add(callerSrcTuple.get(i));
-    }
+  private NTuple<Descriptor> translateToDescTuple(NTuple<Location> locTuple) {
 
-    for (int i = 1; i < calleeInitTuple.size(); i++) {
-      callerInitTuple.add(calleeInitTuple.get(i));
+    NTuple<Descriptor> descTuple = new NTuple<Descriptor>();
+    for (int i = 0; i < locTuple.size(); i++) {
+      descTuple.add(locTuple.get(i).getLocDescriptor());
     }
+    return descTuple;
 
-    return callerInitTuple;
   }
 
   public LocationSummary getLocationSummary(Descriptor d) {
@@ -1279,7 +1520,8 @@ public class LocationInference {
       // inferred composite location
       for (int paramIdx = 0; paramIdx < flowGraph.getNumParameters(); paramIdx++) {
         FlowNode flowNode = flowGraph.getParamFlowNode(paramIdx);
-        CompositeLocation inferredCompLoc = flowNode.getCompositeLocation();
+        CompositeLocation inferredCompLoc =
+            updateCompositeLocation(flowNode.getCompositeLocation());
         // NTuple<Descriptor> descTuple = flowNode.getDescTuple();
         //
         // CompositeLocation assignedCompLoc = flowNode.getCompositeLocation();
@@ -1292,8 +1534,8 @@ public class LocationInference {
         // loc.setLocDescriptor(locDesc);
         // inferredCompLoc = new CompositeLocation(loc);
         // }
-        System.out.println("-paramIdx=" + paramIdx + "   infer=" + inferredCompLoc);
-        System.out.println("-flowNode inferLoc=" + flowNode.getCompositeLocation());
+        System.out.println("-paramIdx=" + paramIdx + "   infer=" + inferredCompLoc + " original="
+            + flowNode.getCompositeLocation());
 
         Descriptor localVarDesc = flowNode.getDescTuple().get(0);
         methodSummary.addMapVarNameToInferCompLoc(localVarDesc, inferredCompLoc);
@@ -1507,10 +1749,10 @@ public class LocationInference {
     Set<Descriptor> keySet = mapDescriptorToHierarchyGraph.keySet();
     for (Iterator iterator = keySet.iterator(); iterator.hasNext();) {
       Descriptor desc = (Descriptor) iterator.next();
+      // System.out.println("SSJAVA: remove redundant edges: " + desc);
       HierarchyGraph simpleHierarchyGraph = getHierarchyGraph(desc).clone();
       simpleHierarchyGraph.setName(desc + "_SIMPLE");
       simpleHierarchyGraph.removeRedundantEdges();
-      // simpleHierarchyGraph.simplifyHierarchyGraph();
       mapDescriptorToSimpleHierarchyGraph.put(desc, simpleHierarchyGraph);
     }
   }
@@ -1536,6 +1778,7 @@ public class LocationInference {
     Set<Descriptor> keySet = mapDescriptorToHierarchyGraph.keySet();
     for (Iterator iterator = keySet.iterator(); iterator.hasNext();) {
       Descriptor desc = (Descriptor) iterator.next();
+      System.out.println("SSJAVA: Constructing Skeleton Hierarchy Graph: " + desc);
       HierarchyGraph simpleGraph = getSimpleHierarchyGraph(desc);
       HierarchyGraph skeletonGraph = simpleGraph.generateSkeletonGraph();
       skeletonGraph.setMapDescToHNode(simpleGraph.getMapDescToHNode());
@@ -1610,7 +1853,6 @@ public class LocationInference {
 
     // do fixed-point analysis
 
-    ssjava.init();
     LinkedList<MethodDescriptor> descriptorListToAnalyze = ssjava.getSortedDescriptors();
 
     // Collections.sort(descriptorListToAnalyze, new
@@ -1683,6 +1925,29 @@ public class LocationInference {
       }
     }
 
+    Set<Descriptor> keySet = mapDescriptorToHierarchyGraph.keySet();
+    for (Iterator iterator = keySet.iterator(); iterator.hasNext();) {
+      Descriptor key = (Descriptor) iterator.next();
+      HierarchyGraph graph = getHierarchyGraph(key);
+
+      Set<HNode> nodeToBeConnected = new HashSet<HNode>();
+      for (Iterator iterator2 = graph.getNodeSet().iterator(); iterator2.hasNext();) {
+        HNode node = (HNode) iterator2.next();
+        if (!node.isSkeleton() && !node.isCombinationNode()) {
+          if (graph.getIncomingNodeSet(node).size() == 0) {
+            nodeToBeConnected.add(node);
+          }
+        }
+      }
+
+      for (Iterator iterator2 = nodeToBeConnected.iterator(); iterator2.hasNext();) {
+        HNode node = (HNode) iterator2.next();
+        System.out.println("NEED TO BE CONNECTED TO TOP=" + node);
+        graph.addEdge(graph.getHNode(TOPDESC), node);
+      }
+
+    }
+
   }
 
   private HierarchyGraph getHierarchyGraph(Descriptor d) {
@@ -1708,62 +1973,101 @@ public class LocationInference {
     // NTuple<Descriptor>
     boolean hasGlobalAccess = false;
     for (Iterator iterator = nodeSet.iterator(); iterator.hasNext();) {
-      FlowNode srcNode = (FlowNode) iterator.next();
-
-      // if the srcNode is started with the global descriptor
-      // need to set as a skeleton node
-      if (!hasGlobalAccess && srcNode.getDescTuple().startsWith(GLOBALDESC)) {
-        hasGlobalAccess = true;
+      FlowNode originalSrcNode = (FlowNode) iterator.next();
+
+      Set<FlowNode> sourceNodeSet = new HashSet<FlowNode>();
+      if (originalSrcNode instanceof FlowReturnNode) {
+        FlowReturnNode rnode = (FlowReturnNode) originalSrcNode;
+        Set<NTuple<Descriptor>> tupleSet = rnode.getTupleSet();
+        for (Iterator iterator2 = tupleSet.iterator(); iterator2.hasNext();) {
+          NTuple<Descriptor> nTuple = (NTuple<Descriptor>) iterator2.next();
+          sourceNodeSet.add(fg.getFlowNode(nTuple));
+          System.out.println("&&&SOURCE fg.getFlowNode(nTuple)=" + fg.getFlowNode(nTuple));
+        }
+      } else {
+        sourceNodeSet.add(originalSrcNode);
       }
 
-      Set<FlowEdge> outEdgeSet = fg.getOutEdgeSet(srcNode);
-      for (Iterator iterator2 = outEdgeSet.iterator(); iterator2.hasNext();) {
-        FlowEdge outEdge = (FlowEdge) iterator2.next();
-        FlowNode dstNode = outEdge.getDst();
+      System.out.println("---sourceNodeSet=" + sourceNodeSet);
+      for (Iterator iterator3 = sourceNodeSet.iterator(); iterator3.hasNext();) {
+        FlowNode srcNode = (FlowNode) iterator3.next();
+
+        // if the srcNode is started with the global descriptor
+        // need to set as a skeleton node
+        if (!hasGlobalAccess && srcNode.getDescTuple().startsWith(GLOBALDESC)) {
+          hasGlobalAccess = true;
+        }
+
+        Set<FlowEdge> outEdgeSet = fg.getOutEdgeSet(originalSrcNode);
+        for (Iterator iterator2 = outEdgeSet.iterator(); iterator2.hasNext();) {
+          FlowEdge outEdge = (FlowEdge) iterator2.next();
+          FlowNode originalDstNode = outEdge.getDst();
+
+          Set<FlowNode> dstNodeSet = new HashSet<FlowNode>();
+          if (originalDstNode instanceof FlowReturnNode) {
+            FlowReturnNode rnode = (FlowReturnNode) originalDstNode;
+            Set<NTuple<Descriptor>> tupleSet = rnode.getTupleSet();
+            for (Iterator iterator4 = tupleSet.iterator(); iterator4.hasNext();) {
+              NTuple<Descriptor> nTuple = (NTuple<Descriptor>) iterator4.next();
+              dstNodeSet.add(fg.getFlowNode(nTuple));
+            }
+          } else {
+            dstNodeSet.add(originalDstNode);
+          }
+          System.out.println("---dstNodeSet=" + dstNodeSet);
+          for (Iterator iterator4 = dstNodeSet.iterator(); iterator4.hasNext();) {
+            FlowNode dstNode = (FlowNode) iterator4.next();
 
-        NTuple<Descriptor> srcNodeTuple = srcNode.getDescTuple();
-        NTuple<Descriptor> dstNodeTuple = dstNode.getDescTuple();
+            NTuple<Descriptor> srcNodeTuple = srcNode.getDescTuple();
+            NTuple<Descriptor> dstNodeTuple = dstNode.getDescTuple();
 
-        if (outEdge.getInitTuple().equals(srcNodeTuple)
-            && outEdge.getEndTuple().equals(dstNodeTuple)) {
+            // if (outEdge.getInitTuple().equals(srcNodeTuple)
+            // && outEdge.getEndTuple().equals(dstNodeTuple)) {
 
-          NTuple<Descriptor> srcCurTuple = srcNode.getCurrentDescTuple();
-          NTuple<Descriptor> dstCurTuple = dstNode.getCurrentDescTuple();
+            NTuple<Descriptor> srcCurTuple = srcNode.getCurrentDescTuple();
+            NTuple<Descriptor> dstCurTuple = dstNode.getCurrentDescTuple();
 
-          if ((srcCurTuple.size() > 1 && dstCurTuple.size() > 1)
-              && srcCurTuple.get(0).equals(dstCurTuple.get(0))) {
+            // System.out.println("-srcCurTuple=" + srcCurTuple + "  dstCurTuple=" + dstCurTuple);
 
-            // value flows between fields
-            Descriptor desc = srcCurTuple.get(0);
-            ClassDescriptor classDesc;
+            if ((srcCurTuple.size() > 1 && dstCurTuple.size() > 1)
+                && srcCurTuple.get(0).equals(dstCurTuple.get(0))) {
 
-            if (desc.equals(GLOBALDESC)) {
-              classDesc = md.getClassDesc();
-            } else {
-              VarDescriptor varDesc = (VarDescriptor) srcCurTuple.get(0);
-              classDesc = varDesc.getType().getClassDesc();
-            }
-            extractFlowsBetweenFields(classDesc, srcNode, dstNode, 1);
+              // value flows between fields
+              Descriptor desc = srcCurTuple.get(0);
+              ClassDescriptor classDesc;
 
-          } else {
-            // value flow between local var - local var or local var - field
+              if (desc.equals(GLOBALDESC)) {
+                classDesc = md.getClassDesc();
+              } else {
+                VarDescriptor varDesc = (VarDescriptor) srcCurTuple.get(0);
+                classDesc = varDesc.getType().getClassDesc();
+              }
+              extractFlowsBetweenFields(classDesc, srcNode, dstNode, 1);
 
-            Descriptor srcDesc = srcCurTuple.get(0);
-            Descriptor dstDesc = dstCurTuple.get(0);
+            } else if (!srcCurTuple.get(0).equals(dstCurTuple.get(0))) {
+              // value flow between local var - local var or local var - field
 
-            methodGraph.addEdge(srcDesc, dstDesc);
+              Descriptor srcDesc = srcCurTuple.get(0);
+              Descriptor dstDesc = dstCurTuple.get(0);
+
+              methodGraph.addEdge(srcDesc, dstDesc);
+
+              if (fg.isParamDesc(srcDesc)) {
+                methodGraph.setParamHNode(srcDesc);
+              }
+              if (fg.isParamDesc(dstDesc)) {
+                methodGraph.setParamHNode(dstDesc);
+              }
 
-            if (fg.isParamDesc(srcDesc)) {
-              methodGraph.setParamHNode(srcDesc);
-            }
-            if (fg.isParamDesc(dstDesc)) {
-              methodGraph.setParamHNode(dstDesc);
             }
 
+            // }
           }
 
         }
+
       }
+
     }
 
     // If the method accesses static fields
@@ -1773,6 +2077,16 @@ public class LocationInference {
     }
     methodGraph.getHNode(GLOBALDESC).setSkeleton(true);
 
+    if (ssjava.getMethodContainingSSJavaLoop().equals(md)) {
+      // if the current method contains the event loop
+      // we need to set all nodes of the hierarchy graph as a skeleton node
+      Set<HNode> hnodeSet = methodGraph.getNodeSet();
+      for (Iterator iterator = hnodeSet.iterator(); iterator.hasNext();) {
+        HNode hnode = (HNode) iterator.next();
+        hnode.setSkeleton(true);
+      }
+    }
+
   }
 
   private MethodSummary getMethodSummary(MethodDescriptor md) {
@@ -1909,7 +2223,8 @@ public class LocationInference {
     if (desc instanceof MethodDescriptor) {
       System.out.println("#EXTRA LOC DECLARATION GEN=" + desc);
 
-      MethodSummary methodSummary = getMethodSummary((MethodDescriptor) desc);
+      MethodDescriptor md = (MethodDescriptor) desc;
+      MethodSummary methodSummary = getMethodSummary(md);
 
       if (!ssjava.getMethodContainingSSJavaLoop().equals(desc)) {
         TypeDescriptor returnType = ((MethodDescriptor) desc).getReturnType();
@@ -1923,7 +2238,9 @@ public class LocationInference {
         }
       }
 
-      rtr += "\n@THISLOC(\"" + methodSummary.getThisLocName() + "\")";
+      if (!md.isStatic()) {
+        rtr += "\n@THISLOC(\"" + methodSummary.getThisLocName() + "\")";
+      }
       rtr += "\n@GLOBALLOC(\"" + methodSummary.getGlobalLocName() + "\")";
 
     }
@@ -2026,6 +2343,9 @@ public class LocationInference {
                 String orgSourceLine = sourceVec.get(varLineNum);
                 int idx =
                     orgSourceLine.indexOf(generateVarDeclaration((VarDescriptor) localVarDesc));
+                System.out.println("idx=" + idx
+                    + "  generateVarDeclaration((VarDescriptor) localVarDesc)="
+                    + generateVarDeclaration((VarDescriptor) localVarDesc));
                 assert (idx != -1);
                 String annoatedStr =
                     orgSourceLine.substring(0, idx) + locAnnotationStr + " "
@@ -2038,7 +2358,8 @@ public class LocationInference {
               int idx =
                   getParamLocation(methodDefStr,
                       generateVarDeclaration((VarDescriptor) localVarDesc));
-
+              System.out.println("methodDefStr=" + methodDefStr + " localVarDesc=" + localVarDesc
+                  + " idx=" + idx);
               assert (idx != -1);
 
               String annoatedStr =
@@ -2112,7 +2433,6 @@ public class LocationInference {
   }
 
   private String generateLocationAnnoatation(CompositeLocation loc) {
-    System.out.println("loc=" + loc);
     String rtr = "";
     // method location
     Location methodLoc = loc.get(0);
@@ -2227,17 +2547,16 @@ public class LocationInference {
 
   }
 
-  private void inferLattices() {
-  }
-
   private void calculateExtraLocations() {
-    LinkedList<MethodDescriptor> descriptorListToAnalyze = ssjava.getSortedDescriptors();
-    for (Iterator iterator = descriptorListToAnalyze.iterator(); iterator.hasNext();) {
+
+    LinkedList<MethodDescriptor> methodDescList = ssjava.getSortedDescriptors();
+    for (Iterator iterator = methodDescList.iterator(); iterator.hasNext();) {
       MethodDescriptor md = (MethodDescriptor) iterator.next();
       if (!ssjava.getMethodContainingSSJavaLoop().equals(md)) {
         calculateExtraLocations(md);
       }
     }
+
   }
 
   private void checkLatticesOfVirtualMethods(MethodDescriptor md) {
@@ -2335,7 +2654,7 @@ public class LocationInference {
 
   private void calculatePCLOC(MethodDescriptor md) {
 
-    System.out.println("#calcualtePCLOC");
+    System.out.println("#CalculatePCLOC");
     MethodSummary methodSummary = getMethodSummary(md);
     FlowGraph fg = getFlowGraph(md);
     Map<Integer, CompositeLocation> mapParamToLoc = methodSummary.getMapParamIdxToInferLoc();
@@ -2344,7 +2663,8 @@ public class LocationInference {
     // PC location is higher than location types of parameters which has incoming flows.
 
     Set<NTuple<Location>> paramLocTupleHavingInFlowSet = new HashSet<NTuple<Location>>();
-    Set<NTuple<Location>> paramLocTupleNOTHavingInFlowSet = new HashSet<NTuple<Location>>();
+    Set<Descriptor> paramDescNOTHavingInFlowSet = new HashSet<Descriptor>();
+    // Set<FlowNode> paramNodeNOThavingInFlowSet = new HashSet<FlowNode>();
 
     int numParams = fg.getNumParameters();
     for (int i = 0; i < numParams; i++) {
@@ -2353,14 +2673,31 @@ public class LocationInference {
       NTuple<Descriptor> paramDescTuple = paramFlowNode.getCurrentDescTuple();
       NTuple<Location> paramLocTuple = translateToLocTuple(md, paramDescTuple);
 
-      if (fg.getIncomingNodeSetByPrefix(prefix).size() > 0) {
+      Set<FlowNode> inNodeToParamSet = fg.getIncomingNodeSetByPrefix(prefix);
+      if (inNodeToParamSet.size() > 0) {
         // parameter has in-value flows
-        paramLocTupleHavingInFlowSet.add(paramLocTuple);
+
+        for (Iterator iterator = inNodeToParamSet.iterator(); iterator.hasNext();) {
+          FlowNode inNode = (FlowNode) iterator.next();
+          Set<FlowEdge> outEdgeSet = fg.getOutEdgeSet(inNode);
+          for (Iterator iterator2 = outEdgeSet.iterator(); iterator2.hasNext();) {
+            FlowEdge flowEdge = (FlowEdge) iterator2.next();
+            if (flowEdge.getEndTuple().startsWith(prefix)) {
+              NTuple<Location> paramLocTupleWithIncomingFlow =
+                  translateToLocTuple(md, flowEdge.getEndTuple());
+              paramLocTupleHavingInFlowSet.add(paramLocTupleWithIncomingFlow);
+            }
+          }
+        }
+
+        // paramLocTupleHavingInFlowSet.add(paramLocTuple);
       } else {
-        paramLocTupleNOTHavingInFlowSet.add(paramLocTuple);
+        // paramNodeNOThavingInFlowSet.add(fg.getFlowNode(paramDescTuple));
+        paramDescNOTHavingInFlowSet.add(prefix);
       }
     }
 
+    System.out.println("paramLocTupleHavingInFlowSet=" + paramLocTupleHavingInFlowSet);
 
     if (paramLocTupleHavingInFlowSet.size() > 0
         && !coversAllParamters(md, fg, paramLocTupleHavingInFlowSet)) {
@@ -2370,36 +2707,26 @@ public class LocationInference {
       NTuple<Location> pcLocTuple =
           generateLocTupleRelativeTo(md, paramLocTupleHavingInFlowSet, PCLOC);
 
-      int pcLocTupleIdx = pcLocTuple.size() - 1;
-      Location pcLoc = pcLocTuple.get(pcLocTupleIdx);
-      Descriptor pcDesc = pcLoc.getLocDescriptor();
-      Descriptor enclosingDesc = pcLocTuple.get(pcLocTupleIdx).getDescriptor();
-
-      HierarchyGraph hierarchyGraph = getHierarchyGraph(enclosingDesc);
-      HNode pcNode = hierarchyGraph.getHNode(pcDesc);
-      pcNode.setSkeleton(true);
-
-      // add ordering relations s.t. PCLOC is higher than all of parameters with incoming flows
-      // and lower than paramters which do not have any incoming flows
-      for (Iterator iterator = paramLocTupleHavingInFlowSet.iterator(); iterator.hasNext();) {
-        NTuple<Location> paramLocTuple = (NTuple<Location>) iterator.next();
-        if (paramLocTuple.size() > pcLocTupleIdx) {
-          Descriptor lowerDesc = paramLocTuple.get(pcLocTupleIdx).getLocDescriptor();
-          hierarchyGraph.addEdge(pcDesc, lowerDesc);
-        }
-      }
+      NTuple<Descriptor> pcDescTuple = translateToDescTuple(pcLocTuple);
+
+      // System.out.println("pcLoc=" + pcLocTuple);
 
-      for (Iterator iterator = paramLocTupleNOTHavingInFlowSet.iterator(); iterator.hasNext();) {
-        NTuple<Location> paramLocTuple = (NTuple<Location>) iterator.next();
-        if (paramLocTuple.size() > pcLocTupleIdx) {
-          Descriptor higherDesc = paramLocTuple.get(pcLocTupleIdx).getLocDescriptor();
-          hierarchyGraph.addEdge(higherDesc, pcDesc);
+      CompositeLocation curPCLoc = methodSummary.getPCLoc();
+      if (curPCLoc.get(0).isTop() || pcLocTuple.size() > curPCLoc.getSize()) {
+        methodSummary.setPCLoc(new CompositeLocation(pcLocTuple));
+
+        // add ordering relations s.t. PCLOC is higher than all flow nodes except the set of
+        // parameters that do not have incoming flows
+        for (Iterator iterator = fg.getNodeSet().iterator(); iterator.hasNext();) {
+          FlowNode node = (FlowNode) iterator.next();
+
+          if (!paramDescNOTHavingInFlowSet.contains(node.getCurrentDescTuple().get(0))) {
+            fg.addValueFlowEdge(pcDescTuple, node.getDescTuple());
+          }
         }
+        fg.getFlowNode(translateToDescTuple(pcLocTuple)).setSkeleton(true);
       }
 
-      System.out.println("pcLoc=" + pcLoc);
-
-      methodSummary.setPCLoc(new CompositeLocation(pcLocTuple));
     }
   }
 
@@ -2446,11 +2773,8 @@ public class LocationInference {
     System.out.println("#calculateRETURNLOC= " + md);
     // calculate a return location:
     // the return location type is lower than all parameters and the location of return values
-
     MethodSummary methodSummary = getMethodSummary(md);
-
     FlowGraph fg = getFlowGraph(md);
-
     Map<Integer, CompositeLocation> mapParamToLoc = methodSummary.getMapParamIdxToInferLoc();
     Set<Integer> paramIdxSet = mapParamToLoc.keySet();
 
@@ -2458,122 +2782,47 @@ public class LocationInference {
       // first, generate the set of return value location types that starts
       // with 'this' reference
 
-      Set<NTuple<Location>> inferFieldReturnLocSet = new HashSet<NTuple<Location>>();
-
       Set<FlowNode> paramFlowNodeFlowingToReturnValueSet = getParamNodeFlowingToReturnValue(md);
-      System.out.println("paramFlowNodeFlowingToReturnValueSet="
-          + paramFlowNodeFlowingToReturnValueSet);
+      // System.out.println("paramFlowNodeFlowingToReturnValueSet="
+      // + paramFlowNodeFlowingToReturnValueSet);
 
-      Set<NTuple<Location>> locFlowingToReturnValueSet = new HashSet<NTuple<Location>>();
+      Set<NTuple<Location>> tupleToBeHigherThanReturnLocSet = new HashSet<NTuple<Location>>();
       for (Iterator iterator = paramFlowNodeFlowingToReturnValueSet.iterator(); iterator.hasNext();) {
         FlowNode fn = (FlowNode) iterator.next();
-
         NTuple<Descriptor> paramDescTuple = fn.getCurrentDescTuple();
-        NTuple<Location> paramLocTuple = translateToLocTuple(md, paramDescTuple);
-
-        locFlowingToReturnValueSet.add(paramLocTuple);
+        tupleToBeHigherThanReturnLocSet.add(translateToLocTuple(md, paramDescTuple));
       }
 
       Set<FlowNode> returnNodeSet = fg.getReturnNodeSet();
       for (Iterator iterator = returnNodeSet.iterator(); iterator.hasNext();) {
         FlowNode returnNode = (FlowNode) iterator.next();
         NTuple<Descriptor> returnDescTuple = returnNode.getCurrentDescTuple();
-        NTuple<Location> returnLocTuple = translateToLocTuple(md, returnDescTuple);
-        locFlowingToReturnValueSet.add(returnLocTuple);
+        tupleToBeHigherThanReturnLocSet.add(translateToLocTuple(md, returnDescTuple));
       }
-      System.out.println("locFlowingToReturnValueSet=" + locFlowingToReturnValueSet);
+      System.out.println("-flow graph's returnNodeSet=" + returnNodeSet);
+      System.out.println("tupleSetToBeHigherThanReturnLoc=" + tupleToBeHigherThanReturnLocSet);
 
       // Here, generates a return location in the method lattice that is lower than the
       // locFlowingToReturnValueSet
       NTuple<Location> returnLocTuple =
-          generateLocTupleRelativeTo(md, locFlowingToReturnValueSet, RLOC);
+          generateLocTupleRelativeTo(md, tupleToBeHigherThanReturnLocSet, RLOC);
 
-      System.out.println("returnLocTuple=" + returnLocTuple);
+      // System.out.println("returnLocTuple=" + returnLocTuple);
+      NTuple<Descriptor> returnDescTuple = translateToDescTuple(returnLocTuple);
+      CompositeLocation curReturnLoc = methodSummary.getRETURNLoc();
+      if (curReturnLoc == null || returnDescTuple.size() > curReturnLoc.getSize()) {
+        methodSummary.setRETURNLoc(new CompositeLocation(returnLocTuple));
 
-      int returnLocTupleIdx = returnLocTuple.size() - 1;
-      Location returnLoc = returnLocTuple.get(returnLocTupleIdx);
-      Descriptor returnDesc = returnLoc.getLocDescriptor();
-      Descriptor enclosingDesc = returnLocTuple.get(returnLocTupleIdx).getDescriptor();
-
-      HierarchyGraph hierarchyGraph = getHierarchyGraph(enclosingDesc);
-      HNode returnNode = hierarchyGraph.getHNode(returnDesc);
-      returnNode.setSkeleton(true);
+        for (Iterator iterator = tupleToBeHigherThanReturnLocSet.iterator(); iterator.hasNext();) {
+          NTuple<Location> higherTuple = (NTuple<Location>) iterator.next();
+          fg.addValueFlowEdge(translateToDescTuple(higherTuple), returnDescTuple);
+        }
+        fg.getFlowNode(returnDescTuple).setSkeleton(true);
 
-      for (Iterator iterator = locFlowingToReturnValueSet.iterator(); iterator.hasNext();) {
-        NTuple<Location> locTuple = (NTuple<Location>) iterator.next();
-        Descriptor higherDesc = locTuple.get(returnLocTupleIdx).getLocDescriptor();
-        hierarchyGraph.addEdge(higherDesc, returnDesc);
       }
 
-      methodSummary.setRETURNLoc(new CompositeLocation(returnLocTuple));
-
-      // skip: for (Iterator iterator = returnNodeSet.iterator(); iterator.hasNext();) {
-      // FlowNode returnNode = (FlowNode) iterator.next();
-      //
-      // NTuple<Descriptor> returnDescTuple = returnNode.getCurrentDescTuple();
-      // NTuple<Location> returnLocTuple = translateToLocTuple(md, returnDescTuple);
-      //
-      // if (returnLocTuple.get(0).getLocDescriptor().equals(md.getThis())) {
-      // // if the location type of the return value matches "this" reference
-      // // then, check whether this return value is equal to/lower than all
-      // // of parameters that possibly flow into the return values
-      // for (Iterator iterator2 = inferParamLocSet.iterator(); iterator2.hasNext();) {
-      // CompositeLocation paramInferLoc = (CompositeLocation) iterator2.next();
-      //
-      // if ((!paramInferLoc.equals(returnLocTuple))
-      // && !isGreaterThan(methodLattice, paramInferLoc, inferReturnLoc)) {
-      // continue skip;
-      // }
-      // }
-      // inferFieldReturnLocSet.add(returnLocTuple);
-      //
-      // }
-      // }
-
-      // if (inferFieldReturnLocSet.size() > 0) {
-      //
-      // // CompositeLocation returnLoc = getLowest(methodLattice, inferFieldReturnLocSet);
-      // CompositeLocation returnLoc = null;
-      // if (returnLoc == null) {
-      // // in this case, assign <'this',bottom> to the RETURNLOC
-      // returnLoc = new CompositeLocation(new Location(md, md.getThis().getSymbol()));
-      // returnLoc.addLocation(new Location(md.getClassDesc(), getLattice(md.getClassDesc())
-      // .getBottomItem()));
-      // }
-      // methodInfo.setReturnLoc(returnLoc);
-      //
-      // } else {
-      // String returnLocSymbol = "RETURNLOC";
-      // CompositeLocation returnLocInferLoc =
-      // new CompositeLocation(new Location(md, returnLocSymbol));
-      // methodInfo.setReturnLoc(returnLocInferLoc);
-      //
-      // for (Iterator iterator = paramIdxSet.iterator(); iterator.hasNext();) {
-      // Integer paramIdx = (Integer) iterator.next();
-      // CompositeLocation inferLoc = mapParamToLoc.get(paramIdx);
-      // String paramLocLocalSymbol = inferLoc.get(0).getLocIdentifier();
-      // if (!methodLattice.isGreaterThan(paramLocLocalSymbol, returnLocSymbol)) {
-      // // TODO
-      // // addRelationHigherToLower(methodLattice, methodInfo,
-      // // paramLocLocalSymbol,
-      // // returnLocSymbol);
-      // }
-      // }
-      //
-      // for (Iterator iterator = returnNodeSet.iterator(); iterator.hasNext();) {
-      // FlowNode returnNode = (FlowNode) iterator.next();
-      // CompositeLocation inferLoc =
-      // generateInferredCompositeLocation(methodInfo, fg.getLocationTuple(returnNode));
-      // if (!isGreaterThan(methodLattice, inferLoc, returnLocInferLoc)) {
-      // // TODO
-      // // addRelation(methodLattice, methodInfo, inferLoc,
-      // // returnLocInferLoc);
-      // }
-      // }
-      //
-      // }
-
     }
+
   }
 
   private void calculateExtraLocations(MethodDescriptor md) {
@@ -2589,7 +2838,7 @@ public class LocationInference {
   private NTuple<Location> generateLocTupleRelativeTo(MethodDescriptor md,
       Set<NTuple<Location>> paramLocTupleHavingInFlowSet, String locNamePrefix) {
 
-    System.out.println("-generateLocTupleRelativeTo=" + paramLocTupleHavingInFlowSet);
+    // System.out.println("-generateLocTupleRelativeTo=" + paramLocTupleHavingInFlowSet);
 
     NTuple<Location> higherLocTuple = new NTuple<Location>();
 
@@ -2601,10 +2850,21 @@ public class LocationInference {
 
     Set<NTuple<Location>> paramLocTupleStartedWithThis = new HashSet<NTuple<Location>>();
 
-    for (Iterator iterator = paramLocTupleHavingInFlowSet.iterator(); iterator.hasNext();) {
+    next: for (Iterator iterator = paramLocTupleHavingInFlowSet.iterator(); iterator.hasNext();) {
       NTuple<Location> paramLocTuple = (NTuple<Location>) iterator.next();
-      if (!paramLocTuple.get(0).getLocDescriptor().equals(thisVarDesc)) {
+      Descriptor paramLocalDesc = paramLocTuple.get(0).getLocDescriptor();
+      if (!paramLocalDesc.equals(thisVarDesc)) {
+
+        Set<FlowNode> inNodeSet = getFlowGraph(md).getIncomingNodeSetByPrefix(paramLocalDesc);
+        for (Iterator iterator2 = inNodeSet.iterator(); iterator2.hasNext();) {
+          FlowNode flowNode = (FlowNode) iterator2.next();
+          if (flowNode.getDescTuple().startsWith(thisVarDesc)) {
+            // System.out.println("paramLocTuple=" + paramLocTuple + " is lower than THIS");
+            continue next;
+          }
+        }
         hasParamNotStartedWithThisRef = true;
+
       } else if (paramLocTuple.size() > 1) {
         paramLocTupleStartedWithThis.add(paramLocTuple);
         if (minSize == 0 || minSize > paramLocTuple.size()) {
@@ -2613,7 +2873,7 @@ public class LocationInference {
       }
     }
 
-    System.out.println("---paramLocTupleStartedWithThis=" + paramLocTupleStartedWithThis);
+    // System.out.println("---paramLocTupleStartedWithThis=" + paramLocTupleStartedWithThis);
     Descriptor enclosingDesc = md;
     if (hasParamNotStartedWithThisRef) {
       // in this case, PCLOC will be the local location
@@ -2626,16 +2886,17 @@ public class LocationInference {
         NTuple<Location> paramLocTuple = null;
         for (Iterator iterator = paramLocTupleStartedWithThis.iterator(); iterator.hasNext();) {
           paramLocTuple = (NTuple<Location>) iterator.next();
-          System.out.println("-----paramLocTuple=" + paramLocTuple + "  idx=" + idx);
+          // System.out.println("-----paramLocTuple=" + paramLocTuple + "  idx=" + idx);
           curLoc = paramLocTuple.get(idx);
           Descriptor locDesc = curLoc.getLocDescriptor();
           locDescSet.add(locDesc);
         }
-        System.out.println("-----locDescSet=" + locDescSet + " idx=" + idx);
+        // System.out.println("-----locDescSet=" + locDescSet + " idx=" + idx);
         if (locDescSet.size() != 1) {
           break;
         }
         Location newLocElement = new Location(curLoc.getDescriptor(), curLoc.getLocDescriptor());
+        System.out.println("newLocElement" + newLocElement);
         higherLocTuple.add(newLocElement);
         enclosingDesc = getClassTypeDescriptor(curLoc.getLocDescriptor());
       }
@@ -2646,7 +2907,6 @@ public class LocationInference {
     NameDescriptor pcLocDesc = new NameDescriptor(pcLocIdentifier);
     Location newLoc = new Location(enclosingDesc, pcLocDesc);
     higherLocTuple.add(newLoc);
-
     System.out.println("---new loc tuple=" + higherLocTuple);
 
     return higherLocTuple;
@@ -2682,7 +2942,7 @@ public class LocationInference {
     for (; iterator.hasNext();) {
       NTuple<Location> curLocTuple = (NTuple<Location>) iterator.next();
       if (isHigherThan(curLocTuple, highest)) {
-        System.out.println(curLocTuple + " is greater than " + highest);
+        // System.out.println(curLocTuple + " is greater than " + highest);
         highest = curLocTuple;
       }
     }
@@ -2692,14 +2952,14 @@ public class LocationInference {
     MethodDescriptor md = (MethodDescriptor) highest.get(0).getDescriptor();
     VarDescriptor thisVarDesc = md.getThis();
 
-    System.out.println("highest=" + highest);
+    // System.out.println("highest=" + highest);
 
     for (Iterator<NTuple<Location>> iter = paramLocTupleHavingInFlowSet.iterator(); iter.hasNext();) {
       NTuple<Location> curLocTuple = iter.next();
 
       if (!curLocTuple.equals(highest) && !hasOrderingRelation(highest, curLocTuple)) {
 
-        System.out.println("add it to the highest set=" + curLocTuple);
+        // System.out.println("add it to the highest set=" + curLocTuple);
         highestSet.add(curLocTuple);
 
       }
@@ -2709,147 +2969,6 @@ public class LocationInference {
 
   }
 
-  private void calculateExtraLocations2(MethodDescriptor md) {
-    // calcualte pcloc, returnloc,...
-
-    SSJavaLattice<String> methodLattice = getMethodLattice(md);
-    MethodLocationInfo methodInfo = getMethodLocationInfo(md);
-    FlowGraph fg = getFlowGraph(md);
-    Set<FlowNode> nodeSet = fg.getNodeSet();
-
-    for (Iterator iterator = nodeSet.iterator(); iterator.hasNext();) {
-      FlowNode flowNode = (FlowNode) iterator.next();
-      if (flowNode.isDeclaratonNode()) {
-        CompositeLocation inferLoc = methodInfo.getInferLocation(flowNode.getDescTuple().get(0));
-        String locIdentifier = inferLoc.get(0).getLocIdentifier();
-        if (!methodLattice.containsKey(locIdentifier)) {
-          methodLattice.put(locIdentifier);
-        }
-
-      }
-    }
-
-    Map<Integer, CompositeLocation> mapParamToLoc = methodInfo.getMapParamIdxToInferLoc();
-    Set<Integer> paramIdxSet = mapParamToLoc.keySet();
-
-    if (!ssjava.getMethodContainingSSJavaLoop().equals(md)) {
-      // calculate the initial program counter location
-      // PC location is higher than location types of all parameters
-      String pcLocSymbol = "PCLOC";
-
-      Set<CompositeLocation> paramInFlowSet = new HashSet<CompositeLocation>();
-
-      for (Iterator iterator = paramIdxSet.iterator(); iterator.hasNext();) {
-        Integer paramIdx = (Integer) iterator.next();
-
-        FlowNode paramFlowNode = fg.getParamFlowNode(paramIdx);
-
-        if (fg.getIncomingFlowNodeSet(paramFlowNode).size() > 0) {
-          // parameter has in-value flows
-          CompositeLocation inferLoc = mapParamToLoc.get(paramIdx);
-          paramInFlowSet.add(inferLoc);
-        }
-      }
-
-      if (paramInFlowSet.size() > 0) {
-        CompositeLocation lowestLoc = getLowest(methodLattice, paramInFlowSet);
-        assert (lowestLoc != null);
-        methodInfo.setPCLoc(lowestLoc);
-      }
-
-    }
-
-    // calculate a return location
-    // the return location type is lower than all parameters and location
-    // types
-    // of return values
-    if (!md.getReturnType().isVoid()) {
-      // first, generate the set of return value location types that starts
-      // with
-      // 'this' reference
-
-      Set<CompositeLocation> inferFieldReturnLocSet = new HashSet<CompositeLocation>();
-
-      Set<FlowNode> paramFlowNode = getParamNodeFlowingToReturnValue(md);
-      Set<CompositeLocation> inferParamLocSet = new HashSet<CompositeLocation>();
-      if (paramFlowNode != null) {
-        for (Iterator iterator = paramFlowNode.iterator(); iterator.hasNext();) {
-          FlowNode fn = (FlowNode) iterator.next();
-          CompositeLocation inferLoc =
-              generateInferredCompositeLocation(methodInfo, getFlowGraph(md).getLocationTuple(fn));
-          inferParamLocSet.add(inferLoc);
-        }
-      }
-
-      Set<FlowNode> returnNodeSet = fg.getReturnNodeSet();
-
-      skip: for (Iterator iterator = returnNodeSet.iterator(); iterator.hasNext();) {
-        FlowNode returnNode = (FlowNode) iterator.next();
-        CompositeLocation inferReturnLoc =
-            generateInferredCompositeLocation(methodInfo, fg.getLocationTuple(returnNode));
-        if (inferReturnLoc.get(0).getLocIdentifier().equals("this")) {
-          // if the location type of the return value matches "this" reference
-          // then, check whether this return value is equal to/lower than all
-          // of
-          // parameters that possibly flow into the return values
-          for (Iterator iterator2 = inferParamLocSet.iterator(); iterator2.hasNext();) {
-            CompositeLocation paramInferLoc = (CompositeLocation) iterator2.next();
-
-            if ((!paramInferLoc.equals(inferReturnLoc))
-                && !isGreaterThan(methodLattice, paramInferLoc, inferReturnLoc)) {
-              continue skip;
-            }
-          }
-          inferFieldReturnLocSet.add(inferReturnLoc);
-
-        }
-      }
-
-      if (inferFieldReturnLocSet.size() > 0) {
-
-        CompositeLocation returnLoc = getLowest(methodLattice, inferFieldReturnLocSet);
-        if (returnLoc == null) {
-          // in this case, assign <'this',bottom> to the RETURNLOC
-          returnLoc = new CompositeLocation(new Location(md, md.getThis().getSymbol()));
-          returnLoc.addLocation(new Location(md.getClassDesc(), getLattice(md.getClassDesc())
-              .getBottomItem()));
-        }
-        methodInfo.setReturnLoc(returnLoc);
-
-      } else {
-        String returnLocSymbol = "RETURNLOC";
-        CompositeLocation returnLocInferLoc =
-            new CompositeLocation(new Location(md, returnLocSymbol));
-        methodInfo.setReturnLoc(returnLocInferLoc);
-
-        for (Iterator iterator = paramIdxSet.iterator(); iterator.hasNext();) {
-          Integer paramIdx = (Integer) iterator.next();
-          CompositeLocation inferLoc = mapParamToLoc.get(paramIdx);
-          String paramLocLocalSymbol = inferLoc.get(0).getLocIdentifier();
-          if (!methodLattice.isGreaterThan(paramLocLocalSymbol, returnLocSymbol)) {
-            // TODO
-            // addRelationHigherToLower(methodLattice, methodInfo,
-            // paramLocLocalSymbol,
-            // returnLocSymbol);
-          }
-        }
-
-        for (Iterator iterator = returnNodeSet.iterator(); iterator.hasNext();) {
-          FlowNode returnNode = (FlowNode) iterator.next();
-          CompositeLocation inferLoc =
-              generateInferredCompositeLocation(methodInfo, fg.getLocationTuple(returnNode));
-          if (!isGreaterThan(methodLattice, inferLoc, returnLocInferLoc)) {
-            // TODO
-            // addRelation(methodLattice, methodInfo, inferLoc,
-            // returnLocInferLoc);
-          }
-        }
-
-      }
-
-    }
-  }
-
   private Set<String> getHigherLocSymbolThan(SSJavaLattice<String> lattice, String loc) {
     Set<String> higherLocSet = new HashSet<String>();
 
@@ -2964,16 +3083,11 @@ public class LocationInference {
     return false;
   }
 
-  private void contributeCalleeFlows(MethodInvokeNode min, MethodDescriptor mdCaller,
-      MethodDescriptor mdCallee) {
-
-    System.out.println("\n##contributeCalleeFlows callee=" + mdCallee + "TO caller=" + mdCaller);
-
-    getSubGlobalFlowGraph(mdCallee);
-
-  }
-
   private GlobalFlowGraph getSubGlobalFlowGraph(MethodDescriptor md) {
+
+    if (!mapMethodDescriptorToSubGlobalFlowGraph.containsKey(md)) {
+      mapMethodDescriptorToSubGlobalFlowGraph.put(md, new GlobalFlowGraph(md));
+    }
     return mapMethodDescriptorToSubGlobalFlowGraph.get(md);
   }
 
@@ -3003,226 +3117,78 @@ public class LocationInference {
           // parameters
 
           Set<FlowNode> localReachSet = calleeFlowGraph.getLocalReachFlowNodeSetFrom(paramNode1);
-          // System.out.println("-param1=" + paramNode1 + " is higher than param2=" + paramNode2);
+          System.out.println("-param1=" + paramNode1 + " is higher than param2=" + paramNode2);
           // System.out.println("-- localReachSet from param1=" + localReachSet);
 
+          NTuple<Descriptor> paramDescTuple1 = paramNode1.getCurrentDescTuple();
+          NTuple<Descriptor> paramDescTuple2 = paramNode2.getCurrentDescTuple();
+
+          System.out.println("-param1CurTuple=" + paramDescTuple1 + " param2CurTuple="
+              + paramDescTuple2);
+          if (paramDescTuple1.get(0).equals(paramDescTuple2.get(0))) {
+            // if two parameters share the same prefix
+            // it already has been assigned to a composite location
+            // so we don't need to add an additional ordering relation caused by these two
+            // paramters.
+            continue;
+          }
+
           if (arg1Tuple.size() > 0 && arg2Tuple.size() > 0 && localReachSet.contains(paramNode2)) {
             // need to propagate an ordering relation s.t. arg1 is higher
             // than arg2
 
+            // add a new flow between the corresponding arguments.
+            callerFlowGraph.addValueFlowEdge(arg1Tuple, arg2Tuple);
+            System.out.println("arg1=" + arg1Tuple + "   arg2=" + arg2Tuple);
+
             // System.out
             // .println("-arg1Tuple=" + arg1Tuple + " is higher than arg2Tuple=" + arg2Tuple);
 
-            // otherwise, flows between method/field locations...
-            callerFlowGraph.addValueFlowEdge(arg1Tuple, arg2Tuple);
-            // System.out.println("arg1=" + arg1Tuple + "   arg2=" + arg2Tuple);
-
           }
 
           System.out.println();
         }
       }
-    }
-    // System.out.println("##\n");
-
-  }
-
-  private void propagateFlowsToCaller(MethodInvokeNode min, MethodDescriptor mdCaller,
-      MethodDescriptor mdCallee) {
-
-    System.out.println("\n##PROPAGATE callee=" + mdCallee + "TO caller=" + mdCaller);
-
-    // if the parameter A reaches to the parameter B
-    // then, add an edge the argument A -> the argument B to the caller's flow
-    // graph
-
-    // TODO
-    // also if a parameter is a composite location and is started with "this"
-    // reference,
-    // need to make sure that the corresponding argument is higher than the
-    // translated location of
-    // the parameter.
-
-    FlowGraph calleeFlowGraph = getFlowGraph(mdCallee);
-    FlowGraph callerFlowGraph = getFlowGraph(mdCaller);
-    int numParam = calleeFlowGraph.getNumParameters();
-
-    for (int i = 0; i < numParam; i++) {
-      for (int k = 0; k < numParam; k++) {
-
-        if (i != k) {
-
-          FlowNode paramNode1 = calleeFlowGraph.getParamFlowNode(i);
-          FlowNode paramNode2 = calleeFlowGraph.getParamFlowNode(k);
-
-          System.out.println("param1=" + paramNode1 + " curDescTuple="
-              + paramNode1.getCurrentDescTuple());
-          System.out.println("param2=" + paramNode2 + " curDescTuple="
-              + paramNode2.getCurrentDescTuple());
-
-          // TODO: deprecated method
-          // NodeTupleSet tupleSetArg1 = getNodeTupleSetByArgIdx(min, i);
-          // NodeTupleSet tupleSetArg2 = getNodeTupleSetByArgIdx(min, k);
-          NodeTupleSet tupleSetArg1 = null;
-          NodeTupleSet tupleSetArg2 = null;
-
-          for (Iterator<NTuple<Descriptor>> iter1 = tupleSetArg1.iterator(); iter1.hasNext();) {
-            NTuple<Descriptor> arg1Tuple = iter1.next();
-
-            for (Iterator<NTuple<Descriptor>> iter2 = tupleSetArg2.iterator(); iter2.hasNext();) {
-              NTuple<Descriptor> arg2Tuple = iter2.next();
-
-              // check if the callee propagates an ordering constraints through
-              // parameters
-
-              Set<FlowNode> localReachSet =
-                  calleeFlowGraph.getLocalReachFlowNodeSetFrom(paramNode1);
-
-              if (localReachSet.contains(paramNode2)) {
-                // need to propagate an ordering relation s.t. arg1 is higher
-                // than arg2
-
-                System.out
-                    .println("-param1=" + paramNode1 + " is higher than param2=" + paramNode2);
-                System.out.println("-arg1Tuple=" + arg1Tuple + " is higher than arg2Tuple="
-                    + arg2Tuple);
-
-                if (!min.getMethod().isStatic()) {
-                  // check if this is the case that values flow to/from the
-                  // current object reference 'this'
-
-                  NTuple<Descriptor> baseTuple = mapMethodInvokeNodeToBaseTuple.get(min);
-                  Descriptor baseRef = baseTuple.get(baseTuple.size() - 1);
-
-                  System.out.println("paramNode1.getCurrentDescTuple()="
-                      + paramNode1.getCurrentDescTuple());
-                  // calculate the prefix of the argument
-
-                  if (arg2Tuple.size() == 1 && arg2Tuple.get(0).equals(baseRef)) {
-                    // in this case, the callee flow causes a caller flow to the
-                    // object whose method
-                    // is invoked.
-
-                    if (!paramNode1.getCurrentDescTuple().startsWith(mdCallee.getThis())) {
-                      // check whether ???
-
-                      NTuple<Descriptor> param1Prefix =
-                          calculatePrefixForParam(callerFlowGraph, calleeFlowGraph, min, arg1Tuple,
-                              paramNode1);
-
-                      if (param1Prefix != null && param1Prefix.startsWith(mdCallee.getThis())) {
-                        // in this case, we need to create a new edge
-                        // 'this.FIELD'->'this'
-                        // but we couldn't... instead we assign a new composite
-                        // location started
-                        // with 'this' reference to the corresponding parameter
-
-                        CompositeLocation compLocForParam1 =
-                            generateCompositeLocation(mdCallee, param1Prefix);
-
-                        System.out
-                            .println("set comp loc=" + compLocForParam1 + " to " + paramNode1);
-                        paramNode1.setCompositeLocation(compLocForParam1);
-
-                        // then, we need to make sure that the corresponding
-                        // argument in the caller
-                        // is required to be higher than or equal to the
-                        // translated parameter
-                        // location
-
-                        NTuple<Descriptor> translatedParamTuple =
-                            translateCompositeLocationToCaller(min, compLocForParam1);
-
-                        // TODO : check if the arg >= the tranlated parameter
-
-                        System.out.println("add a flow edge= " + arg1Tuple + "->"
-                            + translatedParamTuple);
-                        callerFlowGraph.addValueFlowEdge(arg1Tuple, translatedParamTuple);
-
-                        continue;
-
-                      }
-
-                    } else {
-                      // param1 has already been assigned a composite location
-
-                      System.out.println("--param1 has already been assigned a composite location");
-                      CompositeLocation compLocForParam1 = paramNode1.getCompositeLocation();
-                      NTuple<Descriptor> translatedParamTuple =
-                          translateCompositeLocationToCaller(min, compLocForParam1);
-
-                      // TODO : check if the arg >= the tranlated parameter
-
-                      System.out.println("add a flow edge= " + arg1Tuple + "->"
-                          + translatedParamTuple);
-                      callerFlowGraph.addValueFlowEdge(arg1Tuple, translatedParamTuple);
-
-                      continue;
-
-                    }
-
-                  } else if (arg1Tuple.size() == 1 && arg1Tuple.get(0).equals(baseRef)) {
-                    // in this case, the callee flow causes a caller flow
-                    // originated from the object
-                    // whose
-                    // method is invoked.
-
-                    System.out.println("###FROM CASE");
-
-                    if (!paramNode2.getCurrentDescTuple().startsWith(mdCallee.getThis())) {
-
-                      NTuple<Descriptor> param2Prefix =
-                          calculatePrefixForParam(callerFlowGraph, calleeFlowGraph, min, arg2Tuple,
-                              paramNode2);
-
-                      if (param2Prefix != null && param2Prefix.startsWith(mdCallee.getThis())) {
-                        // in this case, we need to create a new edge 'this' ->
-                        // 'this.FIELD' but we couldn't... instead we assign the
-                        // corresponding
-                        // parameter a new composite location started with
-                        // 'this' reference
-
-                        CompositeLocation compLocForParam2 =
-                            generateCompositeLocation(mdCallee, param2Prefix);
-
-                        // System.out.println("set comp loc=" + compLocForParam2
-                        // +
-                        // " to " + paramNode2);
-                        paramNode1.setCompositeLocation(compLocForParam2);
-                        continue;
-                      }
-                    }
-
-                  }
-                }
-
-                // otherwise, flows between method/field locations...
-                callerFlowGraph.addValueFlowEdge(arg1Tuple, arg2Tuple);
-                System.out.println("arg1=" + arg1Tuple + "   arg2=" + arg2Tuple);
-
-              }
+    }
 
-            }
+    // if a parameter has a composite location and the first element of the parameter location
+    // matches the callee's 'this'
+    // we have a more specific constraint: the caller's corresponding argument is higher than the
+    // parameter location which is translated into the caller
 
-          }
-          System.out.println();
+    for (int idx = 0; idx < numParam; idx++) {
+      FlowNode paramNode = calleeFlowGraph.getParamFlowNode(idx);
+      CompositeLocation compLoc = paramNode.getCompositeLocation();
+      if (compLoc != null && compLoc.get(0).getLocDescriptor().equals(min.getMethod().getThis())) {
+        System.out.println("$$$COMPLOC CASE=" + compLoc);
+        NTuple<Descriptor> argTuple = getNodeTupleByArgIdx(min, idx);
+        NTuple<Descriptor> translatedParamTuple =
+            translateCompositeLocationToCaller(idx, min, compLoc);
+        System.out.println("add a flow edge= " + argTuple + "->" + translatedParamTuple);
+        callerFlowGraph.addValueFlowEdge(argTuple, translatedParamTuple);
+
+        Set<NTuple<Location>> pcLocTupleSet = getPCLocTupleSet(min);
+        for (Iterator iterator = pcLocTupleSet.iterator(); iterator.hasNext();) {
+          NTuple<Location> pcLocTuple = (NTuple<Location>) iterator.next();
+          callerFlowGraph.addValueFlowEdge(translateToDescTuple(pcLocTuple), translatedParamTuple);
         }
+
       }
     }
-    System.out.println("##\n");
+
   }
 
-  private NTuple<Descriptor> translateCompositeLocationToCaller(MethodInvokeNode min,
+  private NTuple<Descriptor> translateCompositeLocationToCaller(int idx, MethodInvokeNode min,
       CompositeLocation compLocForParam1) {
+
     NTuple<Descriptor> baseTuple = mapMethodInvokeNodeToBaseTuple.get(min);
 
     NTuple<Descriptor> tuple = new NTuple<Descriptor>();
-
     for (int i = 0; i < baseTuple.size(); i++) {
       tuple.add(baseTuple.get(i));
     }
 
-    for (int i = 1; i < compLocForParam1.getSize(); i++) {
+    for (int i = baseTuple.size(); i < compLocForParam1.getSize(); i++) {
       Location loc = compLocForParam1.get(i);
       tuple.add(loc.getLocDescriptor());
     }
@@ -3296,51 +3262,6 @@ public class LocationInference {
     return newCompLoc;
   }
 
-  private NTuple<Descriptor> calculatePrefixForParam(FlowGraph callerFlowGraph,
-      FlowGraph calleeFlowGraph, MethodInvokeNode min, NTuple<Descriptor> arg1Tuple,
-      FlowNode paramNode1) {
-
-    NTuple<Descriptor> baseTuple = mapMethodInvokeNodeToBaseTuple.get(min);
-    Descriptor baseRef = baseTuple.get(baseTuple.size() - 1);
-    System.out.println("baseRef=" + baseRef);
-
-    FlowNode flowNodeArg1 = callerFlowGraph.getFlowNode(arg1Tuple);
-    List<NTuple<Descriptor>> callerPrefixList = calculatePrefixList(callerFlowGraph, flowNodeArg1);
-    System.out.println("callerPrefixList=" + callerPrefixList);
-
-    List<NTuple<Descriptor>> prefixList = calculatePrefixList(calleeFlowGraph, paramNode1);
-    System.out.println("###prefixList from node=" + paramNode1 + " =" + prefixList);
-
-    List<NTuple<Descriptor>> calleePrefixList =
-        translatePrefixListToCallee(baseRef, min.getMethod(), callerPrefixList);
-
-    System.out.println("calleePrefixList=" + calleePrefixList);
-
-    Set<FlowNode> reachNodeSetFromParam1 = calleeFlowGraph.getReachFlowNodeSetFrom(paramNode1);
-    System.out.println("reachNodeSetFromParam1=" + reachNodeSetFromParam1);
-
-    for (int i = 0; i < calleePrefixList.size(); i++) {
-      NTuple<Descriptor> curPrefix = calleePrefixList.get(i);
-      Set<NTuple<Descriptor>> reachableCommonPrefixSet = new HashSet<NTuple<Descriptor>>();
-
-      for (Iterator iterator2 = reachNodeSetFromParam1.iterator(); iterator2.hasNext();) {
-        FlowNode reachNode = (FlowNode) iterator2.next();
-        if (reachNode.getCurrentDescTuple().startsWith(curPrefix)) {
-          reachableCommonPrefixSet.add(reachNode.getCurrentDescTuple());
-        }
-      }
-
-      if (!reachableCommonPrefixSet.isEmpty()) {
-        System.out.println("###REACHABLECOMONPREFIX=" + reachableCommonPrefixSet
-            + " with curPreFix=" + curPrefix);
-        return curPrefix;
-      }
-
-    }
-
-    return null;
-  }
-
   private List<NTuple<Descriptor>> translatePrefixListToCallee(Descriptor baseRef,
       MethodDescriptor mdCallee, List<NTuple<Descriptor>> callerPrefixList) {
 
@@ -3422,13 +3343,13 @@ public class LocationInference {
 
       if (curDescriptor instanceof VarDescriptor) {
         enclosingDescriptor = md.getClassDesc();
+      } else if (curDescriptor instanceof FieldDescriptor) {
+        enclosingDescriptor = ((FieldDescriptor) curDescriptor).getClassDescriptor();
       } else if (curDescriptor instanceof NameDescriptor) {
         // it is "GLOBAL LOC" case!
         enclosingDescriptor = GLOBALDESC;
-      } else if (curDescriptor instanceof InterDescriptor) {
-        enclosingDescriptor = null;
       } else {
-        enclosingDescriptor = ((FieldDescriptor) curDescriptor).getClassDescriptor();
+        enclosingDescriptor = null;
       }
 
     }
@@ -3746,7 +3667,11 @@ public class LocationInference {
     }
     // _debug_printGraph();
 
-    methodDescList = (LinkedList<MethodDescriptor>) toanalyze_methodDescList.clone();
+  }
+
+  private void constructGlobalFlowGraph() {
+    LinkedList<MethodDescriptor> methodDescList =
+        (LinkedList<MethodDescriptor>) toanalyze_methodDescList.clone();
 
     while (!methodDescList.isEmpty()) {
       MethodDescriptor md = methodDescList.removeLast();
@@ -3755,19 +3680,17 @@ public class LocationInference {
         System.out.println("SSJAVA: Constructing a sub global flow graph: " + md);
 
         GlobalFlowGraph subGlobalFlowGraph = constructSubGlobalFlowGraph(getFlowGraph(md));
-        mapMethodDescriptorToSubGlobalFlowGraph.put(md, subGlobalFlowGraph);
 
         // TODO
         System.out.println("-add Value Flows From CalleeSubGlobalFlowGraph");
         addValueFlowsFromCalleeSubGlobalFlowGraph(md, subGlobalFlowGraph);
-        subGlobalFlowGraph.writeGraph("_SUBGLOBAL");
+        // subGlobalFlowGraph.writeGraph("_SUBGLOBAL");
 
-        System.out.println("-propagate Flows From Callees With No CompositeLocation");
-        propagateFlowsFromCalleesWithNoCompositeLocation(md);
+        // System.out.println("-propagate Flows From Callees With No CompositeLocation");
+        // propagateFlowsFromCalleesWithNoCompositeLocation(md);
 
       }
     }
-
   }
 
   private Set<MethodInvokeNode> getMethodInvokeNodeSet(MethodDescriptor md) {
@@ -3777,83 +3700,6 @@ public class LocationInference {
     return mapMethodDescriptorToMethodInvokeNodeSet.get(md);
   }
 
-  private void constructSubGlobalFlowGraph(MethodDescriptor md) {
-
-    FlowGraph flowGraph = getFlowGraph(md);
-
-    Set<MethodInvokeNode> setMethodInvokeNode = getMethodInvokeNodeSet(md);
-
-    for (Iterator<MethodInvokeNode> iter = setMethodInvokeNode.iterator(); iter.hasNext();) {
-      MethodInvokeNode min = iter.next();
-      propagateFlowsFromMethodInvokeNode(md, min);
-    }
-
-  }
-
-  private void propagateFlowsFromMethodInvokeNode(MethodDescriptor mdCaller, MethodInvokeNode min) {
-    // the transformation for a call site propagates flows through parameters
-    // if the method is virtual, it also grab all relations from any possible
-    // callees
-
-    MethodDescriptor mdCallee = min.getMethod();
-    Set<MethodDescriptor> setPossibleCallees = new HashSet<MethodDescriptor>();
-    if (mdCallee.isStatic()) {
-      setPossibleCallees.add(mdCallee);
-    } else {
-      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);
-    }
-
-    for (Iterator iterator2 = setPossibleCallees.iterator(); iterator2.hasNext();) {
-      MethodDescriptor possibleMdCallee = (MethodDescriptor) iterator2.next();
-      contributeCalleeFlows(min, mdCaller, possibleMdCallee);
-    }
-
-  }
-
-  private void assignCompositeLocation(MethodDescriptor md) {
-
-    FlowGraph flowGraph = getFlowGraph(md);
-
-    Set<FlowNode> nodeSet = flowGraph.getNodeSet();
-
-    next: for (Iterator iterator = nodeSet.iterator(); iterator.hasNext();) {
-      FlowNode flowNode = (FlowNode) iterator.next();
-
-      // assign a composite location only to the local variable
-      if (flowNode.getCurrentDescTuple().size() == 1) {
-
-        List<NTuple<Descriptor>> prefixList = calculatePrefixList(flowGraph, flowNode);
-        Set<FlowNode> reachSet = flowGraph.getReachFlowNodeSetFrom(flowNode);
-
-        for (int i = 0; i < prefixList.size(); i++) {
-          NTuple<Descriptor> curPrefix = prefixList.get(i);
-          Set<NTuple<Descriptor>> reachableCommonPrefixSet = new HashSet<NTuple<Descriptor>>();
-
-          for (Iterator iterator2 = reachSet.iterator(); iterator2.hasNext();) {
-            FlowNode reachNode = (FlowNode) iterator2.next();
-            if (reachNode.getCurrentDescTuple().startsWith(curPrefix)) {
-              reachableCommonPrefixSet.add(reachNode.getCurrentDescTuple());
-            }
-          }
-
-          if (!reachableCommonPrefixSet.isEmpty()) {
-            System.out.println("NEED TO ASSIGN COMP LOC TO " + flowNode + " with prefix="
-                + curPrefix);
-            CompositeLocation newCompLoc = generateCompositeLocation(md, curPrefix);
-            flowNode.setCompositeLocation(newCompLoc);
-            continue next;
-          }
-
-        }
-      }
-
-    }
-
-  }
-
   private void propagateFlowsFromCalleesWithNoCompositeLocation(MethodDescriptor mdCaller) {
 
     // the transformation for a call site propagates flows through parameters
@@ -3888,40 +3734,6 @@ public class LocationInference {
 
   }
 
-  private void propagateFlowsFromCallees(MethodDescriptor mdCaller) {
-
-    // the transformation for a call site propagates flows through parameters
-    // if the method is virtual, it also grab all relations from any possible
-    // callees
-
-    Set<MethodInvokeNode> setMethodInvokeNode =
-        mapMethodDescriptorToMethodInvokeNodeSet.get(mdCaller);
-
-    if (setMethodInvokeNode != null) {
-
-      for (Iterator iterator = setMethodInvokeNode.iterator(); iterator.hasNext();) {
-        MethodInvokeNode min = (MethodInvokeNode) iterator.next();
-        MethodDescriptor mdCallee = min.getMethod();
-        Set<MethodDescriptor> setPossibleCallees = new HashSet<MethodDescriptor>();
-        if (mdCallee.isStatic()) {
-          setPossibleCallees.add(mdCallee);
-        } else {
-          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);
-        }
-
-        for (Iterator iterator2 = setPossibleCallees.iterator(); iterator2.hasNext();) {
-          MethodDescriptor possibleMdCallee = (MethodDescriptor) iterator2.next();
-          propagateFlowsToCaller(min, mdCaller, possibleMdCallee);
-        }
-
-      }
-    }
-
-  }
-
   private void analyzeMethodBody(ClassDescriptor cd, MethodDescriptor md) {
     BlockNode bn = state.getMethodBody(md);
     NodeTupleSet implicitFlowTupleSet = new NodeTupleSet();
@@ -3997,9 +3809,10 @@ public class LocationInference {
     newImplicitTupleSet.addTupleSet(implicitFlowTupleSet);
     newImplicitTupleSet.addTupleSet(condTupleNode);
 
-    if (newImplicitTupleSet.size() > 1) {
+    if (needToGenerateInterLoc(newImplicitTupleSet)) {
       // need to create an intermediate node for the GLB of conditional
       // locations & implicit flows
+
       NTuple<Descriptor> interTuple = getFlowGraph(md).createIntermediateNode().getDescTuple();
       for (Iterator<NTuple<Descriptor>> idxIter = newImplicitTupleSet.iterator(); idxIter.hasNext();) {
         NTuple<Descriptor> tuple = idxIter.next();
@@ -4024,6 +3837,7 @@ public class LocationInference {
   private void analyzeFlowReturnNode(MethodDescriptor md, SymbolTable nametable, ReturnNode rn,
       NodeTupleSet implicitFlowTupleSet) {
 
+    // System.out.println("-analyzeFlowReturnNode=" + rn.printNode(0));
     ExpressionNode returnExp = rn.getReturnExpression();
 
     if (returnExp != null) {
@@ -4061,22 +3875,53 @@ public class LocationInference {
       // add tuples corresponding to the current implicit flows
       currentFlowTupleSet.addTupleSet(implicitFlowTupleSet);
 
-      if (currentFlowTupleSet.size() > 1) {
+      // System.out.println("---currentFlowTupleSet=" + currentFlowTupleSet);
+
+      if (needToGenerateInterLoc(currentFlowTupleSet)) {
         FlowNode meetNode = fg.createIntermediateNode();
         for (Iterator iterator = currentFlowTupleSet.iterator(); iterator.hasNext();) {
           NTuple<Descriptor> currentFlowTuple = (NTuple<Descriptor>) iterator.next();
           fg.addValueFlowEdge(currentFlowTuple, meetNode.getDescTuple());
         }
         fg.addReturnFlowNode(meetNode.getDescTuple());
-      } else if (currentFlowTupleSet.size() == 1) {
-        NTuple<Descriptor> tuple = currentFlowTupleSet.iterator().next();
-        fg.addReturnFlowNode(tuple);
+      } else {
+        // currentFlowTupleSet = removeLiteralTuple(currentFlowTupleSet);
+        for (Iterator iterator = currentFlowTupleSet.iterator(); iterator.hasNext();) {
+          NTuple<Descriptor> currentFlowTuple = (NTuple<Descriptor>) iterator.next();
+          fg.addReturnFlowNode(currentFlowTuple);
+        }
       }
 
     }
 
   }
 
+  private NodeTupleSet removeLiteralTuple(NodeTupleSet inSet) {
+    NodeTupleSet tupleSet = new NodeTupleSet();
+    for (Iterator<NTuple<Descriptor>> iter = inSet.iterator(); iter.hasNext();) {
+      NTuple<Descriptor> tuple = iter.next();
+      if (!tuple.get(0).equals(LITERALDESC)) {
+        tupleSet.addTuple(tuple);
+      }
+    }
+    return tupleSet;
+  }
+
+  private boolean needToGenerateInterLoc(NodeTupleSet tupleSet) {
+    int size = 0;
+    for (Iterator<NTuple<Descriptor>> iter = tupleSet.iterator(); iter.hasNext();) {
+      NTuple<Descriptor> descTuple = iter.next();
+      if (!descTuple.get(0).equals(LITERALDESC)) {
+        size++;
+      }
+    }
+    if (size > 1) {
+      return true;
+    } else {
+      return false;
+    }
+  }
+
   private void analyzeFlowLoopNode(MethodDescriptor md, SymbolTable nametable, LoopNode ln,
       NodeTupleSet implicitFlowTupleSet) {
 
@@ -4091,9 +3936,10 @@ public class LocationInference {
       newImplicitTupleSet.addTupleSet(implicitFlowTupleSet);
       newImplicitTupleSet.addTupleSet(condTupleNode);
 
-      if (newImplicitTupleSet.size() > 1) {
+      if (needToGenerateInterLoc(newImplicitTupleSet)) {
         // need to create an intermediate node for the GLB of conditional
         // locations & implicit flows
+
         NTuple<Descriptor> interTuple = getFlowGraph(md).createIntermediateNode().getDescTuple();
         for (Iterator<NTuple<Descriptor>> idxIter = newImplicitTupleSet.iterator(); idxIter
             .hasNext();) {
@@ -4146,24 +3992,43 @@ public class LocationInference {
       analyzeFlowExpressionNode(md, bn.getVarTable(), ln.getCondition(), condTupleNode, null,
           implicitFlowTupleSet, false);
 
-      // ///////////
-      NTuple<Descriptor> interTuple = getFlowGraph(md).createIntermediateNode().getDescTuple();
+      NodeTupleSet newImplicitTupleSet = new NodeTupleSet();
+      newImplicitTupleSet.addTupleSet(implicitFlowTupleSet);
+      newImplicitTupleSet.addTupleSet(condTupleNode);
 
-      for (Iterator<NTuple<Descriptor>> idxIter = condTupleNode.iterator(); idxIter.hasNext();) {
-        NTuple<Descriptor> tuple = idxIter.next();
-        addFlowGraphEdge(md, tuple, interTuple);
-      }
+      if (needToGenerateInterLoc(newImplicitTupleSet)) {
+        // need to create an intermediate node for the GLB of conditional
+        // locations & implicit flows
+
+        NTuple<Descriptor> interTuple = getFlowGraph(md).createIntermediateNode().getDescTuple();
+        for (Iterator<NTuple<Descriptor>> idxIter = newImplicitTupleSet.iterator(); idxIter
+            .hasNext();) {
+          NTuple<Descriptor> tuple = idxIter.next();
+          addFlowGraphEdge(md, tuple, interTuple);
+        }
+        newImplicitTupleSet.clear();
+        newImplicitTupleSet.addTuple(interTuple);
 
-      for (Iterator<NTuple<Descriptor>> idxIter = implicitFlowTupleSet.iterator(); idxIter
-          .hasNext();) {
-        NTuple<Descriptor> tuple = idxIter.next();
-        addFlowGraphEdge(md, tuple, interTuple);
       }
 
-      NodeTupleSet newImplicitSet = new NodeTupleSet();
-      newImplicitSet.addTuple(interTuple);
-      analyzeFlowBlockNode(md, bn.getVarTable(), ln.getUpdate(), newImplicitSet);
-      analyzeFlowBlockNode(md, bn.getVarTable(), ln.getBody(), newImplicitSet);
+      // ///////////
+      // NTuple<Descriptor> interTuple = getFlowGraph(md).createIntermediateNode().getDescTuple();
+      //
+      // for (Iterator<NTuple<Descriptor>> idxIter = condTupleNode.iterator(); idxIter.hasNext();) {
+      // NTuple<Descriptor> tuple = idxIter.next();
+      // addFlowGraphEdge(md, tuple, interTuple);
+      // }
+      //
+      // for (Iterator<NTuple<Descriptor>> idxIter = implicitFlowTupleSet.iterator(); idxIter
+      // .hasNext();) {
+      // NTuple<Descriptor> tuple = idxIter.next();
+      // addFlowGraphEdge(md, tuple, interTuple);
+      // }
+      //
+      // NodeTupleSet newImplicitSet = new NodeTupleSet();
+      // newImplicitSet.addTuple(interTuple);
+      analyzeFlowBlockNode(md, bn.getVarTable(), ln.getUpdate(), newImplicitTupleSet);
+      analyzeFlowBlockNode(md, bn.getVarTable(), ln.getBody(), newImplicitTupleSet);
       // ///////////
 
       // condTupleNode.addTupleSet(implicitFlowTupleSet);
@@ -4191,12 +4056,14 @@ public class LocationInference {
     newImplicitTupleSet.addTupleSet(implicitFlowTupleSet);
     newImplicitTupleSet.addTupleSet(condTupleNode);
 
-    // System.out.println("condTupleNode=" + condTupleNode);
-    // System.out.println("implicitFlowTupleSet=" + implicitFlowTupleSet);
-    // System.out.println("newImplicitTupleSet=" + newImplicitTupleSet);
+    System.out.println("$$$GGGcondTupleNode=" + condTupleNode.getGlobalLocTupleSet());
 
-    if (newImplicitTupleSet.size() > 1) {
+    // System.out.println("-condTupleNode=" + condTupleNode);
+    // System.out.println("-implicitFlowTupleSet=" + implicitFlowTupleSet);
+    // System.out.println("-newImplicitTupleSet=" + newImplicitTupleSet);
 
+    if (needToGenerateInterLoc(newImplicitTupleSet)) {
+      System.out.println("2");
       // need to create an intermediate node for the GLB of conditional locations & implicit flows
       NTuple<Descriptor> interTuple = getFlowGraph(md).createIntermediateNode().getDescTuple();
       for (Iterator<NTuple<Descriptor>> idxIter = newImplicitTupleSet.iterator(); idxIter.hasNext();) {
@@ -4207,6 +4074,16 @@ public class LocationInference {
       newImplicitTupleSet.addTuple(interTuple);
     }
 
+    GlobalFlowGraph globalFlowGraph = getSubGlobalFlowGraph(md);
+    for (Iterator<NTuple<Location>> iterator = condTupleNode.globalIterator(); iterator.hasNext();) {
+      NTuple<Location> calleeReturnLocTuple = iterator.next();
+      for (Iterator<NTuple<Descriptor>> iter2 = newImplicitTupleSet.iterator(); iter2.hasNext();) {
+        NTuple<Descriptor> callerImplicitTuple = iter2.next();
+        globalFlowGraph.addValueFlowEdge(calleeReturnLocTuple,
+            translateToLocTuple(md, callerImplicitTuple));
+      }
+    }
+
     analyzeFlowBlockNode(md, nametable, isn.getTrueBlock(), newImplicitTupleSet);
 
     if (isn.getFalseBlock() != null) {
@@ -4233,7 +4110,8 @@ public class LocationInference {
 
       // creates edges from RHS to LHS
       NTuple<Descriptor> interTuple = null;
-      if (nodeSetRHS.size() > 1) {
+      if (needToGenerateInterLoc(nodeSetRHS)) {
+
         interTuple = getFlowGraph(md).createIntermediateNode().getDescTuple();
       }
 
@@ -4248,6 +4126,12 @@ public class LocationInference {
         addFlowGraphEdge(md, implicitTuple, tupleLHS);
       }
 
+      GlobalFlowGraph globalFlowGraph = getSubGlobalFlowGraph(md);
+      for (Iterator<NTuple<Location>> iterator = nodeSetRHS.globalIterator(); iterator.hasNext();) {
+        NTuple<Location> calleeReturnLocTuple = iterator.next();
+        globalFlowGraph.addValueFlowEdge(calleeReturnLocTuple, translateToLocTuple(md, tupleLHS));
+      }
+
     }
 
   }
@@ -4270,7 +4154,7 @@ public class LocationInference {
     // note that expression node can create more than one flow node
     // nodeSet contains of flow nodes
     // base is always assigned to null except the case of a name node!
-
+    System.out.println("en=" + en.printNode(0));
     NTuple<Descriptor> flowTuple;
 
     switch (en.kind()) {
@@ -4311,7 +4195,7 @@ public class LocationInference {
       break;
 
     case Kind.LiteralNode:
-      analyzeLiteralNode(md, nametable, (LiteralNode) en);
+      analyzeFlowLiteralNode(md, nametable, (LiteralNode) en, nodeSet);
       break;
 
     case Kind.MethodInvokeNode:
@@ -4343,6 +4227,9 @@ public class LocationInference {
     // checkOffsetNode(md, nametable, (OffsetNode)en, td);
     // return null;
 
+    }
+    if (nodeSet != null) {
+      System.out.println("#EXP nodeGLOBALSet=" + nodeSet.getGlobalLocTupleSet());
     }
     return null;
 
@@ -4402,18 +4289,34 @@ public class LocationInference {
     return mapMethodDescToParamNodeFlowsToReturnValue.get(md);
   }
 
-  private void analyzeFlowMethodInvokeNode(MethodDescriptor md, SymbolTable nametable,
+  private Set<NTuple<Location>> getPCLocTupleSet(MethodInvokeNode min) {
+    if (!mapMethodInvokeNodeToPCLocTupleSet.containsKey(min)) {
+      mapMethodInvokeNodeToPCLocTupleSet.put(min, new HashSet<NTuple<Location>>());
+    }
+    return mapMethodInvokeNodeToPCLocTupleSet.get(min);
+  }
+
+  private void analyzeFlowMethodInvokeNode(MethodDescriptor mdCaller, SymbolTable nametable,
       MethodInvokeNode min, NodeTupleSet nodeSet, NodeTupleSet implicitFlowTupleSet) {
 
     System.out.println("analyzeFlowMethodInvokeNode=" + min.printNode(0));
 
+    Set<NTuple<Location>> pcLocTupleSet = getPCLocTupleSet(min);
+    for (Iterator iterator = implicitFlowTupleSet.iterator(); iterator.hasNext();) {
+      NTuple<Descriptor> pcDescTuple = (NTuple<Descriptor>) iterator.next();
+      if (!pcDescTuple.get(0).equals(LITERALDESC)) {
+        // here we don't need to add the literal value as a PC location
+        pcLocTupleSet.add(translateToLocTuple(mdCaller, pcDescTuple));
+      }
+    }
+
     mapMethodInvokeNodeToArgIdxMap.put(min, new HashMap<Integer, NTuple<Descriptor>>());
 
     if (nodeSet == null) {
       nodeSet = new NodeTupleSet();
     }
 
-    MethodDescriptor calleeMethodDesc = min.getMethod();
+    MethodDescriptor mdCallee = min.getMethod();
 
     NameDescriptor baseName = min.getBaseName();
     boolean isSystemout = false;
@@ -4421,20 +4324,33 @@ public class LocationInference {
       isSystemout = baseName.getSymbol().equals("System.out");
     }
 
-    if (!ssjava.isSSJavaUtil(calleeMethodDesc.getClassDesc())
-        && !ssjava.isTrustMethod(calleeMethodDesc) && !isSystemout) {
+    if (!ssjava.isSSJavaUtil(mdCallee.getClassDesc()) && !ssjava.isTrustMethod(mdCallee)
+        && !isSystemout) {
 
-      addMapCallerMethodDescToMethodInvokeNodeSet(md, min);
+      addMapCallerMethodDescToMethodInvokeNodeSet(mdCaller, min);
 
-      FlowGraph calleeFlowGraph = getFlowGraph(calleeMethodDesc);
+      FlowGraph calleeFlowGraph = getFlowGraph(mdCallee);
+      System.out.println("mdCallee=" + mdCallee);
       Set<FlowNode> calleeReturnSet = calleeFlowGraph.getReturnNodeSet();
 
-      // System.out.println("-calleeReturnSet=" + calleeReturnSet);
+      System.out.println("---calleeReturnSet=" + calleeReturnSet);
+
+      // when generating the global flow graph,
+      // we need to add ordering relations from the set of callee return loc tuple to LHS of the
+      // caller assignment
+      for (Iterator iterator = calleeReturnSet.iterator(); iterator.hasNext();) {
+        FlowNode calleeReturnNode = (FlowNode) iterator.next();
+        NTuple<Location> calleeReturnLocTuple =
+            translateToLocTuple(mdCallee, calleeReturnNode.getDescTuple());
+        nodeSet.addGlobalFlowTuple(calleeReturnLocTuple);
+      }
+
+      NodeTupleSet tupleSet = new NodeTupleSet();
 
       if (min.getExpression() != null) {
 
         NodeTupleSet baseNodeSet = new NodeTupleSet();
-        analyzeFlowExpressionNode(md, nametable, min.getExpression(), baseNodeSet, null,
+        analyzeFlowExpressionNode(mdCaller, nametable, min.getExpression(), baseNodeSet, null,
             implicitFlowTupleSet, false);
 
         assert (baseNodeSet.size() == 1);
@@ -4447,20 +4363,23 @@ public class LocationInference {
           for (Iterator iterator = calleeReturnSet.iterator(); iterator.hasNext();) {
             FlowNode returnNode = (FlowNode) iterator.next();
             NTuple<Descriptor> returnDescTuple = returnNode.getDescTuple();
-            if (returnDescTuple.startsWith(calleeMethodDesc.getThis())) {
+            if (returnDescTuple.startsWith(mdCallee.getThis())) {
               // the location type of the return value is started with 'this'
               // reference
               NTuple<Descriptor> inFlowTuple = new NTuple<Descriptor>(baseTuple.getList());
               inFlowTuple.addAll(returnDescTuple.subList(1, returnDescTuple.size()));
-              nodeSet.addTuple(inFlowTuple);
+              // nodeSet.addTuple(inFlowTuple);
+              tupleSet.addTuple(inFlowTuple);
             } else {
               // TODO
               Set<FlowNode> inFlowSet = calleeFlowGraph.getIncomingFlowNodeSet(returnNode);
               // System.out.println("inFlowSet=" + inFlowSet + "   from retrunNode=" + returnNode);
               for (Iterator iterator2 = inFlowSet.iterator(); iterator2.hasNext();) {
                 FlowNode inFlowNode = (FlowNode) iterator2.next();
-                if (inFlowNode.getDescTuple().startsWith(calleeMethodDesc.getThis())) {
-                  nodeSet.addTupleSet(baseNodeSet);
+                if (inFlowNode.getDescTuple().startsWith(mdCallee.getThis())) {
+                  // nodeSet.addTupleSet(baseNodeSet);
+                  tupleSet.addTupleSet(baseNodeSet);
+
                 }
               }
             }
@@ -4468,7 +4387,6 @@ public class LocationInference {
         }
 
       }
-
       // analyze parameter flows
 
       if (min.numArgs() > 0) {
@@ -4484,16 +4402,16 @@ public class LocationInference {
           ExpressionNode en = min.getArg(i);
           int idx = i + offset;
           NodeTupleSet argTupleSet = new NodeTupleSet();
-          analyzeFlowExpressionNode(md, nametable, en, argTupleSet, false);
+          analyzeFlowExpressionNode(mdCaller, nametable, en, argTupleSet, false);
           // if argument is liternal node, argTuple is set to NULL
 
           NTuple<Descriptor> argTuple = new NTuple<Descriptor>();
-          if (argTupleSet.size() > 1) {
+          if (needToGenerateInterLoc(argTupleSet)) {
             NTuple<Descriptor> interTuple =
-                getFlowGraph(md).createIntermediateNode().getDescTuple();
+                getFlowGraph(mdCaller).createIntermediateNode().getDescTuple();
             for (Iterator<NTuple<Descriptor>> idxIter = argTupleSet.iterator(); idxIter.hasNext();) {
               NTuple<Descriptor> tuple = idxIter.next();
-              addFlowGraphEdge(md, tuple, interTuple);
+              addFlowGraphEdge(mdCaller, tuple, interTuple);
             }
             argTuple = interTuple;
           } else if (argTupleSet.size() == 1) {
@@ -4502,18 +4420,53 @@ public class LocationInference {
             argTuple = new NTuple<Descriptor>();
           }
 
+          if (argTuple.size() > 0 && argTuple.get(argTuple.size() - 1).equals(LITERALDESC)) {
+            argTuple = new NTuple<Descriptor>();
+          }
+
+          // if an argument is static value
+          // we need to create an itermediate node so that we could assign a composite location to
+          // that node if needed
+          System.out.println("argTuple=" + argTuple);
+          if (argTuple.size() > 0 && argTuple.get(0).equals(GLOBALDESC)) {
+            System.out.println("***GLOBAL ARG TUPLE CASE=" + argTuple);
+            NTuple<Descriptor> interTuple =
+                getFlowGraph(mdCaller).createIntermediateNode().getDescTuple();
+            addFlowGraphEdge(mdCaller, argTuple, interTuple);
+            argTuple = interTuple;
+          }
+
           addArgIdxMap(min, idx, argTuple);
 
           FlowNode paramNode = calleeFlowGraph.getParamFlowNode(idx);
+
+          // check whether a param node in the callee graph has incoming flows
+          // if it has incoming flows, the corresponding arg should be lower than the current PC
+          // Descriptor prefix = paramNode.getDescTuple().get(0);
+          // if (calleeFlowGraph.getIncomingNodeSetByPrefix(prefix).size() > 0) {
+          // for (Iterator<NTuple<Descriptor>> iterator = implicitFlowTupleSet.iterator(); iterator
+          // .hasNext();) {
+          // NTuple<Descriptor> pcTuple = iterator.next();
+          // System.out.println("add edge pcTuple =" + pcTuple + " -> " + argTuple);
+          // addFlowGraphEdge(md, pcTuple, argTuple);
+          // }
+          // }
+
           if (hasInFlowTo(calleeFlowGraph, paramNode, calleeReturnSet)
-              || calleeMethodDesc.getModifiers().isNative()) {
-            addParamNodeFlowingToReturnValue(calleeMethodDesc, paramNode);
-            nodeSet.addTupleSet(argTupleSet);
+              || mdCallee.getModifiers().isNative()) {
+            addParamNodeFlowingToReturnValue(mdCallee, paramNode);
+            // nodeSet.addTupleSet(argTupleSet);
+            tupleSet.addTupleSet(argTupleSet);
           }
         }
 
       }
 
+      if (mdCallee.getReturnType() != null && !mdCallee.getReturnType().isVoid()) {
+        FlowReturnNode setNode = getFlowGraph(mdCaller).createReturnNode(min);
+        nodeSet.addTuple(setNode.getDescTuple());
+      }
+      
       // propagateFlowsFromCallee(min, md, min.getMethod());
 
       System.out.println("min nodeSet=" + nodeSet);
@@ -4526,7 +4479,7 @@ public class LocationInference {
 
     // Set<FlowNode> reachableSet = fg.getReachFlowNodeSetFrom(inNode);
     Set<FlowNode> reachableSet = fg.getReachableSetFrom(inNode.getDescTuple());
-    System.out.println("inNode=" + inNode + "  reachalbeSet=" + reachableSet);
+    // System.out.println("inNode=" + inNode + "  reachalbeSet=" + reachableSet);
 
     for (Iterator iterator = reachableSet.iterator(); iterator.hasNext();) {
       FlowNode fn = (FlowNode) iterator.next();
@@ -4553,13 +4506,21 @@ public class LocationInference {
     mapIdxToTuple.put(new Integer(idx), argTuple);
   }
 
-  private void analyzeLiteralNode(MethodDescriptor md, SymbolTable nametable, LiteralNode en) {
-
+  private void analyzeFlowLiteralNode(MethodDescriptor md, SymbolTable nametable, LiteralNode en,
+      NodeTupleSet nodeSet) {
+    NTuple<Descriptor> tuple = new NTuple<Descriptor>();
+    tuple.add(LITERALDESC);
+    nodeSet.addTuple(tuple);
   }
 
   private void analyzeFlowArrayAccessNode(MethodDescriptor md, SymbolTable nametable,
       ArrayAccessNode aan, NodeTupleSet nodeSet, boolean isLHS) {
 
+    // System.out.println("analyzeFlowArrayAccessNode aan=" + aan.printNode(0));
+    String currentArrayAccessNodeExpStr = aan.printNode(0);
+    arrayAccessNodeStack.push(aan.printNode(0));
+
+    // System.out.println("-exp=" + aan.getExpression().printNode(0));
     NodeTupleSet expNodeTupleSet = new NodeTupleSet();
     NTuple<Descriptor> base =
         analyzeFlowExpressionNode(md, nametable, aan.getExpression(), expNodeTupleSet, isLHS);
@@ -4567,6 +4528,8 @@ public class LocationInference {
     NodeTupleSet idxNodeTupleSet = new NodeTupleSet();
     analyzeFlowExpressionNode(md, nametable, aan.getIndex(), idxNodeTupleSet, isLHS);
 
+    arrayAccessNodeStack.pop();
+
     if (isLHS) {
       // need to create an edge from idx to array
       for (Iterator<NTuple<Descriptor>> idxIter = idxNodeTupleSet.iterator(); idxIter.hasNext();) {
@@ -4577,11 +4540,44 @@ public class LocationInference {
         }
       }
 
+      GlobalFlowGraph globalFlowGraph = getSubGlobalFlowGraph(md);
+      for (Iterator<NTuple<Location>> iterator = idxNodeTupleSet.globalIterator(); iterator
+          .hasNext();) {
+        NTuple<Location> calleeReturnLocTuple = iterator.next();
+        for (Iterator<NTuple<Descriptor>> arrIter = expNodeTupleSet.iterator(); arrIter.hasNext();) {
+          NTuple<Descriptor> arrTuple = arrIter.next();
+          globalFlowGraph.addValueFlowEdge(calleeReturnLocTuple, translateToLocTuple(md, arrTuple));
+        }
+      }
+
       nodeSet.addTupleSet(expNodeTupleSet);
     } else {
-      nodeSet.addTupleSet(expNodeTupleSet);
-      nodeSet.addTupleSet(idxNodeTupleSet);
+
+      NodeTupleSet nodeSetArrayAccessExp = new NodeTupleSet();
+
+      nodeSetArrayAccessExp.addTupleSet(expNodeTupleSet);
+      nodeSetArrayAccessExp.addTupleSet(idxNodeTupleSet);
+
+      if (arrayAccessNodeStack.isEmpty()
+          || !arrayAccessNodeStack.peek().startsWith(currentArrayAccessNodeExpStr)) {
+
+        if (needToGenerateInterLoc(nodeSetArrayAccessExp)) {
+          NTuple<Descriptor> interTuple = getFlowGraph(md).createIntermediateNode().getDescTuple();
+
+          for (Iterator<NTuple<Descriptor>> iter = nodeSetArrayAccessExp.iterator(); iter.hasNext();) {
+            NTuple<Descriptor> higherTuple = iter.next();
+            addFlowGraphEdge(md, higherTuple, interTuple);
+          }
+          nodeSetArrayAccessExp.clear();
+          nodeSetArrayAccessExp.addTuple(interTuple);
+        }
+      }
+
+      nodeSet.addGlobalFlowTupleSet(idxNodeTupleSet.getGlobalLocTupleSet());
+      nodeSet.addTupleSet(nodeSetArrayAccessExp);
+
     }
+
   }
 
   private void analyzeCreateObjectNode(MethodDescriptor md, SymbolTable nametable,
@@ -4642,6 +4638,10 @@ public class LocationInference {
       // there are two operands
       nodeSet.addTupleSet(leftOpSet);
       nodeSet.addTupleSet(rightOpSet);
+
+      nodeSet.addGlobalFlowTupleSet(leftOpSet.getGlobalLocTupleSet());
+      nodeSet.addGlobalFlowTupleSet(rightOpSet.getGlobalLocTupleSet());
+
       break;
 
     default:
@@ -4653,8 +4653,6 @@ public class LocationInference {
   private NTuple<Descriptor> analyzeFlowNameNode(MethodDescriptor md, SymbolTable nametable,
       NameNode nn, NodeTupleSet nodeSet, NTuple<Descriptor> base, NodeTupleSet implicitFlowTupleSet) {
 
-    // System.out.println("analyzeFlowNameNode=" + nn.printNode(0));
-
     if (base == null) {
       base = new NTuple<Descriptor>();
     }
@@ -4735,10 +4733,13 @@ public class LocationInference {
   private NTuple<Descriptor> analyzeFlowFieldAccessNode(MethodDescriptor md, SymbolTable nametable,
       FieldAccessNode fan, NodeTupleSet nodeSet, NTuple<Descriptor> base,
       NodeTupleSet implicitFlowTupleSet, boolean isLHS) {
+    // System.out.println("analyzeFlowFieldAccessNode=" + fan.printNode(0));
 
+    String currentArrayAccessNodeExpStr = null;
     ExpressionNode left = fan.getExpression();
     TypeDescriptor ltd = left.getType();
     FieldDescriptor fd = fan.getField();
+    ArrayAccessNode aan = null;
 
     String varName = null;
     if (left.kind() == Kind.NameNode) {
@@ -4755,9 +4756,15 @@ public class LocationInference {
 
     NodeTupleSet idxNodeTupleSet = new NodeTupleSet();
 
+    boolean isArrayCase = false;
     if (left instanceof ArrayAccessNode) {
 
-      ArrayAccessNode aan = (ArrayAccessNode) left;
+      isArrayCase = true;
+      aan = (ArrayAccessNode) left;
+
+      currentArrayAccessNodeExpStr = aan.printNode(0);
+      arrayAccessNodeStack.push(currentArrayAccessNodeExpStr);
+
       left = aan.getExpression();
       analyzeFlowExpressionNode(md, nametable, aan.getIndex(), idxNodeTupleSet, base,
           implicitFlowTupleSet, isLHS);
@@ -4775,13 +4782,11 @@ public class LocationInference {
       NTuple<Descriptor> flowFieldTuple = new NTuple<Descriptor>(base.toList());
 
       if (!left.getType().isPrimitive()) {
-
         if (!fd.getSymbol().equals("length")) {
           // array.length access, just have the location of the array
           flowFieldTuple.add(fd);
           nodeSet.removeTuple(base);
         }
-
       }
       getFlowGraph(md).createNewFlowNode(flowFieldTuple);
 
@@ -4790,9 +4795,47 @@ public class LocationInference {
           NTuple<Descriptor> idxTuple = idxIter.next();
           getFlowGraph(md).addValueFlowEdge(idxTuple, flowFieldTuple);
         }
+
+        GlobalFlowGraph globalFlowGraph = getSubGlobalFlowGraph(md);
+        for (Iterator<NTuple<Location>> iterator = idxNodeTupleSet.globalIterator(); iterator
+            .hasNext();) {
+          NTuple<Location> calleeReturnLocTuple = iterator.next();
+          globalFlowGraph.addValueFlowEdge(calleeReturnLocTuple,
+              translateToLocTuple(md, flowFieldTuple));
+        }
+
+      } else {
+
+        // if it is the array case and not the LHS case
+        if (isArrayCase) {
+          arrayAccessNodeStack.pop();
+
+          if (arrayAccessNodeStack.isEmpty()
+              || !arrayAccessNodeStack.peek().startsWith(currentArrayAccessNodeExpStr)) {
+            NodeTupleSet nodeSetArrayAccessExp = new NodeTupleSet();
+
+            nodeSetArrayAccessExp.addTuple(flowFieldTuple);
+            nodeSetArrayAccessExp.addTupleSet(idxNodeTupleSet);
+
+            if (needToGenerateInterLoc(nodeSetArrayAccessExp)) {
+              NTuple<Descriptor> interTuple =
+                  getFlowGraph(md).createIntermediateNode().getDescTuple();
+
+              for (Iterator<NTuple<Descriptor>> iter = nodeSetArrayAccessExp.iterator(); iter
+                  .hasNext();) {
+                NTuple<Descriptor> higherTuple = iter.next();
+                addFlowGraphEdge(md, higherTuple, interTuple);
+              }
+              flowFieldTuple = interTuple;
+            }
+
+            nodeSet.addGlobalFlowTupleSet(idxNodeTupleSet.getGlobalLocTupleSet());
+          }
+
+        }
+
       }
       return flowFieldTuple;
-
     }
 
   }
@@ -4846,7 +4889,7 @@ public class LocationInference {
 
       // creates edges from RHS to LHS
       NTuple<Descriptor> interTuple = null;
-      if (nodeSetRHS.size() > 1) {
+      if (needToGenerateInterLoc(nodeSetRHS)) {
         interTuple = getFlowGraph(md).createIntermediateNode().getDescTuple();
       }
 
@@ -4867,6 +4910,21 @@ public class LocationInference {
         }
       }
 
+      // create global flow edges if the callee gives return value flows to the caller
+      if (nodeSetRHS.globalLocTupleSize() > 0) {
+        GlobalFlowGraph globalFlowGraph = getSubGlobalFlowGraph(md);
+        for (Iterator<NTuple<Location>> iterator = nodeSetRHS.globalIterator(); iterator.hasNext();) {
+          NTuple<Location> calleeReturnLocTuple = iterator.next();
+          for (Iterator<NTuple<Descriptor>> iter2 = nodeSetLHS.iterator(); iter2.hasNext();) {
+            NTuple<Descriptor> callerLHSTuple = iter2.next();
+            globalFlowGraph.addValueFlowEdge(calleeReturnLocTuple,
+                translateToLocTuple(md, callerLHSTuple));
+            System.out.println("$$$ GLOBAL FLOW ADD=" + calleeReturnLocTuple + " -> "
+                + translateToLocTuple(md, callerLHSTuple));
+          }
+        }
+      }
+
     } else {
       // postinc case
 
@@ -4927,7 +4985,7 @@ public class LocationInference {
     String fileName = "lattice_";
     if (md != null) {
       fileName +=
-          cd.getSymbol().replaceAll("[\\W_]", "") + "_" + md.toString().replaceAll("[\\W_]", "");
+      /* cd.getSymbol().replaceAll("[\\W_]", "") + "_" + */md.toString().replaceAll("[\\W_]", "");
     } else {
       fileName += cd.getSymbol().replaceAll("[\\W_]", "");
     }
@@ -5010,14 +5068,16 @@ public class LocationInference {
     bw.write(locName + " [label=\"" + prettyStr + "\"]" + ";\n");
   }
 
-  public void _debug_printGraph() {
+  public void _debug_writeFlowGraph() {
     Set<MethodDescriptor> keySet = mapMethodDescriptorToFlowGraph.keySet();
 
     for (Iterator<MethodDescriptor> iterator = keySet.iterator(); iterator.hasNext();) {
       MethodDescriptor md = (MethodDescriptor) iterator.next();
       FlowGraph fg = mapMethodDescriptorToFlowGraph.get(md);
+      GlobalFlowGraph subGlobalFlowGraph = getSubGlobalFlowGraph(md);
       try {
         fg.writeGraph();
+        subGlobalFlowGraph.writeGraph("_SUBGLOBAL");
       } catch (IOException e) {
         e.printStackTrace();
       }