[PATCH] D18818: [FIX] Adjust execution context of hoisted loads wrt. error domains

Hongbin Zheng via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 6 02:59:05 PDT 2016


etherzhhb added a subscriber: etherzhhb.

================
Comment at: lib/Analysis/ScopInfo.cpp:2130-2133
@@ -2129,3 +2129,6 @@
       DomainMap[ErrorChildBlock] = Empty;
-      isl_set_free(CurrentDomain);
+      if (!ErrorDomainMap.count(ErrorChildBlock))
+        ErrorDomainMap[ErrorChildBlock] = CurrentDomain;
+      else
+        isl_set_free(CurrentDomain);
     }
----------------
Maybe we can just do an insert here:

```
if (!ErrorDomainMap.insert(std::make_pair(ErrorChildBlock, CurrentDomain).second)
  isl_set_free(CurrentDomain);
```


http://reviews.llvm.org/D18818





More information about the llvm-commits mailing list