RFR: Small fixes - shipment one
Robbin Ehn
rehn at openjdk.java.net
Tue Mar 22 14:27:15 UTC 2022
Reducing the number of variables, the number of assignment to each variable, white-spaces, renamed some variables, removed unnecessary template, etc...
Except the removal of template 'bottom', the code is the same. (assert is moved up one level)
Passes 1-3 (no new failures)
It seem to be slightly faster than before, but at least on par.
This is not complete at all, but to avoid merge conflict I want to ship, and also make someone happy about removing a template.
-------------
Commit messages:
- Reverted wrong accidential change
- Fixed assert on uninit var
- Removed unused template
- Rename, whitespace, brackets, assignment
- Removed unused variable
- Use chunk sp directly in if statement
- Renamed some variables
- Don't cache _cont.argsize()
- Fixed init_chunk
- Single if bail-out fast path failure
- ... and 6 more: https://git.openjdk.java.net/loom/compare/16e4713c...475d195a
Changes: https://git.openjdk.java.net/loom/pull/114/files
Webrev: https://webrevs.openjdk.java.net/?repo=loom&pr=114&range=00
Stats: 134 lines in 7 files changed: 19 ins; 27 del; 88 mod
Patch: https://git.openjdk.java.net/loom/pull/114.diff
Fetch: git fetch https://git.openjdk.java.net/loom pull/114/head:pull/114
PR: https://git.openjdk.java.net/loom/pull/114
More information about the loom-dev
mailing list