[libc-commits] [libc] [libc] Revise the definition of `{get, set}rlimit`. (PR #124701)

via libc-commits libc-commits at lists.llvm.org
Mon Jan 27 21:56:44 PST 2025


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-libc

Author: None (c8ef)

<details>
<summary>Changes</summary>

Closes #<!-- -->124633.

Some parameter types in the definition of `{get, set}rlimit` currently do not match the standard. This patch resolves the issue.
ref: https://man7.org/linux/man-pages/man2/getrlimit.2.html

---
Full diff: https://github.com/llvm/llvm-project/pull/124701.diff


1 Files Affected:

- (modified) libc/include/sys/resource.yaml (+3-1) 


``````````diff
diff --git a/libc/include/sys/resource.yaml b/libc/include/sys/resource.yaml
index 3652d6d490a49b..e7645a3b4f0b3e 100644
--- a/libc/include/sys/resource.yaml
+++ b/libc/include/sys/resource.yaml
@@ -12,10 +12,12 @@ functions:
       - POSIX
     return_type: int
     arguments:
+      - type: int
       - type: struct rlimit *
   - name: setrlimit
     standards:
       - POSIX
     return_type: int
     arguments:
-      - type: const struct rlimit
+      - type: int
+      - type: const struct rlimit *

``````````

</details>


https://github.com/llvm/llvm-project/pull/124701


More information about the libc-commits mailing list