[PATCH] D91804: [NewPM] Add pipeline EP callback after initial frontend cleanup

Yuanfang Chen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 23 16:34:40 PST 2020


ychen added a comment.

In D91804#2412277 <https://reviews.llvm.org/D91804#2412277>, @asbirlea wrote:

> I don't think "ModuleOptimizer" in the name makes sense, the association can come from the "Early" part. Either keep the naming as is, or "EarlyAfterStart" if you want to define order against the other callbacks.
> ychen@ so you have a better naming suggestion?

Having a second look, yeah, I agree that "ModuleOptimizer" in the name does not make sense. `Early` is also vague about where it is called. How about just `ModuleSimplificationEPCallback`?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D91804



More information about the llvm-commits mailing list