[PATCH] D88701: [AArch64] Prefer prologues with sp adjustments merged into stp/ldp for WinCFI

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 2 13:42:13 PDT 2020


efriedma added a comment.

Yes, the dependency chain of sp is one instruction longer, and there's one extra arithmetic op on some cores.  Those double if you count the epilogue that has the same issue.

Not sure how significant that is in practice, but I suspect it's observable in code with a lot of small functions.

Maybe it makes sense to distinguish between -O2 vs. -Os here?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D88701



More information about the llvm-commits mailing list