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

Reply via email to