[Lldb-commits] [PATCH] D140615: [LLDB][LoongArch] Delete the s9 register alias definition

2022-12-23 Thread Hui Li via Phabricator via lldb-commits
lh03061238 created this revision. lh03061238 added reviewers: SixWeining, wangleiat, xen0n, xry111, MaskRay, seehearfeel, DavidSpickett. Herald added subscribers: jeroen.dobbelaere, StephenFan. Herald added a project: All. lh03061238 requested review of this revision. Herald added a project: LLDB.

[Lldb-commits] [PATCH] D140616: [LLDB][LoongArch] Optimize EmulateInstructionLoongArch related code

2022-12-23 Thread Hui Li via Phabricator via lldb-commits
lh03061238 created this revision. lh03061238 added reviewers: SixWeining, wangleiat, xen0n, xry111, MaskRay, seehearfeel, DavidSpickett. Herald added a subscriber: StephenFan. Herald added a project: All. lh03061238 requested review of this revision. Herald added a project: LLDB. Herald added a su

[Lldb-commits] [PATCH] D140616: [LLDB][LoongArch] Optimize EmulateInstructionLoongArch related code

2022-12-23 Thread Lu Weining via Phabricator via lldb-commits
SixWeining added inline comments. Comment at: lldb/source/Plugins/Instruction/LoongArch/EmulateInstructionLoongArch.cpp:202 bool EmulateInstructionLoongArch::EmulateBEQZ(uint32_t inst) { - if (IsLoongArch64()) -return EmulateBEQZ64(inst); - else -return false; + retu

[Lldb-commits] [PATCH] D140623: [LLDB] Fix for libc++ atomic allowing modification of contained value

2022-12-23 Thread Pavel Kosov via Phabricator via lldb-commits
kpdev42 created this revision. kpdev42 added reviewers: DavidSpickett, clayborg, k8stone, davide. kpdev42 added a project: LLDB. Herald added a subscriber: JDevlieghere. Herald added a project: All. kpdev42 requested review of this revision. Herald added a subscriber: lldb-commits. Repository: r

[Lldb-commits] [PATCH] D140624: [LLDB] Fix for libc++ map allowing modification of contained value

2022-12-23 Thread Pavel Kosov via Phabricator via lldb-commits
kpdev42 created this revision. kpdev42 added reviewers: davide, DavidSpickett, clayborg, k8stone. kpdev42 added a project: LLDB. Herald added a subscriber: JDevlieghere. Herald added a project: All. kpdev42 requested review of this revision. Herald added a subscriber: lldb-commits. Repository: r

[Lldb-commits] [PATCH] D140630: [lldb-vscode] Add data breakpoint support

2022-12-23 Thread Callum Macmillan via Phabricator via lldb-commits
cimacmillan created this revision. Herald added a project: All. cimacmillan requested review of this revision. Herald added a project: LLDB. Herald added a subscriber: lldb-commits. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D140630 Files: lldb/packages/Python/lldbsuite/tes

[Lldb-commits] [PATCH] D139379: [llvm][dwwarf] Change CU/TU index to 64-bit

2022-12-23 Thread David Blaikie via Phabricator via lldb-commits
dblaikie added a comment. In D139379#3972876 , @ayermolo wrote: > In D139379#3972871 , @dblaikie > wrote: > >> Perhaps the change to use accessors could be removed, now that you've used >> it to find all the pla

[Lldb-commits] [PATCH] D139379: [llvm][dwwarf] Change CU/TU index to 64-bit

2022-12-23 Thread Alexander Yermolovich via Phabricator via lldb-commits
ayermolo updated this revision to Diff 485156. ayermolo added a comment. Removed generic setters for Index, and changed back to variables Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D139379/new/ https://reviews.llvm.org/D139379 Files: bolt/lib/

[Lldb-commits] [PATCH] D139379: [llvm][dwwarf] Change CU/TU index to 64-bit

2022-12-23 Thread Alexander Yermolovich via Phabricator via lldb-commits
ayermolo added a comment. In D139379#4015569 , @dblaikie wrote: > In D139379#3972876 , @ayermolo > wrote: > >> In D139379#3972871 , @dblaikie >> wrote: >> >>> Perhaps th

[Lldb-commits] [PATCH] D139379: [llvm][dwwarf] Change CU/TU index to 64-bit

2022-12-23 Thread Alexander Yermolovich via Phabricator via lldb-commits
ayermolo updated this revision to Diff 485160. ayermolo added a comment. rebase Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D139379/new/ https://reviews.llvm.org/D139379 Files: bolt/lib/Core/DebugData.cpp bolt/lib/Rewrite/DWARFRewriter.cpp

[Lldb-commits] [PATCH] D140616: [LLDB][LoongArch] Optimize EmulateInstructionLoongArch related code

2022-12-23 Thread Hui Li via Phabricator via lldb-commits
lh03061238 added inline comments. Comment at: lldb/source/Plugins/Instruction/LoongArch/EmulateInstructionLoongArch.cpp:202 bool EmulateInstructionLoongArch::EmulateBEQZ(uint32_t inst) { - if (IsLoongArch64()) -return EmulateBEQZ64(inst); - else -return false; + retu

[Lldb-commits] [PATCH] D140616: [LLDB][LoongArch] Optimize EmulateInstructionLoongArch related code

2022-12-23 Thread Hui Li via Phabricator via lldb-commits
lh03061238 updated this revision to Diff 485178. lh03061238 added a comment. Change (IsLoongArch64()) to IsLoongArch64() in some functions CHANGES SINCE LAST ACTION https://reviews.llvm.org/D140616/new/ https://reviews.llvm.org/D140616 Files: lldb/source/Plugins/Instruction/LoongArch/Emula