[PATCH] D33525: [ThinLTO] Migrate ThinLTOBitcodeWriter to the new PM.

Tim Shen via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu May 25 12:14:51 PDT 2017


timshen added a comment.

In https://reviews.llvm.org/D33525#764251, @chandlerc wrote:

> (focusing on the LLVM side of this review for now)
>
> Can you add an LLVM-based test? Can you add this to `lib/Passes/PassRegistry.def`?


I see that not all passes are registered (BitcodeWriterPass). Is it an oversight?

Does a pass have to be default constructible, in order to be registered easily?


https://reviews.llvm.org/D33525





More information about the cfe-commits mailing list