[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