[PATCH] D47108: Add -fforce-emit-vtables

Krzysztof Pszeniczny via Phabricator via llvm-commits llvm-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 llvm-commits mailing list