[PATCH] D33525: [ThinLTO] Migrate ThinLTOBitcodeWriter to the new PM.
Tim Shen via Phabricator via llvm-commits
llvm-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 llvm-commits
mailing list