RFR: 8199604: Rename CardTableModRefBS to CardTableBarrierSet

[Erik Österlund]

Hi Stefan,

Thank you for the review!

Thanks,
/Erik

On 2018-03-16 16:06, Stefan Karlsson wrote:
> Looks good.
>
> StefanK
>
> On 2018-03-16 13:17, Erik Österlund wrote:
>> Hi,
>>
>> After collapsing barrier sets, it seems like CardTableModRefBS is the 
>> only barrier set that encodes "ModRef" in its name and uses the "BS" 
>> suffix instead of spelling out "BarrierSet".
>>
>> This is a weird inconsistency that I have solved by renaming 
>> CardTableModRefBS to CardTableBarrierSet.
>>
>> Files were renamed like this:
>> parCardTableModRefBS.cpp => cmsCardTable.cpp (this file contains a 
>> bunch of implementations for the card table used by CMS and is in the 
>> CMS directory)
>> cardTableModRefBS.cpp => cardTableBarrierSet.cpp
>> cardTableModRefBS.hpp => cardTableBarrierSet.hpp
>> cardTableModRefBS.inline.hpp => cardTableBarrierSet.inline.hpp
>>
>> I have performed an automatic search and replace to rename 
>> CardTableModRefBS to CardTableBarrierSet, and manually clicked 
>> through everything to make sure that space alignment, comments, etc 
>> all look as they should. I have also checked that the SA agent is not 
>> affected and run hs-tier1 on this changeset.
>>
>> Despite the fix being a bit lengthy, I would still prefer to consider 
>> it trivial (unless somebody opposes that) as I am only renaming a 
>> class in a rather mechanical way. Hope we can agree about that.
>>
>> Webrev:
>> http://cr.openjdk.java.net/~eosterlund/8199604/webrev.00/
>>
>> Bug:
>> https://bugs.openjdk.java.net/browse/JDK-8199604
>>
>> Testing:
>> mach5 hs-tier1-3
>>
>> Thanks,
>> /Erik