[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