WebKit Bugzilla
New
Browse
Search+
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
NEW
258798
[GTK] Build failure: comparison of unsigned expression in ‘>= 0’ is always true
https://bugs.webkit.org/show_bug.cgi?id=258798
Summary
[GTK] Build failure: comparison of unsigned expression in ‘>= 0’ is always true
Kdwk
Reported
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.
Attachments
Add attachment
proposed patch, testcase, etc.
Kdwk
Comment 1
2023-08-22 03:54:34 PDT
This is still happening with aarch64 builds of WebKitGTK.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug