ravitheja added a comment.
The reason assembly unwind did not work is because if the ebp register is set
to 0 at that time, so the CFA is not available. In the case of clang the frame
pointer is not set by any function, which is why the formula does not work.
(lldb) image show-unwind -n __kerne
Author: abhishek
Date: Wed Dec 2 03:40:17 2015
New Revision: 254499
URL: http://llvm.org/viewvc/llvm-project?rev=254499&view=rev
Log:
PTRACE ABI to read FXSAVE area for 32-bit inferior
Summary:
- Problem occurs when:
-- 32-bit inferiors run on x86_32 machine and
the architecture does
Author: tberghammer
Date: Wed Dec 2 05:35:54 2015
New Revision: 254502
URL: http://llvm.org/viewvc/llvm-project?rev=254502&view=rev
Log:
Revert "Added support for -gmodule debugging when debug info is left in the .o
files on Darwin."
The commit caused a test failure on the linux buildbot in
Tes
Author: tberghammer
Date: Wed Dec 2 05:58:51 2015
New Revision: 254504
URL: http://llvm.org/viewvc/llvm-project?rev=254504&view=rev
Log:
Fix "process load/unload" on android
On android the symbols exposed by libdl (dlopen, dlclose, dlerror)
prefixed by "__dl_". This change moves the handling of
This revision was automatically updated to reflect the committed changes.
Closed by commit rL254504: Fix "process load/unload" on android (authored by
tberghammer).
Changed prior to commit:
http://reviews.llvm.org/D11465?vs=41491&id=41608#toc
Repository:
rL LLVM
http://reviews.llvm.org/D114
Author: tberghammer
Date: Wed Dec 2 07:36:17 2015
New Revision: 254507
URL: http://llvm.org/viewvc/llvm-project?rev=254507&view=rev
Log:
XFAIL TestReturnValue for linux i386 clang 3.5-3.6
The test failing most likely because clang 3.5 and 3.6 uses an incorrect
ABI for returning small structs fro
Hi Abhishek,
After this change TestReturnValue.py is failing on Linux with i386 inferior
when compiling the inferior with clang-3.5 (it passes with gcc and with ToT
clang). I marked the test XFAIL for that specific configuration to get the
buildbot green but can you take a look?
My suspicion is t
Hi Tamas
Yes, it was a bug in clang that I had submitted some months ago (ID: 23491) and
it was fixed. But the fix is not present in clang 3.5/3.6. Hence, the failure
is occurring.
Thanks & Regards
Abhishek Aggarwal
From: Tamas Berghammer [mailto:tbergham...@google.com]
Sent: Wednesday, Decemb
tberghammer created this revision.
tberghammer added reviewers: labath, clayborg.
tberghammer added a subscriber: lldb-commits.
Herald added subscribers: danalbert, tberghammer, emaste.
Change Platform::LoadImage to copy the file to the remote platform
The new implementation mimic the behavior of
Author: mohit.bhakkad
Date: Wed Dec 2 11:45:02 2015
New Revision: 254522
URL: http://llvm.org/viewvc/llvm-project?rev=254522&view=rev
Log:
[LLDB][MIPS] fix watchpoint searched on client side for same masked variables
Reviewers: clayborg.
Subscribers: jaydeep, bhushan, sagar, nitesh.jain,lldb-com
This revision was automatically updated to reflect the committed changes.
Closed by commit rL254522: [LLDB][MIPS] fix watchpoint searched on client side
for same masked variables (authored by mohit.bhakkad).
Changed prior to commit:
http://reviews.llvm.org/D15106?vs=41486&id=41644#toc
Reposito
clayborg requested changes to this revision.
clayborg added a comment.
This revision now requires changes to proceed.
"process load" should probably be changed to have an options that allows us to
specify where the shared library needs to be installed:
(lldb) process load --install-path=/usr/l
clayborg added a comment.
The --install-path option is optional and doesn't need to be specified, but if
it is specified, we need all platforms, even the host platform, to install the
shared library to this location prior to loading it.
http://reviews.llvm.org/D15152
___
jingham added a subscriber: jingham.
jingham added a comment.
It seems like an awful lot of logic is duplicated between the POSIX & Android
versions of LoadImage when all that really changes is the name of the function
you are calling. Is it possible to centralize this some more?
Repository:
labath added a subscriber: labath.
labath added a comment.
In http://reviews.llvm.org/D11465#300581, @jingham wrote:
> It seems like an awful lot of logic is duplicated between the POSIX & Android
> versions of LoadImage when all that really changes is the name of the
> function you are calling
jingham added a subscriber: jingham.
jingham added a comment.
We use process load to load images that are already extant (and system
libraries at that.) So it's got to be possible to load a library into a target
process without trying to get the image over to the target. And given how slow
so
Author: tfiala
Date: Wed Dec 2 12:48:38 2015
New Revision: 254530
URL: http://llvm.org/viewvc/llvm-project?rev=254530&view=rev
Log:
Adds candidate formatter for replacing legacy summary results.
Also cleans up some usages of strings where symbolic names
were safer and made more sense.
Try a tes
So the question is why is "S" being found in "std" when we are essentially
looking for "::S"?? Do we have a "using namespace std;" in the source? We
recently added support for using all of the using directives in the user code
so it might be doing a "using namespace std;" into the expression du
ovyalov closed this revision.
ovyalov added a comment.
Submitted as r250474
http://reviews.llvm.org/D13754
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
Author: zturner
Date: Wed Dec 2 13:00:52 2015
New Revision: 254533
URL: http://llvm.org/viewvc/llvm-project?rev=254533&view=rev
Log:
Use sub-commands instead of --mode={client,server}.
This is more pythonic and allows a more idiomatic way of getting
detailed usage information for each individual
So I was able to repro this with the following code:
#include
#include
namespace a
{
struct S
{
int s;
int ss;
};
}
int main (int argc, char const *argv[], char const *envp[])
{
a::S s = { 11, 22 };
puts("stop here"); // evaluate expression: e
Never mind, top of tree LLDB is working correctly. I ran our Apple built LLDB
that had a different version of this -gmodule fix for an Apple specific version
we did and it shows the same issue so I was led to believe this existed for a
while now, but it is definitely due to the -gmodule patch.
Author: tfiala
Date: Wed Dec 2 15:12:17 2015
New Revision: 254546
URL: http://llvm.org/viewvc/llvm-project?rev=254546&view=rev
Log:
Make --results-file stdout implied if unspecified when using a results
formatter.
Also cleans up pylint warnings (stock settings) in the modified function.
Modifi
Author: tfiala
Date: Wed Dec 2 15:45:15 2015
New Revision: 254550
URL: http://llvm.org/viewvc/llvm-project?rev=254550&view=rev
Log:
Candidate fixes for python2/3 compatible string handling in pickling support.
Modified:
lldb/trunk/packages/Python/lldbsuite/test/dotest_channels.py
lldb/tr
zturner created this revision.
zturner added a reviewer: tfiala.
zturner added a subscriber: lldb-commits.
http://reviews.llvm.org/D15162
Files:
packages/Python/lldbsuite/test/dotest_channels.py
packages/Python/lldbsuite/test/test_results.py
Index: packages/Python/lldbsuite/test/test_results
Lgtm
-Todd
> On Dec 2, 2015, at 2:28 PM, Zachary Turner wrote:
>
> zturner created this revision.
> zturner added a reviewer: tfiala.
> zturner added a subscriber: lldb-commits.
>
> http://reviews.llvm.org/D15162
>
> Files:
> packages/Python/lldbsuite/test/dotest_channels.py
> packages/Pyth
tfiala added a subscriber: tfiala.
tfiala added a comment.
Lgtm
-Todd
http://reviews.llvm.org/D15162
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
Author: zturner
Date: Wed Dec 2 17:07:33 2015
New Revision: 254563
URL: http://llvm.org/viewvc/llvm-project?rev=254563&view=rev
Log:
Fix test result serialization to use bytes.
Modified:
lldb/trunk/packages/Python/lldbsuite/test/dotest_channels.py
lldb/trunk/packages/Python/lldbsuite/tes
tfiala added a comment.
Okay I also had a chance to run it with the patch locally, and I didn't see any
issues.
http://reviews.llvm.org/D15162
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/
jasonmolenda added a comment.
Interesting. The routine is
0xf7fd9d70 <+0>: pushl %ecx
0xf7fd9d71 <+1>: pushl %edx
0xf7fd9d72 <+2>: pushl %ebp
0xf7fd9d73 <+3>: movl %esp, %ebp
0xf7fd9d75 <+5>: sysenter
[...]
0xf7fd9d7e <+14>: int$0x80
--> 0xf7fd9d80 <+16>: popl %ebp
0xf7fd9
ovyalov created this revision.
ovyalov added reviewers: clayborg, tberghammer.
ovyalov added a subscriber: lldb-commits.
Debug sections for JIT-ed code are stored in-memory and loaded as part of
module's image once __jit_debug_register_code is called -
if we cannot load a section by its base loa
sivachandra updated this revision to Diff 41706.
sivachandra added a comment.
Rebased and updated to take in DWO changes.
http://reviews.llvm.org/D12809
Files:
include/lldb/Symbol/ClangASTContext.h
include/lldb/Symbol/CompilerDeclContext.h
include/lldb/Symbol/GoASTContext.h
include/lldb
sivachandra added a comment.
PTaL
I finally got time to work on this. I have now rebased it and updated it take
in DWO changes.
I understand this change only enhances debug experience when GCC is used as the
DWARF producer (the targeted functionality already works as expected when the
producer
sivachandra added a comment.
A test for this already exists: TestCallStdStringFunction.py
http://reviews.llvm.org/D12809
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
Author: mohit.bhakkad
Date: Wed Dec 2 22:56:16 2015
New Revision: 254588
URL: http://llvm.org/viewvc/llvm-project?rev=254588&view=rev
Log:
[LLDB] Switch to assembly view if source is moved
Reviewers: clayborg, jingham, jasonmolenda.
Subscribers: jaydeep, bhushan, sagar, nitesh.jain,lldb-commits.
This revision was automatically updated to reflect the committed changes.
Closed by commit rL254588: [LLDB] Switch to assembly view if source is moved
(authored by mohit.bhakkad).
Changed prior to commit:
http://reviews.llvm.org/D12877?vs=38799&id=41714#toc
Repository:
rL LLVM
http://review
dawn created this revision.
dawn added a reviewer: jingham.
dawn added a subscriber: lldb-commits.
dawn set the repository for this revision to rL LLVM.
This patch fixes setting breakpoints on symbol for variants of C and Pascal
where the language is "unknown" within the filtering process added i
Author: jaydeep
Date: Thu Dec 3 00:41:24 2015
New Revision: 254590
URL: http://llvm.org/viewvc/llvm-project?rev=254590&view=rev
Log:
[LLDB][MIPS] Using enum instead of a constant to fetch PC and CAUSE registers.
SUMMARY:
Using enum instead of a constant to fetch PC and CAUSE registers.
M
38 matches
Mail list logo