[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