RFR: 8348858: [leyden] Bump the default code buffer sizes to store more generated code [v3]

Aleksey Shipilev shade at openjdk.org
Thu Jan 30 12:01:37 UTC 2025


> Due to current prototype limitation, we cannot yet store the generated code that has the expanded code buffer. I tried to address that directly, but I think relocations disagree with the whole thing, so this implementation limitation stays for a bit longer. I turned the bailout due to that cause from `info` into `warning`.
> 
> On `JavacBenchApp 50`, this causes us to lose 700 (!) C2 compiled methods from the SCC! We can dodge significant part of the hit by bumping the default code buffer sizes, and thus making buffers less likely to require resizing, and thus allowing to store more code in SCC. 
> 
> Additional testing:
>  - [x] Linux x86_64 server fastdebug, `runtime/cds`

Aleksey Shipilev has updated the pull request incrementally with two additional commits since the last revision:

 - Actually, static_assert with two arguments should be fine
 - Remove static_assert, cannot be compiled well with ADLC

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

Changes:
  - all: https://git.openjdk.org/leyden/pull/28/files
  - new: https://git.openjdk.org/leyden/pull/28/files/0fe629c1..df9a6ba1

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

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

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


More information about the leyden-dev mailing list