RFR: 8307190: Refactor ref_at methods in Constant Pool [v2]

Andrey Turbanov aturbanov at openjdk.org
Thu May 18 10:51:51 UTC 2023


On Wed, 17 May 2023 17:40:01 GMT, Matias Saavedra Silva <matsaave at openjdk.org> wrote:

>> In anticipation of [JDK-8301996](https://bugs.openjdk.org/browse/JDK-8301996), some of the accessors in constantpool.cpp need to be updated. The CPCache rework introduces multiple new meanings to the index argument passed to these functions, so they need to be restructured in a way that facilitates different paths depending on the input. For this enhancement, the bytecode is propagated by the callers to determine how to handle the index. Thanks to this and JDK-8307306, `bool uncached` is no longer needed in these functions.
>> 
>> Tests have been altered to suit the changes to JVMCI. Verified with tier1-5 tests.
>
> Matias Saavedra Silva has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains six additional commits since the last revision:
> 
>  - Fixed javadoc and test cleanup
>  - Merge branch 'master' into refactor_ref_at_8307190
>  - Fixed comments and copyright
>  - Changed compilerToVM methods
>  - Coleen comments
>  - 8307190: Refactor ref_at methods in Constant Pool

test/hotspot/jtreg/compiler/jvmci/compilerToVM/LookupSignatureInPoolTest.java line 104:

> 102:         Asserts.assertTrue(index != ConstantPoolTestsHelper.NO_CP_CACHE_PRESENT, "the class must have been rewritten");
> 103:         // Select an arbitrary bytecode of the type associated with the Constant pool entry
> 104:         switch(cpType) {

Suggestion:

        switch (cpType) {

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

PR Review Comment: https://git.openjdk.org/jdk/pull/13872#discussion_r1197675937


More information about the hotspot-dev mailing list