[PATCH] D82423: [X86] Rename O3-pipeline.ll to opt-pipeline.ll and add O1/O2 command lines.

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 24 09:09:54 PDT 2020


craig.topper added a comment.

In D82423#2111112 <https://reviews.llvm.org/D82423#2111112>, @lebedev.ri wrote:

> I do not understand why we can't just have `llvm/test/CodeGen/X86/O{0,1,2,3}-pipeline.ll`, like there already is for plain opt?


I was just trying to make it so updating the pipeline didn’t require editing 3 files. We also have changes to this test in our downstream code base so I was trying to make it just work when we synced this change.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D82423





More information about the llvm-commits mailing list