Merge pull request #22062 from ClickHouse/datetime64-ubsan

Fix UBSan report in TransformDateTime64

src/Functions/TransformDateTime64.h

@@ -49,8 +49,11 @@ public:
     {}
 
     template <typename ... Args>
-    inline auto execute(const DateTime64 & t, Args && ... args) const
+    inline auto NO_SANITIZE_UNDEFINED execute(const DateTime64 & t, Args && ... args) const
     {
+        /// Type conversion from float to integer may be required.
+        /// We are Ok with implementation specific result for out of range and denormals conversion.
+
         if constexpr (TransformHasExecuteOverload_v<DateTime64, decltype(scale_multiplier), Args...>)
         {
             return wrapped_transform.execute(t, scale_multiplier, std::forward<Args>(args)...);

tests/queries/0_stateless/01773_datetime64_add_ubsan.reference

+0
+0

tests/queries/0_stateless/01773_datetime64_add_ubsan.sql

+-- The result is unspecified but UBSan should not argue.
+SELECT ignore(addHours(now64(3), inf)) FROM numbers(2);