LGTM.
Tested:
Ubuntu 14.04 x86_64, clang-3.5-built lldb.
http://reviews.llvm.org/D5135
___
lldb-commits mailing list
lldb-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits
Author: tfiala
Date: Mon Sep 8 10:57:14 2014
New Revision: 217382
URL: http://llvm.org/viewvc/llvm-project?rev=217382view=rev
Log:
Linux/FreeBSD local debugging: allow redirection to pts for POSIX process.
See http://reviews.llvm.org/D5135 for more details.
Change by Zephyr Zhao.
Modified:
Submitted here:
```
svn commit
Sendingsource/Plugins/Process/POSIX/ProcessPOSIX.cpp
Sendingsource/Plugins/Process/POSIX/ProcessPOSIX.h
Transmitting file data ..
Committed revision 217382.
```
http://reviews.llvm.org/D5135
___
No worries, trying this again with patch set 2.
http://reviews.llvm.org/D5135
___
lldb-commits mailing list
lldb-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits
Author: tfiala
Date: Mon Sep 8 11:44:17 2014
New Revision: 217387
URL: http://llvm.org/viewvc/llvm-project?rev=217387view=rev
Log:
Modify call-function and rdar-12437442 tests to clean up all files produced
during test run.
Modified:
with remote-debug sessions.
___
lldb-commits mailing list
lldb-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits
--
Todd Fiala | Software Engineer | tfi...@google.com | 650-943-3180
.
___
lldb-commits mailing list
lldb-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits
--
Todd Fiala | Software Engineer | tfi...@google.com | 650-943-3180
___
lldb-commits
:
___
lldb-commits mailing list
lldb-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits
--
Todd Fiala | Software Engineer | tfi...@google.com | 650-943-3180
___
lldb-commits mailing list
lldb-commits
don't write
thread.GetNativeThread(), you're guaranteed to only be using generic
methods.
___
lldb-commits mailing list
lldb-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits
--
Todd Fiala | Software Engineer | tfi
Trying this out locally now.
http://reviews.llvm.org/D5110
___
lldb-commits mailing list
lldb-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits
(), you're guaranteed to only be using generic
methods.
___
lldb-commits mailing list
lldb-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits
--
Todd Fiala | Software Engineer | tfi...@google.com | 650-943-3180
entirely. Then
comments about that review started coming in on this thread. Anyway, look
here: http://reviews.llvm.org/D5198
On Fri, Sep 5, 2014 at 4:35 PM, Todd Fiala tfi...@google.com wrote:
Hey Zachary - am I looking in the right spot? The top of the thread is
talking about this:
http
remote debugging over a slow connection.
___
lldb-commits mailing list
lldb-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits
--
Todd Fiala | Software Engineer | tfi...@google.com | 650-943-3180
Author: tfiala
Date: Thu Sep 4 09:16:19 2014
New Revision: 217153
URL: http://llvm.org/viewvc/llvm-project?rev=217153view=rev
Log:
Use the process' ReadCStringFromMemory from
DYLDRendezvous::ReadStringFromMemory.
Tested:
Ubuntu 14.04 x86_64, clang-3.5-built lldb.
MacOSX 10.9.4,
be provided to build*
functions above) which
contains OS-specific build flags.
___
lldb-commits mailing list
lldb-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits
--
Todd Fiala | Software Engineer | tfi
has been setup), which means we can’t load any shared object yet. AFAICT,
this can only happen during remote debugging, as attaching to a live
process with lldb directly would necessarily attach after the entry point.
On Sep 4, 2014, at 3:32 PM, Todd Fiala tfi...@google.com wrote:
(I've also
LGTM, thanks for latest changes.
http://reviews.llvm.org/D5089
___
lldb-commits mailing list
lldb-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits
Author: tfiala
Date: Tue Sep 2 09:50:01 2014
New Revision: 216907
URL: http://llvm.org/viewvc/llvm-project?rev=216907view=rev
Log:
Linux ARM64: add ProcessMonitor-related RegisterContext support.
See http://reviews.llvm.org/D5089 for more details.
Change by Paul Osmialowski.
Added:
Tested:
* Ubuntu 14.04 x86_64, clang 3.5-built lldb.
* MacOSX 10.9.4, Xcode6-Beta6-built lldb.
http://reviews.llvm.org/D5089
___
lldb-commits mailing list
lldb-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits
Closed with commit:
```
svn commit
Sendingsource/Plugins/Process/POSIX/CMakeLists.txt
Sendingsource/Plugins/Process/POSIX/POSIXThread.cpp
Adding
source/Plugins/Process/POSIX/RegisterContextPOSIXProcessMonitor_arm64.cpp
Adding
-commits
--
Todd Fiala | Software Engineer | tfi...@google.com | 650-943-3180
___
lldb-commits mailing list
lldb-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits
Ok.
In here:
svn commit
Sendingsource/Host/CMakeLists.txt
Transmitting file data .
Committed revision 216948.
On Tue, Sep 2, 2014 at 2:15 PM, Stephane Sezer s...@fb.com wrote:
Nope. I don’t have committer access.
On Sep 2, 2014, at 2:12 PM, Todd Fiala tfi...@google.com wrote
must have put the source/Utility directory in the list of
include directories?
J
On Aug 28, 2014, at 7:57 AM, Todd Fiala tfi...@google.com wrote:
Attaching the final diff I'm using over here (needed minor rebasing
against TOT, r216668).
On Thu, Aug 28, 2014 at 7:40 AM, Todd Fiala tfi
://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits
--
Todd Fiala | Software Engineer | tfi...@google.com | 650-943-3180
___
lldb-commits mailing list
lldb-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits
Author: tfiala
Date: Fri Aug 29 11:01:35 2014
New Revision: 216737
URL: http://llvm.org/viewvc/llvm-project?rev=216737view=rev
Log:
lldb - Register Context Linux ARM64
Yet another step toward ARM64 support. With this commit, lldb-gdbserver started
on ARM64 target can be accessed by lldb running
Committed revision 216737.
On Fri, Aug 29, 2014 at 7:46 AM, Todd Fiala tfi...@google.com wrote:
(And thanks for checking it out!)
On Fri, Aug 29, 2014 at 7:46 AM, Todd Fiala tfi...@google.com wrote:
Is this necessary in RegisterInfos_arm64.h?
+#include ../../../Utility
See comments here, Paul:
http://lists.cs.uiuc.edu/pipermail/lldb-commits/Week-of-Mon-20140825/012666.html
I'll wait to see how you adjust this per comments, then get it in.
http://reviews.llvm.org/D4430
___
lldb-commits mailing list
://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits
--
Todd Fiala | Software Engineer | tfi...@google.com | 650-943-3180
___
lldb-commits mailing list
lldb-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits
I just put this in here:
svn commit
Sendingsource/Plugins/Process/gdb-remote/GDBRemoteCommunication.cpp
Transmitting file data .
Committed revision 216745.
On Thu, Aug 28, 2014 at 10:51 PM, Todd Fiala todd.fi...@gmail.com wrote:
I'm going to put this in since I haven't heard any
Author: tfiala
Date: Fri Aug 29 12:10:31 2014
New Revision: 216745
URL: http://llvm.org/viewvc/llvm-project?rev=216745view=rev
Log:
Add arbitrary command line flags to llgs/debugserver startup for local
debugging.
This patch accepts environment variables of the form:
The modification to
source/Plugins/Process/gdb-remote/GDBRemoteCommunication.cpp is not appropriate
in this patch. It already went in as a separate change (llvm.org svn r216745
here:
http://lists.cs.uiuc.edu/pipermail/lldb-commits/Week-of-Mon-20140825/012685.html).
Author: tfiala
Date: Fri Aug 29 12:35:57 2014
New Revision: 216748
URL: http://llvm.org/viewvc/llvm-project?rev=216748view=rev
Log:
Consolidate UnixSignals setting/getting in Process.
See http://reviews.llvm.org/D5108 for details.
This change does the following:
* eliminates the
Put it up here:
svn commit
Sendinginclude/lldb/Host/Host.h
Sendinginclude/lldb/Target/Process.h
Sendinginclude/lldb/lldb-private-forward.h
Sendinglldb.xcodeproj/project.pbxproj
Sendingsource/Host/common/Host.cpp
Sendingsource/Host/freebsd/Host.cpp
/mailman/listinfo/lldb-commits
--
Todd Fiala | Software Engineer | tfi...@google.com | 650-943-3180
___
lldb-commits mailing list
lldb-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits
Author: tfiala
Date: Fri Aug 29 15:14:21 2014
New Revision: 216766
URL: http://llvm.org/viewvc/llvm-project?rev=216766view=rev
Log:
lldb - fix misleading valid target indexes are error message
See http://reviews.llvm.org/D4430 for more details.
Change by Paul Osmialowski.
Modified:
svn commit
Sendingsource/Commands/CommandObjectTarget.cpp
Transmitting file data .
Committed revision 216766.
http://reviews.llvm.org/D4430
___
lldb-commits mailing list
lldb-commits@cs.uiuc.edu
Author: tfiala
Date: Fri Aug 29 15:32:18 2014
New Revision: 216770
URL: http://llvm.org/viewvc/llvm-project?rev=216770view=rev
Log:
Fix a test broken by r216564.
llgs Linux is no longer doing the translation to some gdb fixed signal
numbers. This change modifies the test method to take in the
, 2014 at 10:11 AM, Todd Fiala todd.fi...@gmail.com wrote:
Author: tfiala
Date: Wed Aug 27 12:11:56 2014
New Revision: 216564
URL: http://llvm.org/viewvc/llvm-project?rev=216564view=rev
Log:
NativeThreadLinux: remove translation of some Linux signals to gdb signals.
I copied this originally
No need to rebase this one. Testing it now.
http://reviews.llvm.org/D4381
___
lldb-commits mailing list
lldb-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits
svn commit
Sendinginclude/lldb/Core/ArchSpec.h
Sendingsource/Core/ArchSpec.cpp
Transmitting file data ..
Committed revision 216668.
http://reviews.llvm.org/D4381
___
lldb-commits mailing list
lldb-commits@cs.uiuc.edu
Small tweak - a flag from another branch snuck in. Harmless for the change but
still not intended to be there.
http://reviews.llvm.org/D5108
Files:
include/lldb/Host/Host.h
include/lldb/Target/Process.h
include/lldb/lldb-private-forward.h
lldb.xcodeproj/project.pbxproj
Author: tfiala
Date: Thu Aug 28 16:21:39 2014
New Revision: 216686
URL: http://llvm.org/viewvc/llvm-project?rev=216686view=rev
Log:
Ensure the Host wait4 thread is able to show pid in thread name.
For hosts that cannot support long thread names, shrink the host
wait4 thread name so that it does
I'm going to put this in since I haven't heard any feedback on it.
On Tue, Jul 29, 2014 at 10:02 AM, Todd Fiala todd.fi...@gmail.com wrote:
BTW - I'm using LLDB_DEBUGSERVER_* as the prefix because there are already
some more-specialized environment variables for similar tasks that start
Author: tfiala
Date: Wed Aug 27 11:05:26 2014
New Revision: 216553
URL: http://llvm.org/viewvc/llvm-project?rev=216553view=rev
Log:
Add Linux support for get thread area on ARM64 using ProcessMonitor debugging.
See http://reviews.llvm.org/D5073.
Change by Paul Osmialowski.
Modified:
LGTM.
Tested:
Ubuntu 14.04 x86_64, clang-3.5-built lldb.
http://reviews.llvm.org/D5073
___
lldb-commits mailing list
lldb-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits
Testing now. Already reviewed by Greg Clayton.
http://reviews.llvm.org/D5070
___
lldb-commits mailing list
lldb-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits
As Greg already called out, LGTM.
Just giving it a test run on MacOSX, then will check in.
http://reviews.llvm.org/D5069
___
lldb-commits mailing list
lldb-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits
Author: tfiala
Date: Wed Aug 27 11:21:41 2014
New Revision: 216556
URL: http://llvm.org/viewvc/llvm-project?rev=216556view=rev
Log:
lldb AArch64 should be listed as yet another 64-bit architecture in
HostInfoBase.
See http://reviews.llvm.org/D5070.
Change by Paul Osmialowski.
Modified:
Testing now, change looks good.
http://reviews.llvm.org/D5078
___
lldb-commits mailing list
lldb-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits
Tested:
Ubuntu 14.04 x86_64, clang-3.5-built lldb.
http://reviews.llvm.org/D5078
___
lldb-commits mailing list
lldb-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits
Author: tfiala
Date: Wed Aug 27 11:32:02 2014
New Revision: 216559
URL: http://llvm.org/viewvc/llvm-project?rev=216559view=rev
Log:
lldb Missing ARM64 breakpoint opcode for ProcessPOSIX added.
See http://reviews.llvm.org/D5078.
Change by Paul Osmialowski.
Modified:
svn commit
Sendingsource/Plugins/Process/POSIX/ProcessPOSIX.cpp
Transmitting file data .
Committed revision 216559.
http://reviews.llvm.org/D5078
___
lldb-commits mailing list
lldb-commits@cs.uiuc.edu
Author: tfiala
Date: Wed Aug 27 12:11:56 2014
New Revision: 216564
URL: http://llvm.org/viewvc/llvm-project?rev=216564view=rev
Log:
NativeThreadLinux: remove translation of some Linux signals to gdb signals.
I copied this originally based on what debugserver was doing. This appears to
be
source/Plugins/Process/Utility/RegisterContextPOSIX_arm64.cpp
source/Plugins/Process/Utility/RegisterContextPOSIX_arm64.h
___
lldb-commits mailing list
lldb-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits
--
Todd
Hey Paul,
Trying to catch up on the patches that didn't make it in yet. This looks like
one of them. Can you shoot me an email with all the changes that are not in
yet? Can you also add me as reviewers on them (user tfiala) and rebase them
against lldb top of tree? Thanks!
-Todd
Author: tfiala
Date: Wed Aug 27 17:58:26 2014
New Revision: 216630
URL: http://llvm.org/viewvc/llvm-project?rev=216630view=rev
Log:
Add some logging around Process attaching and inferior exec handling.
Modified:
lldb/trunk/include/lldb/Target/Process.h
Author: tfiala
Date: Tue Aug 26 12:19:10 2014
New Revision: 216465
URL: http://llvm.org/viewvc/llvm-project?rev=216465view=rev
Log:
Refactored existing gdb-remote qProcessInfo tests into separate python file.
I'm about to add some more qProcessInfo tests so I wanted to first pull them out
of the
Author: tfiala
Date: Tue Aug 26 13:21:02 2014
New Revision: 216470
URL: http://llvm.org/viewvc/llvm-project?rev=216470view=rev
Log:
Fix llgs to send triple for non-Apple platforms and lldb to interpret correctly.
This change addresses this bug:
http://llvm.org/bugs/show_bug.cgi?id=20755
This
there to be single call to initialize everything and terminate
everything and this is done via the Debugger calls.
On Aug 21, 2014, at 3:41 PM, Todd Fiala todd.fi...@gmail.com wrote:
Author: tfiala
Date: Thu Aug 21 17:41:25 2014
New Revision: 216238
URL: http://llvm.org/viewvc/llvm
There might be an issue there.
PluginManager uses the file system initialization. But I think Debugger
depends on HostInfo being initialized first.
I'll look and see if I can detangle that.
On Tue, Aug 26, 2014 at 11:43 AM, Todd Fiala tfi...@google.com wrote:
Ah ok. I'll move those
:52 AM, Todd Fiala tfi...@google.com wrote:
Ok - but - are we okay with moving PluginManager::Initialize() within
there? If so, we're good. Because HostInfo::Initialize() needs to come
*before* PluginManager::Initialize().
Seems reasonable, I just need to peek where we're initializing
Putting Ed on this :-)
http://reviews.llvm.org/D4918
___
lldb-commits mailing list
lldb-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits
___
lldb-commits mailing list
lldb-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits
--
Todd Fiala | Software Engineer | tfi...@google.com | 650-943-3180
___
lldb-commits mailing list
lldb-commits
Great, will do - thanks Jason!
On Mon, Aug 25, 2014 at 2:44 PM, Jason Molenda jmole...@apple.com wrote:
Ah sorry I missed that patch.
Yes, this looks correct, please commit.
On Aug 25, 2014, at 1:41 PM, Todd Fiala tfi...@google.com wrote:
Bump!
On Fri, Aug 15, 2014 at 3:30 PM
Submitted.
svn commit
Sendingsource/Symbol/DWARFCallFrameInfo.cpp
Transmitting file data .
Committed revision 216409.
On Mon, Aug 25, 2014 at 2:46 PM, Todd Fiala tfi...@google.com wrote:
Great, will do - thanks Jason!
On Mon, Aug 25, 2014 at 2:44 PM, Jason Molenda jmole
Er I'll get it in... eek..
On Fri, Aug 22, 2014 at 9:49 AM, Todd Fiala tfi...@google.com wrote:
I'm going to test this now. If it all looks good, I'll ge tit in.
On Tue, Aug 19, 2014 at 5:01 PM, Tong Shen endlessr...@google.com wrote:
Thanks Jason!
I will finish this patch and let's
0 and write off those .1% cases where it isn't correct instead of
trying to get too fancy with the assembly analysis code.
On Jul 29, 2014, at 4:17 PM, Todd Fiala tfi...@google.com
wrote:
Hey Jason,
Do you have any feedback on this?
Thanks!
-Todd
On Fri
source/Plugins/Process/Linux/ProcessMonitor.cpp
--
Todd Fiala | Software Engineer | tfi...@google.com | 650-943-3180
___
lldb-commits mailing list
lldb-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits
Hey Paul - I'll be looking to get your patches in today (I was OOO
yesterday).
http://reviews.llvm.org/D4803
___
lldb-commits mailing list
lldb-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits
LGTM.
Tested:
Ubuntu 14.04 x86_64: clang-3.5-built lldb. All local tests passed.
http://reviews.llvm.org/D4803
___
lldb-commits mailing list
lldb-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits
svn commit
Sendingsource/Plugins/Process/Linux/NativeProcessLinux.cpp
Sendingsource/Plugins/Process/Linux/ProcessMonitor.cpp
Transmitting file data ..
Committed revision 216185.
http://reviews.llvm.org/D4803
___
lldb-commits mailing
___
lldb-commits mailing list
lldb-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits
--
Todd Fiala | Software Engineer | tfi...@google.com | 650-943-3180
___
lldb-commits mailing list
lldb-commits
.
Submitted here:
svn commit
Sendingsource/Core/Communication.cpp
Transmitting file data .
Committed revision 216194.
On Thu, Aug 21, 2014 at 9:54 AM, Todd Fiala tfi...@google.com wrote:
Giving this a final test now.
-Todd
On Wed, Aug 20, 2014 at 4:53 PM, Alex Pepper apep
Awesome - fixes an issue in my branch as well :-) Thanks, Jason!
On Thu, Aug 21, 2014 at 4:22 PM, Jason Molenda jmole...@apple.com wrote:
Author: jmolenda
Date: Thu Aug 21 18:22:33 2014
New Revision: 216243
URL: http://llvm.org/viewvc/llvm-project?rev=216243view=rev
Log:
Two small fixes
Looking at this now.
http://reviews.llvm.org/D4803
___
lldb-commits mailing list
lldb-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits
Look at this now (in isolation from http://reviews.llvm.org/D4803).
http://reviews.llvm.org/D4969
___
lldb-commits mailing list
lldb-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits
Same with this - did this go in while I was on vacation?
Looking to close out some reviews that are still listed as open.
http://reviews.llvm.org/D4580
___
lldb-commits mailing list
lldb-commits@cs.uiuc.edu
Thanks. I'll get that in and be done with it (wipes hands clean...) ;-)
On Tue, Aug 19, 2014 at 10:27 AM, jing...@apple.com wrote:
That looks right to me.
Jim
On Aug 18, 2014, at 9:07 PM, Todd Fiala tfi...@google.com wrote:
Hey Jim,
Does this patch adequately address your
Author: tfiala
Date: Tue Aug 19 12:40:43 2014
New Revision: 215996
URL: http://llvm.org/viewvc/llvm-project?rev=215996view=rev
Log:
Adjust process launch --disable-aslr to take true/false value.
This change modifies the 'process launch' --disable-aslr option to take a
boolean argument. If the
Submitted:
tfiala-macpro2:lldb tfiala$ svn commit
Sendinginclude/lldb/Target/Process.h
Sendingsource/Commands/CommandObjectProcess.cpp
Sendingsource/Target/Process.cpp
Transmitting file data ...
Committed revision 215996.
On Tue, Aug 19, 2014 at 10:28 AM, Todd Fiala tfi
.
http://reviews.llvm.org/D4803
--
Todd Fiala | Software Engineer | tfi...@google.com | 650-943-3180
___
lldb-commits mailing list
lldb-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits
Ok - that sounds good. Would you like to pull those out and put it in this
patch, since they seem to be atomically related?
http://reviews.llvm.org/D4803
___
lldb-commits mailing list
lldb-commits@cs.uiuc.edu
Okay - thanks Paul. Closing out. Listing as accepted with understanding this
already went in (albeit slightly differently).
http://reviews.llvm.org/D4579
___
lldb-commits mailing list
lldb-commits@cs.uiuc.edu
Ok - marking as accepted and closing out (with understanding this functionality
already went in via another patch).
http://reviews.llvm.org/D4580
___
lldb-commits mailing list
lldb-commits@cs.uiuc.edu
+ AppleObjCClassDescriptorV2.cpp
+ AppleObjCTypeEncodingParser.cpp
)
___
lldb-commits mailing list
lldb-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits
--
Todd Fiala | Software Engineer | tfi...@google.com | 650
AppleThreadPlanStepThroughObjCTrampoline.cpp
+ AppleObjCClassDescriptorV2.cpp
+ AppleObjCTypeEncodingParser.cpp
)
___
lldb-commits mailing list
lldb-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits
--
Todd Fiala | Software
accordingly. In the event that the setting is not
explicitly made on the process launch command line, then the value is
retrieved from the target.disable-aslr setting (i.e. settings show
target.disable-aslr).
--
Todd Fiala | Software Engineer | tfi...@google.com | 650-943-3180
Index: include/lldb
Hey Jim,
Does this patch adequately address your comments on the earlier patch?
Thanks!
-Todd
On Mon, Aug 18, 2014 at 4:14 PM, Todd Fiala tfi...@google.com wrote:
Adjust process launch --disable-aslr to take true/false value.
This change modifies the 'process launch' --disable-aslr option
This change also adds the Plugins/Process/FreeBSD files to the Xcode project.
http://reviews.llvm.org/D4941
___
lldb-commits mailing list
lldb-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits
Ok cool, I'll get this in after removing the dupe of the system flag.
http://reviews.llvm.org/D4941
___
lldb-commits mailing list
lldb-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits
Thanks Todd - I'm playing around with the Xcode build now and just noticed
this today.
My pleasure, Ed!
http://reviews.llvm.org/D4941
___
lldb-commits mailing list
lldb-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits
Hey Zachary,
LGTM.
Tested:
Ubuntu 14.04 x86_64 - clang-built lldb.
MacOSX 10.9.4 - Xcode 6 Beta 5-built lldb.
Test Failures:
Unrelated - seeing TestCallStopAndContinue.py fail on both Linux and MacOSX
occasionally both without and with the patch. Should not block.
Hey Zach,
I agree with Ed's comments. If you want to do any minor fixup there, I'll give
the final a run on Linux (unless you want to do that part yourself - either way
is fine).
http://reviews.llvm.org/D4816
___
lldb-commits mailing list
Author: tfiala
Date: Tue Aug 12 12:02:07 2014
New Revision: 215458
URL: http://llvm.org/viewvc/llvm-project?rev=215458view=rev
Log:
llgs: corrected Linux signal reception notification for SIGABRT, SIGSEGV and
their ilk.
Added llgs/debugserver gdb-remote tests around SIGABRT and SIGSEGV signal
Ok caught Jason's response after writing that last one.
What state do you see this in, Jason?
-Todd
On Aug 12, 2014, at 5:05 PM, Todd Fiala tfi...@google.com wrote:
Yep, will do later tonight.
-Todd
On Aug 12, 2014, at 4:32 PM, Tong Shen endlessr...@google.com wrote:
Got it. So
,
gcc_eip_i386,
On Aug 12, 2014, at 5:05 PM, Todd Fiala tfi...@google.com wrote:
Yep, will do later tonight.
-Todd
On Aug 12, 2014, at 4:32 PM, Tong Shen endlessr...@google.com wrote:
Got it. So much magic going on there.
+Todd Can you check this in? Thanks!
On Tue, Aug 12, 2014
we’re thinking about it for migration
purposes? (Like - we could #ifdef it now correctly for *APPLE* and have it
default to the linux style just to be safe?)
-Todd
On Tue, Aug 12, 2014 at 5:41 PM, Todd Fiala tfi...@google.com wrote:
Ok - see you already did it. Thanks!
On Tue, Aug 12
Just coming back to the world of LLDB. Looks like lots of healthy
discussion on this one!
On Aug 10, 2014 10:30 PM, Matthew Gardiner m...@csr.com wrote:
Thanks Shawn,
If Greg agrees that we need the SyncIOHandler() invocation in the Command
object(s) (not the HandlePrivateEvent code) then
Shen
___
lldb-commits mailing list
lldb-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits
--
Todd Fiala | Software Engineer | tfi...@google.com | 650-943-3180
___
lldb-commits
name in different functions are different
variables.
--
Best Regards, Tong Shen
___
lldb-commits mailing list
lldb-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits
--
Todd Fiala | Software Engineer | tfi
101 - 200 of 375 matches
Mail list logo