[PATCH] D143286: Updated inline multiplier comment.

Jack Kirk via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 3 10:19:32 PST 2023


JackAKirk created this revision.
Herald added subscribers: mattd, gchakrabarti, asavonic, hiraditya.
Herald added a project: All.
JackAKirk requested review of this revision.
Herald added subscribers: llvm-commits, jholewinski.
Herald added a project: LLVM.

Signed-off-by: JackAKirk <jack.kirk at codeplay.com>


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D143286

Files:
  llvm/lib/Target/NVPTX/NVPTXTargetTransformInfo.h


Index: llvm/lib/Target/NVPTX/NVPTXTargetTransformInfo.h
===================================================================
--- llvm/lib/Target/NVPTX/NVPTXTargetTransformInfo.h
+++ llvm/lib/Target/NVPTX/NVPTXTargetTransformInfo.h
@@ -90,7 +90,7 @@
     return true;
   }
 
-  // Increase the inlining cost threshold by a factor of 5, reflecting that
+  // Increase the inlining cost threshold by a factor of 11, reflecting that
   // calls are particularly expensive in NVPTX.
   unsigned getInliningThresholdMultiplier() { return 11; }
 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D143286.494683.patch
Type: text/x-patch
Size: 539 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20230203/ed9e270d/attachment.bin>


More information about the llvm-commits mailing list