8167425: Redundant code in method PerfLongVariant::sample
authorshshahma
Tue, 14 Mar 2017 22:14:33 -0700
changeset 46334 cfbf22d1d4b3
parent 46333 cef2c78aaa56
child 46336 676abd5f6407
8167425: Redundant code in method PerfLongVariant::sample Reviewed-by: dholmes, coleenp, sspitsyn
hotspot/src/share/vm/runtime/perfData.cpp
--- 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,