Bug 258798

Summary: [GTK] Build failure: comparison of unsigned expression in ‘>= 0’ is always true
Product: WebKit Reporter: Kdwk <kdwkleung>
Component: WebKitGTKAssignee: Nobody <webkit-unassigned>
Status: NEW ---    
Severity: Normal CC: bugs-noreply
Priority: P2    
Version: WebKit Nightly Build   
Hardware: PC   
OS: Linux   

Description Kdwk 2023-07-03 03:26:26 PDT
/app/webkit/Source/JavaScriptCore/assembler/testmasm.cpp: In function ‘void {anonymous}::testExtractUnsignedBitfield64()’:
/app/webkit/Source/JavaScriptCore/assembler/testmasm.cpp:1360:21: error: comparison of unsigned expression in ‘>= 0’ is always true [-Werror=type-limits]
 1360 |             if (lsb >= 0 && width > 0 && lsb + width < 64) {
      |                 ~~~~^~~~
/app/webkit/Source/JavaScriptCore/assembler/testmasm.cpp: In function ‘void {anonymous}::testInsertUnsignedBitfieldInZero32()’:
/app/webkit/Source/JavaScriptCore/assembler/testmasm.cpp:1384:21: error: comparison of unsigned expression in ‘>= 0’ is always true [-Werror=type-limits]
 1384 |             if (lsb >= 0 && width > 0 && lsb + width < 32) {
      |                 ~~~~^~~~
/app/webkit/Source/JavaScriptCore/assembler/testmasm.cpp: In function ‘void {anonymous}::testInsertUnsignedBitfieldInZero64()’:
/app/webkit/Source/JavaScriptCore/assembler/testmasm.cpp:1409:21: error: comparison of unsigned expression in ‘>= 0’ is always true [-Werror=type-limits]
 1409 |             if (lsb >= 0 && width > 0 && lsb + width < 64) {
      |                 ~~~~^~~~
/app/webkit/Source/JavaScriptCore/assembler/testmasm.cpp: In function ‘void {anonymous}::testInsertBitField32()’:
/app/webkit/Source/JavaScriptCore/assembler/testmasm.cpp:1435:21: error: comparison of unsigned expression in ‘>= 0’ is always true [-Werror=type-limits]
 1435 |             if (lsb >= 0 && width > 0 && lsb + width < 32) {
      |                 ~~~~^~~~
/app/webkit/Source/JavaScriptCore/assembler/testmasm.cpp: In function ‘void {anonymous}::testInsertBitField64()’:
/app/webkit/Source/JavaScriptCore/assembler/testmasm.cpp:1465:21: error: comparison of unsigned expression in ‘>= 0’ is always true [-Werror=type-limits]
 1465 |             if (lsb >= 0 && width > 0 && lsb + width < 64) {
      |                 ~~~~^~~~
/app/webkit/Source/JavaScriptCore/assembler/testmasm.cpp: In function ‘void {anonymous}::testExtractInsertBitfieldAtLowEnd32()’:
/app/webkit/Source/JavaScriptCore/assembler/testmasm.cpp:1495:21: error: comparison of unsigned expression in ‘>= 0’ is always true [-Werror=type-limits]
 1495 |             if (lsb >= 0 && width > 0 && lsb + width < 32) {
      |                 ~~~~^~~~
/app/webkit/Source/JavaScriptCore/assembler/testmasm.cpp: In function ‘void {anonymous}::testExtractInsertBitfieldAtLowEnd64()’:
/app/webkit/Source/JavaScriptCore/assembler/testmasm.cpp:1525:21: error: comparison of unsigned expression in ‘>= 0’ is always true [-Werror=type-limits]
 1525 |             if (lsb >= 0 && width > 0 && lsb + width < 64) {
      |                 ~~~~^~~~
/app/webkit/Source/JavaScriptCore/assembler/testmasm.cpp: In function ‘void {anonymous}::testClearBitField32()’:
/app/webkit/Source/JavaScriptCore/assembler/testmasm.cpp:1554:21: error: comparison of unsigned expression in ‘>= 0’ is always true [-Werror=type-limits]
 1554 |             if (lsb >= 0 && width > 0 && lsb + width < 32) {
      |                 ~~~~^~~~
/app/webkit/Source/JavaScriptCore/assembler/testmasm.cpp: In function ‘void {anonymous}::testClearBitField64()’:
/app/webkit/Source/JavaScriptCore/assembler/testmasm.cpp:1579:21: error: comparison of unsigned expression in ‘>= 0’ is always true [-Werror=type-limits]
 1579 |             if (lsb >= 0 && width > 0 && lsb + width < 32) {
      |                 ~~~~^~~~
/app/webkit/Source/JavaScriptCore/assembler/testmasm.cpp: In function ‘void {anonymous}::testInsertSignedBitfieldInZero32()’:
/app/webkit/Source/JavaScriptCore/assembler/testmasm.cpp:1676:21: error: comparison of unsigned expression in ‘>= 0’ is always true [-Werror=type-limits]
 1676 |             if (lsb >= 0 && width > 0 && lsb + width < 32) {
      |                 ~~~~^~~~
/app/webkit/Source/JavaScriptCore/assembler/testmasm.cpp: In function ‘void {anonymous}::testInsertSignedBitfieldInZero64()’:
/app/webkit/Source/JavaScriptCore/assembler/testmasm.cpp:1706:21: error: comparison of unsigned expression in ‘>= 0’ is always true [-Werror=type-limits]
 1706 |             if (lsb >= 0 && width > 0 && lsb + width < 64) {
      |                 ~~~~^~~~
/app/webkit/Source/JavaScriptCore/assembler/testmasm.cpp: In function ‘void {anonymous}::testExtractSignedBitfield32()’:
/app/webkit/Source/JavaScriptCore/assembler/testmasm.cpp:1735:21: error: comparison of unsigned expression in ‘>= 0’ is always true [-Werror=type-limits]
 1735 |             if (lsb >= 0 && width > 0 && lsb + width < 32) {
      |                 ~~~~^~~~
/app/webkit/Source/JavaScriptCore/assembler/testmasm.cpp: In function ‘void {anonymous}::testExtractSignedBitfield64()’:
/app/webkit/Source/JavaScriptCore/assembler/testmasm.cpp:1765:21: error: comparison of unsigned expression in ‘>= 0’ is always true [-Werror=type-limits]
 1765 |             if (lsb >= 0 && width > 0 && lsb + width < 64) {
      |                 ~~~~^~~~
/app/webkit/Source/JavaScriptCore/assembler/testmasm.cpp: In function ‘void {anonymous}::testExtractRegister32()’:
/app/webkit/Source/JavaScriptCore/assembler/testmasm.cpp:1796:23: error: comparison of unsigned expression in ‘>= 0’ is always true [-Werror=type-limits]
 1796 |                 if (0 <= lsb && lsb < datasize) {
      |                     ~~^~~~~~
/app/webkit/Source/JavaScriptCore/assembler/testmasm.cpp: In function ‘void {anonymous}::testExtractRegister64()’:
/app/webkit/Source/JavaScriptCore/assembler/testmasm.cpp:1833:23: error: comparison of unsigned expression in ‘>= 0’ is always true [-Werror=type-limits]
 1833 |                 if (0 <= lsb && lsb < datasize) {
      |                     ~~^~~~~~
/app/webkit/Source/JavaScriptCore/assembler/testmasm.cpp: In lambda function:
/app/webkit/Source/JavaScriptCore/assembler/testmasm.cpp:3013:37: error: suggest parentheses around ‘-’ in operand of ‘&’ [-Werror=parentheses]
 3013 |         return value & (1ull << 48) - 1;
      |                        ~~~~~~~~~~~~~^~~
cc1plus: all warnings being treated as errors
[51/4336] Building CXX object Source/WebCor...p/GraphicsContextGLTextureMapperANGLE.cpp.o
ninja: build stopped: subcommand failed.
Comment 1 Kdwk 2023-08-22 03:54:34 PDT
This is still happening with aarch64 builds of WebKitGTK.