[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.



More information about the cfe-commits mailing list