[PATCH] D59834: [ARM] Don't confuse the scheduler for very large VLDMDIA etc.

Peter Smith via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 27 03:14:10 PDT 2019


peter.smith accepted this revision.
peter.smith added a comment.
This revision is now accepted and ready to land.

I've tentatively marked this as LGTM, if there are any disagreements please feel free to override. If I've understood correctly the only uses of that function are from the ARMSchedule*.td files which will treat larger number of registers as if they were the maximum size available. I think that this, or reverting the previous patch, are the most plausible short term ways to fix the bots (I've checked I can do a 2-stage build).


Repository:
  rL LLVM

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

https://reviews.llvm.org/D59834





More information about the llvm-commits mailing list