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

Pavel Labath via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Mon Dec 5 01:49:46 PST 2016


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
+            (arch.GetTriple().isOSBinFormatELF() ||
              arch.GetMachine() == llvm::Triple::hexagon)) {
           SymbolContextList temp_sc_list;
----------------
I think we can remove the hexagon part as well. Ted, I presume hexagon uses ELF files?


Repository:
  rL LLVM

https://reviews.llvm.org/D27380





More information about the lldb-commits mailing list