[PATCH] D124212: [sanitizer] Use canonical syscalls everywhere
Evgenii Stepanov via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Apr 21 17:09:03 PDT 2022
eugenis added inline comments.
================
Comment at: compiler-rt/lib/sanitizer_common/sanitizer_platform.h:305
#ifndef SANITIZER_USES_CANONICAL_LINUX_SYSCALLS
-# if (defined(__aarch64__) || defined(__riscv) || defined(__hexagon__)) && \
- SANITIZER_LINUX
-# define SANITIZER_USES_CANONICAL_LINUX_SYSCALLS 1
-# else
-# define SANITIZER_USES_CANONICAL_LINUX_SYSCALLS 0
-# endif
+# define SANITIZER_USES_CANONICAL_LINUX_SYSCALLS SANITIZER_LINUX
#endif
----------------
pcc wrote:
> Can we remove this macro and replace it with SANITIZER_LINUX everywhere?
Sure. I wanted to preserve some context in the source, but that does not really matter.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D124212/new/
https://reviews.llvm.org/D124212
More information about the llvm-commits
mailing list