RFR: Some more descriptive names, removed redundant asserts. Request for comments. [v5]

Coleen Phillimore coleenp at openjdk.java.net
Fri Apr 8 00:58:47 UTC 2022


> Also, there's an if statement already for chunk_available which the compiler shouldn't have trouble optimizing.  Otherwise we should refactor freeze_fast.
> Tested locally with make test TEST=jdk/jdk/internal/vm/Continuation since only names.

Coleen Phillimore has updated the pull request incrementally with one additional commit since the last revision:

  Minor work to move patch_unextended_pc to platform dependent and get other platforms to compile (except x86)

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

Changes:
  - all: https://git.openjdk.java.net/loom/pull/139/files
  - new: https://git.openjdk.java.net/loom/pull/139/files/55bde526..5dfc7f7a

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=loom&pr=139&range=04
 - incr: https://webrevs.openjdk.java.net/?repo=loom&pr=139&range=03-04

  Stats: 148 lines in 24 files changed: 94 ins; 46 del; 8 mod
  Patch: https://git.openjdk.java.net/loom/pull/139.diff
  Fetch: git fetch https://git.openjdk.java.net/loom pull/139/head:pull/139

PR: https://git.openjdk.java.net/loom/pull/139


More information about the loom-dev mailing list