[PATCH] D71334: [AArch64] Enable clustering memory accesses to fixed stack objects

Jay Foad via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 6 07:11:55 PST 2020


foad added a comment.

> The usage of `TII->getMemScale` is a bit weird since it makes it look like it's a non-static member function. No need to change it unless you think it would help.

It's a bit weird, but much shorter than `AArch64InstrInfo::getMemScale`! I don't have strong opinions about which is better so I'll leave it as-is.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71334





More information about the llvm-commits mailing list