[PATCH] D47108: Add -fforce-emit-vtables
Krzysztof Pszeniczny via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon May 21 14:02:27 PDT 2018
amharc added a comment.
I think that `MarkVTableUsed` should be called somewhere in Sema (possibly `ActOnFinishCXXMemberDecls`?) if `ForceEmitVTables` is on. This probably requires making `ForceEmitVTables` a `LangOption` in addition to it being a `CodeGenOption`.
This causes the above example to compile correctly. Moreover, I have checked that 7zip and Bullet (the llvm test suite benchmarks which failed previously) build and execute correctly with the above change.
Repository:
rL LLVM
https://reviews.llvm.org/D47108
More information about the cfe-commits
mailing list