[PATCH] D38279: [MachineScheduler] Enable latency heuristic based on scheduled lat.

[Jonas Paulsson via Phabricator via llvm-commits]

jonpa added a comment.

In https://reviews.llvm.org/D38279#945029, @fhahn wrote:

> In https://reviews.llvm.org/D38279#937959, @fhahn wrote:
>
> > In https://reviews.llvm.org/D38279#937309, @jonpa wrote:
> >
> > > This makes sense to me as well, but I wonder why Issued gets a different value depending on top/bot ?
> >
> >
> >
>
>
> Did my response make sense as well? It's probably not extremely well worded :/


Well.. :-)  I must admit I am somewhat confused still... Is this difference the same regardless of in-order / out-of-order and issue width?

I tried this patch on SystemZ, and it seems it may bring just a little improvement if anything (SystemZ is doing bottom-up-only pre-ra).


https://reviews.llvm.org/D38279