Summary: | sys-devel/clang-10.x fails test: clang-tidy/checkers/abseil-duration-factory-scale.cpp | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Thomas Deutschmann (RETIRED) <whissi> |
Component: | Current packages | Assignee: | LLVM support project <llvm> |
Status: | CONFIRMED --- | ||
Severity: | normal | CC: | mgorny |
Priority: | Normal | Keywords: | TESTFAILURE |
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Attachments: | build.log |
Description
Thomas Deutschmann (RETIRED)
2020-06-28 22:04:32 UTC
Feel free to remove blocker if you believe this test failure shouldn't block stabilization. Yeah, let's resolve it later. FTR, exact failure from the log: FAIL: Clang Tools :: clang-tidy/checkers/abseil-duration-factory-scale.cpp (62 of 885) ******************** TEST 'Clang Tools :: clang-tidy/checkers/abseil-duration-factory-scale.cpp' FAILED ******************** Script: -- : 'RUN: at line 1'; /var/tmp/portage/sys-devel/clang-10.0.0/temp/python3.7/bin/python /var/tmp/portage/sys-devel/clang-10.0.0/work/x/y/clang/tools/extra/test/../test/clang-tidy/check_clang_tidy.py /var/tmp/portage/sys-devel/clang-10.0.0/work/x/y/clang/tools/extra/test/clang-tidy/checkers/abseil-duration-factory-scale.cpp abseil-duration-factory-scale /var/tmp/portage/sys-devel/clang-10.0.0/work/x/y/clang-abi_x86_32.x86/tools/extra/test/clang-tidy/checkers/Output/abseil-duration-factory-scale.cpp.tmp -- -- -I/var/tmp/portage/sys-devel/clang-10.0.0/work/x/y/clang/tools/extra/test/clang-tidy/checkers/Inputs -- Exit Code: 1 Command Output (stdout): -- Running ['clang-tidy', '/var/tmp/portage/sys-devel/clang-10.0.0/work/x/y/clang-abi_x86_32.x86/tools/extra/test/clang-tidy/checkers/Output/abseil-duration-factory-scale.cpp.tmp.cpp', '-fix', '--checks=-*,abseil-duration-factory-scale', '-format-style=none', '--', '-I/var/tmp/portage/sys-devel/clang-10.0.0/work/x/y/clang/tools/extra/test/clang-tidy/checkers/Inputs', '-std=c++11', '-nostdinc++']... ------------------------ clang-tidy output ----------------------- 25 warnings generated. /var/tmp/portage/sys-devel/clang-10.0.0/work/x/y/clang-abi_x86_32.x86/tools/extra/test/clang-tidy/checkers/Output/abseil-duration-factory-scale.cpp.tmp.cpp:12:7: warning: use ZeroDuration() for zero-length time intervals [abseil-duration-factory-scale] d = absl::Hours(0); ^~~~~~~~~~~~~~ absl::ZeroDuration() /var/tmp/portage/sys-devel/clang-10.0.0/work/x/y/clang-abi_x86_32.x86/tools/extra/test/clang-tidy/checkers/Output/abseil-duration-factory-scale.cpp.tmp.cpp:12:7: note: FIX-IT applied suggested code changes /var/tmp/portage/sys-devel/clang-10.0.0/work/x/y/clang-abi_x86_32.x86/tools/extra/test/clang-tidy/checkers/Output/abseil-duration-factory-scale.cpp.tmp.cpp:15:7: warning: use ZeroDuration() for zero-length time intervals [abseil-duration-factory-scale] d = absl::Minutes(0); ^~~~~~~~~~~~~~~~ absl::ZeroDuration() /var/tmp/portage/sys-devel/clang-10.0.0/work/x/y/clang-abi_x86_32.x86/tools/extra/test/clang-tidy/checkers/Output/abseil-duration-factory-scale.cpp.tmp.cpp:15:7: note: FIX-IT applied suggested code changes /var/tmp/portage/sys-devel/clang-10.0.0/work/x/y/clang-abi_x86_32.x86/tools/extra/test/clang-tidy/checkers/Output/abseil-duration-factory-scale.cpp.tmp.cpp:18:7: warning: use ZeroDuration() for zero-length time intervals [abseil-duration-factory-scale] d = absl::Seconds(0); ^~~~~~~~~~~~~~~~ absl::ZeroDuration() /var/tmp/portage/sys-devel/clang-10.0.0/work/x/y/clang-abi_x86_32.x86/tools/extra/test/clang-tidy/checkers/Output/abseil-duration-factory-scale.cpp.tmp.cpp:18:7: note: FIX-IT applied suggested code changes /var/tmp/portage/sys-devel/clang-10.0.0/work/x/y/clang-abi_x86_32.x86/tools/extra/test/clang-tidy/checkers/Output/abseil-duration-factory-scale.cpp.tmp.cpp:21:7: warning: use ZeroDuration() for zero-length time intervals [abseil-duration-factory-scale] d = absl::Milliseconds(0); ^~~~~~~~~~~~~~~~~~~~~ absl::ZeroDuration() /var/tmp/portage/sys-devel/clang-10.0.0/work/x/y/clang-abi_x86_32.x86/tools/extra/test/clang-tidy/checkers/Output/abseil-duration-factory-scale.cpp.tmp.cpp:21:7: note: FIX-IT applied suggested code changes /var/tmp/portage/sys-devel/clang-10.0.0/work/x/y/clang-abi_x86_32.x86/tools/extra/test/clang-tidy/checkers/Output/abseil-duration-factory-scale.cpp.tmp.cpp:24:7: warning: use ZeroDuration() for zero-length time intervals [abseil-duration-factory-scale] d = absl::Microseconds(0); ^~~~~~~~~~~~~~~~~~~~~ absl::ZeroDuration() /var/tmp/portage/sys-devel/clang-10.0.0/work/x/y/clang-abi_x86_32.x86/tools/extra/test/clang-tidy/checkers/Output/abseil-duration-factory-scale.cpp.tmp.cpp:24:7: note: FIX-IT applied suggested code changes /var/tmp/portage/sys-devel/clang-10.0.0/work/x/y/clang-abi_x86_32.x86/tools/extra/test/clang-tidy/checkers/Output/abseil-duration-factory-scale.cpp.tmp.cpp:27:7: warning: use ZeroDuration() for zero-length time intervals [abseil-duration-factory-scale] d = absl::Nanoseconds(0); ^~~~~~~~~~~~~~~~~~~~ absl::ZeroDuration() /var/tmp/portage/sys-devel/clang-10.0.0/work/x/y/clang-abi_x86_32.x86/tools/extra/test/clang-tidy/checkers/Output/abseil-duration-factory-scale.cpp.tmp.cpp:27:7: note: FIX-IT applied suggested code changes /var/tmp/portage/sys-devel/clang-10.0.0/work/x/y/clang-abi_x86_32.x86/tools/extra/test/clang-tidy/checkers/Output/abseil-duration-factory-scale.cpp.tmp.cpp:30:7: warning: use ZeroDuration() for zero-length time intervals [abseil-duration-factory-scale] d = absl::Seconds(0.0); ^~~~~~~~~~~~~~~~~~ absl::ZeroDuration() /var/tmp/portage/sys-devel/clang-10.0.0/work/x/y/clang-abi_x86_32.x86/tools/extra/test/clang-tidy/checkers/Output/abseil-duration-factory-scale.cpp.tmp.cpp:30:7: note: FIX-IT applied suggested code changes /var/tmp/portage/sys-devel/clang-10.0.0/work/x/y/clang-abi_x86_32.x86/tools/extra/test/clang-tidy/checkers/Output/abseil-duration-factory-scale.cpp.tmp.cpp:33:7: warning: use ZeroDuration() for zero-length time intervals [abseil-duration-factory-scale] d = absl::Seconds(0x0.000001p-126f); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ absl::ZeroDuration() /var/tmp/portage/sys-devel/clang-10.0.0/work/x/y/clang-abi_x86_32.x86/tools/extra/test/clang-tidy/checkers/Output/abseil-duration-factory-scale.cpp.tmp.cpp:33:7: note: FIX-IT applied suggested code changes /var/tmp/portage/sys-devel/clang-10.0.0/work/x/y/clang-abi_x86_32.x86/tools/extra/test/clang-tidy/checkers/Output/abseil-duration-factory-scale.cpp.tmp.cpp:36:7: warning: use ZeroDuration() for zero-length time intervals [abseil-duration-factory-scale] d = absl::Seconds(int{0}); ^~~~~~~~~~~~~~~~~~~~~ absl::ZeroDuration() /var/tmp/portage/sys-devel/clang-10.0.0/work/x/y/clang-abi_x86_32.x86/tools/extra/test/clang-tidy/checkers/Output/abseil-duration-factory-scale.cpp.tmp.cpp:36:7: note: FIX-IT applied suggested code changes /var/tmp/portage/sys-devel/clang-10.0.0/work/x/y/clang-abi_x86_32.x86/tools/extra/test/clang-tidy/checkers/Output/abseil-duration-factory-scale.cpp.tmp.cpp:39:7: warning: use ZeroDuration() for zero-length time intervals [abseil-duration-factory-scale] d = absl::Seconds(int64_t{0}); ^~~~~~~~~~~~~~~~~~~~~~~~~ absl::ZeroDuration() /var/tmp/portage/sys-devel/clang-10.0.0/work/x/y/clang-abi_x86_32.x86/tools/extra/test/clang-tidy/checkers/Output/abseil-duration-factory-scale.cpp.tmp.cpp:39:7: note: FIX-IT applied suggested code changes /var/tmp/portage/sys-devel/clang-10.0.0/work/x/y/clang-abi_x86_32.x86/tools/extra/test/clang-tidy/checkers/Output/abseil-duration-factory-scale.cpp.tmp.cpp:42:7: warning: use ZeroDuration() for zero-length time intervals [abseil-duration-factory-scale] d = absl::Seconds(float{0}); ^~~~~~~~~~~~~~~~~~~~~~~ absl::ZeroDuration() /var/tmp/portage/sys-devel/clang-10.0.0/work/x/y/clang-abi_x86_32.x86/tools/extra/test/clang-tidy/checkers/Output/abseil-duration-factory-scale.cpp.tmp.cpp:42:7: note: FIX-IT applied suggested code changes /var/tmp/portage/sys-devel/clang-10.0.0/work/x/y/clang-abi_x86_32.x86/tools/extra/test/clang-tidy/checkers/Output/abseil-duration-factory-scale.cpp.tmp.cpp:47:7: warning: internal duration scaling can be removed [abseil-duration-factory-scale] d = absl::Seconds(30 * 60); ^~~~~~~~~~~~~~~~~~~~~~ absl::Minutes(30) /var/tmp/portage/sys-devel/clang-10.0.0/work/x/y/clang-abi_x86_32.x86/tools/extra/test/clang-tidy/checkers/Output/abseil-duration-factory-scale.cpp.tmp.cpp:47:7: note: FIX-IT applied suggested code changes /var/tmp/portage/sys-devel/clang-10.0.0/work/x/y/clang-abi_x86_32.x86/tools/extra/test/clang-tidy/checkers/Output/abseil-duration-factory-scale.cpp.tmp.cpp:50:7: warning: internal duration scaling can be removed [abseil-duration-factory-scale] d = absl::Seconds(60 * 30); ^~~~~~~~~~~~~~~~~~~~~~ absl::Minutes(30) /var/tmp/portage/sys-devel/clang-10.0.0/work/x/y/clang-abi_x86_32.x86/tools/extra/test/clang-tidy/checkers/Output/abseil-duration-factory-scale.cpp.tmp.cpp:50:7: note: FIX-IT applied suggested code changes /var/tmp/portage/sys-devel/clang-10.0.0/work/x/y/clang-abi_x86_32.x86/tools/extra/test/clang-tidy/checkers/Output/abseil-duration-factory-scale.cpp.tmp.cpp:55:7: warning: internal duration scaling can be removed [abseil-duration-factory-scale] d = absl::Seconds(60 * 30 * 60); ^~~~~~~~~~~~~~~~~~~~~~~~~~~ absl::Minutes(60 * 30) /var/tmp/portage/sys-devel/clang-10.0.0/work/x/y/clang-abi_x86_32.x86/tools/extra/test/clang-tidy/checkers/Output/abseil-duration-factory-scale.cpp.tmp.cpp:55:7: note: FIX-IT applied suggested code changes /var/tmp/portage/sys-devel/clang-10.0.0/work/x/y/clang-abi_x86_32.x86/tools/extra/test/clang-tidy/checkers/Output/abseil-duration-factory-scale.cpp.tmp.cpp:61:7: warning: internal duration scaling can be removed [abseil-duration-factory-scale] d = absl::Milliseconds(30 * 1000); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~ absl::Seconds(30) /var/tmp/portage/sys-devel/clang-10.0.0/work/x/y/clang-abi_x86_32.x86/tools/extra/test/clang-tidy/checkers/Output/abseil-duration-factory-scale.cpp.tmp.cpp:61:7: note: FIX-IT applied suggested code changes /var/tmp/portage/sys-devel/clang-10.0.0/work/x/y/clang-abi_x86_32.x86/tools/extra/test/clang-tidy/checkers/Output/abseil-duration-factory-scale.cpp.tmp.cpp:69:7: warning: internal duration scaling can be removed [abseil-duration-factory-scale] d = absl::Seconds(5 * 3600); ^~~~~~~~~~~~~~~~~~~~~~~ absl::Hours(5) /var/tmp/portage/sys-devel/clang-10.0.0/work/x/y/clang-abi_x86_32.x86/tools/extra/test/clang-tidy/checkers/Output/abseil-duration-factory-scale.cpp.tmp.cpp:69:7: note: FIX-IT applied suggested code changes /var/tmp/portage/sys-devel/clang-10.0.0/work/x/y/clang-abi_x86_32.x86/tools/extra/test/clang-tidy/checkers/Output/abseil-duration-factory-scale.cpp.tmp.cpp:72:7: warning: internal duration scaling can be removed [abseil-duration-factory-scale] d = absl::Microseconds(5 * 1e6); ^~~~~~~~~~~~~~~~~~~~~~~~~~~ absl::Seconds(5) /var/tmp/portage/sys-devel/clang-10.0.0/work/x/y/clang-abi_x86_32.x86/tools/extra/test/clang-tidy/checkers/Output/abseil-duration-factory-scale.cpp.tmp.cpp:72:7: note: FIX-IT applied suggested code changes /var/tmp/portage/sys-devel/clang-10.0.0/work/x/y/clang-abi_x86_32.x86/tools/extra/test/clang-tidy/checkers/Output/abseil-duration-factory-scale.cpp.tmp.cpp:78:7: warning: internal duration scaling can be removed [abseil-duration-factory-scale] d = absl::Microseconds(5 * 1000000); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ absl::Seconds(5) /var/tmp/portage/sys-devel/clang-10.0.0/work/x/y/clang-abi_x86_32.x86/tools/extra/test/clang-tidy/checkers/Output/abseil-duration-factory-scale.cpp.tmp.cpp:78:7: note: FIX-IT applied suggested code changes clang-tidy applied 18 of 18 suggested fixes. Suppressed 7 warnings (7 with check filters). ------------------------------------------------------------------ ------------------------------ Fixes ----------------------------- --- /var/tmp/portage/sys-devel/clang-10.0.0/work/x/y/clang-abi_x86_32.x86/tools/extra/test/clang-tidy/checkers/Output/abseil-duration-factory-scale.cpp.tmp.cpp.orig 2020-06-28 23:45:25.676093497 +0200 +++ /var/tmp/portage/sys-devel/clang-10.0.0/work/x/y/clang-abi_x86_32.x86/tools/extra/test/clang-tidy/checkers/Output/abseil-duration-factory-scale.cpp.tmp.cpp 2020-06-28 23:45:25.702761011 +0200 @@ -9,56 +9,56 @@ absl::Duration d; // Zeroes - d = absl::Hours(0); + d = absl::ZeroDuration(); // // - d = absl::Minutes(0); + d = absl::ZeroDuration(); // // - d = absl::Seconds(0); + d = absl::ZeroDuration(); // // - d = absl::Milliseconds(0); + d = absl::ZeroDuration(); // // - d = absl::Microseconds(0); + d = absl::ZeroDuration(); // // - d = absl::Nanoseconds(0); + d = absl::ZeroDuration(); // // - d = absl::Seconds(0.0); + d = absl::ZeroDuration(); // // - d = absl::Seconds(0x0.000001p-126f); + d = absl::ZeroDuration(); // // - d = absl::Seconds(int{0}); + d = absl::ZeroDuration(); // // - d = absl::Seconds(int64_t{0}); + d = absl::ZeroDuration(); // // - d = absl::Seconds(float{0}); + d = absl::ZeroDuration(); // // // Fold seconds into minutes - d = absl::Seconds(30 * 60); + d = absl::Minutes(30); // // - d = absl::Seconds(60 * 30); + d = absl::Minutes(30); // // // Try a few more exotic multiplications - d = absl::Seconds(60 * 30 * 60); + d = absl::Minutes(60 * 30); // // d = absl::Seconds(1e-3 * 30); // // - d = absl::Milliseconds(30 * 1000); + d = absl::Seconds(30); // // d = absl::Milliseconds(30 * 0.001); @@ -66,16 +66,16 @@ // // Multiple steps - d = absl::Seconds(5 * 3600); + d = absl::Hours(5); // // - d = absl::Microseconds(5 * 1e6); + d = absl::Seconds(5); // // d = absl::Seconds(5 * 1e-6); // // - d = absl::Microseconds(5 * 1000000); + d = absl::Seconds(5); // // ------------------------------------------------------------------ FileCheck failed: /var/tmp/portage/sys-devel/clang-10.0.0/work/x/y/clang/tools/extra/test/clang-tidy/checkers/abseil-duration-factory-scale.cpp:60:19: error: CHECK-FIXES: expected string not found in input // CHECK-FIXES: absl::Milliseconds(30); ^ /var/tmp/portage/sys-devel/clang-10.0.0/work/x/y/clang-abi_x86_32.x86/tools/extra/test/clang-tidy/checkers/Output/abseil-duration-factory-scale.cpp.tmp.cpp:56:3: note: scanning from here // ^ /var/tmp/portage/sys-devel/clang-10.0.0/work/x/y/clang-abi_x86_32.x86/tools/extra/test/clang-tidy/checkers/Output/abseil-duration-factory-scale.cpp.tmp.cpp:64:7: note: possible intended match here d = absl::Milliseconds(30 * 0.001); ^ -- Command Output (stderr): -- + : 'RUN: at line 1' + /var/tmp/portage/sys-devel/clang-10.0.0/temp/python3.7/bin/python /var/tmp/portage/sys-devel/clang-10.0.0/work/x/y/clang/tools/extra/test/../test/clang-tidy/check_clang_tidy.py /var/tmp/portage/sys-devel/clang-10.0.0/work/x/y/clang/tools/extra/test/clang-tidy/checkers/abseil-duration-factory-scale.cpp abseil-duration-factory-scale /var/tmp/portage/sys-devel/clang-10.0.0/work/x/y/clang-abi_x86_32.x86/tools/extra/test/clang-tidy/checkers/Output/abseil-duration-factory-scale.cpp.tmp -- -- -I/var/tmp/portage/sys-devel/clang-10.0.0/work/x/y/clang/tools/extra/test/clang-tidy/checkers/Inputs Traceback (most recent call last): File "/var/tmp/portage/sys-devel/clang-10.0.0/work/x/y/clang/tools/extra/test/../test/clang-tidy/check_clang_tidy.py", line 249, in <module> main() File "/var/tmp/portage/sys-devel/clang-10.0.0/work/x/y/clang/tools/extra/test/../test/clang-tidy/check_clang_tidy.py", line 245, in main run_test_once(args, extra_args) File "/var/tmp/portage/sys-devel/clang-10.0.0/work/x/y/clang/tools/extra/test/../test/clang-tidy/check_clang_tidy.py", line 171, in run_test_once stderr=subprocess.STDOUT) File "/usr/lib/python3.7/subprocess.py", line 411, in check_output **kwargs).stdout File "/usr/lib/python3.7/subprocess.py", line 512, in run output=stdout, stderr=stderr) subprocess.CalledProcessError: Command '['FileCheck', '-input-file=/var/tmp/portage/sys-devel/clang-10.0.0/work/x/y/clang-abi_x86_32.x86/tools/extra/test/clang-tidy/checkers/Output/abseil-duration-factory-scale.cpp.tmp.cpp', '/var/tmp/portage/sys-devel/clang-10.0.0/work/x/y/clang/tools/extra/test/clang-tidy/checkers/abseil-duration-factory-scale.cpp', '-check-prefixes=CHECK-FIXES', '-strict-whitespace']' returned non-zero exit status 1. -- ******************** *** Bug 743829 has been marked as a duplicate of this bug. *** Still failing in 11.0.0. |