condy marked 3 inline comments as done.
condy added inline comments.

================
Comment at: llvm/lib/Transforms/Instrumentation/AddressSanitizer.cpp:2556
+  uint64_t RZ = 0;
+  if (SizeInBytes < MinRZ / 2) {
+    // Reduce redzone size for small size objects, e.g. int, char[1]. MinRZ is
----------------
vitalybuka wrote:
> MaskRay wrote:
> > I wonder whether 16-byte array should use this optimization as well (i.e. 
> > `<` => `<=`)
> > 
> > @vitalybuka 
> Agree, <= cutoff looks better here
@MaskRay @vitalybuka Applied


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D102469

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to