[PATCH] [ASan] Allow the users of SymbolizationLoop to make use of the -dsym_hint option in llvm-symbolizer

Sergey Matveev earthdok at google.com
Fri Nov 21 05:38:04 PST 2014


================
Comment at: projects/compiler-rt/lib/asan/scripts/asan_symbolize.py:364
@@ +363,3 @@
+    #     |last_llvm_symbolizer|);
+    #  2. otherwise check if we've seen this hint already; if so,
+    #     reuse |last_llvm_symbolizer| which has the full set of hints;
----------------
s/this hint/all of the hints for this binary/

================
Comment at: projects/compiler-rt/lib/asan/scripts/asan_symbolize.py:371
@@ +370,3 @@
+      if self.system == 'Darwin' and self.dsym_hint_producer:
+        dsym_hints = self.dsym_hint_producer(binary)
+        for dsym_hint in dsym_hints:
----------------
Whoa whoa, let's tune the C++ down a bit :P

```
dsym_hints = set(self.dsym_hint_producer(binary))
use_last_symbolizer = bool(dsym_hints - self.dsym_hints)
self.dsym_hints |= dsym_hints
```

Also I suggest to rename this to dsym_hints_for_binary.

http://reviews.llvm.org/D6310






More information about the llvm-commits mailing list