[PATCH] D79385: NFC: Simplify O1 pass pipeline construction.

Eric Christopher via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 5 22:05:52 PDT 2020


echristo added a comment.

In D79385#2021922 <https://reviews.llvm.org/D79385#2021922>, @chandlerc wrote:

> As much as I'm not thrilled about the duplication, the number of comments I have below about the exact O1 <https://reviews.llvm.org/owners/package/1/> pipeline sure make it seem valuable. Unsure how you feel about addressing these here, later, etc.


Later :) I wanted to get the actual separation as a noop and then make actual substantial changes as I could do them.

I have a couple more pending patches as well. That said, appreciate the specific suggestions - I think the new format makes it much easier to really see what's going on here (and yes, lots of room for cleanup/changes/investigation).

-eric


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D79385





More information about the llvm-commits mailing list