On Mon, 21 Aug 2023 21:16:02 GMT, Chris Plummer <cjplum...@openjdk.org> wrote:
>> VM.buildLongFromIntsPD() is not longer used. Remove it. > > Chris Plummer has updated the pull request incrementally with two additional > commits since the last revision: > > - update copyright > - Remove debugging code Marked as reviewed by kevinw (Committer). StackValueCollection.java looks like it mirrors src/hotspot/share/runtime/stackValueCollection.cpp which has long_at which is used rarely, but at least once in all of hotspot. But not an obvious need for it in SA. Looks like the SA floatAt is also unused. So if we don't see a need to maintain a full mirror of stackValueCollection.cpp then it could be trimmed more. 8-) ------------- PR Review: https://git.openjdk.org/jdk/pull/15371#pullrequestreview-1588860176 PR Comment: https://git.openjdk.org/jdk/pull/15371#issuecomment-1687756168