[libc-commits] [llvm] [libc] [libc][NFC] Fix missing LIBC_INLINE + style (PR #73659)

Clement Courbet via libc-commits libc-commits at lists.llvm.org
Tue Nov 28 07:40:20 PST 2023


================
@@ -24,74 +26,74 @@ constexpr unsigned long long ULLONG_MAX = ~0ULL;
 
 template <class T> class numeric_limits {
 public:
-  static constexpr T max();
-  static constexpr T min();
+  LIBC_INLINE static constexpr T max();
+  LIBC_INLINE static constexpr T min();
 };
 
 // TODO: Add numeric_limits specializations as needed for new types.
 
 template <> class numeric_limits<int> {
 public:
-  static constexpr int max() { return INT_MAX; }
-  static constexpr int min() { return INT_MIN; }
+  LIBC_INLINE static constexpr int max() { return INT_MAX; }
----------------
legrosbuffle wrote:

This is already implied by the fact that the function is defined in the class declaration. @frobtech can we fix the clang -tidy to ignore this ?

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


More information about the libc-commits mailing list