[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