[libc][time] Fix -Wshorten-64-to-32 warning

[ilovepi]

This breaks builds of libc with top of tree clang under -Werror.

[ilovepi]

• [libc][time] Fix -Wshorten-64-to-32 warning #132947 👈 (View in Graphite)
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[llvmbot]

@llvm/pr-subscribers-libc

Author: Paul Kirth (ilovepi)

Changes

This breaks builds of libc with top of tree clang under -Werror.

---

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

1 Files Affected:

• (modified) libc/src/time/time_utils.cpp (+4-4)

diff --git a/libc/src/time/time_utils.cpp b/libc/src/time/time_utils.cpp
index 6e06deb30e354..1c519c3ff8aec 100644
--- a/libc/src/time/time_utils.cpp
+++ b/libc/src/time/time_utils.cpp
@@ -102,10 +102,10 @@ cpp::optional<time_t> mktime_internal(const tm *tm_out) {
 
   // TODO: https://github.com/llvm/llvm-project/issues/121962
   // Need to handle timezone and update of tm_isdst.
-  time_t seconds = tm_out->tm_sec +
-                   tm_out->tm_min * time_constants::SECONDS_PER_MIN +
-                   tm_out->tm_hour * time_constants::SECONDS_PER_HOUR +
-                   total_days * time_constants::SECONDS_PER_DAY;
+  time_t seconds = static_cast<time_t>(
+      tm_out->tm_sec + tm_out->tm_min * time_constants::SECONDS_PER_MIN +
+      tm_out->tm_hour * time_constants::SECONDS_PER_HOUR +
+      total_days * time_constants::SECONDS_PER_DAY);
   return seconds;
 }
 

[ilovepi]

Merge activity

• Mar 25, 11:51 AM EDT: A user started a stack merge that includes this pull request via Graphite.
• Mar 25, 11:53 AM EDT: A user merged this pull request with Graphite.