[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