r202437 - Remove an assertion that no longer holds. <rdar://problem/16135814>

Bob Wilson bob.wilson at apple.com
Thu Feb 27 13:59:17 PST 2014


Author: bwilson
Date: Thu Feb 27 15:59:17 2014
New Revision: 202437

URL: http://llvm.org/viewvc/llvm-project?rev=202437&view=rev
Log:
Remove an assertion that no longer holds. <rdar://problem/16135814>

In r201528, I changed the PGO instrumentation counter for a "do" loop to not
include the fall-through count. That fall-through count is included later, b
it means that this assertion may fail for "do" loops.

Modified:
    cfe/trunk/lib/CodeGen/CodeGenPGO.h

Modified: cfe/trunk/lib/CodeGen/CodeGenPGO.h
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/CodeGen/CodeGenPGO.h?rev=202437&r1=202436&r2=202437&view=diff
==============================================================================
--- cfe/trunk/lib/CodeGen/CodeGenPGO.h (original)
+++ cfe/trunk/lib/CodeGen/CodeGenPGO.h Thu Feb 27 15:59:17 2014
@@ -186,7 +186,6 @@ public:
   /// within the region. The adjusted count, then, is the value of the counter
   /// at the end of the region.
   uint64_t getAdjustedCount() const {
-    assert((Adjust > 0 || (uint64_t)(-Adjust) <= Count) && "Negative count");
     return Count + Adjust;
   }
 





More information about the cfe-commits mailing list