[PATCH] D76274: [Alignment] Add alignTo with skew parameter
Clement Courbet via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Mar 17 03:44:10 PDT 2020
courbet accepted this revision.
courbet added inline comments.
This revision is now accepted and ready to land.
================
Comment at: llvm/include/llvm/Support/Alignment.h:192
+inline uint64_t alignTo(uint64_t Size, Align A, uint64_t Skew) {
+ const uint64_t value = A.value();
+ Skew %= value;
----------------
`Value`
================
Comment at: llvm/unittests/Support/AlignmentTest.cpp:129
+TEST(AlignmentTest, AlignToWithSkew) {
+ EXPECT_EQ(alignTo(5, Align(8), 7), 7U);
+ EXPECT_EQ(alignTo(17, Align(8), 1), 17U);
----------------
Add a test for `0` ?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D76274/new/
https://reviews.llvm.org/D76274
More information about the llvm-commits
mailing list