Changes

[lldb] NFC: less nesting in SearchFilter.cpp
I was working on SearchFilter.cpp and felt it a bit too complex in some
cases in terms of nesting and logic flow.
Reviewers: teemperor, JDevlieghere
Subscribers: lldb-commits
Tags: #lldb
Differential Revision: https://reviews.llvm.org/D71022

[lldb][NFC] Migrate FileSpec::Dump to raw_ostream

[clangd] (take 2) Try harder to find a plausible `clang` as argv0,
particularly on Mac.
Summary: This was originally committed in
88bccded8fa169481fa367debf5ec615640635a1, and reverted in
93f77617abba512d2861e2fc50ce385883f587b6.
This version is now much more testable: the "detect toolchain
properties" part is still not tested but also not active in tests. All
the command manipulation based on the detected properties is directly
tested, and also not active in other tests.
Fixes https://github.com/clangd/clangd/issues/211 Fixes
https://github.com/clangd/clangd/issues/178
Reviewers: kbobyrev, ilya-biryukov
Subscribers: mgorny, ormris, cfe-commits, usaxena95, kadircet, arphaman,
jkorous, MaskRay
Tags: #clang
Differential Revision: https://reviews.llvm.org/D71029

gn build: Merge 99768b243cd

[clang-tidy] Fix PR26274
Summary: This commit fixes http://llvm.org/PR26274 in a simpler and more
correct way than 4736d63f752f8d13f4c6a9afd558565c32119718 did. See 
https://reviews.llvm.org/D69855#1767089 for details.
Reviewers: gribozavr, aaron.ballman, gribozavr2
Reviewed By: aaron.ballman, gribozavr2
Subscribers: gribozavr2, merge_guards_bot, xazax.hun, cfe-commits
Tags: #clang
Differential Revision: https://reviews.llvm.org/D70974

[lldb/DWARF] Remove DWARFDebugRangesBase abstract class
now that we use llvm to parse debug_rnglists, this abstraction is not 
useful.

[lldb/IRExecutionUnit] Stop searching based on demangled names
Summary: This was causing problems on linux, where we'd end up calling
the deleting destructor instead of a regular one (because they have the
same demangled name), making a lot of mischief in the process.
The only place where this was necessary (according to the test suite, at 
least) was to call a base structor instead of a complete one, but this 
is now handled in a more targeted fashion.
TestCallOverriddenMethod is now re-enabled as it now passes reliably.
Reviewers: teemperor, JDevlieghere
Subscribers: lldb-commits
Tags: #lldb
Differential Revision: https://reviews.llvm.org/D70722

[lldb] Migrate VMRange::Dump to raw_ostream

[lldb/DWARF] Fix DW_AT_rnglists_base handling for dwo files
the value of DW_AT_rnglists_base of the skeleton unit is for that unit 
alone (e.g. used in DW_AT_ranges of the unit DIE) and should not apply 
to the split unit.
The split unit has a hardcoded range list base value -- we should 
initialize range list code whenever we detect a nonempty 
debug_rnglists.dwo section.

[lldb][NFC] Remove ability to pass a custom printf format to
DataExtractor::PutToLog
This is luckily not used anywhere.

[lldb/DWARF] Fix DW_AT_addr_base & DW_AT_low_pc interaction
In DWARF5 DW_AT_low_pc (and DW_AT_entry_pc, and possibly others) can use 
DW_FORM_addrx to refer to the address indirectly. This means we need to 
have processed the DW_AT_addr_base attribute before we can do anything 
with these.
Since we were processing the unit attributes serially, this created a 
problem in cases where the DW_AT_addr_base comes after DW_AT_low_pc -- 
we would end up computing the wrong unit base address, which also 
corrupted any values which later depended on that (for instance range 
lists). Clang currently always emits DW_AT_addr_base last.
The fix is simple -- process DW_AT_addr_base first, regardless of its 
position in the attribute list.