[PATCH] D31901: LTO: Pass SF_Executable flag through to InputFile::Symbol

[Peter Collingbourne via Phabricator via llvm-commits]

pcc added a comment.

In https://reviews.llvm.org/D31901#723759, @tobiasvk wrote:

> In https://reviews.llvm.org/D31901#723155, @pcc wrote:
>
> > Agreed with @davide that this needs a test case. I am working on a change to llvm-lto2 that adds a feature for dumping the LTO symbol table, which should allow you to test this. @tobiasvk would you be fine with waiting until that change lands?
>
>
> Yes, absolutely agree we need a test case. It's just currently not possible to test it inside llvm without the linker, so I couldn't add one. It would be great if you could add a symbol table dump feature to llvm-lto2. Please add me to the review when you get to it, so we can merge this once your patch has landed.


This is https://reviews.llvm.org/D31920, I added you as a subscriber.


https://reviews.llvm.org/D31901