> Two of the Valhalla bits are protected by 64/32 bit checks, but the other two 
> are not. This also makes the top comment incorrect for 32-bit versions. I 
> propose some small tweaks to make it less confusing for reviewers / readers 
> of the code.

Stefan Karlsson has updated the pull request with a new target base due to a 
merge or a rebase. The pull request now contains seven commits:

 - Merge remote-tracking branch 'valhalla/lworld' into 
8380870_lworld_32_bit_mark_word_tweaks
 - Fix markWord gtest usage of is_neutral()
 - 8380870: [lworld] More unification of 32-bit code in markWord.hpp
 - Fix test_preservedMarks.cpp
 - Review fparain 2
 - Review fparain
 - 8380802: [lworld] Remove larval bit from markWord

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

Changes: https://git.openjdk.org/valhalla/pull/2260/files
  Webrev: https://webrevs.openjdk.org/?repo=valhalla&pr=2260&range=02
  Stats: 36 lines in 3 files changed: 13 ins; 14 del; 9 mod
  Patch: https://git.openjdk.org/valhalla/pull/2260.diff
  Fetch: git fetch https://git.openjdk.org/valhalla.git pull/2260/head:pull/2260

PR: https://git.openjdk.org/valhalla/pull/2260

Reply via email to