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

Dave Green via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 15 05:23:08 PDT 2019


dmgreen added a comment.

Sorry guys, I was distracted by some other work.

Florian, Nice one looking into the issues. I had tried some quick experiments around adjusting the priority of tryPressure in tryCandidate, but didn't get it to do much. I think because as you said the limits were a lot higher enough to kick in. My further ideas were more in the direction of adjusting the register allocation hints to pick higher registers less for instructions that can be shrunk into T1 instructions. Or (re-)adjusting the schedule taking into account things like CPSR live ranges, again for which instructions can and can't be shrank.

I've created a bug report in PR41884. Liek I tried to put into the comment, hopefully this is something that we don't need for long.


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

https://reviews.llvm.org/D61882





More information about the llvm-commits mailing list