RFR: 8305056: Avoid unaligned access in emit_intX methods if it's unsupported [v7]
Andrew Haley
aph at openjdk.org
Mon Apr 24 11:22:57 UTC 2023
On Fri, 21 Apr 2023 13:50:57 GMT, Vladimir Kempik <vkempik at openjdk.org> wrote:
>> Please review this change which attempts to eliminate unaligned memory stores generated by emit_int16/32/64 methods on some platforms.
>>
>> Primary aim is risc-v platform. But I had to change some code in ppc/arm32/x86 to prevent possible perf degradation.
>
> Vladimir Kempik has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains 10 additional commits since the last revision:
>
> - Merge
> - Rewrite few more helpers
> - Add APH's suggestions and remove some whitespace
> - Rework the fix to use memcpy in codeBuffer.hpp
> - Reduce code duplication
> - Fix 32-bit archs
> - Fix typo
> - change long to ulong in type convertion
> - Fix includes
> - 8305056: Avoid unaligned access in emit_intX methods if not enabled
Looks good to me.
-------------
Marked as reviewed by aph (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/13227#pullrequestreview-1397743486
More information about the hotspot-compiler-dev
mailing list