Page MenuHomePhabricator

lanza (Nathan Lanza)
User

Projects

User does not belong to any projects.

User Details

User Since
Oct 20 2017, 4:34 PM (169 w, 1 d)

Recent Activity

Wed, Jan 13

lanza added inline comments to D94547: [lld-macho] Run ObjCContractPass during LTO.
Wed, Jan 13, 6:42 PM · Restricted Project
lanza added a comment to D94547: [lld-macho] Run ObjCContractPass during LTO.

215d59e7b0b14

Wed, Jan 13, 6:41 PM · Restricted Project

Tue, Jan 12

lanza added a reviewer for D94547: [lld-macho] Run ObjCContractPass during LTO: mehdi_amini.
Tue, Jan 12, 3:57 PM · Restricted Project
lanza added a comment to D94547: [lld-macho] Run ObjCContractPass during LTO.

This goes right before codegen in ThinLTOCodeGenerator. TM.addPassesToEmitFile is the entry point to the llvm backend. So the legacy implementation has this pass ran immediately after the front-end is finished and right before the backend starts. I imagine doing the same here makes the most sense.

Tue, Jan 12, 3:57 PM · Restricted Project

Thu, Jan 7

lanza abandoned D93926: [lldb] Don't remove the lldb.debugger var after exiting the interpreter.

Great, that does indeed seem to work properly. Thanks, Jonas!

Thu, Jan 7, 10:11 PM · Restricted Project
lanza added a comment to D93926: [lldb] Don't remove the lldb.debugger var after exiting the interpreter.

I guess if the intention of that is maintain the debugger instance around it should work, but at the moment it segfaults pretty quick with Xcode's lldb using:

Thu, Jan 7, 9:41 PM · Restricted Project
lanza added a comment to D93953: NFC: Remove simple_ilist comment mentioning ilist/iplist allocating.

On a similar note, I noticed elsewhere after fixing this that you mentioned in a commit message that you intended to remove iplist and ilist and rename as owning_ilist (or something of that sorts). Is that still on the table? Either way, the docs should be updated all around to account for either path name as there are still references to the pre-2016 implementation (e.g. https://llvm.org/docs/ProgrammersManual.html#dss-iplist).

Thu, Jan 7, 9:23 PM · Restricted Project
lanza added a comment to D93926: [lldb] Don't remove the lldb.debugger var after exiting the interpreter.

What's the boundaries of the stable API, then? This was a public API that was removed and broke a plugin I used for vim (and I'll be this isn't the only case, just I'm maybe the only one who has worked on lldb before whose tool broke). The author used threading.Thread(someFunc, (debugger,)) to listen on a socket for fetch requests from lldb outside of the prompt. Not the most beautiful of implementations, but it worked for years on top of a promised public stable API.

Thu, Jan 7, 9:15 PM · Restricted Project

Wed, Dec 30

lanza added a reviewer for D93953: NFC: Remove simple_ilist comment mentioning ilist/iplist allocating: dexonsmith.
Wed, Dec 30, 7:35 PM · Restricted Project
lanza requested review of D93953: NFC: Remove simple_ilist comment mentioning ilist/iplist allocating.
Wed, Dec 30, 7:34 PM · Restricted Project

Tue, Dec 29

lanza added a reviewer for D93926: [lldb] Don't remove the lldb.debugger var after exiting the interpreter: JDevlieghere.
Tue, Dec 29, 4:12 PM · Restricted Project
lanza updated the diff for D93926: [lldb] Don't remove the lldb.debugger var after exiting the interpreter.

did it backwards

Tue, Dec 29, 4:01 PM · Restricted Project
lanza requested review of D93926: [lldb] Don't remove the lldb.debugger var after exiting the interpreter.
Tue, Dec 29, 3:59 PM · Restricted Project

Tue, Dec 22

lanza added a comment to D93273: [CodeGen][ObjC] Destroy callee-destroyed arguments in the caller function when the receiver is nil.

Hey @ahatanak, I've ran into this problem in my companies projects and was wondering what the status of this patch is?

Tue, Dec 22, 4:24 PM · Restricted Project

Dec 9 2020

lanza added a comment to D92856: [libc++] Use TARGET_FILE instead of TARGET_SONAME_FILE when generating ABI lists.

Yup, this worked. Thank you, Louis! I can also verify that the test suite passed on centOS, Darwin and Windows with a pretty standard build.

Dec 9 2020, 1:58 PM · Restricted Project
lanza accepted D92856: [libc++] Use TARGET_FILE instead of TARGET_SONAME_FILE when generating ABI lists.
Dec 9 2020, 1:57 PM · Restricted Project
lanza closed D92705: [llvm] Update WinMsvc.cmake's fms-compatability to match llvm's prereqs.

Sorry, this was landed already but I forgot to put the Differential Revision: in! Closing it now. As far as the Windows changes you guys are talking about I'll leave up to you -- I'm not a Windows dev and was just maintaining a windows CI oncall last week.

Dec 9 2020, 1:13 PM · Restricted Project

Dec 4 2020

lanza committed rG54971c3a7559: [llvm] Update WinMsvc.cmake's fms-compatability to match llvm's prereqs (authored by lanza).
[llvm] Update WinMsvc.cmake's fms-compatability to match llvm's prereqs
Dec 4 2020, 4:35 PM
lanza updated subscribers of D92707: [AArch64][GlobalISel] Narrow 128-bit regs to 64-bit regs in emitTestBit.
Dec 4 2020, 4:29 PM · Restricted Project
lanza added a comment to D92707: [AArch64][GlobalISel] Narrow 128-bit regs to 64-bit regs in emitTestBit.

Thanks, Jessica!

Dec 4 2020, 4:25 PM · Restricted Project
lanza updated subscribers of D92707: [AArch64][GlobalISel] Narrow 128-bit regs to 64-bit regs in emitTestBit.
Dec 4 2020, 4:25 PM · Restricted Project
lanza added reviewers for D92705: [llvm] Update WinMsvc.cmake's fms-compatability to match llvm's prereqs: jfb, aganea.
Dec 4 2020, 3:57 PM · Restricted Project
lanza requested review of D92705: [llvm] Update WinMsvc.cmake's fms-compatability to match llvm's prereqs.
Dec 4 2020, 3:54 PM · Restricted Project

Dec 3 2020

lanza added a comment to D92541: [libcxx] Don't generate cxx-abilist for some sets of targets.

Seems pretty trivially that this just requires LIBCXX_INCLUDE_TESTS and LIBCXX_ENABLE_SHARED for Windows. The error is:

Dec 3 2020, 1:19 PM · Restricted Project

Dec 2 2020

lanza requested review of D92541: [libcxx] Don't generate cxx-abilist for some sets of targets.
Dec 2 2020, 9:27 PM · Restricted Project
lanza requested review of D92532: [IRSimilarity] Don't copy the Mapper for createCandidatesFromSuffixTree.
Dec 2 2020, 6:16 PM · Restricted Project
lanza removed a reviewer for D92524: [IRSimilarity] Strip out the findSimilarity call from the constructor: zer0.
Dec 2 2020, 5:27 PM · Restricted Project
lanza added a reviewer for D92524: [IRSimilarity] Strip out the findSimilarity call from the constructor: plotfi.
Dec 2 2020, 5:27 PM · Restricted Project
lanza added reviewers for D92524: [IRSimilarity] Strip out the findSimilarity call from the constructor: AndrewLitteken, paquette, zer0.
Dec 2 2020, 5:04 PM · Restricted Project
lanza requested review of D92524: [IRSimilarity] Strip out the findSimilarity call from the constructor.
Dec 2 2020, 4:38 PM · Restricted Project

Nov 30 2020

lanza accepted D91874: [GNU ObjC] Fix a regression listing methods twice..

LGTM!

Nov 30 2020, 11:49 AM · Restricted Project

Nov 20 2020

lanza added a reverting change for rG137ff7331705: [lldb] add a missing dependency on intrinsics_gen: rG33c79f76af75: Revert "[lldb] add a missing dependency on intrinsics_gen".
Nov 20 2020, 1:03 PM
lanza committed rG33c79f76af75: Revert "[lldb] add a missing dependency on intrinsics_gen" (authored by lanza).
Revert "[lldb] add a missing dependency on intrinsics_gen"
Nov 20 2020, 1:03 PM
lanza added a reverting change for D90857: [lldb] add a missing dependency on intrinsics_gen: rG33c79f76af75: Revert "[lldb] add a missing dependency on intrinsics_gen".
Nov 20 2020, 1:03 PM · Restricted Project

Nov 6 2020

lanza committed rG137ff7331705: [lldb] add a missing dependency on intrinsics_gen (authored by rmaz).
[lldb] add a missing dependency on intrinsics_gen
Nov 6 2020, 10:20 AM
lanza closed D90857: [lldb] add a missing dependency on intrinsics_gen.
Nov 6 2020, 10:19 AM · Restricted Project

Nov 5 2020

lanza added reviewers for D90857: [lldb] add a missing dependency on intrinsics_gen: JDevlieghere, xiaobai.
Nov 5 2020, 12:58 PM · Restricted Project

Oct 3 2020

lanza committed rGfcb0ab59335b: [clang][NFC] Change a mention of `objc_static_protocol` to `non_runtime` (authored by lanza).
[clang][NFC] Change a mention of `objc_static_protocol` to `non_runtime`
Oct 3 2020, 11:04 AM
lanza added inline comments to D75574: RFC: Implement objc_direct_protocol attribute to remove protocol metadata.
Oct 3 2020, 10:36 AM · Restricted Project

Oct 2 2020

lanza committed rG14f6bfcb52e7: [clang] Implement objc_non_runtime_protocol to remove protocol metadata (authored by lanza).
[clang] Implement objc_non_runtime_protocol to remove protocol metadata
Oct 2 2020, 2:38 PM
lanza closed D75574: RFC: Implement objc_direct_protocol attribute to remove protocol metadata.
Oct 2 2020, 2:38 PM · Restricted Project
lanza updated the diff for D75574: RFC: Implement objc_direct_protocol attribute to remove protocol metadata.

Clean clang-tidy warnings before landing

Oct 2 2020, 2:35 PM · Restricted Project
lanza updated the diff for D75574: RFC: Implement objc_direct_protocol attribute to remove protocol metadata.

Comments

Oct 2 2020, 11:05 AM · Restricted Project
lanza added inline comments to D75574: RFC: Implement objc_direct_protocol attribute to remove protocol metadata.
Oct 2 2020, 11:04 AM · Restricted Project

Oct 1 2020

lanza requested review of D75574: RFC: Implement objc_direct_protocol attribute to remove protocol metadata.
Oct 1 2020, 9:18 PM · Restricted Project
lanza updated the diff for D75574: RFC: Implement objc_direct_protocol attribute to remove protocol metadata.

Update with John's suggestions

Oct 1 2020, 9:12 PM · Restricted Project

Sep 24 2020

lanza added inline comments to D75574: RFC: Implement objc_direct_protocol attribute to remove protocol metadata.
Sep 24 2020, 5:22 PM · Restricted Project
lanza added a comment to D75574: RFC: Implement objc_direct_protocol attribute to remove protocol metadata.

Fixed and added a test under the REDUNDANCY prefix.

Sep 24 2020, 5:19 PM · Restricted Project
lanza updated the diff for D75574: RFC: Implement objc_direct_protocol attribute to remove protocol metadata.

Fix duplicate inheritance issue

Sep 24 2020, 5:17 PM · Restricted Project

Sep 23 2020

lanza added a comment to D75574: RFC: Implement objc_direct_protocol attribute to remove protocol metadata.

@theraven @rjmccall should be ready for review whenever you guys are ready!

Sep 23 2020, 11:46 PM · Restricted Project
lanza updated the diff for D75574: RFC: Implement objc_direct_protocol attribute to remove protocol metadata.

Update

Sep 23 2020, 11:45 PM · Restricted Project

Sep 16 2020

lanza added a comment to D75574: RFC: Implement objc_direct_protocol attribute to remove protocol metadata.

Hmm, I thought we actually just generated a bogus definition for the protocol when it was forward-declared; really, this is better behavior that I expected. Regardless, I don't think it's worthwhile to diagnose this more strongly than a warning because of the history of not doing so.

Sep 16 2020, 1:24 PM · Restricted Project

Sep 8 2020

lanza added a comment to D75574: RFC: Implement objc_direct_protocol attribute to remove protocol metadata.

I don't think it'll actually error out at link time: protocol objects get emitted eagerly on use, cross-module linking is just a code-size optimization. This actually has caused longstanding problems.

Sep 8 2020, 11:29 PM · Restricted Project
lanza added a comment to D75574: RFC: Implement objc_direct_protocol attribute to remove protocol metadata.

A concern that has come up while rewriting this for the listed concerns is forward declared protocols that are defined as non_runtime.

Sep 8 2020, 4:06 PM · Restricted Project

Sep 2 2020

lanza added a comment to D86952: [MIRVRegNamer] MachineInstr StableHashing..

Other than the one comment LGTM.

Sep 2 2020, 10:44 AM · Restricted Project

Aug 28 2020

lanza abandoned D86331: Fix a cmake failure on Windows when LLVM_ENABLE_TERMINFO is set.

Looks like it, I did this around the same time the patch landed. Thanks!

Aug 28 2020, 10:33 AM · Restricted Project

Aug 20 2020

lanza added a reviewer for D86331: Fix a cmake failure on Windows when LLVM_ENABLE_TERMINFO is set: phosek.
Aug 20 2020, 10:08 PM · Restricted Project
lanza requested review of D86331: Fix a cmake failure on Windows when LLVM_ENABLE_TERMINFO is set.
Aug 20 2020, 10:07 PM · Restricted Project

Aug 19 2020

lanza added a comment to D86049: RFC: Implement optional exportable wrapper function generation for objc_direct methods..

This change provides a codegen options flag to clang -fobjc-export-direct-method-wrappers to generate the wrapper functions that begin with the prefix objc_direct_wrapper and are marked as attribute__((alwaysinline)). This way within a link unit the wrapper functions should be inlined away at their call sites, but across a dylib boundary the wrapper call is used.

Aug 19 2020, 9:50 AM · Restricted Project

Aug 11 2020

lanza added a reviewer for D85586: Force Remove Attribute: jmolloy.

Adding jmolloy, he seems to be the original author.

Aug 11 2020, 3:22 PM · Restricted Project

Aug 4 2020

lanza added a comment to D75574: RFC: Implement objc_direct_protocol attribute to remove protocol metadata.

No problem! Thank you, John!

Aug 4 2020, 11:28 AM · Restricted Project

Aug 3 2020

lanza added a comment to D84269: [lldb] Add some example type anotations to python.swig.

Sounds good, just sent out a message to the mailing list.

Aug 3 2020, 3:11 PM · Restricted Project
lanza added a comment to D75574: RFC: Implement objc_direct_protocol attribute to remove protocol metadata.

ping @rjmccall. Any update on a timeline for this review process? Thanks!

Aug 3 2020, 2:12 PM · Restricted Project

Jul 21 2020

Herald added a project to D84269: [lldb] Add some example type anotations to python.swig: Restricted Project.
Jul 21 2020, 1:55 PM · Restricted Project

Apr 30 2020

lanza added a comment to D70885: [lldb] Use explicit lldb commands on tests.

Sorry for bump to this old diff, but I agree with both Jim and Greg -- we shouldn't be importing your ~/.lldbinit, but tests shouldn't depend on there never being another br s. This change should land as breakpoint set.

Apr 30 2020, 5:11 PM · Restricted Project
lanza added a comment to D75574: RFC: Implement objc_direct_protocol attribute to remove protocol metadata.

@rjmccall Hey John, I sent the proposal to the addresses I was pointed to but haven't heard back in multiple weeks. Any update on this?

Apr 30 2020, 5:11 PM · Restricted Project

Apr 14 2020

lanza closed D77106: [cmake] Only set deps for an ExternalProject if the type is executable or library.

Closed by https://reviews.llvm.org/rG7f5fe30a150e7e87d3fbe4da4ab0e76ec38b40b9

Apr 14 2020, 12:24 PM · Restricted Project

Apr 11 2020

lanza added a comment to D75574: RFC: Implement objc_direct_protocol attribute to remove protocol metadata.

If someone writes up a short proposal for this, with motivation and impact, we'd be happy to present it internally.

Apr 11 2020, 10:07 AM · Restricted Project

Apr 1 2020

lanza committed rG7f5fe30a150e: [cmake] Only set deps for an ExternalProject if the type is executable or… (authored by lanza).
[cmake] Only set deps for an ExternalProject if the type is executable or…
Apr 1 2020, 8:42 PM
lanza closed D77284: [cmake] Only set deps for an ExternalProject if the type is executable or library.
Apr 1 2020, 8:42 PM · Restricted Project
lanza created D77284: [cmake] Only set deps for an ExternalProject if the type is executable or library.
Apr 1 2020, 8:42 PM · Restricted Project
lanza updated the diff for D77106: [cmake] Only set deps for an ExternalProject if the type is executable or library.

fix message

Apr 1 2020, 11:49 AM · Restricted Project
lanza added inline comments to D77106: [cmake] Only set deps for an ExternalProject if the type is executable or library.
Apr 1 2020, 11:49 AM · Restricted Project
lanza updated the diff for D77106: [cmake] Only set deps for an ExternalProject if the type is executable or library.

update

Apr 1 2020, 11:49 AM · Restricted Project

Mar 30 2020

lanza added a reviewer for D77106: [cmake] Only set deps for an ExternalProject if the type is executable or library: xiaobai.
Mar 30 2020, 6:02 PM · Restricted Project
lanza added reviewers for D77106: [cmake] Only set deps for an ExternalProject if the type is executable or library: beanz, smeenai.
Mar 30 2020, 6:02 PM · Restricted Project
lanza created D77106: [cmake] Only set deps for an ExternalProject if the type is executable or library.
Mar 30 2020, 5:30 PM · Restricted Project

Mar 25 2020

lanza updated the diff for D75574: RFC: Implement objc_direct_protocol attribute to remove protocol metadata.

Reword commit

Mar 25 2020, 4:49 PM · Restricted Project
lanza updated the diff for D75574: RFC: Implement objc_direct_protocol attribute to remove protocol metadata.

Rename and address some issues

Mar 25 2020, 4:49 PM · Restricted Project

Mar 7 2020

lanza added a comment to D75574: RFC: Implement objc_direct_protocol attribute to remove protocol metadata.

Adding some more knowledgeable reviewers for comments on your RFC. I pointed out a few minor nits, but I'll hold off on a technical review until the ObjC-specific details are worked out and there is buy-in on the feature.

Mar 7 2020, 3:09 PM · Restricted Project

Mar 3 2020

lanza created D75574: RFC: Implement objc_direct_protocol attribute to remove protocol metadata.
Mar 3 2020, 3:00 PM · Restricted Project

Jan 8 2020

lanza abandoned D63526: [NativePDB] Read inc-link thunks from /* LINKER */ module and create symbols.
Jan 8 2020, 1:00 PM

Dec 18 2019

lanza abandoned D64692: [NativePDB] Change GetOrCreateFunctionDecl to private and introduce CreateFunctionDecl.
Dec 18 2019, 4:33 PM
lanza abandoned D64691: [NativePDB] Make GetParentDeclContext return an lldb type.
Dec 18 2019, 4:33 PM
lanza abandoned D64690: [NativePDB] Make GetOrCreateDeclContextForUid return an lldb type.
Dec 18 2019, 4:33 PM
lanza closed D71034: [lldb-vscode] Added support for ‘totalFrames’ field in StackTraces response .

This was landed.

Dec 18 2019, 4:33 PM · Restricted Project
lanza abandoned D64693: [NativePDB] Change GetOrCreateBlockDecl to private and introduce CreateBlockDecl.
Dec 18 2019, 4:33 PM
lanza added a comment to D71633: [lldb-vscode] Only close the debuggers in/out when DAP is over stdin/out.

Great point! Fixed.

Dec 18 2019, 11:26 AM · Restricted Project
lanza updated the diff for D71633: [lldb-vscode] Only close the debuggers in/out when DAP is over stdin/out.

fixup according to Greg's requests

Dec 18 2019, 11:26 AM · Restricted Project

Dec 17 2019

lanza created D71633: [lldb-vscode] Only close the debuggers in/out when DAP is over stdin/out.
Dec 17 2019, 3:34 PM · Restricted Project

Dec 9 2019

lanza committed rG6dad5e441db5: The field ‘totalFrames’ which is total number of frames available, is mandatory… (authored by serhiy.redko).
The field ‘totalFrames’ which is total number of frames available, is mandatory…
Dec 9 2019, 10:53 AM

Dec 5 2019

lanza added a comment to D71034: [lldb-vscode] Added support for ‘totalFrames’ field in StackTraces response .

LGTM. Any concerns @clayborg?

Dec 5 2019, 1:25 PM · Restricted Project

Sep 4 2019

lanza committed rL370978: Request commit access for lanza.
Request commit access for lanza
Sep 4 2019, 1:46 PM

Aug 27 2019

lanza committed rL370106: [Platform/Android] Read the adb server from an env variable if set.
[Platform/Android] Read the adb server from an env variable if set
Aug 27 2019, 1:19 PM
lanza committed rG0c01d920512c: [Platform/Android] Read the adb server from an env variable if set (authored by lanza).
[Platform/Android] Read the adb server from an env variable if set
Aug 27 2019, 1:17 PM
lanza closed D66689: [Platform/Android] Read the adb server from an env variable if set.
Aug 27 2019, 1:14 PM · Restricted Project

Aug 23 2019

lanza created D66689: [Platform/Android] Read the adb server from an env variable if set.
Aug 23 2019, 4:05 PM · Restricted Project

Jul 23 2019

lanza accepted D65155: [lldb] Remove Xcode project legacy.

Not directly related, but there's some other legacy cruft I figure are valid for removal. e.g.

utils/sync-source
misc/grep-svn-log.py
git-svn/convert.py
Jul 23 2019, 9:44 AM · Restricted Project, Restricted Project

Jul 22 2019

lanza added inline comments to D65109: [LLDB] Remove the Xcode project.
Jul 22 2019, 1:26 PM · Restricted Project, Restricted Project

Jul 21 2019

lanza closed D64689: [NativePDB] Make GetOrCreateDeclForUid return an lldb CompilerDecl.
Jul 21 2019, 1:45 AM