Page MenuHomePhabricator

jasonmolenda (Jason Molenda)
User

Projects

User does not belong to any projects.

User Details

User Since
Sep 2 2014, 12:05 PM (254 w, 21 h)

Recent Activity

Yesterday

jasonmolenda accepted D63667: Support __kernel_rt_sigreturn in frame initialization.

Sorry yes, I should have marked this as approved. I don't have a linux machine handy so I haven't looked into the problem Jan reports with the change -- fwiw if you turn on unwind logging ('log enable lldb unwind' before you backtrace at a stop), it will tell you which unwind rule it is using to find the caller's return pc value as it goes up the stack. It's a little tricky to read at first, but once you get the hang of it you can usually spot the problem quickly and understand why lldb is behaving the way it is. But just looking at the source changes you're making, I'm fine with this.

Tue, Jul 16, 2:20 PM · Restricted Project

Mon, Jul 15

jasonmolenda committed rGc245249b7bd0: Update some file changes, but there's a dependency loop so it doesn't quite… (authored by jasonmolenda).
Update some file changes, but there's a dependency loop so it doesn't quite…
Mon, Jul 15, 5:00 PM
jasonmolenda added a comment to D63667: Support __kernel_rt_sigreturn in frame initialization.

Ah, regarding the S bit, looks good to me. I think I misunderstood the original problem you were working on. On Darwin systems, we have our friend sigtramp. When an async interrupt is received, the kernel saves the register context to stack and then calls sigtramp which does some instructions and then calls the trap handler function. Backing up the pc by 1 for __sigtramp is fine, because it did a normal CALL instruction to the trap handler.

Mon, Jul 15, 3:06 PM · Restricted Project
jasonmolenda added a comment to D63667: Support __kernel_rt_sigreturn in frame initialization.

I've updated this with code to recognize the 'S' in the eh_frame augmentation and record it in a new bool member of UnwindPlan, m_plan_is_for_signal_trap. I haven't hooked up any consumers of the new bit; as you say, with the current code flow we don't parse the eh_frame info until after we've decided whether the current frame is a trap handler frame or not.

I took a look at behaves_like_zeroth_frame. In my case, we're not getting to the point of consulting it, because before checking it, if we are processing a trap handler frame or if AlwaysRelyOnEHInfo returns true (both of which are true in my case), we use the EH Frame unwind plan if it is valid at the current address, and that valid at current address check is already passing. I'm somewhat reluctant to fiddle with the behaves_like_zeroth_frame bit speculatively. Note that with this change, it will get set for the next frame above __kernel_rt_sigreturn (because the current test checks if the next-lower frame is an eTrapHandler frame), which I think is where we want the async processing.

Mon, Jul 15, 3:01 PM · Restricted Project

Mon, Jul 8

jasonmolenda accepted D64362: Remove lldb-perf.

Thanks Jonas, yes we need to come up with a perf testing infrastructure in lldb but there isn't a lot of value in this original attempt from c. 2013.

Mon, Jul 8, 2:21 PM · Restricted Project, Restricted Project

Tue, Jul 2

jasonmolenda committed rG37aeca39ba2d: Update cmake build setup so lldb doesn't link against the DebugSymbols… (authored by jasonmolenda).
Update cmake build setup so lldb doesn't link against the DebugSymbols…
Tue, Jul 2, 4:41 PM
jasonmolenda committed rGaab389170202: add symbols/declvendor.cpp. (authored by jasonmolenda).
add symbols/declvendor.cpp.
Tue, Jul 2, 4:08 PM

Wed, Jun 26

jasonmolenda committed rG1659276beec2: Support nested target.xml register definition files, lack of reg group markers. (authored by jasonmolenda).
Support nested target.xml register definition files, lack of reg group markers.
Wed, Jun 26, 3:02 PM
jasonmolenda committed rGddf025e8dc46: Use the // integer divide operator in these target definition files, like… (authored by jasonmolenda).
Use the // integer divide operator in these target definition files, like…
Wed, Jun 26, 2:43 PM

Tue, Jun 25

jasonmolenda created D63802: Handle nested register definition xml files from the remote serial protocol stub.
Tue, Jun 25, 7:21 PM · Restricted Project, Restricted Project

Mon, Jun 24

jasonmolenda committed rG868a394bb60d: Don't link against the DebugSymbols private framework; try to dlopen + dlsym… (authored by jasonmolenda).
Don't link against the DebugSymbols private framework; try to dlopen + dlsym…
Mon, Jun 24, 3:10 PM
jasonmolenda committed rGb5e86e0b5108: Add windows abi plugin, breakpointprecondition. (authored by jasonmolenda).
Add windows abi plugin, breakpointprecondition.
Mon, Jun 24, 2:50 PM

Fri, Jun 21

jasonmolenda added a comment to D63667: Support __kernel_rt_sigreturn in frame initialization.

This looks good. Over in RegisterContextLLDB::GetFullUnwindPlanForFrame there's a behaves_like_zeroth_frame bool with similar setup -- in that case, it's trying to decide what type of UnwindPlan to pick for the frame, either one that is synchronous ("valid only at call sites / throwable locations") or asynchronous ("valid at every instruction"). For a behaves_like_zeroth_frame it needs to pick an asynchronous unwind plan. You may want to look at that too.

Fri, Jun 21, 2:32 PM · Restricted Project

Jun 13 2019

jasonmolenda accepted D63310: Make crashlog.py less noisy.

Nice.

Jun 13 2019, 5:38 PM · Restricted Project, Restricted Project

Jun 12 2019

jasonmolenda committed rG0e197bcb6b82: Re-land r363103 ("When reading ObjC class table, use new SPI if it is avail")… (authored by jasonmolenda).
Re-land r363103 ("When reading ObjC class table, use new SPI if it is avail")…
Jun 12 2019, 2:44 PM

Jun 11 2019

jasonmolenda committed rGdd4bed3d7a85: Back out r363103 ("When reading ObjC class table, use new SPI if it is avail")… (authored by jasonmolenda).
Back out r363103 ("When reading ObjC class table, use new SPI if it is avail")…
Jun 11 2019, 5:59 PM
jasonmolenda committed rG1a7362f33e1f: When reading ObjC class table, use new SPI if it is avail (authored by jasonmolenda).
When reading ObjC class table, use new SPI if it is avail
Jun 11 2019, 2:29 PM
jasonmolenda committed rGeb2e0c3844d3: Update AuxVector.cpp (authored by jasonmolenda).
Update AuxVector.cpp
Jun 11 2019, 2:18 PM

Jun 4 2019

jasonmolenda committed rGc93b99589f7e: Call abs to avoid signed/unsigned comparison warning. (authored by jasonmolenda).
Call abs to avoid signed/unsigned comparison warning.
Jun 4 2019, 3:46 PM

Jun 3 2019

jasonmolenda added inline comments to D62732: [WIP][RISCV] Initial port of LLDB for RISC-V.
Jun 3 2019, 4:30 PM · Restricted Project
jasonmolenda committed rGc3ea7c66fec0: Add support for mid-function epilogues on x86 that end in a non-local jump. (authored by jasonmolenda).
Add support for mid-function epilogues on x86 that end in a non-local jump.
Jun 3 2019, 3:34 PM

May 31 2019

jasonmolenda created D62764: Detect x86 mid-function epilogues that end in a jump.
May 31 2019, 7:05 PM · Restricted Project, Restricted Project

May 22 2019

jasonmolenda added a comment to D62221: [lldb-server][LLGS] Support 'g' packets.

Does your feature print *all* the registers including floating point/vector registers? Or just the main general purpose registers? In debugserver, we expedite the register values for the stopping thread:

May 22 2019, 5:59 PM · Restricted Project, Restricted Project

May 21 2019

jasonmolenda added inline comments to D62213: [ABI] Implement Windows ABI for x86_64.
May 21 2019, 7:33 PM · Restricted Project, Restricted Project
jasonmolenda added inline comments to D62221: [lldb-server][LLGS] Support 'g' packets.
May 21 2019, 3:19 PM · Restricted Project, Restricted Project

May 16 2019

jasonmolenda accepted D62015: Make sure GetObjectDescription falls back to the Objective-C runtime..
May 16 2019, 11:06 AM · Restricted Project
jasonmolenda added a comment to D62015: Make sure GetObjectDescription falls back to the Objective-C runtime..

My one concern is how this will work in a swift-lldb where we have a swift ValueObject, the print description fails, and we fall back to the ObjC print description. That's a change in behavior, right? I imagine worst case is that the objc print description will fail to produce anything and we'll have the same behavior as we do today.

May 16 2019, 10:14 AM · Restricted Project

May 10 2019

jasonmolenda committed rGabcb1215c964: Change the disabling of packet logging to be in TearDownHook lambdas. (authored by jasonmolenda).
Change the disabling of packet logging to be in TearDownHook lambdas.
May 10 2019, 4:24 PM
jasonmolenda committed rG18ba8947a6c2: Ted pointed out that some of test tests that are enabling packet logging when… (authored by jasonmolenda).
Ted pointed out that some of test tests that are enabling packet logging when…
May 10 2019, 4:04 PM

May 9 2019

jasonmolenda accepted D61732: FuncUnwinders: Add a new "SymbolFile" unwind plan.

Looks good to me.

May 9 2019, 5:48 PM · Restricted Project
jasonmolenda accepted D61733: Breakpad: Generate unwind plans from STACK CFI records.
May 9 2019, 5:34 PM · Restricted Project

May 2 2019

jasonmolenda committed rGc9f44cd4b6e5: RegisterContextLLDB::GetFullUnwindPlanForFrame has four verbose logging… (authored by jasonmolenda).
RegisterContextLLDB::GetFullUnwindPlanForFrame has four verbose logging…
May 2 2019, 4:51 PM
jasonmolenda committed rG3af3900ee7f9: Upstreaming an apple local patch by Frederic Riss. (authored by jasonmolenda).
Upstreaming an apple local patch by Frederic Riss.
May 2 2019, 4:17 PM
jasonmolenda abandoned D60957: Read ObjC class names in one large read, instead of reading them individually.

I had some more revisions on this patch, but I finally understood that I was solving the wrong problem with this patchset. lldb should not fetch class names up front when scanning the objective c class tables, they should only be read lazily as we evaluate an expression. The fact that we're reading names during the table scan is the problem that needs to be fixed.

May 2 2019, 3:18 PM · Restricted Project

Apr 26 2019

jasonmolenda accepted D61218: Fix a stack-smasher in PlatformMacOSX::GetSDKDirectory().

LGTM.

Apr 26 2019, 5:29 PM · Restricted Project, Restricted Project

Apr 25 2019

jasonmolenda updated the diff for D60957: Read ObjC class names in one large read, instead of reading them individually.

Fix one small bug in the jitted expressions. Increase the estimated size of class names that lldb uses to pre-allocate the string pool buffer. Added a method to read the string pool out of the inferior, to reduce code duplication, and have it read the last name to get the full size of the string pool correct.

Apr 25 2019, 6:42 PM · Restricted Project
jasonmolenda committed rG1cb2c21fb4bd: Another use of the interactive lldb.debugger. (authored by jasonmolenda).
Another use of the interactive lldb.debugger.
Apr 25 2019, 1:46 PM
jasonmolenda committed rG3775794812e5: Two tests were using the interactive convenience variable lldb.debugger. They… (authored by jasonmolenda).
Two tests were using the interactive convenience variable lldb.debugger. They…
Apr 25 2019, 1:03 PM

Apr 24 2019

jasonmolenda committed rGd129ee34a51e: add postfixexpression.cpp. (authored by jasonmolenda).
add postfixexpression.cpp.
Apr 24 2019, 12:49 PM

Apr 22 2019

jasonmolenda committed rG759805fd8a6c: One small tweak to LocateExecutableScriptingResources - I was still stat'ing… (authored by jasonmolenda).
One small tweak to LocateExecutableScriptingResources - I was still stat'ing…
Apr 22 2019, 6:12 PM
jasonmolenda committed rG9270a201b9d1: Add a small check to PlatformDarwin::LoadScriptingResourceForModule which reads… (authored by jasonmolenda).
Add a small check to PlatformDarwin::LoadScriptingResourceForModule which reads…
Apr 22 2019, 6:04 PM
jasonmolenda committed rGa38b8c8abce0: Fix a bug in my change to ModulesDidLoad in r357955. In the process of hoisting… (authored by jasonmolenda).
Fix a bug in my change to ModulesDidLoad in r357955. In the process of hoisting…
Apr 22 2019, 3:42 PM
jasonmolenda accepted D60829: FuncUnwinders: remove "current_offset" from function arguments.

Sorry for the delay in looking at this. It looks like a good change, an improvement on the original for sure. I've never seen multiple eh_frame FDEs for a single function, so that's fine. The .ARM.exidx format and Apple's compact unwind format can theoretically have multiple different unwind plans for a single function - but I've never seen it done in practice. More commonly, a single unwind plan will be used for multiple functions that have the same unwind state. (these two formats don't describe prologue/epilogues - they only describe the unwind state in the middle of the function where exceptions can be thrown)

Apr 22 2019, 3:08 PM · Restricted Project

Apr 21 2019

jasonmolenda updated the summary of D60957: Read ObjC class names in one large read, instead of reading them individually.
Apr 21 2019, 11:20 PM · Restricted Project
jasonmolenda updated the summary of D60957: Read ObjC class names in one large read, instead of reading them individually.
Apr 21 2019, 11:20 PM · Restricted Project
jasonmolenda created D60957: Read ObjC class names in one large read, instead of reading them individually.
Apr 21 2019, 11:19 PM · Restricted Project
jasonmolenda accepted D60949: UnwindPlan: pretty-print dwarf expressions.

very nice.

Apr 21 2019, 9:44 PM · Restricted Project

Apr 17 2019

jasonmolenda added a comment to D60829: FuncUnwinders: remove "current_offset" from function arguments.

Thanks for digging in to this Pavel. I haven't had time to look over the patch today, but I'd like to; I'll make time for it in the next couple days if that's OK.

Apr 17 2019, 9:45 PM · Restricted Project

Apr 15 2019

jasonmolenda accepted D60655: Fix typo in ArmUnwindInfo::GetUnwindPlan.

Agreed, this change is correct. I double checked this in the http://infocenter.arm.com/help/topic/com.arm.doc.ihi0038b/IHI0038B_ehabi.pdf doc (mod 24 Nov 2015) and the only difference in Section 9.3 "Frame unwinding instructions" Table 4 and this code is in the comment - they now refer to this as 'Pop VFP double-precision registers D[8]-D[8+nnn] saved (as if) by VPUSH (see remark d)'. But there's no benefit in updating this one comment while the others are using the older comments (this code was originally written against the 30 Nov 2012 version of the doc).

Apr 15 2019, 2:36 PM · Restricted Project, Restricted Project

Apr 8 2019

jasonmolenda committed rG1724a179e7ae: Rename Target::GetSharedModule to Target::GetOrCreateModule. (authored by jasonmolenda).
Rename Target::GetSharedModule to Target::GetOrCreateModule.
Apr 8 2019, 4:02 PM

Apr 5 2019

jasonmolenda updated the diff for D60172: Renamed Target::GetSharedModule to AddModule, allow for ModulesDidLoad to be delayed when batch adding Modules.

Added a testcase, TestModuleLoadedNotifys.

Apr 5 2019, 8:39 PM · Restricted Project

Apr 4 2019

jasonmolenda added a comment to D60172: Renamed Target::GetSharedModule to AddModule, allow for ModulesDidLoad to be delayed when batch adding Modules.

Thanks Pavel, I'll convert this to use Expected<> while I'm working on it. I'm writing a test case right now, and looking at how DynamicPluginDarwin works more closely for adding & removing binaries, e.g. there's also no way to batch remove libraries and have a group broadcast notification be sent. There was also a bug in my original patch for calling LoadScriptingResourceForModule() to find a python resource file in a dSYM bundle - it is currently called from Target::ModuleUpdated when a binary is added to the target's ModuleList, but I'm suppressing that call with this change, so I need to move that over to ModulesDidLoad.

Apr 4 2019, 6:34 PM · Restricted Project

Apr 3 2019

jasonmolenda updated the diff for D60172: Renamed Target::GetSharedModule to AddModule, allow for ModulesDidLoad to be delayed when batch adding Modules.

Remove const bool notify's. Rename method to Target::GetOrCreateModule. Refine the method headerdoc a bit.

Apr 3 2019, 3:32 PM · Restricted Project
jasonmolenda added a comment to D60172: Renamed Target::GetSharedModule to AddModule, allow for ModulesDidLoad to be delayed when batch adding Modules.

OK looked things over a bit.

Apr 3 2019, 2:36 PM · Restricted Project

Apr 2 2019

jasonmolenda added a comment to D60172: Renamed Target::GetSharedModule to AddModule, allow for ModulesDidLoad to be delayed when batch adding Modules.

Thanks for looking the patch over.

Apr 2 2019, 9:20 PM · Restricted Project
jasonmolenda created D60172: Renamed Target::GetSharedModule to AddModule, allow for ModulesDidLoad to be delayed when batch adding Modules.
Apr 2 2019, 7:51 PM · Restricted Project
jasonmolenda committed rGaa107ca3a55f: Avoid macro redefinition error if HAVE_LIBCOMPRESSION is already defined. (authored by jasonmolenda).
Avoid macro redefinition error if HAVE_LIBCOMPRESSION is already defined.
Apr 2 2019, 6:16 PM

Apr 1 2019

jasonmolenda added inline comments to D60022: [Process] Fix WriteMemory return value.
Apr 1 2019, 11:26 AM · Restricted Project, Restricted Project

Mar 27 2019

jasonmolenda accepted D59896: [ObjectFileMachO] Disable memory caching for savecore..

Looks good. Removing that printf is good. Could you also remove the printf("mach_header:...") in the same function?

Mar 27 2019, 4:08 PM · Restricted Project, Restricted Project

Mar 26 2019

jasonmolenda added a comment to D55718: [ARC] Basic support in gdb-remote process plugin.

As long as the numbers _can_ still come from the GDB server, I am all for that. If they are not supplied, then we use arch defaults. That allows new system/arch bringups where the info might be changing often, and also allows the info to be locked down. The architecture plug-ins are a great place for this. We should also move the breakpoint opcode logic into the architecture plug-ins.

Mar 26 2019, 12:59 PM · Restricted Project
jasonmolenda added a comment to D55718: [ARC] Basic support in gdb-remote process plugin.

It would be really nice to get the GDB remote server for ARC so that it vends the correct regs to begin with, but if that isn't possible I guess we need to do what we are doing here. I would really like to not see more architectures have to add code to ProcessGDBRemote.cpp at all. If that isn't possible this solution will need to be used.

Mar 26 2019, 12:14 PM · Restricted Project
jasonmolenda accepted D59495: Fix an out-of-bounds error in RegisterContextDarwin_arm64.

Sorry for not weighing in earlier, yes this is good.

Mar 26 2019, 12:02 PM · Restricted Project

Mar 20 2019

jasonmolenda committed rG9d760a0a76e6: Change the logging on ptrace(PT_KILL) in MachProcess::Kill to log if… (authored by jasonmolenda).
Change the logging on ptrace(PT_KILL) in MachProcess::Kill to log if…
Mar 20 2019, 4:01 PM
jasonmolenda committed rGdb5edebea266: Update DWARF files. (authored by jasonmolenda).
Update DWARF files.
Mar 20 2019, 4:00 PM

Mar 18 2019

jasonmolenda committed rG2d5e7adf26f4: A target definition file that may work for Aarch32 Cortex-M target processor… (authored by jasonmolenda).
A target definition file that may work for Aarch32 Cortex-M target processor…
Mar 18 2019, 2:39 PM

Mar 13 2019

jasonmolenda committed rG97866d74db72: Reorder the operations in DynamicLoaderDarwinKernel::KextImageInfo… (authored by jasonmolenda).
Reorder the operations in DynamicLoaderDarwinKernel::KextImageInfo…
Mar 13 2019, 4:34 PM
jasonmolenda committed rG6a5fa552c715: Update. (authored by jasonmolenda).
Update.
Mar 13 2019, 11:24 AM

Mar 12 2019

jasonmolenda committed rGc02b95e4f9b5: Rework how lldb warngs about kexts that failed to load in a darwin kernel debug… (authored by jasonmolenda).
Rework how lldb warngs about kexts that failed to load in a darwin kernel debug…
Mar 12 2019, 12:42 PM

Mar 11 2019

jasonmolenda accepted D58347: Reinitialize UnwindTable when the SymbolFile changes.

Yeah, I'm ok with this, thanks.

Mar 11 2019, 4:59 PM · Restricted Project

Mar 8 2019

jasonmolenda committed rGcee6c47a62c4: Add parens to force the order of operations in an expression trying to do… (authored by jasonmolenda).
Add parens to force the order of operations in an expression trying to do…
Mar 8 2019, 4:04 PM

Mar 7 2019

jasonmolenda committed rG988332a54a70: Add ASAN llvm build directory variants to get_llvm_bin_dirs(). (authored by jasonmolenda).
Add ASAN llvm build directory variants to get_llvm_bin_dirs().
Mar 7 2019, 8:18 PM

Mar 6 2019

jasonmolenda committed rGa583486065db: When disassembling Aarch64 target and vendor Apple, set the cpu to "apple… (authored by jasonmolenda).
When disassembling Aarch64 target and vendor Apple, set the cpu to "apple…
Mar 6 2019, 7:18 PM
jasonmolenda committed rG2d6e6cbacc72: Remove the warning in DynamicLoaderDarwinKernel::KextImageInfo… (authored by jasonmolenda).
Remove the warning in DynamicLoaderDarwinKernel::KextImageInfo…
Mar 6 2019, 3:47 PM
jasonmolenda accepted D59030: Pass ConstString by value.

Looks good to me.

Mar 6 2019, 1:14 PM · Restricted Project, Restricted Project

Mar 5 2019

jasonmolenda committed rG4cc9ff124554: Change the scanning algorithm in DynamicLoaderDarwinKernel… (authored by jasonmolenda).
Change the scanning algorithm in DynamicLoaderDarwinKernel…
Mar 5 2019, 6:45 PM
jasonmolenda committed rGb459f182e83e: Re-commit logging for SBCompileUnit::GetNumLineEntries. (authored by jasonmolenda).
Re-commit logging for SBCompileUnit::GetNumLineEntries.
Mar 5 2019, 6:32 PM
jasonmolenda committed rGf228b2c2efcc: Revert logging addition to SBCompileUnit::GetNumLineEntries, causing lldb-mi… (authored by jasonmolenda).
Revert logging addition to SBCompileUnit::GetNumLineEntries, causing lldb-mi…
Mar 5 2019, 2:18 PM
jasonmolenda committed rG14c1e08ff57b: Add logging to SBCompileUnit::GetNumLineEntries. (authored by jasonmolenda).
Add logging to SBCompileUnit::GetNumLineEntries.
Mar 5 2019, 11:46 AM
jasonmolenda added a comment to D58962: Sanity check --max-gdbserver-port.

LGTM fwiw I think the --min-gdbserver-port and --max-gdbserver-port options were primarily/only used for iOS etc testsuite runs, where a fixed set of ports is relayed between the devices. I was having some problems with lldb-server in platform mode on these devices and thought it would be nice to have a standalone implementation of the platform protocol packets, so I wrote one a couple months ago and that's what we're using now for this environment. I should probably upstream it, I think it would work on linux too, it doesn't use any of lldb/llvm so it has several areas that are primitive because I was going for "good enough" and moving on, e.g. it has a dumb logging class and the networking could use some work, etc. I want to keep this completely free of lldb/llvm dependencies so it's a little bit of an oddball.

Mar 5 2019, 9:28 AM · Restricted Project

Mar 4 2019

jasonmolenda accepted D58912: [debugserver] Fix IsUserReady thread filtering.

Looks good, that was my only concern.

Mar 4 2019, 3:51 PM · Restricted Project
jasonmolenda added a comment to D58912: [debugserver] Fix IsUserReady thread filtering.

Will this hide a thread that jumps through a null function pointer? That's the only user process case where a pc of 0 needs to be reported to the developer.

Mar 4 2019, 3:39 PM · Restricted Project

Feb 28 2019

jasonmolenda committed rG7cdaf1e06bb5: Update com.apple.diagnosticd.diagnostic entitlement name to the newer com.apple. (authored by jasonmolenda).
Update com.apple.diagnosticd.diagnostic entitlement name to the newer com.apple.
Feb 28 2019, 7:35 PM
jasonmolenda committed rGe772052edf6c: Increase timeout in Symbols::DownloadObjectAndSymbolFile from 30 seconds to 120… (authored by jasonmolenda).
Increase timeout in Symbols::DownloadObjectAndSymbolFile from 30 seconds to 120…
Feb 28 2019, 7:25 PM
jasonmolenda committed rGf8fad6ca5bc6: Symbols.cpp in unittest moved too. (authored by jasonmolenda).
Symbols.cpp in unittest moved too.
Feb 28 2019, 3:37 PM

Feb 27 2019

jasonmolenda committed rG321f80e23f04: Move Symbols.cpp files. (authored by jasonmolenda).
Move Symbols.cpp files.
Feb 27 2019, 5:27 PM

Feb 19 2019

jasonmolenda added inline comments to D58347: Reinitialize UnwindTable when the SymbolFile changes.
Feb 19 2019, 12:39 PM · Restricted Project

Feb 18 2019

jasonmolenda committed rG461c4168787b: One more fix while I'm looking at this - remove the unused IsSBProcess method… (authored by jasonmolenda).
One more fix while I'm looking at this - remove the unused IsSBProcess method…
Feb 18 2019, 2:18 PM
jasonmolenda committed rG870c0a648b51: Ah, misunderstood Jonas' feedback - fix this so we'll do the right thing when… (authored by jasonmolenda).
Ah, misunderstood Jonas' feedback - fix this so we'll do the right thing when…
Feb 18 2019, 2:14 PM
jasonmolenda committed rG5044316d162d: Clean up an unused variable warning when building this for mac native. (authored by jasonmolenda).
Clean up an unused variable warning when building this for mac native.
Feb 18 2019, 2:10 PM

Feb 15 2019

jasonmolenda committed rG3bf883eac95b: Add some unconditional logging on the failure points when attaching to a… (authored by jasonmolenda).
Add some unconditional logging on the failure points when attaching to a…
Feb 15 2019, 2:40 PM

Feb 10 2019

jasonmolenda accepted D57928: Fix x86 return pattern detection.

Thanks for fixing this and adding a testcase. I don't know why 0xc9 leave was here; it's handled over in x86AssemblyInspectionEngine::leave_pattern_p. Do you have commit access?

Feb 10 2019, 12:01 AM · Restricted Project

Feb 8 2019

jasonmolenda committed rG3b68761dc3e0: Tiny fix spotted by static analyzer; GetPath() returns a std::string, we get a… (authored by jasonmolenda).
Tiny fix spotted by static analyzer; GetPath() returns a std::string, we get a…
Feb 8 2019, 3:36 PM

Feb 5 2019

jasonmolenda committed rG8130bf67baa7: Add a warning to GDBRemoteRegisterContext (if packet logging enabled) if the… (authored by jasonmolenda).
Add a warning to GDBRemoteRegisterContext (if packet logging enabled) if the…
Feb 5 2019, 8:08 PM
jasonmolenda accepted D57745: [x64] Process the B field of the REX prefix correctly for the PUSH and POP instructions.

The change looks good to me, thanks for fixing this. I'm not sure you're testing what you meant to test in TestPopRBPWithREX because you're pushing/popping r13. You could get the unwind state at byte offset 2 (after the push r13 has executed) and see that you can GetRegisterInfo() r13, and the unwind state at byte offset 4 and verify that you can't GetRegisterInfo() r13. That's a good test to make sure we handle the B bit correctly.

Feb 5 2019, 4:11 PM · Restricted Project

Jan 23 2019

jasonmolenda added a comment to D57011: Refactor HAVE_LIBCOMPRESSION and related code in GDBRemoteCommunication.

Sorry for the delay in checking this out. It works fine. You need to add a #include "lldb/Host/Config.h" to GDBRemoteCommunication.h.

Jan 23 2019, 3:34 PM

Jan 22 2019

jasonmolenda added a comment to D57011: Refactor HAVE_LIBCOMPRESSION and related code in GDBRemoteCommunication.

I'll test this later today. The cleanup looks good; the goal of the most recent changes was to make HAVE_LIBCOMPESSION always enabled on Darwin systems - the setting would occasionally get lost in the Xcode project settings and we'd have slow channels using uncompressed communication until we looked into the perf regressions. After this sequence happened twice, I wanted to make it always-on (it also needed to be conditional for a while because these API were only added to Darwin ~4 years ago)

Jan 22 2019, 12:07 PM

Dec 12 2018

jasonmolenda accepted D55607: Make crashlog.py work when a .dSYM is present, but a binary is missing.

Yep, looks good. It would be nice if we found a dSYM with a Spotlight search (mdfind) if we could look NEXT to the dSYM bundle and see if there is a real binary, and load that. But this is a good first step, and it gets us the source-level information for symbolicating the crash. More advanced crashlog users may want to look at the assembly instructions around the crash site, and for that we need the actual binary.

Dec 12 2018, 12:21 PM

Dec 6 2018

jasonmolenda added a comment to D55399: If we still have all_image_infos use it in DynamicLoaderMacOSDYLD to detect exec's.

I don't feel strongly about it, your choice. But if I misunderstood it this time, I'll likely misunderstand it again in the future. :)

Dec 6 2018, 5:15 PM
jasonmolenda accepted D55399: If we still have all_image_infos use it in DynamicLoaderMacOSDYLD to detect exec's.

LGTM.

Dec 6 2018, 4:46 PM

Nov 12 2018

jasonmolenda accepted D54460: Don't keep a global ABI plugin per architecture.

LGTM, my guess is that this was Greg's code originally. He may have made a singleton object out of caution.

Nov 12 2018, 6:06 PM