RFR: [leyden] Do not enable -XX:AOTMode=on by default [v2]

[Ioi Lam]

> During early Leyden development, we decided to make `-XX:AOTMode=on` the default to make it easier to diagnose mismatches between training and production runs. However, this make it impossible to run the `make test JTREG=AOT_JDK...` tests:
> 
> 
> $ make test JTREG=AOT_JDK=onestep TEST=open/test/hotspot/jtreg/runtime/invokedynamic
> [...]
> Running test 'jtreg:open/test/hotspot/jtreg/runtime/invokedynamic'
> [0.010s][error][aot] An error has occurred while processing the AOT cache. Run with -Xlog:aot for details.
> [0.010s][error][aot] Mismatched values for property jdk.module.addexports: java.base/jdk.internal.foreign=ALL-UNNAMED,java.base/jdk.internal.misc=ALL-UNNAMED specified during runtime but not during dump time
> [0.010s][error][aot] Disabling optimized module handling
> [0.010s][error][aot] AOT cache has aot-linked classes. It cannot be used when archived full module graph is not used.
> [0.011s][error][aot] Unable to map shared spaces
> failed to get JDK properties:
> 
> 
> Now people should be familiar with using the AOT cache, so this nanny mode isn't necessary more.

Ioi Lam has updated the pull request incrementally with one additional commit since the last revision:

  Fixed test failure

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

Changes:
  - all: https://git.openjdk.org/leyden/pull/82/files
  - new: https://git.openjdk.org/leyden/pull/82/files/73b82f01..6bdfac02

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=leyden&pr=82&range=01
 - incr: https://webrevs.openjdk.org/?repo=leyden&pr=82&range=00-01

  Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod
  Patch: https://git.openjdk.org/leyden/pull/82.diff
  Fetch: git fetch https://git.openjdk.org/leyden.git pull/82/head:pull/82

PR: https://git.openjdk.org/leyden/pull/82