RFR: 8278302: [s390] Implement fast-path for ASCII-compatible CharsetEncoders

Tyler Steele duke at openjdk.java.net
Mon Jan 17 18:55:26 UTC 2022


On Mon, 17 Jan 2022 11:40:48 GMT, Lutz Schmidt <lucy at openjdk.org> wrote:

>> If you prefer it, the test result files are also accessible here:
>> https://ibm.box.com/s/n1l6e1vpncpsqqvzgbm3czpulmibu8oz
>
> @backwaterred I have updated the PR with a commit fixing the issue found during testing. The only code change in this commit is in line #80, where I removed the "int", turning the declaration/definition into an assignment. All else is just code comments. 
> 
> I would appreciate very much if you could run the tests once again. There is one more thing: could you, or somebody else in your team, please review the change? Thx.

@RealLucy I am happy to re-run the tests and review your changes. I believe you will need a second reviewer as well. Hopefully we can encourage someone from the mailing lists to come check out the PR, and I will also ask around.

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

PR: https://git.openjdk.java.net/jdk/pull/6738


More information about the hotspot-compiler-dev mailing list