[PATCH] D28491: [AArch64] Add new subtarget feature to fuse AES crypto operations

Evandro Menezes via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 12 07:52:59 PST 2017


evandro added a comment.

> No, since they are pretty opaque.  But the pseudo `MOVaddr` is expanded into the pair `ADRP/ADD` only after the RA.  On A57, it's important to schedule them back to back, e.g., by running the `MISched` after the RA instead of before.

Or rather, I wonder why pseudo expansion is happening this late, when they are very simple instrs in AArch64.  Methinks that expanding them sooner would expose them to more optimizations, yes?


Repository:
  rL LLVM

https://reviews.llvm.org/D28491





More information about the llvm-commits mailing list