[libc-commits] [libc] [libc] [Task] Enable most libc disabled warnings (PR #122835)

Nick Desaulniers via libc-commits libc-commits at lists.llvm.org
Tue Jan 14 11:17:03 PST 2025


================
@@ -72,9 +72,9 @@ long int check_primitives(int first, ...) {
   count += args.next_var<unsigned long>();
   count += args.next_var<long long>();
   count += args.next_var<unsigned long long>();
-  count += args.next_var<double>();
-  count += args.next_var<double>();
-  count += args.next_var<long double>();
+  count += static_cast<long>(args.next_var<double>());
+  count += static_cast<long>(args.next_var<double>());
+  count += static_cast<long>(args.next_var<long double>());
----------------
nickdesaulniers wrote:

Mind changing the return type and type of `count` to just `long`?  `int` is implied, and isn't specified generally throughout llvm.

Same for `check_struct_type` below.

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


More information about the libc-commits mailing list