-
Bug
-
Resolution: Not an Issue
-
P4
-
14
The changes in JDK-8233702 added a clamp() function to clamp a value between given min and max, with the assert checking whether min <= max.
Attempting to use this method in
methodData.cpp:
int MethodData::compute_extra_data_count(int data_size, int empty_bc_count, bool needs_speculative_traps) {
933: int extra_data_count = MIN2(empty_bc_count, MAX2(4, (empty_bc_count * 30) / 100));
caused the assert to fail (assuming extra_data_count should be bounded by [4...empy_bc_count]) which indicates a potential issue with the calculation, i.e. unexpected return values.
Please look at the method/calculation and if the clamp() method should be used, and if so use it after fixing the code.
Attempting to use this method in
methodData.cpp:
int MethodData::compute_extra_data_count(int data_size, int empty_bc_count, bool needs_speculative_traps) {
933: int extra_data_count = MIN2(empty_bc_count, MAX2(4, (empty_bc_count * 30) / 100));
caused the assert to fail (assuming extra_data_count should be bounded by [4...empy_bc_count]) which indicates a potential issue with the calculation, i.e. unexpected return values.
Please look at the method/calculation and if the clamp() method should be used, and if so use it after fixing the code.
- relates to
-
JDK-8235192 [JVMCI] remove JVMCI version of MethodData::compute_extra_data_count
-
- Open
-
-
JDK-8233702 Introduce helper function to clamp value to range
-
- Resolved
-