Patch fixes scheduling of FP load instructions with pre/post increment adding WriteAdr for address operand.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Comment Actions
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.