[PATCH] D20957: [JumpThreading] Prevent dangling pointer problems in BranchProbabilityInfo

Sanjoy Das via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 8 12:14:31 PDT 2016


sanjoy added subscribers: congh, dnovillo.

================
Comment at: lib/Analysis/BranchProbabilityInfo.cpp:644
@@ +643,3 @@
+  for (auto I = Probs.begin(), E = Probs.end(); I != E; ++I) {
+    auto Key = I->first;
+    if (Key.first == BB)
----------------
I see, but I'm not sure that this won't be a compile time regression.  Maybe @dnovillo or @congh (names that show up on git-blame) will have an idea?

================
Comment at: unittests/Analysis/BlockFrequencyInfoTest.cpp:58
@@ +57,3 @@
+
+  void releaseBFI() {
+    // Prevent AssertingVH from failing.
----------------
In that case, let's call it `destroyBFI`?


http://reviews.llvm.org/D20957





More information about the llvm-commits mailing list