[PATCH] D32721: Accept archive files with no symbol table instad of warning on them.

Davide Italiano via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 1 16:15:44 PDT 2017


davide added subscribers: rafael, davide.
davide requested changes to this revision.
davide added a comment.
This revision now requires changes to proceed.

hmm, I'm not entirely sure I agree with this one. (cc @rafael)
This has the drawback of of having to scan all the members in an archive so it's potentially expensive.
We could maybe take the hit because the LTO time is dominated by opt/codegen but I'm under the assumption the cost should be evaluated.
Also, I'm really worried about semantic changes, like two members with the same symbol name producing different results.


https://reviews.llvm.org/D32721





More information about the llvm-commits mailing list