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

Clement Courbet via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 6 23:29:33 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:

Thanks for the explanations. Can we put these in the doc (libc/docs/code_style.rst) ? There are probably going to be other people like me who are going to wonder about that, and those latter explanations make a lot of sense.

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


More information about the llvm-commits mailing list