[PATCH] D153679: [LegacyPM] Remove RewriteSymbolsLegacyPass

Steven Johnson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 26 13:14:39 PDT 2023


srj added a comment.

In D153679#4450286 <https://reviews.llvm.org/D153679#4450286>, @nikic wrote:

> I think we should revert this change for now and keep the pass until we have NewPM support for the codegen pipeline. @aeubanks What do you think?

Halide contributor Andrew Adams commented elsewhere: "It's an old commit. Hard to say if anyone is using the feature that this PR removes." I suspect that we can live without it until the entire API migration is done. (Is there an ETA for that? We've been hearing for years that it will happen...)


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D153679/new/

https://reviews.llvm.org/D153679



More information about the llvm-commits mailing list