Skip to content

Commit b92c0fa

Browse files
committed
[VecOps] Remove outdated IsSmall helper function in tests
This is a follow-up on 2605710, which is required to make the `vecops_rvec.cxx` tests pass on ARM64 with GCC 14. That commit changed from using the `ROOT::Detail::VecOps::` namespace explicitly for `IsSmall` to a `using` statement. However, there was also a remnant `IsSmall` function with an outdated implementation in the test file itself, which was accidentally picked up by the tests then.
1 parent 0a78e9c commit b92c0fa

File tree

1 file changed

+0
-7
lines changed

1 file changed

+0
-7
lines changed

math/vecops/test/vecops_rvec.cxx

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1493,13 +1493,6 @@ TEST(VecOps, Construct)
14931493
EXPECT_TRUE(fourVects[2] == ref2);
14941494
}
14951495

1496-
bool IsSmall(const RVec<int> &v)
1497-
{
1498-
// the first array element is right after the 3 data members of SmallVectorBase
1499-
return reinterpret_cast<std::uintptr_t>(v.begin()) - reinterpret_cast<std::uintptr_t>(&v) ==
1500-
sizeof(void *) + 2 * sizeof(int);
1501-
}
1502-
15031496
// this is a regression test for https://github.com/root-project/root/issues/6796
15041497
TEST(VecOps, MemoryAdoptionAndClear)
15051498
{

0 commit comments

Comments
 (0)