[PATCH] D153679: [LegacyPM] Remove RewriteSymbolsLegacyPass

Arthur Eubanks via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 26 13:22:26 PDT 2023


aeubanks added a comment.

we could revert, but as you've alluded to I doubt that halide users are actually using this pass, for example I removed it from LLVM in https://reviews.llvm.org/D99707 without complaint

I'd remove the halide use and see if anybody complains, and if somebody does we can revert this patch

you could also add the corresponding new PM pass to the optimization pipeline that halide runs


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