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

Coleen Phillimore coleenp at openjdk.org
Wed May 17 20:19:57 UTC 2023


On Wed, 17 May 2023 20:08:58 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

>> 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/LookupNameAndTypeRefIndexInPoolTest.java line 115:
> 
>> 113:             break;
>> 114:           default:
>> 115:             throw new Error("Unexpected consant pool entry");
> 
> Typo: consant

I wonder if this should be refactored into a little function toOpcodeType() or something like that.  It probably should be for these tests.

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

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


More information about the hotspot-dev mailing list