-
Bug
-
Resolution: Fixed
-
P4
-
None
-
b08
Issue | Fix Version | Assignee | Priority | Status | Resolution | Resolved In Build |
---|---|---|---|---|---|---|
JDK-8293426 | 15.0.9 | Yuri Nesterenko | P4 | Resolved | Fixed | b04 |
JDK-8293582 | 13.0.13 | Yuri Nesterenko | P4 | Resolved | Fixed | b04 |
JDK-8304851 | 11.0.20-oracle | Fairoz Matte | P4 | Resolved | Fixed | b01 |
JDK-8267863 | 11.0.12 | Yasumasa Suenaga | P4 | Resolved | Fixed | b05 |
I saw some warnings in JFR code when I tried to build OpenJDK with GCC 10.1.1 on Fedora 32:
```
/home/ysuenaga/OpenJDK/jdk/src/hotspot/share/jfr/utilities/jfrHashtable.hpp:86:14: warning: comparison of unsigned expression in '>= 0' is always true [-Wtype-limits]
86 | assert(h >= 0 && h < _table_size, "Illegal hash value");
| ~~^~~~
```
```
/home/ysuenaga/OpenJDK/jdk/src/hotspot/share/jfr/utilities/jfrDoublyLinkedList.hpp:138:17: warning: comparison of unsigned expression in '>= 0' is always true [-Wtype-limits]
138 | assert(_count >= 0, "invariant");
```
Both `h` and `_cound` are unsigned type, so we can remove them.
```
/home/ysuenaga/OpenJDK/jdk/src/hotspot/share/jfr/utilities/jfrHashtable.hpp:86:14: warning: comparison of unsigned expression in '>= 0' is always true [-Wtype-limits]
86 | assert(h >= 0 && h < _table_size, "Illegal hash value");
| ~~^~~~
```
```
/home/ysuenaga/OpenJDK/jdk/src/hotspot/share/jfr/utilities/jfrDoublyLinkedList.hpp:138:17: warning: comparison of unsigned expression in '>= 0' is always true [-Wtype-limits]
138 | assert(_count >= 0, "invariant");
```
Both `h` and `_cound` are unsigned type, so we can remove them.
- backported by
-
JDK-8267863 GCC 10 warnings -Wtype-limits with JFR code
- Resolved
-
JDK-8293426 GCC 10 warnings -Wtype-limits with JFR code
- Resolved
-
JDK-8293582 GCC 10 warnings -Wtype-limits with JFR code
- Resolved
-
JDK-8304851 GCC 10 warnings -Wtype-limits with JFR code
- Resolved
- links to
-
Commit openjdk/jdk13u-dev/29b5ddce
-
Commit openjdk/jdk15u-dev/bcc79e20
-
Review openjdk/jdk13u-dev/399
-
Review openjdk/jdk15u-dev/271
(3 links to)