[PATCH] D60067: [llvm-symbolizer] Add llvm-addr2line.

[James Henderson via Phabricator via llvm-commits]

jhenderson added inline comments.


================
Comment at: lib/DebugInfo/Symbolize/DIPrinter.cpp:85-86
+    OS << Filename << ":" << Info.Line;
+    if (!OutputStyleGNU)
+      OS << ":" << Info.Column;
+    OS << "\n";
----------------
Thinking about it, it might be nice if this was under a new dedicated switch, that can be individually be enabled and disabled, so that llvm-addr2line users can print that without changing the other defaults.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D60067