[PATCH] D46837: [MachineScheduler] Skip an implicit def of a super-reg added by regalloc in findDefIdx.

Javed Absar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 7 03:37:27 PDT 2018


javed.absar added a comment.

In https://reviews.llvm.org/D46837#1124532, @jonpa wrote:

> r334161 fixes this issue for SystemZ by fixing the backend per alternative (3) above.
>
> Is there any interest for (1) or (2), or should I abandon them?


Option (2) seems a decent practical solution. Would a lot of cases fail because of it, and would existing sched-models pick wrong latencies in some cases?


https://reviews.llvm.org/D46837





More information about the llvm-commits mailing list