[PATCH] D20084: [sanitizer] Initial implementation of a Hardened Allocator
Kostya Serebryany via llvm-commits
llvm-commits at lists.llvm.org
Wed May 11 13:51:05 PDT 2016
kcc added inline comments.
================
Comment at: projects/compiler-rt/lib/hardened_allocator/scudo_allocator.cc:80
@@ +79,3 @@
+ // 1st 8 bytes
+ u128 checksum : 16;
+ u128 requested_size : 40;
----------------
vitalybuka wrote:
> I see why you need 128bit for structure.
> Why do you need u128 type for 2 bit members?
I actually like it this way. The intent is more clear.
http://reviews.llvm.org/D20084
More information about the llvm-commits
mailing list