RFR: JDK-8310510: WordsPerLong is wrong for 64-bit platforms

David Holmes dholmes at openjdk.org
Wed Jun 21 12:44:03 UTC 2023


On Wed, 21 Jun 2023 08:08:44 GMT, Thomas Stuefe <stuefe at openjdk.org> wrote:

> Trivial patch.
> 
> We have WordsPerLong defined as this:
> 
> 
> const int WordsPerLong = 2; // Number of stack entries for longs
> 
> 
> which is wrong for 64-bit platforms. Historically used only for ppc/s390.
> 
> Patch removes the unnecessary align (BasicLock is always 2 words, no need to align to 2) and removes WordsPerLong, since it is not needed anymore.

Not sure how this can be wrong given it means "number of stack slots for a Java long" - which is 2. ??

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

PR Comment: https://git.openjdk.org/jdk/pull/14580#issuecomment-1600763390


More information about the hotspot-dev mailing list