[PATCH] AArch64 sanitizer support
Christophe Lyon
christophe.lyon at linaro.org
Tue Feb 11 03:14:00 PST 2014
================
Comment at: libsanitizer/sanitizer_common/sanitizer_linux.cc:96
@@ -95,1 +95,3 @@
uptr internal_open(const char *filename, int flags) {
+#ifdef __aarch64__
+ return internal_syscall(__NR_openat, AT_FDCWD, (uptr)filename, flags);
----------------
Alexander Potapenko wrote:
> Christophe Lyon wrote:
> > Kostya Serebryany wrote:
> > > OMG, these ifdefs again...
> > > What is so special about AArch64 that it needs other syscalls
> > > (where ARM/PowerPC/SPARC/MIPS doesn't need them)?
> > > If there is no way to avoid these ifdefs, they at least deserve a top-level comment in this file.
> > >
> > I discussed that a while ago with AArch64 maintainers, here is what they said:
> >
> > > The AArch64 linux port uses the canonical syscall set as mandated by
> > > the upstream linux community for all new ports. The system calls are
> > > not missing, rather, other architectures are using legacy syscalls ;-)
> >
> >
> Is there a list of such syscalls somewhere? We'd better use them on every platform then.
I looked at the wrappers in
glibc/ports/sysdeps/unix/sysv/linux/genericXXX
I am not aware of a real list.
http://llvm-reviews.chandlerc.com/D2732
More information about the llvm-commits
mailing list