[PATCH] D78635: [NFC][TTI] Add TargetCostKind argument to getUserCost

Sam Parker via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 23 02:06:21 PDT 2020


samparker marked an inline comment as done.
samparker added inline comments.


================
Comment at: llvm/include/llvm/Analysis/TargetTransformInfo.h:157
+    TCK_CodeSize,        ///< Instruction code size.
+    TCK_Hybrid           ///< Intersection of size and performance.
   };
----------------
mtrofin wrote:
> Would 'workingset' accurately describe what the 'hybrid' is used for, so a weighed sum of code and latency? I'm having trouble wrapping my mind around the old  'intersection' notion (what would intersecting 2 costs mean?), and 'hybrid', at a first glance, leads me to believe it's about combining all other 3. 
I also don't know what it means, I just quoted it from below :) and I agree that Hybrid is not descriptive. I would probably prefer to be as explicit as possible with something like SizeAndLatencyWeight.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D78635/new/

https://reviews.llvm.org/D78635





More information about the llvm-commits mailing list