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

Ramkumar Ramachandra via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 13 09:27:52 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));
----------------
artagnon wrote:

We do, in general, but this particular case is just a simple cast from one integer type to another, preserving signedness, so I don't think it matters.

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


More information about the llvm-commits mailing list