beanz accepted this revision.
beanz added a comment.

LGTM other than a few style comments. Please clang-format before committing. 
Some of the formatting looks off (although that might just be my phone).



================
Comment at: lib/Support/PrettyStackTrace.cpp:152
+  va_start(ap, format);
+  const int size_or_error = vsnprintf(nullptr, 0, format, ap);
+  va_end(ap);
----------------
Your variable names are very LLDB-esque. Please follow the LLVM style guide, 
which is CamelCase.


Repository:
  rL LLVM

https://reviews.llvm.org/D27683



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to