Skip to content

Commit a0df0a5

Browse files
committed
8340731: Cleanup remaining IA64 references in hotspot code
Reviewed-by: dholmes, aph
1 parent 8485cb1 commit a0df0a5

17 files changed

+25
-81
lines changed

src/hotspot/os/bsd/os_bsd.cpp

+1-5
Original file line numberDiff line numberDiff line change
@@ -229,8 +229,6 @@ size_t os::rss() {
229229
// Cpu architecture string
230230
#if defined(ZERO)
231231
static char cpu_arch[] = ZERO_LIBARCH;
232-
#elif defined(IA64)
233-
static char cpu_arch[] = "ia64";
234232
#elif defined(IA32)
235233
static char cpu_arch[] = "i386";
236234
#elif defined(AMD64)
@@ -1192,8 +1190,6 @@ void * os::dll_load(const char *filename, char *ebuf, int ebuflen) {
11921190
static Elf32_Half running_arch_code=EM_386;
11931191
#elif (defined AMD64)
11941192
static Elf32_Half running_arch_code=EM_X86_64;
1195-
#elif (defined IA64)
1196-
static Elf32_Half running_arch_code=EM_IA_64;
11971193
#elif (defined __powerpc64__)
11981194
static Elf32_Half running_arch_code=EM_PPC64;
11991195
#elif (defined __powerpc__)
@@ -1214,7 +1210,7 @@ void * os::dll_load(const char *filename, char *ebuf, int ebuflen) {
12141210
static Elf32_Half running_arch_code=EM_68K;
12151211
#else
12161212
#error Method os::dll_load requires that one of following is defined:\
1217-
IA32, AMD64, IA64, __powerpc__, ARM, S390, ALPHA, MIPS, MIPSEL, PARISC, M68K
1213+
IA32, AMD64, __powerpc__, ARM, S390, ALPHA, MIPS, MIPSEL, PARISC, M68K
12181214
#endif
12191215

12201216
// Identify compatibility class for VM's architecture and library's architecture

src/hotspot/os/linux/hugepages.cpp

+1-2
Original file line numberDiff line numberDiff line change
@@ -55,8 +55,7 @@ static size_t scan_default_hugepagesize() {
5555

5656
// large_page_size on Linux is used to round up heap size. x86 uses either
5757
// 2M or 4M page, depending on whether PAE (Physical Address Extensions)
58-
// mode is enabled. AMD64/EM64T uses 2M page in 64bit mode. IA64 can use
59-
// page as large as 1G.
58+
// mode is enabled. AMD64/EM64T uses 2M page in 64bit mode.
6059
//
6160
// Here we try to figure out page size by parsing /proc/meminfo and looking
6261
// for a line with the following format:

src/hotspot/os/linux/os_linux.cpp

+10-23
Original file line numberDiff line numberDiff line change
@@ -461,26 +461,17 @@ bool os::Linux::get_tick_information(CPUPerfTicks* pticks, int which_logical_cpu
461461
}
462462

463463
#ifndef SYS_gettid
464-
// i386: 224, ia64: 1105, amd64: 186, sparc: 143
465-
#ifdef __ia64__
466-
#define SYS_gettid 1105
464+
// i386: 224, amd64: 186, sparc: 143
465+
#if defined(__i386__)
466+
#define SYS_gettid 224
467+
#elif defined(__amd64__)
468+
#define SYS_gettid 186
469+
#elif defined(__sparc__)
470+
#define SYS_gettid 143
467471
#else
468-
#ifdef __i386__
469-
#define SYS_gettid 224
470-
#else
471-
#ifdef __amd64__
472-
#define SYS_gettid 186
473-
#else
474-
#ifdef __sparc__
475-
#define SYS_gettid 143
476-
#else
477-
#error define gettid for the arch
478-
#endif
479-
#endif
480-
#endif
472+
#error "Define SYS_gettid for this architecture"
481473
#endif
482-
#endif
483-
474+
#endif // SYS_gettid
484475

485476
// pid_t gettid()
486477
//
@@ -1778,8 +1769,6 @@ void * os::dll_load(const char *filename, char *ebuf, int ebuflen) {
17781769
static Elf32_Half running_arch_code=EM_386;
17791770
#elif (defined AMD64) || (defined X32)
17801771
static Elf32_Half running_arch_code=EM_X86_64;
1781-
#elif (defined IA64)
1782-
static Elf32_Half running_arch_code=EM_IA_64;
17831772
#elif (defined __sparc) && (defined _LP64)
17841773
static Elf32_Half running_arch_code=EM_SPARCV9;
17851774
#elif (defined __sparc) && (!defined _LP64)
@@ -1812,7 +1801,7 @@ void * os::dll_load(const char *filename, char *ebuf, int ebuflen) {
18121801
static Elf32_Half running_arch_code=EM_LOONGARCH;
18131802
#else
18141803
#error Method os::dll_load requires that one of following is defined:\
1815-
AARCH64, ALPHA, ARM, AMD64, IA32, IA64, LOONGARCH64, M68K, MIPS, MIPSEL, PARISC, __powerpc__, __powerpc64__, RISCV, S390, SH, __sparc
1804+
AARCH64, ALPHA, ARM, AMD64, IA32, LOONGARCH64, M68K, MIPS, MIPSEL, PARISC, __powerpc__, __powerpc64__, RISCV, S390, SH, __sparc
18161805
#endif
18171806

18181807
// Identify compatibility class for VM's architecture and library's architecture
@@ -2719,8 +2708,6 @@ void os::get_summary_cpu_info(char* cpuinfo, size_t length) {
27192708
strncpy(cpuinfo, "ARM", length);
27202709
#elif defined(IA32)
27212710
strncpy(cpuinfo, "x86_32", length);
2722-
#elif defined(IA64)
2723-
strncpy(cpuinfo, "IA64", length);
27242711
#elif defined(PPC)
27252712
strncpy(cpuinfo, "PPC64", length);
27262713
#elif defined(RISCV)

src/hotspot/os/posix/signals_posix.cpp

-7
Original file line numberDiff line numberDiff line change
@@ -960,10 +960,6 @@ static bool get_signal_code_description(const siginfo_t* si, enum_sigcode_desc_t
960960
{ SIGILL, ILL_PRVREG, "ILL_PRVREG", "Privileged register." },
961961
{ SIGILL, ILL_COPROC, "ILL_COPROC", "Coprocessor error." },
962962
{ SIGILL, ILL_BADSTK, "ILL_BADSTK", "Internal stack error." },
963-
#if defined(IA64) && defined(LINUX)
964-
{ SIGILL, ILL_BADIADDR, "ILL_BADIADDR", "Unimplemented instruction address" },
965-
{ SIGILL, ILL_BREAK, "ILL_BREAK", "Application Break instruction" },
966-
#endif
967963
{ SIGFPE, FPE_INTDIV, "FPE_INTDIV", "Integer divide by zero." },
968964
{ SIGFPE, FPE_INTOVF, "FPE_INTOVF", "Integer overflow." },
969965
{ SIGFPE, FPE_FLTDIV, "FPE_FLTDIV", "Floating-point divide by zero." },
@@ -977,9 +973,6 @@ static bool get_signal_code_description(const siginfo_t* si, enum_sigcode_desc_t
977973
#if defined(AIX)
978974
// no explanation found what keyerr would be
979975
{ SIGSEGV, SEGV_KEYERR, "SEGV_KEYERR", "key error" },
980-
#endif
981-
#if defined(IA64) && !defined(AIX)
982-
{ SIGSEGV, SEGV_PSTKOVF, "SEGV_PSTKOVF", "Paragraph stack overflow" },
983976
#endif
984977
{ SIGBUS, BUS_ADRALN, "BUS_ADRALN", "Invalid address alignment." },
985978
{ SIGBUS, BUS_ADRERR, "BUS_ADRERR", "Nonexistent physical address." },

src/hotspot/share/opto/chaitin.cpp

-1
Original file line numberDiff line numberDiff line change
@@ -956,7 +956,6 @@ void PhaseChaitin::gather_lrg_masks( bool after_aggressive ) {
956956
// Each entry is reg_pressure_per_value,number_of_regs
957957
// RegL RegI RegFlags RegF RegD INTPRESSURE FLOATPRESSURE
958958
// IA32 2 1 1 1 1 6 6
959-
// IA64 1 1 1 1 1 50 41
960959
// SPARC 2 2 2 2 2 48 (24) 52 (26)
961960
// SPARCV9 2 2 2 2 2 48 (24) 52 (26)
962961
// AMD64 1 1 1 1 1 14 15

src/hotspot/share/opto/generateOptoStub.cpp

-4
Original file line numberDiff line numberDiff line change
@@ -224,10 +224,6 @@ void GraphKit::gen_stub(address C_function,
224224
store_to_memory(control(), adr_sp, null(), T_ADDRESS, MemNode::unordered);
225225
// Clear last_Java_pc
226226
store_to_memory(control(), adr_last_Java_pc, null(), T_ADDRESS, MemNode::unordered);
227-
#if (defined(IA64) && !defined(AIX))
228-
Node* adr_last_Java_fp = basic_plus_adr(top(), thread, in_bytes(JavaThread::last_Java_fp_offset()));
229-
store_to_memory(control(), adr_last_Java_fp, null(), T_ADDRESS, MemNode::unordered);
230-
#endif
231227

232228
// For is-fancy-jump, the C-return value is also the branch target
233229
Node* target = map()->in(TypeFunc::Parms);

src/hotspot/share/opto/memnode.hpp

+2-2
Original file line numberDiff line numberDiff line change
@@ -196,7 +196,7 @@ class LoadNode : public MemNode {
196196
// non-pinned LoadNode by the pinned LoadNode.
197197
ControlDependency _control_dependency;
198198

199-
// On platforms with weak memory ordering (e.g., PPC, Ia64) we distinguish
199+
// On platforms with weak memory ordering (e.g., PPC) we distinguish
200200
// loads that can be reordered, and such requiring acquire semantics to
201201
// adhere to the Java specification. The required behaviour is stored in
202202
// this field.
@@ -566,7 +566,7 @@ class LoadNKlassNode : public LoadNNode {
566566
// Store value; requires Store, Address and Value
567567
class StoreNode : public MemNode {
568568
private:
569-
// On platforms with weak memory ordering (e.g., PPC, Ia64) we distinguish
569+
// On platforms with weak memory ordering (e.g., PPC) we distinguish
570570
// stores that can be reordered, and such requiring release semantics to
571571
// adhere to the Java specification. The required behaviour is stored in
572572
// this field.

src/hotspot/share/opto/output.cpp

+1-1
Original file line numberDiff line numberDiff line change
@@ -383,7 +383,7 @@ bool PhaseOutput::need_stack_bang(int frame_size_in_bytes) const {
383383
bool PhaseOutput::need_register_stack_bang() const {
384384
// Determine if we need to generate a register stack overflow check.
385385
// This is only used on architectures which have split register
386-
// and memory stacks (ie. IA64).
386+
// and memory stacks.
387387
// Bang if the method is not a stub function and has java calls
388388
return (C->stub_function() == nullptr && C->has_java_calls());
389389
}

src/hotspot/share/runtime/abstract_vm_version.cpp

-1
Original file line numberDiff line numberDiff line change
@@ -187,7 +187,6 @@ const char* Abstract_VM_Version::vm_release() {
187187
#define CPU AARCH64_ONLY("aarch64") \
188188
AMD64_ONLY("amd64") \
189189
IA32_ONLY("x86") \
190-
IA64_ONLY("ia64") \
191190
S390_ONLY("s390") \
192191
RISCV64_ONLY("riscv64")
193192
#endif // !ZERO

src/hotspot/share/runtime/deoptimization.cpp

+1-5
Original file line numberDiff line numberDiff line change
@@ -607,9 +607,6 @@ Deoptimization::UnrollBlock* Deoptimization::fetch_unroll_info_helper(JavaThread
607607
// where it will be very difficult to figure out what went wrong. Better
608608
// to die an early death here than some very obscure death later when the
609609
// trail is cold.
610-
// Note: on ia64 this guarantee can be fooled by frames with no memory stack
611-
// in that it will fail to detect a problem when there is one. This needs
612-
// more work in tiger timeframe.
613610
guarantee(array->unextended_sp() == unpack_sp, "vframe_array_head must contain the vframeArray to unpack");
614611

615612
int number_of_frames = array->frames();
@@ -2342,8 +2339,7 @@ JRT_ENTRY(void, Deoptimization::uncommon_trap_inner(JavaThread* current, jint tr
23422339
}
23432340

23442341
// Setting +ProfileTraps fixes the following, on all platforms:
2345-
// 4852688: ProfileInterpreter is off by default for ia64. The result is
2346-
// infinite heroic-opt-uncommon-trap/deopt/recompile cycles, since the
2342+
// The result is infinite heroic-opt-uncommon-trap/deopt/recompile cycles, since the
23472343
// recompile relies on a MethodData* to record heroic opt failures.
23482344

23492345
// Whether the interpreter is producing MDO data or not, we also need

src/hotspot/share/runtime/frame.hpp

+3-4
Original file line numberDiff line numberDiff line change
@@ -164,10 +164,9 @@ class frame {
164164
void patch_pc(Thread* thread, address pc);
165165

166166
// Every frame needs to return a unique id which distinguishes it from all other frames.
167-
// For sparc and ia32 use sp. ia64 can have memory frames that are empty so multiple frames
168-
// will have identical sp values. For ia64 the bsp (fp) value will serve. No real frame
169-
// should have an id() of null so it is a distinguishing value for an unmatchable frame.
170-
// We also have relationals which allow comparing a frame to anoth frame's id() allow
167+
// For sparc and ia32 use sp.
168+
// No real frame should have an id() of null so it is a distinguishing value for an unmatchable frame.
169+
// We also have relationals which allow comparing a frame to another frame's id() allowing
171170
// us to distinguish younger (more recent activation) from older (less recent activations)
172171
// A null id is only valid when comparing for equality.
173172

src/hotspot/share/runtime/javaCalls.cpp

-5
Original file line numberDiff line numberDiff line change
@@ -116,11 +116,6 @@ JavaCallWrapper::~JavaCallWrapper() {
116116
ThreadStateTransition::transition_from_java(_thread, _thread_in_vm);
117117

118118
// State has been restored now make the anchor frame visible for the profiler.
119-
// Do this after the transition because this allows us to put an assert
120-
// the Java->vm transition which checks to see that stack is not walkable
121-
// on sparc/ia64 which will catch violations of the resetting of last_Java_frame
122-
// invariants (i.e. _flags always cleared on return to Java)
123-
124119
_thread->frame_anchor()->copy(&_anchor);
125120

126121
// Release handles after we are marked as being inside the VM again, since this

src/hotspot/share/runtime/os.hpp

+1-2
Original file line numberDiff line numberDiff line change
@@ -859,8 +859,7 @@ class os: AllStatic {
859859
// We don't attempt to become a debugger, so we only follow frames if that
860860
// does not require a lookup in the unwind table, which is part of the binary
861861
// file but may be unsafe to read after a fatal error. So on x86, we can
862-
// only walk stack if %ebp is used as frame pointer; on ia64, it's not
863-
// possible to walk C stack without having the unwind table.
862+
// only walk stack if %ebp is used as frame pointer.
864863
static bool is_first_C_frame(frame *fr);
865864
static frame get_sender_for_C_frame(frame *fr);
866865

src/hotspot/share/runtime/relocator.cpp

+1-1
Original file line numberDiff line numberDiff line change
@@ -200,7 +200,7 @@ bool Relocator::handle_code_changes() {
200200
bool Relocator::is_opcode_lookupswitch(Bytecodes::Code bc) {
201201
switch (bc) {
202202
case Bytecodes::_tableswitch: return false;
203-
case Bytecodes::_lookupswitch: // not rewritten on ia64
203+
case Bytecodes::_lookupswitch:
204204
case Bytecodes::_fast_linearswitch: // rewritten _lookupswitch
205205
case Bytecodes::_fast_binaryswitch: return true; // rewritten _lookupswitch
206206
default: ShouldNotReachHere();

src/hotspot/share/utilities/elfFuncDescTable.cpp

+2-3
Original file line numberDiff line numberDiff line change
@@ -34,11 +34,10 @@ ElfFuncDescTable::ElfFuncDescTable(FILE* file, Elf_Shdr shdr, int index) :
3434
_section(file, shdr), _file(file), _index(index) {
3535
assert(file, "null file handle");
3636
// The actual function address (i.e. function entry point) is always the
37-
// first value in the function descriptor (on IA64 and PPC64 they look as follows):
37+
// first value in the function descriptor (on PPC64 they look as follows):
3838
// PPC64: [function entry point, TOC pointer, environment pointer]
39-
// IA64 : [function entry point, GP (global pointer) value]
4039
// Unfortunately 'shdr.sh_entsize' doesn't always seem to contain this size (it's zero on PPC64) so we can't assert
41-
// assert(IA64_ONLY(2) PPC64_ONLY(3) * sizeof(address) == shdr.sh_entsize, "Size mismatch for '.opd' section entries");
40+
// assert(PPC64_ONLY(3) * sizeof(address) == shdr.sh_entsize, "Size mismatch for '.opd' section entries");
4241

4342
_status = _section.status();
4443
}

src/hotspot/share/utilities/elfFuncDescTable.hpp

+2-3
Original file line numberDiff line numberDiff line change
@@ -35,9 +35,8 @@
3535

3636
/*
3737

38-
On PowerPC-64 (and other architectures like for example IA64) a pointer to a
39-
function is not just a plain code address, but instead a pointer to a so called
40-
function descriptor (which is simply a structure containing 3 pointers).
38+
On PowerPC-64 a pointer to a function is not just a plain code address, but instead a pointer
39+
to a so-called function descriptor (which is simply a structure containing 3 pointers).
4140
This fact is also reflected in the ELF ABI for PowerPC-64.
4241

4342
On architectures like x86 or SPARC, the ELF symbol table contains the start

src/hotspot/share/utilities/macros.hpp

-12
Original file line numberDiff line numberDiff line change
@@ -458,18 +458,6 @@
458458
#define NOT_IA32(code) code
459459
#endif
460460

461-
// This is a REALLY BIG HACK, but on AIX <sys/systemcfg.h> unconditionally defines IA64.
462-
// At least on AIX 7.1 this is a real problem because 'systemcfg.h' is indirectly included
463-
// by 'pthread.h' and other common system headers.
464-
465-
#if defined(IA64) && !defined(AIX)
466-
#define IA64_ONLY(code) code
467-
#define NOT_IA64(code)
468-
#else
469-
#define IA64_ONLY(code)
470-
#define NOT_IA64(code) code
471-
#endif
472-
473461
#ifdef AMD64
474462
#define AMD64_ONLY(code) code
475463
#define NOT_AMD64(code)

0 commit comments

Comments
 (0)