This revision was automatically updated to reflect the committed changes.
Closed by commit rL257460: Add clang::Type::Pipe to ClangASTContext (authored
by labath).
Changed prior to commit:
http://reviews.llvm.org/D16055?vs=44456&id=44607#toc
Repository:
rL LLVM
http://reviews.llvm.org/D1605
Author: labath
Date: Tue Jan 12 02:51:28 2016
New Revision: 257460
URL: http://llvm.org/viewvc/llvm-project?rev=257460&view=rev
Log:
Add clang::Type::Pipe to ClangASTContext
Summary:
Clang recently added support for an OpenCL pipe type. Adding the new type to
relevant switches to
avoid warnings.
Author: ravitheja
Date: Tue Jan 12 04:08:41 2016
New Revision: 257465
URL: http://llvm.org/viewvc/llvm-project?rev=257465&view=rev
Log:
Fix for TestNoreturnUnwind.py on i386
Summary:
The testcase TestNoreturnUnwind.py was failing
because the unwind from the vdso library was not
successful for cla
Author: tberghammer
Date: Tue Jan 12 04:41:43 2016
New Revision: 257469
URL: http://llvm.org/viewvc/llvm-project?rev=257469&view=rev
Log:
Try to fix the mips64 build after rL257447
Modified:
lldb/trunk/source/Plugins/Process/Linux/NativeRegisterContextLinux_mips64.cpp
Modified:
lldb/trunk/
ldrumm requested a review of this revision.
ldrumm added a comment.
bump
Comment at:
source/Plugins/ExpressionParser/Clang/ClangExpressionParser.cpp:85-97
@@ -83,1 +84,15 @@
+namespace {
+void debugStringVector(Log *log, const std::vector& vec,
const char *name)
+{
+
labath added a comment.
Could you explain what is the nature of the failure in the vdso parsing?
Otherwise it's quite hard to see what's the relationship of your patch to it,
as it seems that the ObjectFile should be able to handle files which are not
fully contained within the initial read (co
labath added a comment.
In http://reviews.llvm.org/D16107#324594, @labath wrote:
> Could you explain what is the nature of the failure in the vdso parsing?
Strike that. I see that you have linked the bug # in the title...
http://reviews.llvm.org/D16107
_
tberghammer added a comment.
Can we calculate the size of the object file based on elf headers and the
section headers instead of using the /proc file system?
I think a good algorithm would be to take the maximum of the following 2
expression:
Start of section header + Number of section head
spyffe added a comment.
I'm a little concerned that LanguageRuntime::GetOverrideExprOptions() appears
to generate a new set of options from whole cloth rather than using the
existing set as a starting point.
Specifically, since your use case is wanting to override the calling
convention, I thi
Author: sas
Date: Tue Jan 12 13:02:41 2016
New Revision: 257502
URL: http://llvm.org/viewvc/llvm-project?rev=257502&view=rev
Log:
Increase use of svr4 packets to improve performance on POSIX remotes
Summary:
Allows the remote to enumerate the link map when adding and removing
shared libraries, so
ovyalov added a comment.
vdso issue is related to ELF format and I'm wondering whether we can keep the
fix contained in ObjectFileELF - for example, if a section that we're trying to
read is beyond the boundaries of memory buffer we can read its content from
inferior's memory using ObjectFile::
Looking at it now.
On Tue, Jan 12, 2016 at 11:25 AM, Siva Chandra wrote:
> This breaks the build:
> http://lab.llvm.org:8011/builders/lldb-x86_64-ubuntu-14.04-buildserver/builds/4487
>
> On Tue, Jan 12, 2016 at 11:02 AM, Stephane Sezer via lldb-commits
> wrote:
>> Author: sas
>> Date: Tue Jan 12
Author: sas
Date: Tue Jan 12 14:22:58 2016
New Revision: 257510
URL: http://llvm.org/viewvc/llvm-project?rev=257510&view=rev
Log:
Fix build breakage of r257502.
Modified:
lldb/trunk/source/Plugins/DynamicLoader/POSIX-DYLD/DYLDRendezvous.cpp
Modified: lldb/trunk/source/Plugins/DynamicLoader/P
Author: zturner
Date: Tue Jan 12 14:45:29 2016
New Revision: 257513
URL: http://llvm.org/viewvc/llvm-project?rev=257513&view=rev
Log:
Fix TestThreadJump on Windows.
The system can create threads for a system threadpool, so there is
no guarantee that the thread that is stopped is thread 1. So use
ldrumm added a comment.
In http://reviews.llvm.org/D15527#324919, @spyffe wrote:
> I'm a little concerned that LanguageRuntime::GetOverrideExprOptions() appears
> to generate a new set of options from whole cloth rather than using the
> existing set as a starting point.
>
> Specifically, since
sivachandra created this revision.
sivachandra added a reviewer: zturner.
sivachandra added a subscriber: lldb-commits.
http://reviews.llvm.org/D16125
Files:
packages/Python/lldbsuite/test/functionalities/thread/jump/TestThreadJump.py
Index:
packages/Python/lldbsuite/test/functionalities/thre
This isn't going to work either, because Windows doesn't the name=
portion. God I really hate these tests that do substring matching.
I guess there's no easy way to guarantee that the main.cpp: is
on the same frame as the breakpoint. Can you change it to:
substrs = ['stopped',
'main.cpp:{}'
sivachandra updated this revision to Diff 44674.
sivachandra added a comment.
Address comment
http://reviews.llvm.org/D16125
Files:
packages/Python/lldbsuite/test/functionalities/thread/jump/TestThreadJump.py
Index:
packages/Python/lldbsuite/test/functionalities/thread/jump/TestThreadJump.p
zturner accepted this revision.
zturner added a comment.
This revision is now accepted and ready to land.
If this passes on your side, go ahead. Just from looking, I think it should
pass on Windows.
http://reviews.llvm.org/D16125
___
lldb-commits
Author: sivachandra
Date: Tue Jan 12 16:33:19 2016
New Revision: 257531
URL: http://llvm.org/viewvc/llvm-project?rev=257531&view=rev
Log:
[TestThreadJump] Adjust match sub-string after recent change.
Reviewers: zturner
Subscribers: zturner, lldb-commits
Differential Revision: http://reviews.llv
zturner created this revision.
zturner added a reviewer: amccarth.
zturner added a subscriber: lldb-commits.
There were a number of problems preventing this from working:
1. The SWIG typemaps for converting Python lists to and from C++
arrays were not updated for Python 3. So they
amccarth accepted this revision.
amccarth added a comment.
This revision is now accepted and ready to land.
I don't know enough about SWIG to give that a thorough review.
Comment at: source/Host/windows/ProcessLauncherWindows.cpp:38
@@ +37,3 @@
+cur_entry += strlen(cur_e
bhushan closed this revision.
bhushan added a comment.
Closed by commit http://reviews.llvm.org/rL257442
Repository:
rL LLVM
http://reviews.llvm.org/D16051
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/ma
bhushan closed this revision.
bhushan added a comment.
Closed by commit http://reviews.llvm.org/rL257441
Repository:
rL LLVM
http://reviews.llvm.org/D16046
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/ma
Author: jmolenda
Date: Tue Jan 12 22:08:10 2016
New Revision: 257569
URL: http://llvm.org/viewvc/llvm-project?rev=257569&view=rev
Log:
Add a small refinement to the qSymbol:: support in lldb.
This is a packet which allows the remote gdb stub to ask for the address
of a symbol in the process. lldb
nitesh.jain created this revision.
nitesh.jain added reviewers: clayborg, ovyalov.
nitesh.jain added subscribers: jaydeep, bhushan, sagar, mohit.bhakkad,
lldb-commits.
nitesh.jain set the repository for this revision to rL LLVM.
When incorrect type used for 'char' then (at least) one of the expre
26 matches
Mail list logo