[libc-commits] [libc] [libc] List `rand` and `srand` as supported on the GPU (PR #96757)

via libc-commits libc-commits at lists.llvm.org
Wed Jun 26 05:06:57 PDT 2024


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-libc

Author: Joseph Huber (jhuber6)

<details>
<summary>Changes</summary>

Summary:
I initially didn't report these as supported because they didn't provide
expected behavior and were very wasteful. The recent patch moved them to
a lock-free atomic implementation so they can now actually be used.


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


2 Files Affected:

- (modified) libc/docs/gpu/support.rst (+2) 
- (modified) libc/src/stdlib/rand_util.cpp (+1-1) 


``````````diff
diff --git a/libc/docs/gpu/support.rst b/libc/docs/gpu/support.rst
index 31bbdc41090be..1044b005de0ee 100644
--- a/libc/docs/gpu/support.rst
+++ b/libc/docs/gpu/support.rst
@@ -190,6 +190,8 @@ strtold        |check|
 strtoll        |check|
 strtoul        |check|
 strtoull       |check|
+srand          |check|
+rand           |check|
 =============  =========  ============
 
 inttypes.h
diff --git a/libc/src/stdlib/rand_util.cpp b/libc/src/stdlib/rand_util.cpp
index 81ee358935716..ff3478db70003 100644
--- a/libc/src/stdlib/rand_util.cpp
+++ b/libc/src/stdlib/rand_util.cpp
@@ -12,7 +12,7 @@
 
 namespace LIBC_NAMESPACE {
 
-//  C standard 7.10p2: If 'rand' is called before 'srand' it is to
+// C standard 7.10p2: If 'rand' is called before 'srand' it is to
 // proceed as if the 'srand' function was called with a value of '1'.
 cpp::Atomic<unsigned long> rand_next = 1;
 

``````````

</details>


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


More information about the libc-commits mailing list