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

Kostya Serebryany via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 9 12:41:45 PST 2018


kcc added inline comments.


================
Comment at: compiler-rt/lib/hwasan/hwasan_report.cc:68
+
+InternalMmapVector<char> *ScopedReport::error_message_ptr_;
+BlockingMutex ScopedReport::error_message_lock_;
----------------
eugenis wrote:
> 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.
> 
ack. 
I'd prefer to add append() method to InternalMmapVector then and use it here


https://reviews.llvm.org/D54284





More information about the llvm-commits mailing list