[PATCH] elf: Suppress clang warning on tst-unique4
Sam James
sam@gentoo.org
Fri Dec 27 06:47:13 GMT 2024
Florian Weimer <fweimer@redhat.com> writes:
> * H. J. Lu:
>
>> From: Adhemerval Zanella <adhemerval.zanella@linaro.org>
>>
>> clang warns that the instantiation of the variable is required, but no
>> definition is available. They are implemented on tst-unique4lib.so.
>>
>> Checked on x86_64-linux-gnu.
>> ---
>> elf/tst-unique4.cc | 6 ++++++
>> elf/tst-unique4lib.cc | 2 +-
>> 2 files changed, 7 insertions(+), 1 deletion(-)
>>
>> diff --git a/elf/tst-unique4.cc b/elf/tst-unique4.cc
>> index 575c70d3a1..ab727b8dbb 100644
>> --- a/elf/tst-unique4.cc
>> +++ b/elf/tst-unique4.cc
>> @@ -2,7 +2,12 @@
>> #include "tst-unique4.h"
>>
>> #include <cstdio>
>> +#include <libc-diag.h>
>>
>> +/* clang warns that the instatiation of the variable is required, but no
>> + definition is available. They are implemented on tst-unique4lib.so. */
>> +DIAG_PUSH_NEEDS_COMMENT_CLANG;
>> +DIAG_IGNORE_NEEDS_COMMENT_CLANG (13, "-Wundefined-var-template");
>> static int a[24] =
>> {
>> S<1>::i, S<2>::i, S<3>::i, S<4>::i, S<5>::i, S<6>::i, S<7>::i, S<8>::i,
>> @@ -10,6 +15,7 @@ static int a[24] =
>> S<16>::i, S<17>::i, S<18>::i, S<19>::i, S<20>::i, S<21>::i, S<22>::i,
>> S<23>::i, S<24>::i
>> };
>> +DIAG_POP_NEEDS_COMMENT_CLANG;
>
> Could a portable version of this use explicit template instantiations?
https://inbox.sourceware.org/libc-alpha/CAMe9rOpagS=d=HBERSWcTcimrQg23kDOtzD5erkNWfU1_9S6NQ@mail.gmail.com/
thanks,
sam
More information about the Libc-alpha
mailing list