[PATCH] D139859: [llvm-symbolize] Allow finding location of a symbol

[Thorsten via Phabricator via llvm-commits]

tschuett added inline comments.


================
Comment at: llvm/include/llvm/DebugInfo/DWARF/DWARFContext.h:91
 
+  Optional<StringMap<std::vector<DILineInfo>>> SymbolLocations;
+
----------------
Please use `std::optional`. The LLVM variant is going away.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D139859/new/

https://reviews.llvm.org/D139859