caio/clang-p2996
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Remove weak-linked symbols for SBBreakpointListImpl

Browse Source 
Summary:
Similar to SBStructuredData's Impl class, SBBreakpointListImpl was
getting weak-link exported in the lldb namespace. This change list fixes
that by moving out of the lldb public namespace, which removes it from
public export visibility.

Fixes:
rdar://28960344

Reviewers: jingham

Subscribers: lldb-commits

Differential Revision: https://reviews.llvm.org/D26553

llvm-svn: 286631
This commit is contained in:
Todd Fiala
2016-11-11 21:06:40 +00:00
parent cbd54304a3
commit 653e3f4e19
3 changed files with 8 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
lldb/source/API/SBBreakpoint.cpp
View File

@@ -712,11 +712,11 @@ SBBreakpoint::GetNumBreakpointLocationsFromEvent(const lldb::SBEvent &event) {
}
// This is simple collection of breakpoint id's and their target.
class lldb::SBBreakpointListImpl {
class SBBreakpointListImpl {
public:
SBBreakpointListImpl(SBTarget &target) : m_target_wp() {
if (target.IsValid())
m_target_wp = target.GetSP();
SBBreakpointListImpl(lldb::TargetSP target_sp) : m_target_wp() {
if (target_sp && target_sp->IsValid())
m_target_wp = target_sp;
}
~SBBreakpointListImpl() = default;
@@ -796,7 +796,7 @@ private:
};
SBBreakpointList::SBBreakpointList(SBTarget &target)
: m_opaque_sp(new lldb::SBBreakpointListImpl(target)) {}
: m_opaque_sp(new SBBreakpointListImpl(target.GetSP())) {}
SBBreakpointList::~SBBreakpointList() {}