Windows gcc OpenCL link errors: multiple definition

[noobie-iv]

SET PATH=^
d:\Programs\Prog\w64devkit-14.2.0\bin;^
d:\Programs\Prog\CMake-3.27.4\bin

SET DEPS=^
d:\Lib\boost\1.83.0\w64devkit-14.2.0;^
d:\Lib\Eigen\3.4.0;^
d:\Lib\libjpeg-turbo\3.1.0\w64devkit-14.2.0-Release;^
d:\Lib\libpng\1.6.47\w64devkit-14.2.0-Release;^
d:\Lib\libtiff\4.7.0\w64devkit-14.2.0-Release;^
d:\Lib\OpenCL\2024.10.24\w64devkit-14.2.0-Release;^
d:\Lib\Qt\5.12.12\w64devkit-14.2.0-Release;^
d:\Lib\zlib\1.3.1\w64devkit-14.2.0-Release

CMAKE^
 -S .\src^
 -B .\build\w64devkit-14.2.0-Release^
 -G "MinGW Makefiles"^
 -DCMAKE_BUILD_TYPE=Release^
 -DCMAKE_PREFIX_PATH:PATH=%DEPS%^
 -DENABLE_OPENGL:BOOL=ON^
 -DENABLE_OPENCL:BOOL=ON

CMAKE^
 --build .\build\w64devkit-14.2.0-Release^
 --parallel

500 lines of errors:

D:\Programs\Prog\gcc\bin/ld.exe: ../libopencl.a(OpenCLAffineTransform.cpp.obj):OpenCLAffineTransform.cpp:(.data$_ZN2cl8Platform8default_E+0x0): multiple definition of `.weak._ZN2cl8Platform8default_E._ZNK2cl5Error4whatEv'; CMakeFiles\opencl_tests.dir/objects.a(TestCopy.cpp.obj):TestCopy.cpp:(.data$_ZN2cl8Platform8default_E+0x0): first defined here
D:\Programs\Prog\gcc\bin/ld.exe: ../libopencl.a(OpenCLAffineTransform.cpp.obj):OpenCLAffineTransform.cpp:(.data$_ZN2cl8Platform20default_initialized_E+0x0): multiple definition of `.weak._ZN2cl8Platform20default_initialized_E._ZNK2cl5Error4whatEv'; CMakeFiles\opencl_tests.dir/objects.a(TestCopy.cpp.obj):TestCopy.cpp:(.data$_ZN2cl8Platform20default_initialized_E+0x0): first defined here
D:\Programs\Prog\gcc\bin/ld.exe: ../libopencl.a(OpenCLAffineTransform.cpp.obj):OpenCLAffineTransform.cpp:(.data$_ZN2cl6Device14default_error_E+0x0): multiple definition of `.weak._ZN2cl6Device14default_error_E._ZNK2cl5Error4whatEv'; CMakeFiles\opencl_tests.dir/objects.a(TestCopy.cpp.obj):TestCopy.cpp:(.data$_ZN2cl6Device14default_error_E+0x0): first defined here
D:\Programs\Prog\gcc\bin/ld.exe: ../libopencl.a(OpenCLAffineTransform.cpp.obj):OpenCLAffineTransform.cpp:(.data$_ZN2cl6Device8default_E+0x0): multiple definition of `.weak._ZN2cl6Device8default_E._ZNK2cl5Error4whatEv'; CMakeFiles\opencl_tests.dir/objects.a(TestCopy.cpp.obj):TestCopy.cpp:(.data$_ZN2cl6Device8default_E+0x0): first defined here
D:\Programs\Prog\gcc\bin/ld.exe: ../libopencl.a(OpenCLAffineTransform.cpp.obj):OpenCLAffineTransform.cpp:(.data$_ZN2cl6Device20default_initialized_E+0x0): multiple definition of `.weak._ZN2cl6Device20default_initialized_E._ZNK2cl5Error4whatEv'; CMakeFiles\opencl_tests.dir/objects.a(TestCopy.cpp.obj):TestCopy.cpp:(.data$_ZN2cl6Device20default_initialized_E+0x0): first defined here
...

Solution: alibaba/MNN#530

[zvezdochiot]

alibaba/MNN#530 say:

#if defined(_MSC_VER)
    #define CL_HPP_DEFINE_STATIC_MEMBER_ __declspec(selectany)
#else
    #ifdef _WIN32
        #define CL_HPP_DEFINE_STATIC_MEMBER_ inline
    #else
        #define CL_HPP_DEFINE_STATIC_MEMBER_ __attribute__((weak))
    #endif // _WIN32
#endif // !_MSC_VER

[plzombie]

@zvezdochiot @noobie-iv В версии SDK 2023.12.14, что у меня, сейчас вот такой код

#if __cplusplus >= 201703L
# define CL_HPP_DEFINE_STATIC_MEMBER_ inline
#elif defined(_MSC_VER)
# define CL_HPP_DEFINE_STATIC_MEMBER_ __declspec(selectany)
#elif defined(__MINGW32__)
# define CL_HPP_DEFINE_STATIC_MEMBER_ __attribute__((selectany))
#else
# define CL_HPP_DEFINE_STATIC_MEMBER_ __attribute__((weak))
#endif // !_MSC_VER

[zvezdochiot]

@plzombie say:

такой код

Переформатирую:

#if __cplusplus >= 201703L
    #define CL_HPP_DEFINE_STATIC_MEMBER_ inline
#elif defined(_MSC_VER)
    #define CL_HPP_DEFINE_STATIC_MEMBER_ __declspec(selectany)
#elif defined(__MINGW32__)
    #define CL_HPP_DEFINE_STATIC_MEMBER_ __attribute__((selectany))
#else
    #define CL_HPP_DEFINE_STATIC_MEMBER_ __attribute__((weak))
#endif // !_MSC_VER

[plzombie]

Я не понимаю, зачем вообще там OpenCL лежит в папке src/acceleration/opencl/khronos/, если его всё равно нужно при сборке указывать. Надо или обновить, или выкинуть этот код и переписать под SDK

//
// Copyright (c) 2020 The Khronos Group Inc.
//
// Licensed under the Apache License, Version 2.0 (the "License");
// you may not use this file except in compliance with the License.
// You may obtain a copy of the License at
//
//    http://www.apache.org/licenses/LICENSE-2.0
//
// Unless required by applicable law or agreed to in writing, software
// distributed under the License is distributed on an "AS IS" BASIS,
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
// See the License for the specific language governing permissions and
// limitations under the License.
//

#include <CL/opencl.hpp>
#pragma message("cl2.hpp has been renamed to opencl.hpp to make it clear that it supports all versions of OpenCL. Please include opencl.hpp directly.")

Собственно, заменить cl2.hpp на opencl.hpp, или не заменять (для совместимости). А эту папку просто удалить (со всеми ссылками в cmake).

[zvezdochiot]

@plzombie say:

А эту папку просто удалить (со всеми ссылками в cmake).

Вообще не вопрос. Ежели будет собираться, так лишнего и не надо.

[plzombie]

# We ship Khronos versions of OpenCL headers to work-around issues
# with NVidia's OpenCL headers. Even the Khronos headers weren't
# problem-free. The winning combination was cl2.hpp from OpenCL 2.1
# and the rest of the headers from OpenCL 1.1.
FILE(GLOB khronos_headers khronos/CL/*.h khronos/CL/*.hpp)

Вот меня в папке коммент смущает. Надо потестить ещё будет с включённым и отключённым OpenCL

[noobie-iv]

@plzombie

зачем вообще там OpenCL
Вот меня в папке коммент смущает

Там и про фотошоп такие же обидные комментарии были. И под все библиотеки самодельные cmake-проекты, взамен оригинальных. Предполагалось, видимо, раз в год это все рихтовать, или вообще заново переделывать. Колхозные cmake мы уже выбросили, когда первая сборка под win была. Наверное, и остальное можно следом отправлять.

[zvezdochiot]

@noobie-iv say:

остальное можно следом отправлять.

Это касается и crash_reporter. Он для чего сейчас в репе? Надобности в нём совершенно никакой. Всё же на git.

PS: А каким образом libtiff вообще в секции OpenCL оказался? Что то здесь совсем нечисто.

[noobie-iv]

@zvezdochiot @plzombie

Вырезал старые заголовки OpenCL: 88ebfdf. Компилится, работает, разницы не вижу. Пара вопросов:

1. Пришлось добавить параметр OPENCL_VERSION=1.10 для конфигурации cmake при сборке, потому что заголовки требовали пользовательский макрос CL_TARGET_OPENCL_VERSION, и без него выставляли версию в 3.0. Возможно, это завышенные требования, и на всякий случай собираться надо со значением пониже.
2. В файле OpenCLGaussBlur.cpp пришлось вырезать в трех местах вызовы device.getInfo<CL_DEVICE_HOST_UNIFIED_MEMORY>, потому что CL_DEVICE_HOST_UNIFIED_MEMORY объявлен устаревшим. На что его заменить - не нагуглил сходу. Возможно, отломалась какая-нибудь оптимизация.

[zvezdochiot]

@noobie-iv say:

Вырезал старые заголовки OpenCL

@plzombie должон что то знать. Он там boost вырезал, должен был и на "особенности" opencl натыкаться.

[plzombie]

@zvezdochiot Я тут другой баг заметил. Геометрические искажения стоит на "Без искажений". Нажимаю на пакетную обработку, ставится "Кривые" и надо руками обратно сбрасывать. Пересохранение не помогает. Есть только на одном старом проекте.