[PATCH] D32802: Add checks so that -pre-RA-sched=list-ilp does not crash on SystemZ (Bug 32723).

Jonas Paulsson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 9 22:57:34 PDT 2017


jonpa added a comment.

In https://reviews.llvm.org/D32802#749860, @MatzeB wrote:

> In https://reviews.llvm.org/D32802#749855, @jonpa wrote:
>
> > In https://reviews.llvm.org/D32802#749851, @MatzeB wrote:
> >
> > > Just out of interested: What's the interested in these selection dag schedulers, I though they are deprecated and only SourceListDAGScheduler and possibly SchedulerDAGFast are of interest with the MachineScheduler around.
> > >
> > > Is this because of the same reasons (or same benchmark?) that motivated https://reviews.llvm.org/D32563 ?
> >
> >
> > This was just a bug I discovered with llvm-stress. As long as this option is around, it shouldn't mean that the compiler crashes when it is used, right?  (Perhaps an exit with some kind of message explaining that this scheduler is unsupported might due instead...?)
>
>
> llvm-stress is using commandline options to change the selection dag scheduler?


I was trying different llc options to find more bugs with llvm-stress. This way I would get a bigger set of different inputs to the various CodeGen passes. Changing isel scheduler is one way of changing things around.


https://reviews.llvm.org/D32802





More information about the llvm-commits mailing list