aboutsummaryrefslogtreecommitdiff
path: root/llvm/lib/Analysis/ModuleSummaryAnalysis.cpp
diff options
context:
space:
mode:
authorAlex Bradbury <asb@igalia.com>2023-12-06 15:30:48 +0000
committerGitHub <noreply@github.com>2023-12-06 15:30:48 +0000
commitb7173652163fa968e1df47fa211ce8a549489abd (patch)
tree7b5d7036e73cbeb64ce0a4f42d023df5e047b98a /llvm/lib/Analysis/ModuleSummaryAnalysis.cpp
parent10f7801cff26732b9a757ed42a1e2b89021a61d8 (diff)
downloadllvm-b7173652163fa968e1df47fa211ce8a549489abd.zip
llvm-b7173652163fa968e1df47fa211ce8a549489abd.tar.gz
llvm-b7173652163fa968e1df47fa211ce8a549489abd.tar.bz2
[MachineScheduler][NFCI] Add Offset and OffsetIsScalable args to shouldClusterMemOps (#73778)
These are picked up from getMemOperandsWithOffsetWidth but weren't then being passed through to shouldClusterMemOps, which forces backends to collect the information again if they want to use the kind of heuristics typically used for the similar shouldScheduleLoadsNear function (e.g. checking the offset is within 1 cache line). This patch just adds the parameters, but doesn't attempt to use them. There is potential to use them in the current PPC and AArch64 shouldClusterMemOps implementation, and I intend to use the offset in the heuristic for RISC-V. I've left these for future patches in the interest of being as incremental as possible. As noted in the review and in an inline FIXME, an ElementCount-style abstraction may later be used to condense these two parameters to one argument. ElementCount isn't quite suitable as it doesn't support negative offsets.
Diffstat (limited to 'llvm/lib/Analysis/ModuleSummaryAnalysis.cpp')
0 files changed, 0 insertions, 0 deletions