[llvm] MathExtras/test: increase coverage (PR #95425)

Fred Grim via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 13 09:10:27 PDT 2024


================
@@ -183,15 +185,34 @@ TEST(MathExtras, NextPowerOf2) {
   EXPECT_EQ(256u, NextPowerOf2(128));
 }
 
-TEST(MathExtras, alignTo) {
+TEST(MathExtras, AlignTo) {
   EXPECT_EQ(8u, alignTo(5, 8));
   EXPECT_EQ(24u, alignTo(17, 8));
   EXPECT_EQ(0u, alignTo(~0LL, 8));
----------------
feg208 wrote:

this is somewhat a llvm style thing but do we prefer reinterpret/static_cast to c-style casts?

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


More information about the llvm-commits mailing list