> We have the strange situation where calling `t.isAlive()` on a 
> `java.lang.Thread` `t`, will call into the VM (via `alive()` then 
> `isAlive0()`) where the VM then examines the `eetop` field of `t` to extract 
> its `JavaThread` pointer and compare it to null. We can simply read `eetop` 
> directly in `Thread.alive()`:
> 
> boolean alive() {
>   return eetop != 0;
> } 
> 
> I also updated a comment in relation to `eetop`.
> 
> Testing: tiers 1-3
> 
> Thanks

David Holmes has updated the pull request incrementally with one additional 
commit since the last revision:

  Fix typo

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/13287/files
  - new: https://git.openjdk.org/jdk/pull/13287/files/a02431a9..10022a06

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=13287&range=04
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=13287&range=03-04

  Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod
  Patch: https://git.openjdk.org/jdk/pull/13287.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/13287/head:pull/13287

PR: https://git.openjdk.org/jdk/pull/13287

Reply via email to