[PATCH] D65775: [Sanitizer] Linux explicitally migrate shadow mapping to Transparent Huge Page
David CARLIER via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Aug 6 12:16:47 PDT 2019
devnexen marked an inline comment as done.
devnexen added inline comments.
================
Comment at: compiler-rt/lib/sanitizer_common/sanitizer_posix_libcdep.cpp:77
+ int rerrno = 0;
+ if (internal_iserror(r, &rerrno) && rerrno == EINVAL)
+ return false;
----------------
vitalybuka wrote:
> I guess this should be:
> ```
> if (internal_iserror(r, &rerrno) && rerrno == EINVAL)
> return true;
> ```
>
> we get EINVAL for "never"?
Why true ?
Ah correct maybe it is more likely EAGAIN.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D65775/new/
https://reviews.llvm.org/D65775
More information about the llvm-commits
mailing list