[PATCH] D38588: Clear LastMappingSymbols and LastEMS(Info) when resetting the ARM(AArch64)ELFStreamer

Yichao Yu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 5 11:45:17 PDT 2017


yuyichao added a comment.

> we might want at least an assert here.

Do you know to make it an assertion failure instead of a segfault?
The failure is actually very simple. The LastEMSInfo contains a NULL pointer so any reference from it segfaults. Is dereferencing pointers/unique_ptr's something we add assert for?


https://reviews.llvm.org/D38588





More information about the llvm-commits mailing list