[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