[PATCH] D50940: [sanitizer] Change Mmap*NoAccess to return MMAP_FAILED (~(uptr)0) on error

[Aleksey Shlyapnikov via Phabricator via llvm-commits]

alekseyshl added a comment.

I do support the unification of the return value, and if no call site is currently care for the actual syscall return value, I'd say, let's make all of them return pointer, nullptr or die. MMAP_FAILED might be a reasonable common ground too, but seems like nullptr is used in many places already and Windows returns NULL on failure, so a bit more unification across platforms here.


Repository:
  rCRT Compiler Runtime

https://reviews.llvm.org/D50940