[PATCH] D145265: [Pipeline] Remove GlobalCleanupPM

Nikita Popov via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Mar 24 11:23:33 PDT 2023


nikic added a comment.

I think this (removing GlobalCleanupPM) makes sense at a high level, but this is a pretty substantial pipeline change and will need more thorough evaluation (e.g. looking at IR diffs on test suite). For example, it is not at all obvious to me that moving InstCombine before EarlyCSE is harmless, given the impact this may have on one-use heuristics.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D145265



More information about the cfe-commits mailing list