| Message ID | 20180411161452.25043-3-valentin.korenblit@smile.fr |
|---|---|
| State | Superseded |
| Headers | show |
| Series | llvm for mesa3d | expand |
Hello, On Wed, 11 Apr 2018 18:14:50 +0200, Valentin Korenblit wrote: > diff --git a/package/Config.in b/package/Config.in > index 62770790a1..f376fad3a2 100644 > --- a/package/Config.in > +++ b/package/Config.in > @@ -283,6 +283,7 @@ comment "Graphic libraries" > source "package/fbv/Config.in" > source "package/freerdp/Config.in" > source "package/imagemagick/Config.in" > + source "package/libclc/Config.in" > source "package/linux-fusion/Config.in" > source "package/lite/Config.in" > source "package/mesa3d/Config.in" > diff --git a/package/libclc/Config.in b/package/libclc/Config.in > new file mode 100644 > index 0000000000..a0f388c4e1 > --- /dev/null > +++ b/package/libclc/Config.in > @@ -0,0 +1,22 @@ > +config BR2_PACKAGE_LIBCLC > + bool "libclc" > + depends on BR2_PACKAGE_LLVM_ARCH_SUPPORTS > + depends on BR2_HOST_GCC_AT_LEAST_4_8 > + depends on BR2_TOOLCHAIN_GCC_AT_LEAST_4_8 > + depends on BR2_TOOLCHAIN_HAS_THREADS > + depends on BR2_INSTALL_LIBSTDCPP > + depends on !BR2_TOOLCHAIN_HAS_GCC_BUG_64735 # std::shared_future > + depends on !BR2_STATIC_LIBS > + depends on BR2_USE_WCHAR # std::wstring You seem to have replicated here all the dependencies of the target LLVM package. However, you only depend on host-llvm and host-clang. Are you sure all those dependencies are needed by libclc ? > +# There are only two releases: release_35 and release_38, but the last > +# commit is from 2 years ago. Master has some recent activity. > +LIBCLC_VERSION = 00236279a293b3737dee08c14f25923a889d2795 > +LIBCLC_SITE = https://git.llvm.org/git/libclc > +LIBCLC_SITE_METHOD = git > +LIBCLC_LICENSE = NCSA > +LIBCLC_LICENSE_FILES = LICENSE.TXT > + > +LIBCLC_DEPENDENCIES = host-clang host-llvm > +LIBCLC_INSTALL_STAGING = YES > + > +# C++ compiler is used to build a small tool (prepare-builtins) for the host. > +# It must be built with the C++ compiler from the host > +LIBCLC_CONF_OPTS = --with-llvm-config=$(HOST_DIR)/usr/bin/llvm-config \ > + --prefix="/usr" \ > + --includedir="/usr/local/include" \ I'm not a big fan of /usr/local/include. Perhaps we should use something really special like /usr/share/libclc/ or something like this ? > + --pkgconfigdir="/usr/lib/pkgconfig" \ > + --with-cxx-compiler=$(HOSTCXX) Weird for a target package to use the host compiler, no ? > +define LIBCLC_CONFIGURE_CMDS > + (cd $(@D); $(TARGET_CONFIGURE_OPTS) ./configure.py $(LIBCLC_CONF_OPTS)) Is the configure.py script compatible with both Python 2.x and Python 3.x ? Best regards, Thomas
Hello Thomas, On 28/04/2018 15:06, Thomas Petazzoni wrote: > Hello, > > On Wed, 11 Apr 2018 18:14:50 +0200, Valentin Korenblit wrote: > >> diff --git a/package/Config.in b/package/Config.in >> index 62770790a1..f376fad3a2 100644 >> --- a/package/Config.in >> +++ b/package/Config.in >> @@ -283,6 +283,7 @@ comment "Graphic libraries" >> source "package/fbv/Config.in" >> source "package/freerdp/Config.in" >> source "package/imagemagick/Config.in" >> + source "package/libclc/Config.in" >> source "package/linux-fusion/Config.in" >> source "package/lite/Config.in" >> source "package/mesa3d/Config.in" >> diff --git a/package/libclc/Config.in b/package/libclc/Config.in >> new file mode 100644 >> index 0000000000..a0f388c4e1 >> --- /dev/null >> +++ b/package/libclc/Config.in >> @@ -0,0 +1,22 @@ >> +config BR2_PACKAGE_LIBCLC >> + bool "libclc" >> + depends on BR2_PACKAGE_LLVM_ARCH_SUPPORTS >> + depends on BR2_HOST_GCC_AT_LEAST_4_8 >> + depends on BR2_TOOLCHAIN_GCC_AT_LEAST_4_8 >> + depends on BR2_TOOLCHAIN_HAS_THREADS >> + depends on BR2_INSTALL_LIBSTDCPP >> + depends on !BR2_TOOLCHAIN_HAS_GCC_BUG_64735 # std::shared_future >> + depends on !BR2_STATIC_LIBS >> + depends on BR2_USE_WCHAR # std::wstring > You seem to have replicated here all the dependencies of the target > LLVM package. However, you only depend on host-llvm and host-clang. Are > you sure all those dependencies are needed by libclc ? Actually, it wouldn't make sense to have libclc without target LLVM, but it is true that it is not a build dependency, so I'll remove them. >> +# There are only two releases: release_35 and release_38, but the last >> +# commit is from 2 years ago. Master has some recent activity. >> +LIBCLC_VERSION = 00236279a293b3737dee08c14f25923a889d2795 >> +LIBCLC_SITE = https://git.llvm.org/git/libclc >> +LIBCLC_SITE_METHOD = git >> +LIBCLC_LICENSE = NCSA >> +LIBCLC_LICENSE_FILES = LICENSE.TXT >> + >> +LIBCLC_DEPENDENCIES = host-clang host-llvm >> +LIBCLC_INSTALL_STAGING = YES >> + >> +# C++ compiler is used to build a small tool (prepare-builtins) for the host. >> +# It must be built with the C++ compiler from the host >> +LIBCLC_CONF_OPTS = --with-llvm-config=$(HOST_DIR)/usr/bin/llvm-config \ >> + --prefix="/usr" \ >> + --includedir="/usr/local/include" \ > I'm not a big fan of /usr/local/include. Perhaps we should use > something really special like /usr/share/libclc/ or something like > this ? Perfect, I'll change it to that directory >> + --pkgconfigdir="/usr/lib/pkgconfig" \ >> + --with-cxx-compiler=$(HOSTCXX) > Weird for a target package to use the host compiler, no ? Yes, it is used only in the target utils/prepare-builtins.o, all the other stuff is compiled with host-clang. >> +define LIBCLC_CONFIGURE_CMDS >> + (cd $(@D); $(TARGET_CONFIGURE_OPTS) ./configure.py $(LIBCLC_CONF_OPTS)) > Is the configure.py script compatible with both Python 2.x and Python > 3.x ? I believe it is, but I'll check it. > Best regards, > > Thomas Thanks, Valentin
Hello, On Mon, 30 Apr 2018 09:54:10 +0200, Valentin Korenblit wrote: > >> diff --git a/package/libclc/Config.in b/package/libclc/Config.in > >> new file mode 100644 > >> index 0000000000..a0f388c4e1 > >> --- /dev/null > >> +++ b/package/libclc/Config.in > >> @@ -0,0 +1,22 @@ > >> +config BR2_PACKAGE_LIBCLC > >> + bool "libclc" > >> + depends on BR2_PACKAGE_LLVM_ARCH_SUPPORTS > >> + depends on BR2_HOST_GCC_AT_LEAST_4_8 > >> + depends on BR2_TOOLCHAIN_GCC_AT_LEAST_4_8 > >> + depends on BR2_TOOLCHAIN_HAS_THREADS > >> + depends on BR2_INSTALL_LIBSTDCPP > >> + depends on !BR2_TOOLCHAIN_HAS_GCC_BUG_64735 # std::shared_future > >> + depends on !BR2_STATIC_LIBS > >> + depends on BR2_USE_WCHAR # std::wstring > > You seem to have replicated here all the dependencies of the target > > LLVM package. However, you only depend on host-llvm and host-clang. Are > > you sure all those dependencies are needed by libclc ? > > Actually, it wouldn't make sense to have libclc without target LLVM, but it > is true that it is not a build dependency, so I'll remove them. Is target LLVM a runtime dependency for libclc ? Could you explain a bit the interaction between libclc and target LLVM, so that we can decide what is the best way to express this dependency ? > >> +# C++ compiler is used to build a small tool (prepare-builtins) for the host. > >> +# It must be built with the C++ compiler from the host > >> +LIBCLC_CONF_OPTS = --with-llvm-config=$(HOST_DIR)/usr/bin/llvm-config \ > >> + --prefix="/usr" \ > >> + --includedir="/usr/local/include" \ > > I'm not a big fan of /usr/local/include. Perhaps we should use > > something really special like /usr/share/libclc/ or something like > > this ? > > Perfect, I'll change it to that directory Thanks! > >> + --pkgconfigdir="/usr/lib/pkgconfig" \ > >> + --with-cxx-compiler=$(HOSTCXX) > > Weird for a target package to use the host compiler, no ? > > Yes, it is used only in the target utils/prepare-builtins.o, all the > other stuff is compiled with host-clang. OK. Thanks! Thomas
Hi Thomas, On 30/04/2018 10:02, Thomas Petazzoni wrote: > Hello, > > On Mon, 30 Apr 2018 09:54:10 +0200, Valentin Korenblit wrote: > >>>> diff --git a/package/libclc/Config.in b/package/libclc/Config.in >>>> new file mode 100644 >>>> index 0000000000..a0f388c4e1 >>>> --- /dev/null >>>> +++ b/package/libclc/Config.in >>>> @@ -0,0 +1,22 @@ >>>> +config BR2_PACKAGE_LIBCLC >>>> + bool "libclc" >>>> + depends on BR2_PACKAGE_LLVM_ARCH_SUPPORTS >>>> + depends on BR2_HOST_GCC_AT_LEAST_4_8 >>>> + depends on BR2_TOOLCHAIN_GCC_AT_LEAST_4_8 >>>> + depends on BR2_TOOLCHAIN_HAS_THREADS >>>> + depends on BR2_INSTALL_LIBSTDCPP >>>> + depends on !BR2_TOOLCHAIN_HAS_GCC_BUG_64735 # std::shared_future >>>> + depends on !BR2_STATIC_LIBS >>>> + depends on BR2_USE_WCHAR # std::wstring >>> You seem to have replicated here all the dependencies of the target >>> LLVM package. However, you only depend on host-llvm and host-clang. Are >>> you sure all those dependencies are needed by libclc ? >> Actually, it wouldn't make sense to have libclc without target LLVM, but it >> is true that it is not a build dependency, so I'll remove them. > Is target LLVM a runtime dependency for libclc ? Could you explain a > bit the interaction between libclc and target LLVM, so that we can > decide what is the best way to express this dependency ? Yes, it is a runtime dependency. libclc is a library that provides device builtin functions defined in the OpenCL specification in LLVM IR bitcode. At some point, they will need to be processed by the LLVM code generator, AMDGPU backend in this case, as this library is required by the OpenCL implementation of Mesa (Clover) which targets GPUs using either R600 or RadeonSI Gallium drivers. > >>>> +# C++ compiler is used to build a small tool (prepare-builtins) for the host. >>>> +# It must be built with the C++ compiler from the host >>>> +LIBCLC_CONF_OPTS = --with-llvm-config=$(HOST_DIR)/usr/bin/llvm-config \ >>>> + --prefix="/usr" \ >>>> + --includedir="/usr/local/include" \ >>> I'm not a big fan of /usr/local/include. Perhaps we should use >>> something really special like /usr/share/libclc/ or something like >>> this ? >> Perfect, I'll change it to that directory > Thanks! > >>>> + --pkgconfigdir="/usr/lib/pkgconfig" \ >>>> + --with-cxx-compiler=$(HOSTCXX) >>> Weird for a target package to use the host compiler, no ? >> Yes, it is used only in the target utils/prepare-builtins.o, all the >> other stuff is compiled with host-clang. > OK. > > Thanks! > > Thomas Best regards, Valentin
diff --git a/DEVELOPERS b/DEVELOPERS index da1c6fa58e..cc68bf2487 100644 --- a/DEVELOPERS +++ b/DEVELOPERS @@ -1907,6 +1907,7 @@ F: package/tstools/ N: Valentin Korenblit <valentin.korenblit@smile.fr> F: package/clang/ +F: package/libclc/ F: package/llvm/ N: Vanya Sergeev <vsergeev@gmail.com> diff --git a/package/Config.in b/package/Config.in index 62770790a1..f376fad3a2 100644 --- a/package/Config.in +++ b/package/Config.in @@ -283,6 +283,7 @@ comment "Graphic libraries" source "package/fbv/Config.in" source "package/freerdp/Config.in" source "package/imagemagick/Config.in" + source "package/libclc/Config.in" source "package/linux-fusion/Config.in" source "package/lite/Config.in" source "package/mesa3d/Config.in" diff --git a/package/libclc/Config.in b/package/libclc/Config.in new file mode 100644 index 0000000000..a0f388c4e1 --- /dev/null +++ b/package/libclc/Config.in @@ -0,0 +1,22 @@ +config BR2_PACKAGE_LIBCLC + bool "libclc" + depends on BR2_PACKAGE_LLVM_ARCH_SUPPORTS + depends on BR2_HOST_GCC_AT_LEAST_4_8 + depends on BR2_TOOLCHAIN_GCC_AT_LEAST_4_8 + depends on BR2_TOOLCHAIN_HAS_THREADS + depends on BR2_INSTALL_LIBSTDCPP + depends on !BR2_TOOLCHAIN_HAS_GCC_BUG_64735 # std::shared_future + depends on !BR2_STATIC_LIBS + depends on BR2_USE_WCHAR # std::wstring + help + libclc is an open source, BSD licensed implementation of + the library requirements of the OpenCL C programming language, + as specified by the OpenCL 1.1 Specification. + + http://libclc.llvm.org/ + +comment "libclc needs a toolchain w/ wchar, threads, C++, gcc >= 4.8, host gcc >= 4.8, dynamic library" + depends on BR2_PACKAGE_LLVM_ARCH_SUPPORTS + depends on !BR2_TOOLCHAIN_HAS_THREADS || !BR2_INSTALL_LIBSTDCPP || \ + !BR2_HOST_GCC_AT_LEAST_4_8 || !BR2_TOOLCHAIN_GCC_AT_LEAST_4_8 \ + || BR2_STATIC_LIBS || !BR2_USE_WCHAR diff --git a/package/libclc/libclc.hash b/package/libclc/libclc.hash new file mode 100644 index 0000000000..c5bf5a668c --- /dev/null +++ b/package/libclc/libclc.hash @@ -0,0 +1,3 @@ +# locally calculated +sha256 e2f1f294f93695f2d1f87ccb9760231b435702dd0c27abeb38baa97186674d9e libclc-00236279a293b3737dee08c14f25923a889d2795.tar.gz +sha256 c66aa55d8478b45f1018500af9f433dd271444ad304d683dd49ff2a42f30dad1 LICENSE.TXT diff --git a/package/libclc/libclc.mk b/package/libclc/libclc.mk new file mode 100644 index 0000000000..93726a22b5 --- /dev/null +++ b/package/libclc/libclc.mk @@ -0,0 +1,42 @@ +################################################################################ +# +# libclc +# +################################################################################ + +# There are only two releases: release_35 and release_38, but the last +# commit is from 2 years ago. Master has some recent activity. +LIBCLC_VERSION = 00236279a293b3737dee08c14f25923a889d2795 +LIBCLC_SITE = https://git.llvm.org/git/libclc +LIBCLC_SITE_METHOD = git +LIBCLC_LICENSE = NCSA +LIBCLC_LICENSE_FILES = LICENSE.TXT + +LIBCLC_DEPENDENCIES = host-clang host-llvm +LIBCLC_INSTALL_STAGING = YES + +# C++ compiler is used to build a small tool (prepare-builtins) for the host. +# It must be built with the C++ compiler from the host +LIBCLC_CONF_OPTS = --with-llvm-config=$(HOST_DIR)/usr/bin/llvm-config \ + --prefix="/usr" \ + --includedir="/usr/local/include" \ + --pkgconfigdir="/usr/lib/pkgconfig" \ + --with-cxx-compiler=$(HOSTCXX) + +define LIBCLC_CONFIGURE_CMDS + (cd $(@D); $(TARGET_CONFIGURE_OPTS) ./configure.py $(LIBCLC_CONF_OPTS)) +endef + +define LIBCLC_BUILD_CMDS + $(TARGET_MAKE_ENV) $(MAKE) -C $(@D) +endef + +define LIBCLC_INSTALL_TARGET_CMDS + $(TARGET_MAKE_ENV) $(MAKE) -C $(@D) DESTDIR=$(TARGET_DIR) install +endef + +define LIBCLC_INSTALL_STAGING_CMDS + $(TARGET_MAKE_ENV) $(MAKE) -C $(@D) DESTDIR=$(STAGING_DIR) install +endef + +$(eval $(generic-package))
This patch provides libclc, an open source implementation of the library requirements of the OpenCL C programming language, as specified by the OpenCL 1.1 Specification. It is intended to be used with Mesa Clover. It needs to be compiled with host-clang, as it generates LLVM IR bitcode files containing device builtin functions for each target. Currently, libclc supports AMDGCN, R600 and NVPTX targets. As OpenCL kernels are built dynamically on the target using libclang and libLLVM, it is necessary to have clc headers on the target. Buildroot removes /usr/include in its target-finalize step, so clc headers are installed to /usr/local/include. Signed-off-by: Valentin Korenblit <valentin.korenblit@smile.fr> --- DEVELOPERS | 1 + package/Config.in | 1 + package/libclc/Config.in | 22 ++++++++++++++++++++++ package/libclc/libclc.hash | 3 +++ package/libclc/libclc.mk | 42 ++++++++++++++++++++++++++++++++++++++++++ 5 files changed, 69 insertions(+) create mode 100644 package/libclc/Config.in create mode 100644 package/libclc/libclc.hash create mode 100644 package/libclc/libclc.mk