RFR: 8263968: CDS: java/lang/ModuleLayer.EMPTY_LAYER should be singleton

[David Holmes]

On Mon, 22 Mar 2021 19:40:19 GMT, Aleksei Voitylov <avoitylov at openjdk.org> wrote:

> With CDS and G1, test api/java_lang/ModuleLayer/Boot.html fails in JDK 16. 
> 
> After JDK-8253081 with CDS enabled two instances of empty layer appear in the runtime. One comes from default initialization of java/lang/ModuleLayer.EMPTY_LAYER, another one comes from CDS archive and is returned by ModuleLayer.boot().parents().get(0). The fix makes java/lang/ModuleLayer.EMPTY_LAYER a singleton with both CDS on and off, similar to java/lang/module/Configuration.EMPTY_CONFIGURATION.
> 
> Testing: JCK 16, jtreg (including newly added test), pre-submit GitHub actions tests.

Seems reasonable.

Though I hadn't realized CDS intruded into the Java layer that way!

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

Marked as reviewed by dholmes (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/3131