[PATCH] D46907: [llvm-mca] Introduce a sequential container of Stages

Matt Davis via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 15 15:22:01 PDT 2018


mattd added inline comments.


================
Comment at: tools/llvm-mca/Backend.cpp:38
+Stage &Backend::getInitialStage() {
+  assert(Stages.size() > 0 && "No initial pipeline stage registered.");
+  assert(Stages[0] && "Invalid initial pipeline stage.");
----------------
This case should never occur, but since we expose this as a public api routine, I want to be super-confident that users are using the backend correctly, e.g., an initial stage is always available.


https://reviews.llvm.org/D46907





More information about the llvm-commits mailing list