[libc] Add Convenience syscall Template Function
authorAlex Brachet <alexbrachetmialot@gmail.com>
Thu, 13 Feb 2020 19:39:16 +0000 (14:39 -0500)
committerAlex Brachet <alexbrachetmialot@gmail.com>
Thu, 13 Feb 2020 19:39:16 +0000 (14:39 -0500)
Summary: This patch adds a convenience template function so we don't need to cast all types to `long` manually when calling `__llvm_libc::syscall`.

Reviewers: sivachandra, MaskRay, gchatelet

Reviewed By: sivachandra

Subscribers: libc-commits, tschuett

Differential Revision: https://reviews.llvm.org/D74530

libc/config/linux/x86_64/syscall.h.inc
libc/test/config/linux/x86_64/syscall_test.cpp

index 4659aa0..ee3b5e5 100644 (file)
@@ -94,6 +94,13 @@ __attribute__((always_inline)) inline long syscall(long __number, long __arg1,
   return retcode;
 }
 
+template <typename... Ts>
+__attribute__((always_inline)) inline long syscall(long __number, Ts... ts) {
+  static_assert(sizeof...(Ts) <= 6, "Too many arguments for syscall");
+  return syscall(__number, (long)ts...);
+}
+
+
 #undef SYSCALL_CLOBBER_LIST
 
 } // namespace __llvm_libc
index efab144..8db7a85 100644 (file)
@@ -36,4 +36,9 @@ TEST(X86_64_SyscallTest, APITest) {
       [](long n, long a1, long a2, long a3, long a4, long a5, long a6) {
         return __llvm_libc::syscall(n, a1, a2, a3, a4, a5, a6);
       });
+
+  std::function<long(long, void *)> notLongType(
+      [](long n, void *a1) {
+        return __llvm_libc::syscall(n, a1);
+      });
 }