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