Add -address option to symbolizer

[khemant]

Summary

When llvm-symbolizer is passed a list of address, its nice to have an option which will print address corresponding line information. This improves human readability.

[samsonov]

Please see http://llvm.org/docs/Phabricator.html on how to upload patches with more context.

I don't understand why you need to pass the value of "-address" option down to LLVMSymbolizer class - you can just handle it inside main(). I would also prefer option name to be more verbose like "-print-address".

[samsonov]

test/tools/llvm-symbolizer/addr.test

1 ↗

(On Diff #36755)

Where did you get addr.exe from? I'd prefer to either use existing executable file, or commit the source file with description of build procedure. (the former is better).

[khemant]

You are right. I did not pay much attention to the driver.

[khemant]

Option is handled inside main. Source is present as comment in test case.

[samsonov]

LGTM

test/tools/llvm-symbolizer/sym.test

18

Please watch the Windows bots after the commit - you may need to relax slashes into "slash or backslash" here.

docs/CommandGuide/llvm-symbolizer.rst

  location, look for the debug info at the .dSYM path provided via the
  ``-dsym-hint`` flag. This flag can be used multiple times.
 
+.. option:: -print-address
+ Print address before the source code location. Defaults to false.
 
 EXIT STATUS
 -----------

test/tools/llvm-symbolizer/Inputs/addr.inp

+0x40054d

test/tools/llvm-symbolizer/sym.test

+#Source:
+##include <stdio.h>
+#static inline int inc (int *a) {
+#  printf ("%d\n",(*a)++);
+#  return (*a)++;
+#}
+#
+#int main () {
+#  int x = 1;
+#  return inc(&x);
+#}
+#Build as : clang -g -O2 addr.c
+
+RUN: llvm-symbolizer -inlining -print-address -obj=%p/Inputs/addr.exe < %p/Inputs/addr.inp | FileCheck %s
+
+#CHECK: 0x40054d
+#CHECK: main
+#CHECK: /prj/qct/sunray-austin/scratch/khemant/x.c:9:0
+

tools/llvm-symbolizer/llvm-symbolizer.cpp

 ClDsymHint("dsym-hint", cl::ZeroOrMore,
            cl::desc("Path to .dSYM bundles to search for debug info for the "
                     "object files"));
+static cl::opt<bool>
+    ClPrintAddress("print-address", cl::init(false),
+                   cl::desc("Show address before line information"));
 
 static bool parseCommand(bool &IsData, std::string &ModuleName,
                          uint64_t &ModuleOffset) {

     std::string Result =
         IsData ? Symbolizer.symbolizeData(ModuleName, ModuleOffset)
                : Symbolizer.symbolizeCode(ModuleName, ModuleOffset);
+    if (ClPrintAddress) {
+      outs() << "0x";
+      outs().write_hex(ModuleOffset);
+      outs() << "\n";
+    }
     outs() << Result << "\n";
     outs().flush();
   }