[Lldb-commits] [PATCH] D27380: [lldb] Update the check for Linux or FreeBSD in SymbolFileDWARF::FindFunctions.

2016-12-05 Thread Alexander Shaposhnikov via Phabricator via lldb-commits
alexshap updated this revision to Diff 80278. alexshap added a comment. Address comments Repository: rL LLVM https://reviews.llvm.org/D27380 Files: source/Plugins/ObjectFile/ELF/ObjectFileELF.cpp source/Plugins/SymbolFile/DWARF/SymbolFileDWARF.cpp Index:

[Lldb-commits] [PATCH] D27380: [lldb] Update the check for Linux or FreeBSD in SymbolFileDWARF::FindFunctions.

2016-12-05 Thread Pavel Labath via Phabricator via lldb-commits
labath added a subscriber: ted. labath added a comment. It looks reasonable to me, but I don't know the full context, so I'd wait until others can comment on this as well. Comment at: source/Plugins/SymbolFile/DWARF/SymbolFileDWARF.cpp:2590 +

[Lldb-commits] [PATCH] D27380: [lldb] Update the check for Linux or FreeBSD in SymbolFileDWARF::FindFunctions.

2016-12-02 Thread Alexander Shaposhnikov via Phabricator via lldb-commits
alexshap created this revision. alexshap added reviewers: clayborg, spyffe. alexshap added a subscriber: lldb-commits. alexshap set the repository for this revision to rL LLVM. This diff 1. Adds a comment to ObjectFileELF.cpp about the current approach to determining the OS. 2. Replaces the