[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 09:10:19 PDT 2017
yuyichao added a comment.
> Why do you need to call this here and in `reset()`?
What do you mean by "and in `reset`"? (This **is** `reset` function). In case there's confusion, the two part of the patch are on two targets and they are unrelated though should be fixing the same issue.
I'm clearing it because IIUC the mapping shouldn't be shared between emission of multiple object (and according to the comment, this is what the `reset` function should be used for).
https://reviews.llvm.org/D38588
More information about the llvm-commits
mailing list