[PATCH] D94383: [SystemZ] Don't crash with -misched-cutoff
Ulrich Weigand via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Jan 12 02:00:36 PST 2021
uweigand added a comment.
I don't think we need to bother with skipping advancing the hazard state. I believe the main point of the cutoff is to avoid combinatorial explosion where there are many instructions to schedule and at each step there are many candidates to consider. Advancing the hazard state doesn't consider candidates and is therefore just a linear pass over instructions.
I'd simply clear the Available list once in ::initalize. That's where other MachineSchedStrategy implementations also clear their respective queues.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D94383/new/
https://reviews.llvm.org/D94383
More information about the llvm-commits
mailing list