[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