[llvm] ADT: Improve DenseMap documentation for insertion APIs (PR #172177)

Jakub Kuderski via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 16 06:07:33 PST 2025


================
@@ -208,9 +208,9 @@ class DenseMapBase : public DebugEpochBase {
     return ValueT();
   }
 
-  // Return the entry with the specified key, or \p Default. This variant is
-  // useful, because `lookup` cannot be used with non-default-constructible
-  // values.
+  /// Returns the mapped value for Val if present, otherwise Default.
----------------
kuhar wrote:

I still think it'd be better to change the argument name from Val to Key

https://github.com/llvm/llvm-project/pull/172177


More information about the llvm-commits mailing list