Integrated: 8256387: Unexpected result if patching an entire instruction on AArch64

[Eric Liu]

On Tue, 17 Nov 2020 06:14:37 GMT, Eric Liu <github.com+10482586+erik1iu at openjdk.org> wrote:

> This patch fixed some potential risks in assembler_aarch64.hpp.
> 
> According to the C standard, shift operation is undefined if the shift
> count greater than or equals to the length in bits of the promoted left
> operand.
> 
> In assembler_aarch64.hpp, there are some utility functions for easily
> operating the encoded instructions. E.g.
> 
>         Instruction_aarch64::patch(address, int, int, uint64_t)
> 
> All those functions use `(1U << nbits) - 1` to calculate mask which may
> have some potential risks if `nbits` equals 32. That would be an
> unexpected result if someone intends to deal with an entire instruction.
> 
> To fix this issue, this patch simply uses `1ULL` to replace `1U`.

This pull request has now been integrated.

Changeset: f1d6e8db
Author:    Eric Liu <eric.c.liu at arm.com>
Committer: Aleksey Shipilev <shade at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/f1d6e8db
Stats:     7 lines in 1 file changed: 0 ins; 0 del; 7 mod

8256387: Unexpected result if patching an entire instruction on AArch64

Reviewed-by: shade, aph

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

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