[PATCH] Simplify Symbolizer::SymbolizePC() interface.
Alexey Samsonov
vonosmas at gmail.com
Wed Nov 26 11:22:20 PST 2014
================
Comment at: lib/sanitizer_common/sanitizer_symbolizer_libbacktrace.cc:86
@@ -85,2 +85,3 @@
struct SymbolizeCodeData {
+ SymbolizedFrame *res;
----------------
earthdok wrote:
> We have SymbolizeCode(), SymbolizeData() and struct SymbolizeCodeData. Confusing. Besides, "Data" doesn't really tell you anything. This should be renamed to something like SymbolizeCodeResult.
Renamed to SymbolizeCodeCallbackArg.
================
Comment at: lib/sanitizer_common/sanitizer_symbolizer_libbacktrace.cc:87
@@ -86,4 +86,3 @@
struct SymbolizeCodeData {
- AddressInfo *frames;
- uptr n_frames;
- uptr max_frames;
+ SymbolizedFrame *res;
+ SymbolizedFrame *last;
----------------
earthdok wrote:
> I'm not happy with this name. Maybe "result" or "first"? I'd prefer the latter.
Done
http://reviews.llvm.org/D6394
More information about the llvm-commits
mailing list