[PATCH] D54284: [hwasan] Add entire report to abort message on Android.

Evgenii Stepanov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 9 09:05:28 PST 2018


eugenis added inline comments.


================
Comment at: compiler-rt/lib/hwasan/hwasan_report.cc:68
+
+InternalMmapVector<char> *ScopedReport::error_message_ptr_;
+BlockingMutex ScopedReport::error_message_lock_;
----------------
kcc wrote:
> why not InternalScopedString ? 
I don't like InternalScopedString. Its append method truncates the string instead of resizing the storage, and also does unnecessary format string parsing.



https://reviews.llvm.org/D54284





More information about the llvm-commits mailing list