[llvm-bugs] [Bug 36663] New: [llvm-mca] Make the pipeline configurable.

via llvm-bugs llvm-bugs at lists.llvm.org
Fri Mar 9 07:01:03 PST 2018


https://bugs.llvm.org/show_bug.cgi?id=36663

            Bug ID: 36663
           Summary: [llvm-mca] Make the pipeline configurable.
           Product: new-bugs
           Version: unspecified
          Hardware: PC
                OS: Linux
            Status: NEW
          Severity: enhancement
          Priority: P
         Component: new bugs
          Assignee: unassignedbugs at nondot.org
          Reporter: courbet at google.com
                CC: llvm-bugs at lists.llvm.org

Following my comment on the initial review:
https://reviews.llvm.org/D43951?id=136547#inline-384487

There is a lot of coupling between the driver
(RunCycle()/NotifyCycleBegin()/NotifyCycleEnd()) and the pipeline itself. This
should be split into a driver class ad a configurable pipeline.
With the current design the backend has to know about all the possible
pipelines and expose all the quantities that listeners might be interested in
(GetNumXXXStalls(), NotifyInstructionXXX()). This does not scale well to more
pipeline components (in particular to simulate the frontend). I think it would
be better to manipulate IB/HWS/DU though a common interface and give this
interface a way to notify the listeners.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-bugs/attachments/20180309/ea33e0bc/attachment.html>


More information about the llvm-bugs mailing list