[PATCH] D70108: [TTI] Added TTI pass queries for max load/store-per-memory-intrinsic.
Matt Arsenault via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Jan 22 08:12:10 PST 2020
arsenm added a comment.
Still needs address space parameters
================
Comment at: llvm/include/llvm/Analysis/TargetTransformInfo.h:899-901
+ /// This function returns the maximum number of store operations permitted
+ /// to glue together during lowering of llvm.memcpy. The value is set by
+ // the target at the performance threshold for such a replacement.
----------------
hussainjk wrote:
> arsenm wrote:
> > I don't understand what I means by glue here. Like SelectionDAG glue? I don't think this is a useful concept to expose to the IR
> AFAIK this refers to groups of stores getting chained together within the memcpy expansion, with there being no guarantees otherwise about consecutive scheduling.
This should avoid referring to SelectionDAG specifics
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D70108/new/
https://reviews.llvm.org/D70108
More information about the llvm-commits
mailing list