[PATCH] D35275: [Sanitizers] ASan/MSan/LSan allocators set errno on failure.

Vitaly Buka via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 12 16:16:34 PDT 2017


vitalybuka added a comment.

In https://reviews.llvm.org/D35275#807348, @alekseyshl wrote:

> In https://reviews.llvm.org/D35275#807144, @vitalybuka wrote:
>
> > Does this need to be a single patch?
>
>
> No, it does not have to be a single patch, would you prefer to see per-sanitizer patches? My idea was that this way it is easier to see the similarities, but I'm fine splitting it into multiple patches.


errno = errno_ENOMEM; and ptr_check changes seems trivial to be in one patch, but replacement MsanReallocate to msan_* probably deserve separate one.


https://reviews.llvm.org/D35275





More information about the llvm-commits mailing list