[Lldb-commits] [PATCH] D111715: [WIP] [lldb] change name demangling to be consistent between windows and linx

2021-10-16 Thread Nico Weber via Phabricator via lldb-commits
thakis added a comment. I don't have an opinion on this change and I don't mind the demangler change, but isn't the type information helpful? The mangled itanium name doesn't include type information which is why it's not printed, but the mangled ms name does include it. But as I said, I don't

[Lldb-commits] [PATCH] D111632: [lldb] Split ParseSingleMember into Obj-C property and normal member/ivar parsing code.

2021-10-16 Thread Raphael Isemann via Phabricator via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG60b96aa65e59: [lldb] Split ParseSingleMember into Obj-C property and normal member/ivar… (authored by teemperor). Herald added a subscriber: lldb-commits. Changed prior to commit: https://reviews.llvm.o

[Lldb-commits] [lldb] 60b96aa - [lldb] Split ParseSingleMember into Obj-C property and normal member/ivar parsing code.

2021-10-16 Thread Raphael Isemann via lldb-commits
Author: Raphael Isemann Date: 2021-10-16T14:20:04+02:00 New Revision: 60b96aa65e5959361e9edea15b3591f90988ccad URL: https://github.com/llvm/llvm-project/commit/60b96aa65e5959361e9edea15b3591f90988ccad DIFF: https://github.com/llvm/llvm-project/commit/60b96aa65e5959361e9edea15b3591f90988ccad.dif

[Lldb-commits] [PATCH] D111715: [WIP] [lldb] change name demangling to be consistent between windows and linx

2021-10-16 Thread Lasse Folger via Phabricator via lldb-commits
lassefolger added a comment. created a NFC for formatting: https://reviews.llvm.org/D111934 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D111715/new/ https://reviews.llvm.org/D111715 ___ lldb-commits mai

[Lldb-commits] [PATCH] D111934: [NFC] clang format change

2021-10-16 Thread Lasse Folger via Phabricator via lldb-commits
lassefolger created this revision. lassefolger added reviewers: teemperor, werat. lassefolger requested review of this revision. Herald added projects: LLDB, LLVM. Herald added subscribers: llvm-commits, lldb-commits. clang format on some demangling files Repository: rG LLVM Github Monorepo h

[Lldb-commits] [PATCH] D111715: [WIP] [lldb] change name demangling to be consistent between windows and linx

2021-10-16 Thread Lasse Folger via Phabricator via lldb-commits
lassefolger updated this revision to Diff 380155. lassefolger added a comment. add tests for the change and adjust description to include examples Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D111715/new/ https://reviews.llvm.org/D111715 Files:

[Lldb-commits] [PATCH] D111908: [lldb] [Utility] Remove Status::WasInterrupted() along with its only use

2021-10-16 Thread Michał Górny via Phabricator via lldb-commits
mgorny added a comment. FTR, this doesn't make `^c` work any better or worse. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D111908/new/ https://reviews.llvm.org/D111908 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.l

[Lldb-commits] [PATCH] D111491: [lldb] [gdb-remote] Remove HardcodeARMRegisters() hack

2021-10-16 Thread Michał Górny via Phabricator via lldb-commits
mgorny added a comment. Gentle ping. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D111491/new/ https://reviews.llvm.org/D111491 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-com