RFR(S): 8159817: Convert FreeRegionList_test to GTest

Kim Barrett kim.barrett at oracle.com
Thu Oct 20 18:59:11 UTC 2016


> On Oct 19, 2016, at 5:19 PM, Kirill Zhaldybin <kirill.zhaldybin at oracle.com> wrote:
> 
> Dear all, Kim,
> 
> After the discussion on Gtest and a few fixes which improved assert/guarantee behavior if they were triggered in gtest I made a new version of this fix.
> 
> Here are a new WebRev: http://cr.openjdk.java.net/~kzhaldyb/webrevs/JDK-8159817/webrev.03/
> 
> Changes:
> 1. Instead of new macro I added a check which prevents the test to be run if G1 is not the current GC
> 2. Since verify_list is uses guarantee it could be considered safe to be used in gtests.
> 
> Could you please let me know your opinion?

Looks good.

Just one tiny nit; no need for a new webrev for this:

------------------------------------------------------------------------------ 
test/native/gc/g1/test_freeRegionList.cpp
  46   G1RegionToSpaceMapper* bot_storage =
  47       G1RegionToSpaceMapper::create_mapper(bot_rs,

Line 47 indented 4; should be 2.

------------------------------------------------------------------------------




More information about the hotspot-gc-dev mailing list