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



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to