[PATCH] D48496: [llvm-mca] Rename Backend to Pipeline. NFC.

[Andrea Di Biagio via Phabricator via llvm-commits]

andreadb added inline comments.


================
Comment at: tools/llvm-mca/Pipeline.h:11
 ///
-/// This file implements an OoO backend for the llvm-mca tool.
+/// This file implements an OoO pipeline for the llvm-mca tool.
 ///
----------------
courbet wrote:
> andreadb wrote:
> > Remove the OoO part. I don't think it makes much sense when applied to pipeline.
> I would keep it, just to make it clear to the reader that  it's not strictly a linear DAG.
The Pipeline (as a container of stages) is not out-of-order. Instead, the hardware backend is OoO. I don't think it is semantically correct to say that the Pipeline is out-of-order.
We could rephrase this as "a Stage pipeline for an OoO processor". Or something similar.


https://reviews.llvm.org/D48496