changeset 46334:cfbf22d1d4b3

8167425: Redundant code in method PerfLongVariant::sample Reviewed-by: dholmes, coleenp, sspitsyn
author shshahma
date Tue, 14 Mar 2017 22:14:33 -0700
parents cef2c78aaa56
children 676abd5f6407
files hotspot/src/share/vm/runtime/perfData.cpp
diffstat 1 files changed, 0 insertions(+), 9 deletions(-) [+]
line wrap: on
line diff
--- a/hotspot/src/share/vm/runtime/perfData.cpp	Wed Dec 14 09:14:16 2016 +0100
+++ b/hotspot/src/share/vm/runtime/perfData.cpp	Tue Mar 14 22:14:33 2017 -0700
@@ -213,18 +213,9 @@
 }
 
 void PerfLongVariant::sample() {
-
-  // JJJ - This should not happen.  Maybe the first sample is taken
-  // while the _sample_helper is being null'ed out.
-  // assert(_sample_helper != NULL || _sampled != NULL, "unexpected state");
-  if (_sample_helper == NULL) return;
-
   if (_sample_helper != NULL) {
     *(jlong*)_valuep = _sample_helper->take_sample();
   }
-  else if (_sampled != NULL) {
-    *(jlong*)_valuep = *_sampled;
-  }
 }
 
 PerfByteArray::PerfByteArray(CounterNS ns, const char* namep, Units u,