Remove invalid assert in CLRLifoSemaphore (#14805)
authorKoundinya Veluri <kouvel@users.noreply.github.com>
Thu, 2 Nov 2017 20:49:11 +0000 (13:49 -0700)
committerGitHub <noreply@github.com>
Thu, 2 Nov 2017 20:49:11 +0000 (13:49 -0700)
commit8c0fd04f78ef2b6325c6be876cc2c2f8e2bc9a1f
treefe3abd38c2934d37d61f13e8478fff1add5327b2
parent2f7596487eb74a9dee0eb1c25d2f0f0047f963c4
Remove invalid assert in CLRLifoSemaphore (#14805)

After https://github.com/dotnet/coreclr/pull/14535, the assert is no longer valid. A thread that times out does not decrement the count of waiters signaled to wake because a timed-out thread does not observe a signal to the object that was waited upon. If there were no other waiters to observe the signal, the count of waiters signaled to wake can become greater than the waiter count. This is valid, and tracks how many signals are pending. I don't think there is a good way to update the assert to reflect this, removed the assert.
src/vm/synch.cpp