RFR(XS): 8194741: CDS broken due to 8194741
dmitry.samersoff at bell-sw.com
Mon Jan 15 15:04:39 UTC 2018
The fix looks good to me.
On 15.01.2018 17:25, Erik Österlund wrote:
> I accidentally broke CDS with 8194741, which affects a bunch of tier 2
> CDS tests.
> The problem is trivial: I missed a branch condition where the resolved
> references object is null that should have gone to the else bit, but did
> not after my change.
> Sorry for breaking this - I seem to have mixed up my test run results
> from tier 1-5, and after that only ran an extra tier 1 sanity check
> before checking in which did not catch this.
> I have run the failing tests locally to verify they work now (and they
> do), and am running tier 1-2 again before meanwhile.
> Here is the original webrev of 8194741 where I messed up the condition
> in constantPool.cpp:330:
More information about the hotspot-runtime-dev