[PATCH] Make asan_symbolize.py not crash on Windows.

Nico Weber thakis at chromium.org
Wed Jan 28 09:18:04 PST 2015


I'll land the indent changes separately, removed that from this change.

In http://reviews.llvm.org/D7227#114653, @earthdok wrote:

> Wouldn't it be better to do this in the Chromium-specific wrapper (src/tools/valgrind/asan_symbolize.py)?


I moved it into SymbolizationLoop, so that the specific wrapper will do the right thing automatically.


================
Comment at: lib/asan/scripts/asan_symbolize.py:474
@@ +473,3 @@
+    for line in logfile:
+      print line,
+    sys.exit(0)
----------------
glider wrote:
> Nit: spare comma?
No, a traiilng comma in a print line makes print not add a \n. This is intentional because line already ends in a \n.

================
Comment at: lib/asan/scripts/asan_symbolize.py:443
@@ +442,3 @@
+    # in sandboxed processes.  This script isn't needed there.
+    sys.exit(0)
+
----------------
err actually, I suppose I should copy stdin to stdout too

http://reviews.llvm.org/D7227

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list