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

[Mircea Trofin via Phabricator via llvm-commits]

mtrofin added inline comments.
Herald added a subscriber: wuzish.


================
Comment at: llvm/include/llvm/Analysis/TargetTransformInfo.h:157
+    TCK_CodeSize,        ///< Instruction code size.
+    TCK_Hybrid           ///< Intersection of size and performance.
   };
----------------
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. 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78635