[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