Skip to content

8313882: Fix -Wconversion warnings in runtime code #15177

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 22 commits into from
Closed

8313882: Fix -Wconversion warnings in runtime code #15177

Changes from 1 commit
Commits
Show all changes
22 commits
Select commit Hold shift + click to select a range
bfeaafc
Fix relocator.cpp
coleenp Aug 1, 2023
3d668be
Fix objectMonitor.hpp - monitor_value is 2
coleenp Aug 1, 2023
b413244
Fix java.cpp
coleenp Aug 2, 2023
4186fee
Fixed sharedRuntime.cpp
coleenp Aug 2, 2023
20545c8
Fix arguments.cpp
coleenp Aug 2, 2023
4421daf
Fixed synchronizer.cpp by fixing _held_monitor_count and _jni_monitor…
coleenp Aug 3, 2023
c6ecce8
Fix deoptimization.cpp (size_of_frames is used to assign to _total_fr…
coleenp Aug 3, 2023
18e3f67
Fix continuationFreezeThaw.cpp
coleenp Aug 3, 2023
070e905
Fix signature.cpp
coleenp Aug 3, 2023
d086870
Fix asserted files in runtime with one warning.
coleenp Aug 3, 2023
48a306f
Fix gtests
coleenp Aug 3, 2023
12295ba
Fix format strings for monitor_counts and fix arguments.cpp differently.
coleenp Aug 4, 2023
4eeeb16
Fix jvmci for held_monitor_count type.
coleenp Aug 4, 2023
a77884e
Make bytecode.hpp index() functions return u2 not long.
coleenp Aug 8, 2023
1af1cfe
Dean's and Patricio's suggested changes.
coleenp Aug 9, 2023
512daa1
Change Arguments::parse_uintx to parse_uint and callers.
coleenp Aug 9, 2023
f496165
Merge branch 'master' into runtime-conversion
coleenp Aug 9, 2023
4366563
Revert histogram sum and weighted_sum changes.
coleenp Aug 9, 2023
51e7190
put checked_cast around getc().
coleenp Aug 9, 2023
17eaec5
put checked_cast around getc().
coleenp Aug 9, 2023
e254b2d
Put arguments back to the tested version.
coleenp Aug 9, 2023
0e21a92
Add a checked_cast.
coleenp Aug 10, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 5 additions & 5 deletions src/hotspot/share/interpreter/bytecode.hpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -274,7 +274,7 @@ class Bytecode_checkcast: public Bytecode {
void verify() const { assert(Bytecodes::java_code(code()) == Bytecodes::_checkcast, "check checkcast"); }

// Returns index
int index() const { return get_index_u2(Bytecodes::_checkcast); };
u2 index() const { return get_index_u2(Bytecodes::_checkcast); };
};

// Abstraction for instanceof
Expand All @@ -284,7 +284,7 @@ class Bytecode_instanceof: public Bytecode {
void verify() const { assert(code() == Bytecodes::_instanceof, "check instanceof"); }

// Returns index
long index() const { return get_index_u2(Bytecodes::_instanceof); };
u2 index() const { return get_index_u2(Bytecodes::_instanceof); };
};

class Bytecode_new: public Bytecode {
Expand All @@ -293,7 +293,7 @@ class Bytecode_new: public Bytecode {
void verify() const { assert(java_code() == Bytecodes::_new, "check new"); }

// Returns index
long index() const { return get_index_u2(Bytecodes::_new); };
u2 index() const { return get_index_u2(Bytecodes::_new); };
};

class Bytecode_multianewarray: public Bytecode {
Expand All @@ -302,7 +302,7 @@ class Bytecode_multianewarray: public Bytecode {
void verify() const { assert(java_code() == Bytecodes::_multianewarray, "check new"); }

// Returns index
long index() const { return get_index_u2(Bytecodes::_multianewarray); };
u2 index() const { return get_index_u2(Bytecodes::_multianewarray); };
};

class Bytecode_anewarray: public Bytecode {
Expand All @@ -311,7 +311,7 @@ class Bytecode_anewarray: public Bytecode {
void verify() const { assert(java_code() == Bytecodes::_anewarray, "check anewarray"); }

// Returns index
long index() const { return get_index_u2(Bytecodes::_anewarray); };
u2 index() const { return get_index_u2(Bytecodes::_anewarray); };
};

// Abstraction for ldc, ldc_w and ldc2_w
Expand Down