[PATCH] Make llvm-symbolizer work on Windows.

Zachary Turner zturner at google.com
Fri Apr 24 11:26:35 PDT 2015


I've addressed all of the comments except for the RAII com initialization comment.  I think that can happen as a separate CL, since it will need to undergo review from others to make sure they're comfortable with the way we're introducing it (since COM is kind of a platform-specific thing, it has to be done carefully).


http://reviews.llvm.org/D9234

Files:
  include/llvm/DebugInfo/PDB/PDBContext.h
  lib/DebugInfo/PDB/CMakeLists.txt
  lib/DebugInfo/PDB/PDBContext.cpp
  tools/llvm-symbolizer/CMakeLists.txt
  tools/llvm-symbolizer/LLVMSymbolize.cpp
  tools/llvm-symbolizer/llvm-symbolizer.cpp

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D9234.24406.patch
Type: text/x-patch
Size: 10162 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150424/a7d704e6/attachment.bin>


More information about the llvm-commits mailing list