[Lldb-commits] [PATCH] D67376: [DWARF] Evaluate DW_OP_entry_value
    Adrian Prantl via Phabricator via lldb-commits 
    lldb-commits at lists.llvm.org
       
    Tue Sep 10 08:41:43 PDT 2019
    
    
  
aprantl added inline comments.
================
Comment at: lldb/source/Plugins/SymbolFile/DWARF/SymbolFileDWARF.cpp:3752
+        LocationInCaller = parse_simple_location(i);
+        break;
+      }
----------------
vsk wrote:
> aprantl wrote:
> > default?
> I don't believe any action is needed in the default case: we do not want to log or report an error.
Doesn't this trigger the switch-doesn't-cover-all-cases warning otherwise?
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D67376/new/
https://reviews.llvm.org/D67376
    
    
More information about the lldb-commits
mailing list