Странный код не собирающийся clang-ом

@alecss131 · Регистрация: 11.08.2017

Author24 — интернет-сервис помощи студентам

Решил поиграться с apache log4cxx. Ради интереса решил собрать с помощью clang (а именно clang-cl). Библиотека использует с++17 и студийным (msvc) компилятором собирается без проблем, с теми же настройками. Уже собранная библиотека (как зависимость) в проекте собирается без проблем с помощью clang
При сборке ругается

Код

error: explicit instantiation of 'shared_ptr' not in a namespace enclosing 'std'

место в коде куда ссылается

C++

1 2	// Instantiate template pointer types passed as parameters LOG4CXX_INSTANTIATE_EXPORTED_PTR(log4cxx::spi::Filter);

и содержимое этого макроса

C++

1	#define LOG4CXX_INSTANTIATE_EXPORTED_PTR(T) template class LOG4CXX_EXPORT std::shared_ptr<T>

LOG4CXX_EXPORT в данном случае макро раскрывающееся в пустоту, возможный вариант __declspec(dllexport)
Как это исправить? Я не понимаю что тут происходит, так как такой конструкции с template не встречал. Гугл особо не помогает, нашел статью что так делать не стоит, это если по ошибке гуглить, а что это за тема не пойму.
IDE подсказывает что макро раскрывается в

C++

1 2	template class std::shared_ptr<log4cxx::spi::Filter>

В моем понимании после template должно идти <...>, пусть даже со специализацией
Я пока временно исправил так, но думаю что это не правильно

C++

1	#define LOG4CXX_INSTANTIATE_EXPORTED_PTR(T) LOG4CXX_EXPORT std::shared_ptr<T>

Странно что этот макрос используется без использования в коде в дальнейшем.

@zayats80888 · 15.05.2023, 12:05

Сообщение от alecss131

В моем понимании после template должно идти <...>, пусть даже со специализацией

Это не специализация, а явное инстанцирование. https://en.cppreference.com/w/... s_template
А ошибка скорее всего связана с тем, что это инстанцирование сделано в другом нэймспэйсе.

Добавлено через 4 минуты

Сообщение от alecss131

Как это исправить?

Перенести эту строчку в глобальный нэймспэйс.

@alecss131 · 15.05.2023, 12:12 **[ТС]**

Сообщение от zayats80888

Перенести эту строчку в глобальный нэймспэйс.

Теперь IDE выдает предупреждение

Код

Clang-Tidy: Modification of 'std' namespace can result in undefined behavior

Помимо этого места еще в 2 местах пришлось вынести 3 использования, но зато теперь собирается. Надеюсь ничего не сломалось

Добавлено через 3 минуты

Сообщение от zayats80888

что это инстанцирование сделано в другом нэймспэйсе.

Было так в случае из первого сообщения темы

C++

namespace log4cxx
{
 
class File;
 
namespace helpers
{
class Pool;
}
 
 
namespace rolling
{
// Instantiate template pointer types passed as parameters
LOG4CXX_INSTANTIATE_EXPORTED_PTR(log4cxx::spi::Filter);
 
//...
//другой код
//...
}
}

Странно что студия такое пропускает

@zayats80888 · 15.05.2023, 12:17

Сообщение от alecss131

Надеюсь ничего не сломалось

Вообще это похоже на экспорт инстанса шаблона, если использовать эту библиотеку с тем же с++ рантаймом, с которым она компилировалась, то проблем быть не должно.

Добавлено через 2 минуты

Не по теме:

Сообщение от alecss131

Странно что студия такое пропускает

А может в студии там экспортировался не std::shared_ptr, а какой-то свой умный указатель? Там других макросов условной компиляции нет?

@TheCalligrapher · 15.05.2023, 16:44

Простейший пример

C++

namespace N
{
  template <typename> struct S {};
}
 
namespace M
{
  template struct N::S<int>;
}
 
int main()
{
}

и Clang 5 тоже пропускает. Ошибка генерируется начиная с Clang 6.

@zayats80888 6340 / 3511 / 1427 Регистрация: 07.02.2019 Сообщений: 8,977
	15.05.2023, 12:05	2
	Сообщение от alecss131 В моем понимании после template должно идти <...>, пусть даже со специализацией Это не специализация, а явное инстанцирование. https://en.cppreference.com/w/... s_template А ошибка скорее всего связана с тем, что это инстанцирование сделано в другом нэймспэйсе. Добавлено через 4 минуты Сообщение от alecss131 Как это исправить? Перенести эту строчку в глобальный нэймспэйс. 1

@zayats80888 6340 / 3511 / 1427 Регистрация: 07.02.2019 Сообщений: 8,977
	15.05.2023, 12:17	4
	Сообщение от alecss131 Надеюсь ничего не сломалось Вообще это похоже на экспорт инстанса шаблона, если использовать эту библиотеку с тем же с++ рантаймом, с которым она компилировалась, то проблем быть не должно. Добавлено через 2 минуты Не по теме: Сообщение от alecss131 Странно что студия такое пропускает А может в студии там экспортировался не `std::shared_ptr`, а какой-то свой умный указатель? Там других макросов условной компиляции нет? 0

	15.05.2023, 16:44