[llvm] MathExtras: avoid unnecessarily widening types (PR #95426)

Jay Foad via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 26 02:03:02 PDT 2024


================
@@ -135,22 +151,22 @@ template <typename T> T reverseBits(T Val) {
 // ambiguity.
 
 /// Return the high 32 bits of a 64 bit value.
-constexpr inline uint32_t Hi_32(uint64_t Value) {
+constexpr uint32_t Hi_32(uint64_t Value) {
----------------
jayfoad wrote:

Nit: I'd prefer doing this as a separate "constexpr implies inline" precommit.

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


More information about the llvm-commits mailing list