[PATCH] D108083: add sanitizer support to hexagon

Sid Manning via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Aug 16 11:31:45 PDT 2021


sidneym added inline comments.


================
Comment at: compiler-rt/lib/sanitizer_common/sanitizer_syscall_linux_hexagon.inc:124
+
+bool internal_iserror(uptr retval, int *rverrno) {
+  if (retval == (uptr)-1) {
----------------
bcain wrote:
> @sidneym Can you confirm that this implementation looks correct?  I thought the convention was -1 for failed syscalls.
Maybe it should be <0 instead.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D108083/new/

https://reviews.llvm.org/D108083



More information about the cfe-commits mailing list