Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Restore lldb build #53951

Merged
merged 2 commits into from
Sep 5, 2018
Merged

Restore lldb build #53951

merged 2 commits into from
Sep 5, 2018

Conversation

tromey
Copy link
Contributor

@tromey tromey commented Sep 4, 2018

commit 6c10142 ("Update LLVM submodule") disabled the lldb build.
This patch updates the lldb and clang submodules to once again build
against the LLVM that is included in the Rust tree, and reverts the
.travis.yml changes from that patch.

commit 6c10142 ("Update LLVM submodule") disabled the lldb build.
This patch updates the lldb and clang submodules to once again build
against the LLVM that is included in the Rust tree, and reverts the
.travis.yml changes from that patch.
@rust-highfive
Copy link
Collaborator

r? @nikomatsakis

(rust_highfive has picked a reviewer for you, use r? to override)

@rust-highfive
Copy link
Collaborator

⚠️ Warning ⚠️

  • These commits modify submodules.

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Sep 4, 2018
@alexcrichton
Copy link
Member

@bors: r+ p=1

Thanks for doing this! Sorry long weekend here so was a bit delayed getting around to it!

@bors
Copy link
Contributor

bors commented Sep 4, 2018

📌 Commit 289da84 has been approved by alexcrichton

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Sep 4, 2018
@bors
Copy link
Contributor

bors commented Sep 4, 2018

⌛ Testing commit 289da84 with merge f7175d56dd1abae937254e7705ab7564b597c3bd...

@bors
Copy link
Contributor

bors commented Sep 4, 2018

💔 Test failed - status-travis

@bors bors added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Sep 4, 2018
@rust-highfive
Copy link
Collaborator

The job dist-i686-apple of your PR failed on Travis (raw log). Through arcane magic we have determined that the following fragments from the build log may contain information about the problem.

Click to expand the log.
[00:04:03]       Memory: 8 GB
[00:04:03]       Boot ROM Version: VMW71.00V.0.B64.1704110547
[00:04:03]       Apple ROM Info: [MS_VM_CERT/SHA1/27d66596a61c48dd3dc7216fd715126e33f59ae7]Welcome to the Virtual Machine
[00:04:03]       SMC Version (system): 2.8f0
[00:04:03]       Serial Number (system): VM6KlCEzuEFh
[00:04:03] 
[00:04:03] hw.ncpu: 4
[00:04:03] hw.byteorder: 1234
[00:04:03] hw.memsize: 8589934592
---
[00:54:02] [ 74%] Building CXX object tools/lldb/source/Plugins/ExpressionParser/Clang/CMakeFiles/lldbPluginExpressionParserClang.dir/ClangExpressionParser.cpp.o
[00:54:03] [ 74%] Building CXX object tools/lldb/source/Core/CMakeFiles/lldbCore.dir/Opcode.cpp.o
[00:54:05] [ 74%] Building CXX object tools/lldb/source/Core/CMakeFiles/lldbCore.dir/PluginManager.cpp.o
[00:54:10] [ 74%] Building CXX object tools/clang/lib/StaticAnalyzer/Checkers/CMakeFiles/clangStaticAnalyzerCheckers.dir/MPI-Checker/MPIFunctionClassifier.cpp.o
[00:54:13] [ 74%] Building CXX object tools/lldb/source/Core/CMakeFiles/lldbCore.dir/RichManglingContext.cpp.o
[00:54:17] [ 74%] Building CXX object tools/lldb/source/Plugins/ExpressionParser/Clang/CMakeFiles/lldbPluginExpressionParserClang.dir/ClangExpressionVariable.cpp.o
[00:54:18] [ 74%] Building CXX object tools/clang/lib/StaticAnalyzer/Checkers/CMakeFiles/clangStaticAnalyzerCheckers.dir/NSAutoreleasePoolChecker.cpp.o
[00:54:18] [ 74%] Building CXX object tools/lldb/source/Core/CMakeFiles/lldbCore.dir/SearchFilter.cpp.o
[00:54:18] [ 74%] Building CXX object tools/lldb/source/Commands/CMakeFiles/lldbCommands.dir/CommandObjectWatchpoint.cpp.o
---
[00:55:12]   bool starts = StartsTerm();
[00:55:12]        ^
[00:55:12] 1 warning generated.
[00:55:12] [ 76%] Building CXX object tools/lldb/source/Plugins/ExpressionParser/Rust/CMakeFiles/lldbPluginExpressionParserRust.dir/RustFunctionCaller.cpp.o
[00:55:12] /Users/travis/build/rust-lang/rust/src/tools/lldb/source/Plugins/ExpressionParser/Rust/RustFunctionCaller.cpp:21:10: fatal error: 'lldb/Core/State.h' file not found
[00:55:12] #include "lldb/Core/State.h"
[00:55:12] 1 error generated.
[00:55:12] 1 error generated.
[00:55:12] make[3]: *** [tools/lldb/source/Plugins/ExpressionParser/Rust/CMakeFiles/lldbPluginExpressionParserRust.dir/RustFunctionCaller.cpp.o] Error 1
[00:55:12] make[2]: *** [tools/lldb/source/Plugins/ExpressionParser/Rust/CMakeFiles/lldbPluginExpressionParserRust.dir/all] Error 2
[00:55:12] make[2]: *** Waiting for unfinished jobs....
[00:55:12] [ 76%] Building CXX object tools/lldb/source/Plugins/ExpressionParser/Clang/CMakeFiles/lldbPluginExpressionParserClang.dir/IRForTarget.cpp.o
[00:55:15] [ 76%] Building CXX object tools/lldb/source/Core/CMakeFiles/lldbCore.dir/ValueObjectConstResultChild.cpp.o
[00:55:17] [ 76%] Building CXX object tools/clang/lib/StaticAnalyzer/Checkers/CMakeFiles/clangStaticAnalyzerCheckers.dir/ObjCAtSyncChecker.cpp.o
[00:55:19] [ 76%] Building CXX object tools/lldb/source/Core/CMakeFiles/lldbCore.dir/ValueObjectConstResultImpl.cpp.o
---
[00:56:33] [ 77%] Building CXX object tools/clang/lib/StaticAnalyzer/Checkers/CMakeFiles/clangStaticAnalyzerCheckers.dir/UndefCapturedBlockVarChecker.cpp.o
[00:56:35] [ 77%] Building CXX object tools/clang/lib/StaticAnalyzer/Checkers/CMakeFiles/clangStaticAnalyzerCheckers.dir/UndefResultChecker.cpp.o
[00:56:37] [ 77%] Building CXX object tools/clang/lib/StaticAnalyzer/Checkers/CMakeFiles/clangStaticAnalyzerCheckers.dir/UndefinedArraySubscriptChecker.cpp.o
[00:56:39] [ 77%] Building CXX object tools/clang/lib/StaticAnalyzer/Checkers/CMakeFiles/clangStaticAnalyzerCheckers.dir/UndefinedAssignmentChecker.cpp.o
[00:56:42] [ 77%] Building CXX object tools/clang/lib/StaticAnalyzer/Checkers/CMakeFiles/clangStaticAnalyzerCheckers.dir/UninitializedObject/UninitializedObjectChecker.cpp.o
[00:56:44] [ 77%] Building CXX object tools/clang/lib/StaticAnalyzer/Checkers/CMakeFiles/clangStaticAnalyzerCheckers.dir/UninitializedObject/UninitializedPointee.cpp.o
[00:56:47] [ 77%] Building CXX object tools/clang/lib/StaticAnalyzer/Checkers/CMakeFiles/clangStaticAnalyzerCheckers.dir/UnreachableCodeChecker.cpp.o
[00:56:51] [ 77%] Building CXX object tools/clang/lib/StaticAnalyzer/Checkers/CMakeFiles/clangStaticAnalyzerCheckers.dir/VforkChecker.cpp.o
[00:56:52] [ 77%] Building CXX object tools/clang/lib/StaticAnalyzer/Checkers/CMakeFiles/clangStaticAnalyzerCheckers.dir/VLASizeChecker.cpp.o
[00:56:53] [ 77%] Building CXX object tools/clang/lib/StaticAnalyzer/Checkers/CMakeFiles/clangStaticAnalyzerCheckers.dir/ValistChecker.cpp.o
[00:56:53] [ 77%] Building CXX object tools/clang/lib/StaticAnalyzer/Checkers/CMakeFiles/clangStaticAnalyzerCheckers.dir/ValistChecker.cpp.o
[00:56:56] [ 77%] Building CXX object tools/clang/lib/StaticAnalyzer/Checkers/CMakeFiles/clangStaticAnalyzerCheckers.dir/VirtualCallChecker.cpp.o
[00:57:05] [ 77%] Linking CXX static library ../../../../../lib/libclangStaticAnalyzerCheckers.a
[00:57:05] [ 77%] Built target clangStaticAnalyzerCheckers
[00:57:05] make[1]: *** [all] Error 2
[00:57:05] command did not execute successfully, got: exit code: 2
[00:57:05] 
[00:57:05] 
[00:57:05] build script failed, must exit now', /Users/travis/.cargo/registry/src/jackfan.us.kg-1ecc6299db9ec823/cmake-0.1.33/src/lib.rs:773:5
[00:57:05]  finished in 2222.084
[00:57:05] travis_fold:end:llvm

[00:57:05] travis_time:end:llvm:start=1536088912901087000,finish=1536091134876538000,duration=2221975451000
[00:57:05] travis_time:end:llvm:start=1536088912901087000,finish=1536091134876538000,duration=2221975451000

[00:57:05] failed to run: /Users/travis/build/rust-lang/rust/build/bootstrap/debug/bootstrap build
[00:57:05] Build completed unsuccessfully in 0:51:49
[00:57:05] make: *** [all] Error 1

The command "stamp sh -x -c "$RUN_SCRIPT"" exited with 2.
travis_time:start:15d6d356
$ date && (curl -fs --head https://google.com | grep ^Date: | sed 's/Date: //g' || true)
---
travis_fold:start:after_failure.2
travis_time:start:1949ad13
$ ls -lat $HOME/Library/Logs/DiagnosticReports/
total 0
drwx------+ 15 travis  staff  510 Jan 25  2018 ..
drwx------   2 travis  staff   68 Dec  6  2017 .
travis_fold:end:after_failure.2
travis_fold:start:after_failure.3
travis_time:start:2a182cea
$ find $HOME/Library/Logs/DiagnosticReports -type f -name '*.crash' -not -name '*.stage2-*.crash' -not -name 'com.apple.CoreSimulator.CoreSimulatorService-*.crash' -exec printf travis_fold":start:crashlog\n\033[31;1m%s\033[0m\n" {} \; -exec head -750 {} \; -exec echo travis_fold":"end:crashlog \; || true
$ find $HOME/Library/Logs/DiagnosticReports -type f -name '*.crash' -not -name '*.stage2-*.crash' -not -name 'com.apple.CoreSimulator.CoreSimulatorService-*.crash' -exec printf travis_fold":start:crashlog\n\033[31;1m%s\033[0m\n" {} \; -exec head -750 {} \; -exec echo travis_fold":"end:crashlog \; || true
travis_time:end:2a182cea:start=1536091136999173000,finish=1536091137020992000,duration=21819000
travis_fold:end:after_failure.3
travis_fold:start:after_failure.4
travis_time:start:10277f94
$ ln -s . checkout && for CORE in obj/cores/core.*; do EXE=$(echo $CORE | sed 's|obj/cores/core\.[0-9]*\.!checkout!\(.*\)|\1|;y|!|/|'); if [ -f "$EXE" ]; then printf travis_fold":start:crashlog\n\033[31;1m%s\033[0m\n" "$CORE"; gdb -q -c "$CORE" "$EXE" -iex 'set auto-load off' -iex 'dir src/' -iex 'set sysroot .' -ex bt -ex q; echo travis_fold":"end:crashlog; fi; done || true
travis_fold:end:after_failure.4
travis_fold:start:after_failure.5
travis_time:start:0360dbe6
travis_time:start:0360dbe6
$ cat ./obj/build/x86_64-unknown-linux-gnu/native/asan/build/lib/asan/clang_rt.asan-dynamic-i386.vers || true
cat: ./obj/build/x86_64-unknown-linux-gnu/native/asan/build/lib/asan/clang_rt.asan-dynamic-i386.vers: No such file or directory
travis_fold:end:after_failure.5
travis_fold:start:after_failure.6
travis_time:start:0a8ed020
$ dmesg | grep -i kill
$ dmesg | grep -i kill
Unable to obtain kernel buffer: Operation not permitted
usage: sudo dmesg
travis_fold:end:after_failure.6

Done. Your build exited with 1.

I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact @TimNN. (Feature Requests)

@tromey
Copy link
Contributor Author

tromey commented Sep 5, 2018

The lldb code definitely looks incorrect -- that file has moved -- but I cannot reproduce it in a local build.

@tromey
Copy link
Contributor Author

tromey commented Sep 5, 2018

Maybe locally it is picking up an installed lldb header.

@tromey
Copy link
Contributor Author

tromey commented Sep 5, 2018

I've fixed the lldb issue and as it is trivial I'm going to see if I can re-r+ this.

@bors r=alexcrichton

@bors
Copy link
Contributor

bors commented Sep 5, 2018

@tromey: 🔑 Insufficient privileges: Not in reviewers

@tromey
Copy link
Contributor Author

tromey commented Sep 5, 2018

@alexcrichton it will need another r+ from you

@alexcrichton
Copy link
Member

@bors: r+ delegate+

@bors
Copy link
Contributor

bors commented Sep 5, 2018

✌️ @tromey can now approve this pull request

@bors
Copy link
Contributor

bors commented Sep 5, 2018

📌 Commit 7bb30ff has been approved by alexcrichton

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Sep 5, 2018
@bors
Copy link
Contributor

bors commented Sep 5, 2018

⌛ Testing commit 7bb30ff with merge 780b0c7...

bors added a commit that referenced this pull request Sep 5, 2018
Restore lldb build

commit 6c10142 ("Update LLVM submodule") disabled the lldb build.
This patch updates the lldb and clang submodules to once again build
against the LLVM that is included in the Rust tree, and reverts the
.travis.yml changes from that patch.
@bors
Copy link
Contributor

bors commented Sep 5, 2018

☀️ Test successful - status-appveyor, status-travis
Approved by: alexcrichton
Pushing 780b0c7 to master...

@bors bors merged commit 7bb30ff into rust-lang:master Sep 5, 2018
@tromey tromey deleted the restore-lldb-build branch September 6, 2018 17:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants