[Lldb-commits] [PATCH] D65864: Remove Module::GetSymbolVendor

2019-08-08 Thread Pavel Labath via Phabricator via lldb-commits
This revision was automatically updated to reflect the committed changes. labath marked an inline comment as done. Closed by commit rL368263: Remove Module::GetSymbolVendor (authored by labath, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to

[Lldb-commits] [PATCH] D65864: Remove Module::GetSymbolVendor

2019-08-08 Thread Pavel Labath via Phabricator via lldb-commits
labath marked 2 inline comments as done. labath added inline comments. Comment at: include/lldb/Core/Module.h:1006 std::atomic m_did_load_objfile{false}; - std::atomic m_did_load_symbol_vendor{false}; + std::atomic m_did_load_symbol_file{false}; std::atomic m_did_set_uuid

[Lldb-commits] [PATCH] D65864: Remove Module::GetSymbolVendor

2019-08-07 Thread Greg Clayton via Phabricator via lldb-commits
clayborg accepted this revision. clayborg added a comment. Optional rename in inline comments, but looks good. Comment at: include/lldb/Core/Module.h:1006 std::atomic m_did_load_objfile{false}; - std::atomic m_did_load_symbol_vendor{false}; + std::atomic m_did_load_symbol_

[Lldb-commits] [PATCH] D65864: Remove Module::GetSymbolVendor

2019-08-07 Thread Pavel Labath via Phabricator via lldb-commits
labath created this revision. labath added reviewers: clayborg, JDevlieghere, jingham. Herald added a subscriber: jfb. Herald added a reviewer: jdoerfert. This patch removes the GetSymbolVendor function, and the various mentions of the SymbolVendor in the Module class. The implementation of GetSym