[PATCH] D70108: [TTI] Added TTI pass queries for max load/store-per-memory-intrinsic.

Hussain Kadhem via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 3 16:57:24 PST 2020


hussainjk added a comment.

In D70108#1802747 <https://reviews.llvm.org/D70108#1802747>, @nemanjai wrote:

> I realize that without any uses of these functions, we can't really add a test case. However, can we at least get a description of why this is useful in the middle end if this patch won't add any users of these functions?
>
> Generally, I am a bit apprehensive about adding functions without callers if we don't have a clear roadmap to when/how these will be used.


I have a pending loop optimization where I want to use some of these values (the memset and memcpy ones) for cost modeling. I included the others because they are lumped together, and it seemed like it would be incomplete to only include the two I need.


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