be more careful about moving check...could break stuff in the future accidentally...
authorbdemsky <bdemsky>
Wed, 10 Nov 2010 02:03:34 +0000 (02:03 +0000)
committerbdemsky <bdemsky>
Wed, 10 Nov 2010 02:03:34 +0000 (02:03 +0000)
Robust/src/IR/Flat/RuntimeConflictResolver.java

index f5ff473a159eb9b38bd6d2e4576eef930188c8b2..c94f2e78e6a0969e124e6090c016e0587fc23aec 100644 (file)
@@ -817,16 +817,17 @@ public class RuntimeConflictResolver {
     String tasksrc=taint.isRBlockTaint()?"(SESEcommon *) record, ":"(SESEcommon *)(((INTPTR)record)|1LL), ";
 
 
-    //int heaprootNum = connectedHRHash.get(taint).id;
     // YUCKY HACK FOR NOW, ALWAYS USE ONE HASH TABLE
     // (CONSVERATIVELY PUT ALL PARAMS INTO ONE CONNECTED COMPONENT)
     // UNTIL WE FIGURE OUT WHY YOU SOMETIMES GET DIFFERENT
     // heaprootNum VALUES WHEN THEY SHOULD BE THE SAME
+    // We should use the commented lines in the if statements...
     int heaprootNum = 0;
-    assert heaprootNum != -1;
     
     //Do call if we need it.
     if(primConfWrite||objConfWrite) {
+      //int heaprootNum = connectedHRHash.get(taint).id;
+      assert heaprootNum != -1;
       int allocSiteID = connectedHRHash.get(taint).getWaitingQueueBucketNum(curr);
       int traverserID = doneTaints.get(taint);
         currCase.append("    int tmpkey"+depth+"=rcr_generateKey("+prefix+");\n");
@@ -835,6 +836,8 @@ public class RuntimeConflictResolver {
       else
         currCase.append("    int tmpvar"+depth+"=rcr_WRITEBINCASE(allHashStructures["+heaprootNum+"], tmpkey"+depth+", "+ tasksrc+strrcr+index+");\n");
     } else if (primConfRead||objConfRead) {
+      //int heaprootNum = connectedHRHash.get(taint).id;
+      assert heaprootNum != -1;
       int allocSiteID = connectedHRHash.get(taint).getWaitingQueueBucketNum(curr);
       int traverserID = doneTaints.get(taint);
       currCase.append("    int tmpkey"+depth+"=rcr_generateKey("+prefix+");\n");