RFR: 8256245: AArch64: Implement Base64 decoding intrinsic [v5]
Dong Bo
dongbo at openjdk.java.net
Wed Apr 7 06:00:45 UTC 2021
On Tue, 6 Apr 2021 14:04:07 GMT, Andrew Haley <aph at openjdk.org> wrote:
>> Dong Bo has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 10 commits:
>>
>> - conflicts resolved
>> - Merge branch 'master' of https://git.openjdk.java.net/jdk into aarch64.base64.decode
>> - resovling conflicts
>> - load data with one ldrw, add JMH tests for error inputs
>> - Merge branch 'master' into aarch64.base64.decode
>> - copyright
>> - trivial fixes
>> - Handling error in SIMD case with loops, combining two non-SIMD cases into one code blob, addressing other comments
>> - Merge branch 'master' into aarch64.base64.decode
>> - 8256245: AArch64: Implement Base64 decoding intrinsic
>
> src/hotspot/cpu/aarch64/stubGenerator_aarch64.cpp line 5800:
>
>> 5798: __ br(Assembler::LT, Process4B);
>> 5799:
>> 5800: // The 1st character of the input can be illegal if the data is MIME encoded.
>
> Why is this sentence here? It is very misleading.
This sentence was used to describe the worst case observed frequently so that readers can understand more easily why the pre-processing non-SIMD code is necessary.
I apologize for being unclear and misleading. The annotations have been modified as suggested.
-------------
PR: https://git.openjdk.java.net/jdk/pull/3228
More information about the core-libs-dev
mailing list