address comments

Author: moar55

libc/src/sys/mman/linux/process_mrelease.cpp

@@ -24,7 +24,7 @@ LLVM_LIBC_FUNCTION(int, process_mrelease, (int pidfd, unsigned int flags)) {
 
   if (ret < 0) {
     libc_errno = static_cast<int>(-ret);
-    return libc_errno;
+    return -1;
   }
 
   return 0;

libc/src/sys/mman/process_mrelease.h

@@ -1,11 +1,10 @@
-//===-- Implementation header for process_mrelease  function -----------------*-
-// C++ -*-===//
+//===-- Implementation header for process_mrelease function --------*- C++ -*-===//
 //
 // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
 // See https://llvm.org/LICENSE.txt for license information.
 // SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
 //
-//===----------------------------------------------------------------------===//
+//===-------------------------------------------------------------------------===//
 
 #ifndef LLVM_LIBC_SRC_SYS_MMAN_PROCESS_MRELEASE_H
 #define LLVM_LIBC_SRC_SYS_MMAN_PROCESS_MRELEASE_H

libc/test/src/sys/mman/linux/process_mrelease_test.cpp

@@ -1,3 +1,10 @@
+//===-- Unittests for process_mrelease ------------------------------------===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
+//
+//===----------------------------------------------------------------------===//
 //===-- Unittests for process_mrelease
 //-------------------------------------===//
 //
@@ -68,5 +75,5 @@ TEST(LlvmLibcMProcessMReleaseTest, ErrorNotKilled) {
 
 TEST(LlvmLibcMProcessMReleaseTest, ErrorNonExistingPidfd) {
 
-  ASSERT_EQ(LIBC_NAMESPACE::process_mrelease(12345, 0), EBADF);
+  ASSERT_EQ(LIBC_NAMESPACE::process_mrelease(-1, 0), EBADF);
 }