Skip to content

Commit 0f426e4

Browse files
eme64chhagedorn
andcommitted
8343685: C2 SuperWord: refactor VPointer with MemPointer
Co-authored-by: Christian Hagedorn <[email protected]> Reviewed-by: kvn, chagedorn
1 parent 6b91985 commit 0f426e4

18 files changed

+2630
-2240
lines changed

src/hotspot/share/classfile/vmSymbols.hpp

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
/*
2-
* Copyright (c) 1997, 2024, Oracle and/or its affiliates. All rights reserved.
2+
* Copyright (c) 1997, 2025, Oracle and/or its affiliates. All rights reserved.
33
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
44
*
55
* This code is free software; you can redistribute it and/or modify it
@@ -356,6 +356,8 @@ class SerializeClosure;
356356
template(jdk_internal_foreign_abi_VMStorage_array_array_signature, "[[Ljdk/internal/foreign/abi/VMStorage;") \
357357
template(jdk_internal_foreign_abi_CallConv, "jdk/internal/foreign/abi/UpcallLinker$CallRegs") \
358358
\
359+
template(jdk_internal_foreign_NativeMemorySegmentImpl, "jdk/internal/foreign/NativeMemorySegmentImpl") \
360+
\
359361
/* Support for JVMCI */ \
360362
JVMCI_VM_SYMBOLS_DO(template, do_alias) \
361363
\

src/hotspot/share/opto/memnode.cpp

Lines changed: 12 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -2859,16 +2859,16 @@ class MergePrimitiveStores : public StackObj {
28592859
return is_trace(TraceMergeStores::Tag::BASIC);
28602860
}
28612861

2862-
bool is_trace_pointer() const {
2863-
return is_trace(TraceMergeStores::Tag::POINTER);
2862+
bool is_trace_pointer_parsing() const {
2863+
return is_trace(TraceMergeStores::Tag::POINTER_PARSING);
28642864
}
28652865

2866-
bool is_trace_aliasing() const {
2867-
return is_trace(TraceMergeStores::Tag::ALIASING);
2866+
bool is_trace_pointer_aliasing() const {
2867+
return is_trace(TraceMergeStores::Tag::POINTER_ALIASING);
28682868
}
28692869

2870-
bool is_trace_adjacency() const {
2871-
return is_trace(TraceMergeStores::Tag::ADJACENCY);
2870+
bool is_trace_pointer_adjacency() const {
2871+
return is_trace(TraceMergeStores::Tag::POINTER_ADJACENCY);
28722872
}
28732873

28742874
bool is_trace_success() const {
@@ -2939,12 +2939,13 @@ bool MergePrimitiveStores::is_adjacent_pair(const StoreNode* use_store, const St
29392939

29402940
ResourceMark rm;
29412941
#ifndef PRODUCT
2942-
const TraceMemPointer trace(is_trace_pointer(),
2943-
is_trace_aliasing(),
2944-
is_trace_adjacency());
2942+
const TraceMemPointer trace(is_trace_pointer_parsing(),
2943+
is_trace_pointer_aliasing(),
2944+
is_trace_pointer_adjacency(),
2945+
true);
29452946
#endif
2946-
const MemPointer pointer_use(use_store NOT_PRODUCT( COMMA trace ));
2947-
const MemPointer pointer_def(def_store NOT_PRODUCT( COMMA trace ));
2947+
const MemPointer pointer_use(use_store NOT_PRODUCT(COMMA trace));
2948+
const MemPointer pointer_def(def_store NOT_PRODUCT(COMMA trace));
29482949
return pointer_def.is_adjacent_to_and_before(pointer_use);
29492950
}
29502951

0 commit comments

Comments
 (0)