[PATCH] D23089: llc: Add -start-before/-stop-before options

Matthias Braun via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 23 14:38:56 PDT 2016


MatzeB added inline comments.

================
Comment at: include/llvm/CodeGen/TargetPassConfig.h:151
@@ +150,3 @@
+           "Start after and start before passes are given");
+    assert(!(StopBefore && StopAfter) &&
+           "Stop after and stop before passed are given");
----------------
qcolombet wrote:
> hfinkel wrote:
> > These asserts should be report_fatal_error; the user might trigger them with bad command-line options (and I assume we still want them in release builds).
> Agreed. A follow-up commit is fine given we use to assert on that already.
I'll leave this as an assert, but will add some error handling/catching for this case to llc.

Reasoning: I'd like to keep the llvm free of (unnecessary) commandline handling logic, that's the frontends job IMO (llc in this case). Ideally we would have all of this start/stop/before/after handling purely in llc, however when I tried to do that it turned out to be quite tricky so I left the code as is and went for the easy enhancement.


Repository:
  rL LLVM

https://reviews.llvm.org/D23089





More information about the llvm-commits mailing list