Integrated: 8293937: x86: Drop LP64 conditions from clearly x86_32 code
Aleksey Shipilev
shade at openjdk.org
Mon Sep 19 06:20:55 UTC 2022
On Fri, 16 Sep 2022 11:10:29 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:
> Noticed this when porting Loom on x86_32. There are `*_x86_32.cpp` files that use `_LP64` as if it matters for them. It does not make sense, as in those files we always have `!_LP64`. We can drop the conditionals and clean the code.
>
> Proof of completeness:
>
>
> $ ack LP64 src/hotspot/ | grep _32
> src/hotspot/cpu/x86/register_x86.hpp:386: NOT_LP64( 8 + ) // FILL0-FILL7 in x86_32.ad
> src/hotspot/cpu/x86/vm_version_x86.hpp:733: return LP64_ONLY(true) NOT_LP64(false); // not implemented on x86_32
This pull request has now been integrated.
Changeset: 357a2cc2
Author: Aleksey Shipilev <shade at openjdk.org>
URL: https://git.openjdk.org/jdk/commit/357a2cc22a72876fc412b4fc99b9da8f05840678
Stats: 46 lines in 2 files changed: 0 ins; 37 del; 9 mod
8293937: x86: Drop LP64 conditions from clearly x86_32 code
Reviewed-by: kvn, stuefe
-------------
PR: https://git.openjdk.org/jdk/pull/10305
More information about the hotspot-compiler-dev
mailing list