jimingham wrote: Reverting those two commits seems to have caused this build failure on Ubuntu:
Step 4 (build) warnings: build (warnings) ../llvm-project/clang/lib/Lex/PPDirectives.cpp:548:28: warning: comparison of integers of different signs: 'unsigned int' and 'int' [-Wsign-compare] 1 warning generated. ../llvm-project/clang/lib/Frontend/TextDiagnostic.cpp:148:36: warning: comparison of integers of different signs: 'int' and 'unsigned int' [-Wsign-compare] 1 warning generated. ../llvm-project/llvm/lib/Target/AMDGPU/AMDGPUIGroupLP.cpp:1079:20: warning: comparison of integers of different signs: 'typename iterator_traits<const SDep *>::difference_type' (aka 'int') and 'unsigned int' [-Wsign-compare] ../llvm-project/llvm/lib/Target/AMDGPU/AMDGPUIGroupLP.cpp:1089:24: warning: comparison of integers of different signs: 'typename iterator_traits<SDep *>::difference_type' (aka 'int') and 'unsigned int' [-Wsign-compare] ../llvm-project/llvm/lib/Target/AMDGPU/AMDGPUIGroupLP.cpp:1120:20: warning: comparison of integers of different signs: 'typename iterator_traits<const SDep *>::difference_type' (aka 'int') and 'unsigned int' [-Wsign-compare] ../llvm-project/llvm/lib/Target/AMDGPU/AMDGPUIGroupLP.cpp:1130:24: warning: comparison of integers of different signs: 'typename iterator_traits<SDep *>::difference_type' (aka 'int') and 'unsigned int' [-Wsign-compare] ../llvm-project/lldb/source/Plugins/Language/CPlusPlus/LibCxx.cpp:1107:7: warning: implicit conversion from 'long long' to 'const std::time_t' (aka 'const long') changes value from -1096193779200 to -977118720 [-Wconstant-conversion] ../llvm-project/lldb/source/Plugins/Language/CPlusPlus/LibCxx.cpp:1109:7: warning: implicit conversion from 'long long' to 'const std::time_t' (aka 'const long') changes value from 971890963199 to 1228354303 [-Wconstant-conversion] ../llvm-project/lld/MachO/SyntheticSections.cpp:2063:25: warning: comparison of integers of different signs: 'int' and 'const uint32_t' (aka 'const unsigned int') [-Wsign-compare] 1 warning generated. Step 6 (test) failure: build (failure) clang: warning: argument unused during compilation: '-fmodules-cache-path=/home/tcwg-buildbot/worker/lldb-arm-ubuntu/build/lldb-test-build.noindex/module-cache-clang/lldb-shell' [-Wunused-command-line-argument] Would you mind taking a look, this is not my area of llvm... https://lab.llvm.org/buildbot/#/builders/17/builds/53025 Thanks! Jim > On May 13, 2024, at 6:22 PM, Zequan Wu ***@***.***> wrote: > > > your commit deleted that file I think, I added it back when I did the revert > (possibly a mistake)... It passes on my macOS system but is failing on Ubuntu > after the revert. I think I'll just disable it for now. > > This change adds the new test, so deleting it as part of the reverting is > expected. > > — > Reply to this email directly, view it on GitHub > <https://github.com/llvm/llvm-project/pull/90663#issuecomment-2109095788>, or > unsubscribe > <https://github.com/notifications/unsubscribe-auth/ADUPVW76LSTVQBSKCNZZU2TZCFROLAVCNFSM6AAAAABHA7HMYWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMBZGA4TKNZYHA>. > You are receiving this because you are on a team that was mentioned. > https://github.com/llvm/llvm-project/pull/90663 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
