[PATCH] D114826: [memprof] Align each rawprofile section to 8b.

Vitaly Buka via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 30 17:09:37 PST 2021


vitalybuka added inline comments.


================
Comment at: compiler-rt/lib/memprof/memprof_rawprofile.cpp:35
+// Returns the size rounded up to 8b.
+u64 alignedSize(const u64 Size) { return Size + Size % 8; }
 } // namespace
----------------
tejohnson wrote:
> Should this be Size + 8 - Size%8?
RoundUpTo(size, 8)

BTW why 8? should this be alignof(T) ?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D114826/new/

https://reviews.llvm.org/D114826



More information about the llvm-commits mailing list