[PATCH] D116361: [SchedModels][CortexA55] Fix scheduling of FP loads

[Dave Green via Phabricator via llvm-commits]

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

Thanks for pulling this part out, it looks good. I've been trying to work out if they really should be 3 microops (so execute over 1.5 cycles), but it appears to be quite difficult to measure. As far as I can tell it maybe shouldn't, but my attempts to use NumMicroOps=0 only made things look worse! The optimization guide wasn't very clear on where exactly the updates happen.

So I'm happy with this as-is. LGTM. Looks like a nice improvement.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D116361