[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