[Lldb-commits] [lldb] r340747 - Disable use-color if the output stream is not a terminal with color support.

2018-08-27 Thread Raphael Isemann via lldb-commits
Author: teemperor Date: Mon Aug 27 08:16:25 2018 New Revision: 340747 URL: http://llvm.org/viewvc/llvm-project?rev=340747&view=rev Log: Disable use-color if the output stream is not a terminal with color support. Summary: LLDB currently only checks the output terminal for color support by looking

[Lldb-commits] [PATCH] D51243: Disable use-color if the output stream is not a terminal with color support.

2018-08-27 Thread Raphael Isemann via Phabricator via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL340747: Disable use-color if the output stream is not a terminal with color support. (authored by teemperor, committed by ). Herald added a subscriber: llvm-commits. Changed prior to commit: https://rev

[Lldb-commits] [lldb] r340748 - Let the CompilerInstance create our clang ASTContext

2018-08-27 Thread Raphael Isemann via lldb-commits
Author: teemperor Date: Mon Aug 27 08:18:33 2018 New Revision: 340748 URL: http://llvm.org/viewvc/llvm-project?rev=340748&view=rev Log: Let the CompilerInstance create our clang ASTContext Summary: Now that we moved the BuiltinContext and SelectorTable to the CompilerInstance, we can also get rid

[Lldb-commits] [PATCH] D51253: Let the CompilerInstance create our clang ASTContext

2018-08-27 Thread Raphael Isemann via Phabricator via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL340748: Let the CompilerInstance create our clang ASTContext (authored by teemperor, committed by ). Herald added a subscriber: llvm-commits. Changed prior to commit: https://reviews.llvm.org/D51253?vs=

[Lldb-commits] [PATCH] D49685: LLDB does not respect platform sysroot when loading core on Linux

2018-08-27 Thread Eugene Birukov via Phabricator via lldb-commits
EugeneBi marked an inline comment as done. EugeneBi added inline comments. Comment at: packages/Python/lldbsuite/test/functionalities/postmortem/elf-core/TestLinuxCore.py:215 +# /home/labath/test/a.out) +tmp_sysroot = "/tmp/lldb_i386_mock_sysroot" +execut

[Lldb-commits] [PATCH] D51208: [DWARF] Fix dwarf5-index-is-used.cpp

2018-08-27 Thread David Blaikie via Phabricator via lldb-commits
dblaikie added a comment. >> But if LLDB has different performance characteristics, or the default should >> be different for other reasons - I'm fine with that. I think I left it on >> for Apple so as not to mess with their stuff because of the MachO/dsym sort >> of thing that's a bit differen

[Lldb-commits] [PATCH] D51319: Use a RAII guard to lock/unlock DisassemblerLLVMC [NFC]

2018-08-27 Thread Raphael Isemann via Phabricator via lldb-commits
teemperor created this revision. teemperor added a reviewer: LLDB. Herald added a subscriber: lldb-commits. The manual lock/unlock calls make my LazyBool refactoring tricky (because now `return` potentially cause deadlocks), so this patch just replaces them with a much safer lock guard that is u

[Lldb-commits] [PATCH] D51319: Use a RAII guard to lock/unlock DisassemblerLLVMC [NFC]

2018-08-27 Thread Vedant Kumar via Phabricator via lldb-commits
vsk added inline comments. Comment at: source/Plugins/Disassembler/llvm/DisassemblerLLVMC.h:81 + struct Guard { +DisassemblerLLVMC *m_instance; +Guard(DisassemblerLLVMC *instance, InstructionLLVMC *inst, This is nice. Do you think it might be even safer

[Lldb-commits] [lldb] r340779 - Fix typo

2018-08-27 Thread Adrian Prantl via lldb-commits
Author: adrian Date: Mon Aug 27 14:46:18 2018 New Revision: 340779 URL: http://llvm.org/viewvc/llvm-project?rev=340779&view=rev Log: Fix typo Modified: lldb/trunk/test/CMakeLists.txt Modified: lldb/trunk/test/CMakeLists.txt URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/test/CMakeLists

[Lldb-commits] [lldb] r340791 - Add a mkdir -p to builddir into lldbtest.py

2018-08-27 Thread Adrian Prantl via lldb-commits
Author: adrian Date: Mon Aug 27 16:06:37 2018 New Revision: 340791 URL: http://llvm.org/viewvc/llvm-project?rev=340791&view=rev Log: Add a mkdir -p to builddir into lldbtest.py Based on how it is executed, it may not have been yet created. Modified: lldb/trunk/lit/Suite/lldbtest.py Modified

[Lldb-commits] [lldb] r340792 - Make the DYLD_INSERT_LIBRARIES workaround for SIP more robut for the various configurations that bots are running

2018-08-27 Thread Adrian Prantl via lldb-commits
Author: adrian Date: Mon Aug 27 16:06:38 2018 New Revision: 340792 URL: http://llvm.org/viewvc/llvm-project?rev=340792&view=rev Log: Make the DYLD_INSERT_LIBRARIES workaround for SIP more robut for the various configurations that bots are running Modified: lldb/trunk/lit/Suite/lldbtest.py M

[Lldb-commits] [PATCH] D51319: Use a RAII guard to control access to DisassemblerLLVMC.

2018-08-27 Thread Raphael Isemann via Phabricator via lldb-commits
teemperor updated this revision to Diff 162775. teemperor retitled this revision from "Use a RAII guard to lock/unlock DisassemblerLLVMC [NFC]" to "Use a RAII guard to control access to DisassemblerLLVMC.". teemperor edited the summary of this revision. teemperor added a comment. - Got rid of Lo

[Lldb-commits] [PATCH] D51319: Use a RAII guard to control access to DisassemblerLLVMC.

2018-08-27 Thread Davide Italiano via Phabricator via lldb-commits
davide accepted this revision. davide added a comment. This revision is now accepted and ready to land. LGTM if Vedant is happy with this. https://reviews.llvm.org/D51319 ___ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.or

[Lldb-commits] [PATCH] D51319: Use a RAII guard to control access to DisassemblerLLVMC.

2018-08-27 Thread Vedant Kumar via Phabricator via lldb-commits
vsk accepted this revision. vsk added a comment. Looks great, thanks! Comment at: source/Plugins/Disassembler/llvm/DisassemblerLLVMC.cpp:176 bool got_op = false; -std::shared_ptr disasm_sp(GetDisassembler()); -if (disasm_sp) { - const ArchSpec &arch = disasm_s