[PATCH] D61882: [ARM] Don't use the Machine Scheduler for cortex-m at minsize

Sjoerd Meijer via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 15 03:33:49 PDT 2019


SjoerdMeijer added a comment.

I agree with this:

> While the patch is a quick fix, I think we should properly address those issues. Would you mind creating a bug report?

This is a workaround for suboptimal decision making elsewhere. Teaching the scheduler new tricks and heuristics is usually not low-hanging fruit. So, since this workaround is very minimal and clean, creating a bug report looks like an excellent way forward to me.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D61882/new/

https://reviews.llvm.org/D61882





More information about the llvm-commits mailing list