[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