[Lldb-commits] [PATCH] D92264: [lldb] [POSIX-DYLD] Update the cached exe path after attach

2020-11-28 Thread Michał Górny via Phabricator via lldb-commits
mgorny created this revision. mgorny added reviewers: labath, emaste, krytarowski. mgorny requested review of this revision. Fix the POSIX-DYLD plugin to update the cached executable path after attaching. Previously, the path was cached in DYLDRendezvous constructor and not updated afterwards. T

[Lldb-commits] [PATCH] D88247: Fix memory leak in SBValue::GetAddress

2020-11-28 Thread Andy Yankovsky via Phabricator via lldb-commits
werat abandoned this revision. werat added a comment. Herald added a subscriber: pengfei. Refactored in D88249 to avoid this problem altogether. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D88247/new/ https://rev

[Lldb-commits] [PATCH] D92103: [ASTImporter] Import the default argument of TemplateTypeParmDecl

2020-11-28 Thread Endre Fülöp via Phabricator via lldb-commits
gamesh411 added a comment. On bitcoin v0.18.1 , there is an assertion introduced by this change. The TU that can be used for reproduction is `src/script/interpreter.cpp`. Assertion message: CTU loaded AST file: /home/gamesh411/bitcoin/src/script

[Lldb-commits] [PATCH] D86671: [clang-tidy] Add new case type to check variables with Hungarian notation

2020-11-28 Thread Aaron Ballman via Phabricator via lldb-commits
aaron.ballman added inline comments. Comment at: clang-tools-extra/clang-tidy/readability/IdentifierNamingCheck.cpp:237 +std::string Val = HNOption.General.lookup(Opt.first); +if (Val.empty()) { + HNOption.General.insert({Opt.first, Opt.second.str()}); -

[Lldb-commits] [PATCH] D91713: [libomptarget] Implement get_device_num for amdgcn, nvptx

2020-11-28 Thread Jon Chesterfield via Phabricator via lldb-commits
JonChesterfield updated this revision to Diff 307215. JonChesterfield added a comment. Herald added subscribers: llvm-commits, libc-commits, libcxx-commits, lldb-commits, Sanitizers, cfe-commits, mravishankar, teijeong, frasercrmck, dexonsmith, awarzynski, rdzhabarov, ecnelises, tatianashp, wenle

[Lldb-commits] [PATCH] D91734: [FastISel] Flush local value map on every instruction

2020-11-28 Thread Paul Robinson via Phabricator via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rGcf1c774d6ace: [FastISel] Flush local value map on ever instruction (authored by probinson). Herald added a project: LLDB. Herald added a subscriber: lldb-commits. Changed prior to commit: https://review

[Lldb-commits] [PATCH] D86292: [LLDB][RISCV] Add RISC-V ArchSpec and rv32/rv64 variant detection

2020-11-28 Thread Luís Marques via Phabricator via lldb-commits
luismarques updated this revision to Diff 307360. luismarques added a comment. Merge tests, using `--docnum`. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D86292/new/ https://reviews.llvm.org/D86292 Files: lldb/include/lldb/Utility/ArchSpec.h