> 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 incrementally with one additional 
commit since the last revision:

  Fix merge

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

Changes:
  - all: https://git.openjdk.org/valhalla/pull/2260/files
  - new: https://git.openjdk.org/valhalla/pull/2260/files/c03631c0..09283185

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=valhalla&pr=2260&range=03
 - incr: https://webrevs.openjdk.org/?repo=valhalla&pr=2260&range=02-03

  Stats: 2 lines in 1 file changed: 2 ins; 0 del; 0 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