RFR: 8072817: CardTableExtension kind() should be BarrierSet::CardTableExtension
kim.barrett at oracle.com
Thu Aug 27 20:45:57 UTC 2015
On Aug 27, 2015, at 2:48 PM, Jesper Wilhelmsson <jesper.wilhelmsson at oracle.com> wrote:
> Hi Kim,
> Looks good in general.
> In src/cpu/aarch64/vm/macroAssembler_aarch64.cpp there is a comma between the two checks:
> + assert(bs->kind() == BarrierSet::CardTableForRS,
> + bs->kind() == BarrierSet::CardTableExtension,
> + "Wrong barrier set kind");
> I assume there should be a ||
> Besides that the patch seems to do what you described below.
You are correct. Unfortunately, jprt doesn’t test the platform that uses that file.
More information about the hotspot-gc-dev