[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