src/hotspot/share/gc/g1/g1RegionToSpaceMapper.cpp

8221260: Initialize more class members on construction, remove some unused ones
Reviewed-by:

*** 1,7 ****
  /*
!  * Copyright (c) 2001, 2018, Oracle and/or its affiliates. All rights reserved.
   * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
   *
   * This code is free software; you can redistribute it and/or modify it
   * under the terms of the GNU General Public License version 2 only, as
   * published by the Free Software Foundation.
--- 1,7 ----
  /*
!  * Copyright (c) 2001, 2019, Oracle and/or its affiliates. All rights reserved.
   * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
   *
   * This code is free software; you can redistribute it and/or modify it
   * under the terms of the GNU General Public License version 2 only, as
   * published by the Free Software Foundation.
***************
*** 203,214 ****
--- 203,216 ----
                                                           size_t alloc_granularity,
                                                           size_t commit_factor,
                                                           MemoryType type) :
    G1RegionToSpaceMapper(rs, actual_size, page_size, alloc_granularity, commit_factor, type),
    _rs(rs),
+   _dram_mapper(NULL),
    _num_committed_dram(0),
    _num_committed_nvdimm(0),
+   _start_index_of_dram(0),
    _page_size(page_size),
    _commit_factor(commit_factor),
    _type(type) {
    assert(actual_size == 2 * MaxHeapSize, "For 2-way heterogenuous heap, reserved space is two times MaxHeapSize");
  }
***************
*** 246,256 ****
      _dram_mapper = new G1RegionsLargerThanCommitSizeMapper(rs_dram, rs_dram.size(), _page_size, _region_granularity, _commit_factor, _type);
    } else {
      _dram_mapper = new G1RegionsSmallerThanCommitSizeMapper(rs_dram, rs_dram.size(), _page_size, _region_granularity, _commit_factor, _type);
    }
  
-   _start_index_of_nvdimm = 0;
    _start_index_of_dram = (uint)(rs_nvdimm.size() / _region_granularity);
    return true;
  }
  
  void G1RegionToHeteroSpaceMapper::commit_regions(uint start_idx, size_t num_regions, WorkGang* pretouch_gang) {
--- 248,257 ----