src/hotspot/share/gc/shenandoah/shenandoahConcurrentMark.cpp

rev 59439 : 8245825: Shenandoah: Remove diagnostic flag ShenandoahConcurrentScanCodeRoots

*** 306,318 ****
      // Can save time not walking through forwarding pointers.
      ShenandoahInitMarkRootsTask<NONE> mark_roots(&root_proc);
      workers->run_task(&mark_roots);
    }
  
-   if (ShenandoahConcurrentScanCodeRoots) {
      clear_claim_codecache();
-   }
  }
  
  void ShenandoahConcurrentMark::update_roots(ShenandoahPhaseTimings::Phase root_phase) {
    assert(ShenandoahSafepoint::is_at_shenandoah_safepoint(), "Must be at a safepoint");
    assert(root_phase == ShenandoahPhaseTimings::full_gc_update_roots ||
--- 306,316 ----
*** 390,400 ****
      _task_queues->register_queue(i, task_queue);
    }
  }
  
  void ShenandoahConcurrentMark::concurrent_scan_code_roots(uint worker_id, ReferenceProcessor* rp) {
!   if (ShenandoahConcurrentScanCodeRoots && claim_codecache()) {
      ShenandoahObjToScanQueue* q = task_queues()->queue(worker_id);
      if (!_heap->unload_classes()) {
        MutexLocker mu(CodeCache_lock, Mutex::_no_safepoint_check_flag);
        // TODO: We can not honor StringDeduplication here, due to lock ranking
        // inversion. So, we may miss some deduplication candidates.
--- 388,398 ----
      _task_queues->register_queue(i, task_queue);
    }
  }
  
  void ShenandoahConcurrentMark::concurrent_scan_code_roots(uint worker_id, ReferenceProcessor* rp) {
!   if (claim_codecache()) {
      ShenandoahObjToScanQueue* q = task_queues()->queue(worker_id);
      if (!_heap->unload_classes()) {
        MutexLocker mu(CodeCache_lock, Mutex::_no_safepoint_check_flag);
        // TODO: We can not honor StringDeduplication here, due to lock ranking
        // inversion. So, we may miss some deduplication candidates.
*** 941,953 ****
      }
    }
  }
  
  bool ShenandoahConcurrentMark::claim_codecache() {
-   assert(ShenandoahConcurrentScanCodeRoots, "must not be called otherwise");
    return _claimed_codecache.try_set();
  }
  
  void ShenandoahConcurrentMark::clear_claim_codecache() {
-   assert(ShenandoahConcurrentScanCodeRoots, "must not be called otherwise");
    _claimed_codecache.unset();
  }
--- 939,949 ----