[PATCH] D54142: [ARM] Cortex-M4 schedule

Dave Green via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 7 14:19:26 PST 2019


dmgreen added a comment.

Hello. This was increasing codesize by more that I'd like, which was why I never committed it. The performance results would probably make it worthwhile, but at -Oz, it's not something people should be paying for.

I had a look into turning this off at minsize, but that felt like a bit of a hack and IIRC there was a test failing because of it. I just re-ran the results and there are some performance regressions I'd like to look into before committing too.


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

https://reviews.llvm.org/D54142





More information about the llvm-commits mailing list