[llvm] [clang] [clang-tools-extra] [Clang][AArch64] Add fix vector types to header into SVE (PR #73258)
David Spickett via cfe-commits
cfe-commits at lists.llvm.org
Wed Dec 13 03:24:36 PST 2023
DavidSpickett wrote:
I think this caused a test suite program to fail to build:
```
cd /home/tcwg-buildbot/worker/clang-aarch64-global-isel/test/sandbox/build/SingleSource/UnitTests && /home/tcwg-buildbot/worker/clang-aarch64-global-isel/stage1.install/bin/llvm-size --format=sysv /home/tcwg-buildbot/worker/clang-aarch64-global-isel/test/sandbox/build/SingleSource/UnitTests/testcase-CGExprConstant > /home/tcwg-buildbot/worker/clang-aarch64-global-isel/test/sandbox/build/SingleSource/UnitTests/testcase-CGExprConstant.size
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~
38 | (value))
| ~~~~~~~~
/home/tcwg-buildbot/worker/clang-aarch64-global-isel/test/test-suite/SingleSource/UnitTests/Float/test_isfpclass.h:28:11: note: expanded from macro 'CHECK_CLASS'
28 | if (!(cond)) { \
| ^~~~
/home/tcwg-buildbot/worker/clang-aarch64-global-isel/test/test-suite/SingleSource/UnitTests/Float/test_isfpclass.h:58:3: error: floating point classification requires argument of floating point type (passed in 'int')
58 | CHECK(fcFinite, x);
| ^~~~~~~~~~~~~~~~~~
/home/tcwg-buildbot/worker/clang-aarch64-global-isel/test/test-suite/SingleSource/UnitTests/Float/test_isfpclass.h:37:60: note: expanded from macro 'CHECK'
37 | CHECK_CLASS(!!((flags)&FPCLASS) == !!__builtin_isfpclass((flags), (value)), \
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~
38 | (value))
| ~~~~~~~~
/home/tcwg-buildbot/worker/clang-aarch64-global-isel/test/test-suite/SingleSource/UnitTests/Float/test_isfpclass.h:28:11: note: expanded from macro 'CHECK_CLASS'
28 | if (!(cond)) { \
| ^~~~
/home/tcwg-buildbot/worker/clang-aarch64-global-isel/test/test-suite/SingleSource/UnitTests/Float/test_isfpclass.h:60:3: error: floating point classification requires argument of floating point type (passed in 'int')
60 | CHECK(fcQNan | fcInf | fcNormal | fcSubnormal | fcZero, x);
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/home/tcwg-buildbot/worker/clang-aarch64-global-isel/test/test-suite/SingleSource/UnitTests/Float/test_isfpclass.h:37:60: note: expanded from macro 'CHECK'
37 | CHECK_CLASS(!!((flags)&FPCLASS) == !!__builtin_isfpclass((flags), (value)), \
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~
38 | (value))
| ~~~~~~~~
/home/tcwg-buildbot/worker/clang-aarch64-global-isel/test/test-suite/SingleSource/UnitTests/Float/test_isfpclass.h:28:11: note: expanded from macro 'CHECK_CLASS'
28 | if (!(cond)) { \
| ^~~~
```
https://lab.llvm.org/buildbot/#/builders/183/builds/18017
I'm guessing it includes a header effected by this change.
https://github.com/llvm/llvm-project/pull/73258
More information about the cfe-commits
mailing list