[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