[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