[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