[PATCH] D82320: [NPM] Attempt to run opt passes specified via -foo-pass under NPM
Yuanfang Chen via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jun 22 18:17:35 PDT 2020
ychen accepted this revision.
ychen added a comment.
This revision is now accepted and ready to land.
In D82320#2107835 <https://reviews.llvm.org/D82320#2107835>, @aeubanks wrote:
> In D82320#2107762 <https://reviews.llvm.org/D82320#2107762>, @ychen wrote:
>
> > This is great! What are these failures look like? Do they look straightforward to fix?
>
>
> Updated the description.
> Most of them are passes either not ported to NPM
I would assume most of these are passes for codegen pipeline?
or don't have the same name in PassRegistry.def as their old name.
> I was thinking about potentially creating some wrapper class around old PM passes but I don't think that would work.
Not for this patch, how about adding the old name in the PassRegistry.def as an alternative name?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D82320/new/
https://reviews.llvm.org/D82320
More information about the llvm-commits
mailing list