[libc-commits] [libc] [libc] Use cpp::numeric_limits in preference to C23 <limits.h> macros (PR #102665)

LLVM Continuous Integration via libc-commits libc-commits at lists.llvm.org
Fri Aug 9 12:50:40 PDT 2024 

llvm-ci wrote:

LLVM Buildbot has detected a new failure on builder `libc-riscv64-debian-fullbuild-dbg` running on `libc-riscv64-debian` while building `libc` at step 4 "annotate".

Full details are available at: https://lab.llvm.org/buildbot/#/builders/183/builds/2517

Here is the relevant piece of the build log for the reference:
```
Step 4 (annotate) failure: 'python ../llvm-zorg/zorg/buildbot/builders/annotated/libc-linux.py ...' (failure)
...
[       OK ] LlvmLibcStdcBitceilUllTest.Ones (20 us)
[ RUN      ] LlvmLibcStdcBitceilUllTest.OneLessThanPowsTwo
[       OK ] LlvmLibcStdcBitceilUllTest.OneLessThanPowsTwo (19 us)
[ RUN      ] LlvmLibcStdcBitceilUllTest.OneMoreThanPowsTwo
[       OK ] LlvmLibcStdcBitceilUllTest.OneMoreThanPowsTwo (19 us)
Ran 4 tests.  PASS: 4  FAIL: 0
[738/988] Building CXX object projects/libc/test/src/stdbit/CMakeFiles/libc.test.src.stdbit.stdc_bit_floor_ui_test.__unit__.__build__.dir/stdc_bit_floor_ui_test.cpp.o
[739/988] Building CXX object projects/libc/test/src/stdbit/CMakeFiles/libc.test.src.stdbit.stdc_leading_zeros_ui_test.__unit__.__build__.dir/stdc_leading_zeros_ui_test.cpp.o
[740/988] Linking CXX executable projects/libc/test/src/stdbit/libc.test.src.stdbit.stdc_leading_zeros_ui_test.__unit__.__build__
[741/988] Running unit test libc.test.src.stdbit.stdc_leading_zeros_ui_test.__unit__
FAILED: projects/libc/test/src/stdbit/CMakeFiles/libc.test.src.stdbit.stdc_leading_zeros_ui_test.__unit__ /home/libc_worker/libc-riscv64-debian/libc-riscv64-debian-fullbuild-dbg/build/projects/libc/test/src/stdbit/CMakeFiles/libc.test.src.stdbit.stdc_leading_zeros_ui_test.__unit__ 
cd /home/libc_worker/libc-riscv64-debian/libc-riscv64-debian-fullbuild-dbg/build/projects/libc/test/src/stdbit && /home/libc_worker/libc-riscv64-debian/libc-riscv64-debian-fullbuild-dbg/build/projects/libc/test/src/stdbit/libc.test.src.stdbit.stdc_leading_zeros_ui_test.__unit__.__build__
[==========] Running 2 tests from 1 test suite.
[ RUN      ] LlvmLibcStdcLeadingZerosUiTest.Zero
/home/libc_worker/libc-riscv64-debian/libc-riscv64-debian-fullbuild-dbg/llvm-project/libc/test/src/stdbit/stdc_leading_zeros_ui_test.cpp:17: FAILURE
      Expected: __llvm_libc_18_0_0_git::stdc_leading_zeros_ui(0U)
      Which is: 32
To be equal to: static_cast<unsigned>(__llvm_libc_18_0_0_git::cpp::numeric_limits<int>::digits)
      Which is: 31
[  FAILED  ] LlvmLibcStdcLeadingZerosUiTest.Zero
[ RUN      ] LlvmLibcStdcLeadingZerosUiTest.OneHot
/home/libc_worker/libc-riscv64-debian/libc-riscv64-debian-fullbuild-dbg/llvm-project/libc/test/src/stdbit/stdc_leading_zeros_ui_test.cpp:24: FAILURE
      Expected: __llvm_libc_18_0_0_git::stdc_leading_zeros_ui(1U << i)
      Which is: 31
To be equal to: __llvm_libc_18_0_0_git::cpp::numeric_limits<int>::digits - i - 1
      Which is: 30
/home/libc_worker/libc-riscv64-debian/libc-riscv64-debian-fullbuild-dbg/llvm-project/libc/test/src/stdbit/stdc_leading_zeros_ui_test.cpp:24: FAILURE
      Expected: __llvm_libc_18_0_0_git::stdc_leading_zeros_ui(1U << i)
      Which is: 30
To be equal to: __llvm_libc_18_0_0_git::cpp::numeric_limits<int>::digits - i - 1
      Which is: 29
/home/libc_worker/libc-riscv64-debian/libc-riscv64-debian-fullbuild-dbg/llvm-project/libc/test/src/stdbit/stdc_leading_zeros_ui_test.cpp:24: FAILURE
      Expected: __llvm_libc_18_0_0_git::stdc_leading_zeros_ui(1U << i)
      Which is: 29
To be equal to: __llvm_libc_18_0_0_git::cpp::numeric_limits<int>::digits - i - 1
      Which is: 28
/home/libc_worker/libc-riscv64-debian/libc-riscv64-debian-fullbuild-dbg/llvm-project/libc/test/src/stdbit/stdc_leading_zeros_ui_test.cpp:24: FAILURE
      Expected: __llvm_libc_18_0_0_git::stdc_leading_zeros_ui(1U << i)
      Which is: 28
To be equal to: __llvm_libc_18_0_0_git::cpp::numeric_limits<int>::digits - i - 1
      Which is: 27
/home/libc_worker/libc-riscv64-debian/libc-riscv64-debian-fullbuild-dbg/llvm-project/libc/test/src/stdbit/stdc_leading_zeros_ui_test.cpp:24: FAILURE
      Expected: __llvm_libc_18_0_0_git::stdc_leading_zeros_ui(1U << i)
      Which is: 27
To be equal to: __llvm_libc_18_0_0_git::cpp::numeric_limits<int>::digits - i - 1
      Which is: 26
/home/libc_worker/libc-riscv64-debian/libc-riscv64-debian-fullbuild-dbg/llvm-project/libc/test/src/stdbit/stdc_leading_zeros_ui_test.cpp:24: FAILURE
      Expected: __llvm_libc_18_0_0_git::stdc_leading_zeros_ui(1U << i)
      Which is: 26
Step 8 (libc-unit-tests) failure: libc-unit-tests (failure)
...
[       OK ] LlvmLibcStdcBitceilUllTest.Ones (20 us)
[ RUN      ] LlvmLibcStdcBitceilUllTest.OneLessThanPowsTwo
[       OK ] LlvmLibcStdcBitceilUllTest.OneLessThanPowsTwo (19 us)
[ RUN      ] LlvmLibcStdcBitceilUllTest.OneMoreThanPowsTwo
[       OK ] LlvmLibcStdcBitceilUllTest.OneMoreThanPowsTwo (19 us)
Ran 4 tests.  PASS: 4  FAIL: 0
[738/988] Building CXX object projects/libc/test/src/stdbit/CMakeFiles/libc.test.src.stdbit.stdc_bit_floor_ui_test.__unit__.__build__.dir/stdc_bit_floor_ui_test.cpp.o
[739/988] Building CXX object projects/libc/test/src/stdbit/CMakeFiles/libc.test.src.stdbit.stdc_leading_zeros_ui_test.__unit__.__build__.dir/stdc_leading_zeros_ui_test.cpp.o
[740/988] Linking CXX executable projects/libc/test/src/stdbit/libc.test.src.stdbit.stdc_leading_zeros_ui_test.__unit__.__build__
[741/988] Running unit test libc.test.src.stdbit.stdc_leading_zeros_ui_test.__unit__
FAILED: projects/libc/test/src/stdbit/CMakeFiles/libc.test.src.stdbit.stdc_leading_zeros_ui_test.__unit__ /home/libc_worker/libc-riscv64-debian/libc-riscv64-debian-fullbuild-dbg/build/projects/libc/test/src/stdbit/CMakeFiles/libc.test.src.stdbit.stdc_leading_zeros_ui_test.__unit__ 
cd /home/libc_worker/libc-riscv64-debian/libc-riscv64-debian-fullbuild-dbg/build/projects/libc/test/src/stdbit && /home/libc_worker/libc-riscv64-debian/libc-riscv64-debian-fullbuild-dbg/build/projects/libc/test/src/stdbit/libc.test.src.stdbit.stdc_leading_zeros_ui_test.__unit__.__build__
[==========] Running 2 tests from 1 test suite.
[ RUN      ] LlvmLibcStdcLeadingZerosUiTest.Zero
/home/libc_worker/libc-riscv64-debian/libc-riscv64-debian-fullbuild-dbg/llvm-project/libc/test/src/stdbit/stdc_leading_zeros_ui_test.cpp:17: FAILURE
      Expected: __llvm_libc_18_0_0_git::stdc_leading_zeros_ui(0U)
      Which is: 32
To be equal to: static_cast<unsigned>(__llvm_libc_18_0_0_git::cpp::numeric_limits<int>::digits)
      Which is: 31
[  FAILED  ] LlvmLibcStdcLeadingZerosUiTest.Zero
[ RUN      ] LlvmLibcStdcLeadingZerosUiTest.OneHot
/home/libc_worker/libc-riscv64-debian/libc-riscv64-debian-fullbuild-dbg/llvm-project/libc/test/src/stdbit/stdc_leading_zeros_ui_test.cpp:24: FAILURE
      Expected: __llvm_libc_18_0_0_git::stdc_leading_zeros_ui(1U << i)
      Which is: 31
To be equal to: __llvm_libc_18_0_0_git::cpp::numeric_limits<int>::digits - i - 1
      Which is: 30
/home/libc_worker/libc-riscv64-debian/libc-riscv64-debian-fullbuild-dbg/llvm-project/libc/test/src/stdbit/stdc_leading_zeros_ui_test.cpp:24: FAILURE
      Expected: __llvm_libc_18_0_0_git::stdc_leading_zeros_ui(1U << i)
      Which is: 30
To be equal to: __llvm_libc_18_0_0_git::cpp::numeric_limits<int>::digits - i - 1
      Which is: 29
/home/libc_worker/libc-riscv64-debian/libc-riscv64-debian-fullbuild-dbg/llvm-project/libc/test/src/stdbit/stdc_leading_zeros_ui_test.cpp:24: FAILURE
      Expected: __llvm_libc_18_0_0_git::stdc_leading_zeros_ui(1U << i)
      Which is: 29
To be equal to: __llvm_libc_18_0_0_git::cpp::numeric_limits<int>::digits - i - 1
      Which is: 28
/home/libc_worker/libc-riscv64-debian/libc-riscv64-debian-fullbuild-dbg/llvm-project/libc/test/src/stdbit/stdc_leading_zeros_ui_test.cpp:24: FAILURE
      Expected: __llvm_libc_18_0_0_git::stdc_leading_zeros_ui(1U << i)
      Which is: 28
To be equal to: __llvm_libc_18_0_0_git::cpp::numeric_limits<int>::digits - i - 1
      Which is: 27
/home/libc_worker/libc-riscv64-debian/libc-riscv64-debian-fullbuild-dbg/llvm-project/libc/test/src/stdbit/stdc_leading_zeros_ui_test.cpp:24: FAILURE
      Expected: __llvm_libc_18_0_0_git::stdc_leading_zeros_ui(1U << i)
      Which is: 27
To be equal to: __llvm_libc_18_0_0_git::cpp::numeric_limits<int>::digits - i - 1
      Which is: 26
/home/libc_worker/libc-riscv64-debian/libc-riscv64-debian-fullbuild-dbg/llvm-project/libc/test/src/stdbit/stdc_leading_zeros_ui_test.cpp:24: FAILURE
      Expected: __llvm_libc_18_0_0_git::stdc_leading_zeros_ui(1U << i)
      Which is: 26

```

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

More information about the libc-commits mailing list