[libc-commits] [libc] [libc] Use explicit cast to time_t in utimes_test. (PR #185307)

via libc-commits libc-commits at lists.llvm.org
Sun Mar 8 10:56:26 PDT 2026


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-libc

Author: Alexey Samsonov (vonosmas)

<details>
<summary>Changes</summary>

This fixes an error on RISCV-32 bot, where time_t is "long long" type (64-bit, as required by POSIX), instead of "long".

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


1 Files Affected:

- (modified) libc/test/src/sys/time/utimes_test.cpp (+2-2) 


``````````diff
diff --git a/libc/test/src/sys/time/utimes_test.cpp b/libc/test/src/sys/time/utimes_test.cpp
index 944c13d8f9170..3c8688ebda8d9 100644
--- a/libc/test/src/sys/time/utimes_test.cpp
+++ b/libc/test/src/sys/time/utimes_test.cpp
@@ -50,7 +50,7 @@ TEST_F(LlvmLibcUtimesTest, ChangeTimesSpecific) {
   // seconds
   ASSERT_EQ(statbuf.st_atim.tv_sec, times[0].tv_sec);
   ASSERT_EQ(statbuf.st_mtim.tv_sec, times[1].tv_sec);
-  ASSERT_GT(statbuf.st_ctim.tv_sec, 0L);
+  ASSERT_GT(statbuf.st_ctim.tv_sec, static_cast<time_t>(0));
 
   // microseconds
   ASSERT_EQ(statbuf.st_atim.tv_nsec,
@@ -61,7 +61,7 @@ TEST_F(LlvmLibcUtimesTest, ChangeTimesSpecific) {
   // legacy way to check seconds
   ASSERT_EQ(statbuf.st_atime, times[0].tv_sec);
   ASSERT_EQ(statbuf.st_mtime, times[1].tv_sec);
-  ASSERT_GT(statbuf.st_ctime, 0L);
+  ASSERT_GT(statbuf.st_ctime, static_cast<time_t>(0));
 
   ASSERT_THAT(LIBC_NAMESPACE::remove(TEST_FILE), Succeeds(0));
 }

``````````

</details>


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


More information about the libc-commits mailing list