message: [SBBreakpoint] Outline some functions to prevent to be exported.
  They're hidden, so all they cause is a linker warning.
  ld: warning: cannot export hidden symbol
  lldb::SBBreakpointNameImpl::operator==(lldb::SBBreakpointNameImpl const&) from
  tools/lldb/source/API/CMakeFiles/liblldb.dir/SBBreakpointName.cpp.o
diff: @@ -37,24 +37,7 @@ namespace lldb
  {
  class SBBreakpointNameImpl {
  public:
  - SBBreakpointNameImpl(SBTarget &sb_target, const char *name)
  - {
  - if (!name || name[0] == '\0')
  - return;
  - m_name.assign(name);
  -
  - if (!sb_target.IsValid())
  - return;
  -
  - TargetSP target_sp = sb_target.GetSP();
  - if (!target_sp)
  - return;
  -
  - m_target_wp = target_sp;
  - }
  -
  - SBBreakpointNameImpl(TargetSP target_sp, const char *name)
  - {
  + SBBreakpointNameImpl(TargetSP target_sp, const char *name) {
  if (!name || name[0] == '\0')
  return;
  m_name.assign(name);
  @@ -65,15 +48,10 @@ public:
  m_target_wp = target_sp;
  }
  - bool operator==(const SBBreakpointNameImpl &rhs) {
  - return m_name == rhs.m_name
  - && m_target_wp.lock() == rhs.m_target_wp.lock();
  - }
  + SBBreakpointNameImpl(SBTarget &sb_target, const char *name);
  + bool operator==(const SBBreakpointNameImpl &rhs);
  + bool operator!=(const SBBreakpointNameImpl &rhs);
  - bool operator!=(const SBBreakpointNameImpl &rhs) {
  - return m_name != rhs.m_name
  - || m_target_wp.lock() != rhs.m_target_wp.lock();
  - }
  // For now we take a simple approach and only keep the name, and relook
  // up the location when we need it.
  @@ -89,19 +67,38 @@ public:
  return !m_name.empty() && m_target_wp.lock();
  }
  - lldb_private::BreakpointName *GetBreakpointName()
  - {
  - if (!IsValid())
  - return nullptr;
  - TargetSP target_sp = GetTarget();
  + lldb_private::BreakpointName *GetBreakpointName() const;
  +
  +private:
  + TargetWP m_target_wp;
  + std::string m_name;
  +};
  +
  +SBBreakpointNameImpl::SBBreakpointNameImpl(SBTarget &sb_target,
  + const char *name) {
  + if (!name || name[0] == '\0')
  + return;
  + m_name.assign(name);
  +
  + if (!sb_target.IsValid())
  + return;
  +
  + TargetSP target_sp = sb_target.GetSP();
  if (!target_sp)
  - return nullptr;
  - Status error;
  - return target_sp->FindBreakpointName(ConstString(m_name), true, error);
  + return;
  +
  + m_target_wp = target_sp;
  }
  - const lldb_private::BreakpointName *GetBreakpointName() const
  - {
  +bool SBBreakpointNameImpl::operator==(const SBBreakpointNameImpl &rhs) {
  + return m_name == rhs.m_name && m_target_wp.lock() == rhs.m_target_wp.lock();
  +}
  +
  +bool SBBreakpointNameImpl::operator!=(const SBBreakpointNameImpl &rhs) {
  + return m_name != rhs.m_name || m_target_wp.lock() != rhs.m_target_wp.lock();
  +}
  +
  +lldb_private::BreakpointName *SBBreakpointNameImpl::GetBreakpointName() const {
  if (!IsValid())
  return nullptr;
  TargetSP target_sp = GetTarget();
  @@ -111,10 +108,6 @@ public:
  return target_sp->FindBreakpointName(ConstString(m_name), true, error);
  }
  -private:
  - TargetWP m_target_wp;
  - std::string m_name;
  -};
  } // namespace lldb
  SBBreakpointName::SBBreakpointName() {}
  