[llvm] [ScheduleDAG] Dirty height/depth in addPred/removePred even for latency zero (PR #102915)

Björn Pettersson via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 21 23:42:14 PDT 2024


bjope wrote:

> Correctness issue or enhancement?

I think that kind of depends on how you see it.
I don't know about any case where this has resulted in miscompile (specially since it has been incorrect in-tree for 15 years). So it is not a correctness issue in that sense.

The goal with the patch is to serve as an optimization/enhancement. It should make the scheduling DAG more _correct_, allowing optimizations based on the scheduling DAG to pick a better schedule when looking at height/depth.

https://github.com/llvm/llvm-project/pull/102915


More information about the llvm-commits mailing list