[llvm] MachineTraceMetrics: always include instr latency in depth (PR #73550)
Ramkumar Ramachandra via llvm-commits
llvm-commits at lists.llvm.org
Tue Nov 28 01:32:51 PST 2023
artagnon wrote:
> This doesn't seem to be true from my inspection of `getOperandLatency`: in fact, `getOperandLatency` falls back to `getInstrLatency` in some cases, and even in cases when it doesn't, the latency of the instruction is implicitly included in its computation. Please correct me in case I've misunderstood something.
Sorry, it includes the instruction latency of DefMI, not UseMI. I'll have to re-think this patch.
https://github.com/llvm/llvm-project/pull/73550
More information about the llvm-commits
mailing list