[PATCH] D50419: [ELF] Skip non-InputSection's in getErrPlace

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 14 16:04:15 PDT 2018


MaskRay abandoned this revision.
MaskRay added a comment.

In https://reviews.llvm.org/D50419#1198702, @grimar wrote:

> It seems caused by https://reviews.llvm.org/rL336106. But I really do not understand how it can happen.
>
> As my comment for the commit says:
>  "We should only have InputSection type of the sections in the vector.
>  (because we already converted MergeInputSection in mergeSections()
>  and combined EhInputSections in combineEhFrameSections())."
>
> What am I missing?


Thanks for the information! I find that `mergeSections()` does not remove non-alive sections as its comments says. I have created https://reviews.llvm.org/D50742

In an internal target, both revisions will fix the invalid cast but https://reviews.llvm.org/D50742 is obviously the way to go.


Repository:
  rLLD LLVM Linker

https://reviews.llvm.org/D50419





More information about the llvm-commits mailing list