[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