-
Notifications
You must be signed in to change notification settings - Fork 159
Samples and Utility libraries #37
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from all commits
Commits
Show all changes
203 commits
Select commit
Hold shift + click to select a range
5c225fa
char * cl_sdk_read_file(char * filename)
Melirius 5ab156b
New line
Melirius cc6cbb9
Error code handling
Melirius 392bf81
Idiomatic OpenCL error handling
Melirius d0be28b
Samples pick up INTERFACE targets
MathiasMagnus 3706c4c
SDK recurses into all submodules
MathiasMagnus 98e810a
Update submodules
MathiasMagnus e9e5bb2
Bump OpenCL-CLHPP commit
MathiasMagnus 0fb8972
Initial Utility library
MathiasMagnus 604f214
Added initial C++ saxpy
MathiasMagnus ddc9f26
Fix index range checks
MathiasMagnus a71d8aa
Initial kernel file
MathiasMagnus ca3c698
WIP
MathiasMagnus 8256516
Fix CLI parsing SDK utility
MathiasMagnus 87b6123
Added context to the queue
MathiasMagnus e23e6d5
Document saxpy sample
MathiasMagnus 3179603
Change to lazy buffer
MathiasMagnus 081135d
Add empty main
MathiasMagnus cbf5b88
Bump submodule hash
MathiasMagnus 38e5ad7
Separate Util and SDK libs
MathiasMagnus 7df3300
Promote OPENCL_SAMPLE_TEST as option
MathiasMagnus c685427
Initial CI
MathiasMagnus 89c6321
Map git SSH protocol to HTTPS
MathiasMagnus ac26453
Disable recurse by runner
MathiasMagnus 94f9711
Bump submodule hashes
MathiasMagnus f6a8484
Add note on beforescript fragment
MathiasMagnus b111cb1
Add options to control building & testing samples
MathiasMagnus 45c37ed
Build samples next to ICD loader
MathiasMagnus 83efeb3
Fetch and use TCLAP
MathiasMagnus 005d158
Specify LIBRARY DESTINATION et al
MathiasMagnus f8bcfa4
Unsupported target_sources INTERFACE on 3.1
MathiasMagnus c0af945
Fix TCLAP handling
MathiasMagnus c47fd0f
Fixed build tree installation
MathiasMagnus 378f62f
Add missing includes
MathiasMagnus 391d940
Resolve dependency cycle
MathiasMagnus 6dfa164
Add missing include
MathiasMagnus 92a6663
Backport to C++14
MathiasMagnus cccdffc
Enable building and testing tests
MathiasMagnus 93ac21b
Clarify config args
MathiasMagnus 72df63c
recursively clone submodules (Mock missing)
MathiasMagnus 4993bd9
Bump submodule hash
MathiasMagnus 5437c05
Descriptive error message
MathiasMagnus 97f4cca
Allow test failure temporarily
MathiasMagnus b31bd2b
cl_context cl_util_get_context(int plat_id, int dev_id, cl_device_typ…
Melirius 272f5c1
WIP: Saxpy in C
Melirius 82b8767
WIP includes
Melirius 3acd160
WIP more includes
Melirius ea9a217
WIP cl_utils_build_program
Melirius 7fc8082
Saxpy in C
Melirius 362e196
Math library to link
Melirius 49dc957
C11 for fopen_s and tmpfile_s
Melirius cbdb44a
Want extension of C lib
Melirius 367f934
CMAKE_C_STANDARD_REQUIRED
Melirius d852e57
No Annex K funcs
Melirius 71540fc
No inline
Melirius 208bae8
Parse options error handling, std::string cl::util::read_text_file(co…
Melirius bf10d55
Uuups
Melirius 3dcd43b
Separate Cargs
Melirius 43754b1
Restructure, error handling in cl::util::read_text_file
Melirius a9c1d9c
Uniform cl_util_ prefix
Melirius ea73555
Bump CLHPP commit hash
MathiasMagnus 65a016b
OpenCL-Headers track develop_stream
MathiasMagnus 0ba1dc9
OpenCL-ICD-Loader track develop_stream
MathiasMagnus 4e35951
Bump OpenCL-ICD-Loader hash
MathiasMagnus b8b8c19
Bump CLHPP hash
MathiasMagnus 1cd9447
OpenCL-CLHPP track develop_stream
MathiasMagnus 21e0dca
Bump CLHPP hash
MathiasMagnus 287b416
Added custom device type CLI arg parse
MathiasMagnus 0f1807a
Renamed Triplet to DeviceTriplet
MathiasMagnus cc771da
Added newline at eof
MathiasMagnus 77b9f59
Minor corrections
MathiasMagnus e51c50a
WIP
MathiasMagnus 01edd00
Reduce sample feature complete
MathiasMagnus 2ef30ae
Update readme
MathiasMagnus 3a695cd
Added event profiling utility
MathiasMagnus 9736913
Added event profiling
MathiasMagnus ae9ddd7
Bump CLHPP hash
MathiasMagnus cc4d879
Simplify index range calculation.
Melirius 4240335
C reduce sample should use 3.0 too
MathiasMagnus 95b24ee
Add copyright notice
MathiasMagnus 2e9ecc1
Add newline at eof
MathiasMagnus 4fdaa5e
Simplify extension selection logic
MathiasMagnus 1bbe93b
Fix size_t handling
MathiasMagnus 1add023
Logically group flush to message
MathiasMagnus c9b012b
Change precedence of extensions
MathiasMagnus b27b46e
FMA in cppSaxpy
Melirius bd026bf
std::logic_error instead of std::exception
Melirius e87fd9b
C error handling, DeviceTriplet
Melirius f442d3a
Reduce C working
Melirius d92c56b
Uniform "OpenCL SDK includes"
Melirius 92a6cbf
Image load/save, compilation options for reduce
Melirius e93626a
Minimum CMake fix
Melirius 9b315a6
Uuups
Melirius 631b53d
Another cl.h capitalizations
Melirius 0050abe
Definition of size_t
Melirius 60549e8
Nicier loops (by Navid)
Melirius 3374a03
Fix bits overlapping in random ints range
Melirius 4b5eda1
Bump Header hash
MathiasMagnus c5d6f2d
ROCm exec stub
MathiasMagnus 2fa1b94
Build using Vcpkg
MathiasMagnus c0c1541
Simplify extension detection
MathiasMagnus 40b0e71
Bump base image version
MathiasMagnus e82d06a
Install root package config
MathiasMagnus 8a7cf63
Don't download TCLAP in CI
MathiasMagnus 279b4c5
No TCLAP artifact
MathiasMagnus 734f24f
SDK consumption tests
MathiasMagnus faae073
Fix missing stage name
MathiasMagnus 65cb98d
Fix compiler passing
MathiasMagnus bdcd26f
Only MSVC should use ISO C secure exts
MathiasMagnus 1096d96
Only use stable ROCm nodes
MathiasMagnus f0abd79
Fix dual-purpose API usage in C Util lib
MathiasMagnus 2214803
Add histogram C++ example
neon60 0f6816a
Update readme
neon60 c16f43f
Merge request findings
neon60 ddeee88
Remove treshold
neon60 046fc79
Findings MR II
neon60 bd3e9b9
Remove barrier
neon60 51a927c
Add one barrier
neon60 e2cc406
Removed unused variable
neon60 acfd8aa
Updated build instructions
MathiasMagnus 71f10f4
InteropWindow stub implementation
MathiasMagnus eb731b5
WIP
MathiasMagnus 46fa5a3
Feature complete sample
MathiasMagnus d3d894d
Added generic windowing options
MathiasMagnus 1a78e73
Fix grammar
MathiasMagnus 2e8d7f9
Conditional CMake minimum bump
MathiasMagnus 804d829
Bump image versions
MathiasMagnus 5557039
Bump minimum CMake ver in CI
MathiasMagnus 6ada8a1
Backport IMPT TGT
MathiasMagnus d76d13b
Remove explicit OpenGL::OpenGL
MathiasMagnus 212bb78
Platform-specific OpenGL
MathiasMagnus 6288873
Update minimum CMake for samples to 3.10
MathiasMagnus 361cbbe
Include X11 headers
MathiasMagnus feadafa
No explicit CTOR call
MathiasMagnus c527075
More precise platform header includes
MathiasMagnus 389bdb8
Install GLU in CI
MathiasMagnus 829ddec
Refine dependency detection
MathiasMagnus e231240
Add deps to Util lib
MathiasMagnus 33ad12a
GCC warning: expression result unused
MathiasMagnus 5a255e0
MSVC warning: unreachable code
MathiasMagnus 99c8046
Fix error variable
MathiasMagnus 71dcc4a
Don't test GUI sample
MathiasMagnus 5695872
Bump image versions
MathiasMagnus acdce5f
Take dep from image
MathiasMagnus 48f5fdc
Declare tests only if both user and sample allows
MathiasMagnus c3cc6cf
WIP
MathiasMagnus 85cac55
WIP
MathiasMagnus b26e5de
Utils and SDK lib separation cleanup
MathiasMagnus 080eae0
Fix Linux linkage
MathiasMagnus 90758db
Add missing source from unity build
MathiasMagnus 091f69e
NOMINMAX
MathiasMagnus 728a631
Don't export cl::Error
MathiasMagnus eed2cea
Update readme
MathiasMagnus 36c3750
remove extra space
MathiasMagnus 5c2b0dd
Remove dead code
MathiasMagnus 07b6d8f
Print errors to cerr
MathiasMagnus 33f4ff5
Working Blur C
Melirius bcf9e59
Bug in Intel CPU runtime?
Melirius 582b431
Dual-pass blur
Melirius 92db64b
Box blur in one and two passes
Melirius b3eabd7
WIP subgroup exchange
Melirius a89c35f
Fixing GCC compilation
Melirius a7169e2
No max defined in GCC
Melirius 978a438
And link m here also
Melirius 64ef158
Variant for review
Melirius 732e091
WIP Gaussian blur
Melirius 177cb27
Gauss dual-pass: simple and subgroup excahnge
Melirius fbccfa1
Pi fix
Melirius 35c7887
Gauss local memory exchange
Melirius b35fdf1
Docs
Melirius b87003e
Docs
Melirius 1d74bf4
File utils, stb fixed, some code cleaning
Melirius 4401909
Correct naming restored
Melirius 8781063
More consistent cl_uint for plat_id and dev_id
Melirius 169b1ef
snprintf for Linux
Melirius 88b62e2
Blur standard image added, error handling in image SDK improved, blur…
Melirius 7428f5e
More cl_uint
Melirius 0c96032
Even more cl_uint
Melirius c19c353
Bug in check_use_work_group_reduce fixed, macros for CLI added
Melirius 184c5de
Error handling rewrite, `const`s
Melirius 8c01a8e
Remove blanks, shift for Collatz
Melirius e31e95c
Integer limits header
Melirius 1b3899a
Execution step added
Melirius 574bfa9
Format and minor CI update (#2)
MathiasMagnus 0812c13
Update README.md
MathiasMagnus b70488f
Update README.md
MathiasMagnus 061a288
Fix RPATH
MathiasMagnus 887655c
Copy kernels into build tree
MathiasMagnus 25d51c5
Fix RPATH for build and install trees
MathiasMagnus 4bbaa82
Handle warnings
MathiasMagnus 901201c
Trigger CI
MathiasMagnus dc1060b
Apply clang format
MathiasMagnus 5eea822
Bump submodule hashes to latest develop_stream
MathiasMagnus 0f53986
Get MacOs same-build include dir fix
MathiasMagnus cade1f3
histogram declare kernel file
MathiasMagnus 6d4e9e4
Bump subproject hashes
MathiasMagnus 2d9d893
Exemplify 64-bit Windows build
MathiasMagnus 036f98b
Add note on DLLs in install tree on Windows
MathiasMagnus 7b99e06
Bump header hash
MathiasMagnus 7bf2436
Cleaner command-line output
MathiasMagnus 3b7fbe4
Remove GitLab CI
MathiasMagnus 19d372c
Remove temporary redirection to Stream HPC repos
MathiasMagnus 87e1d9d
Bump submodule hashes to have SDK base changes
MathiasMagnus File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -4,3 +4,4 @@ install/ | |
| # vim | ||
| *~ | ||
| .*.sw[op] | ||
| .vscode | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,10 @@ | ||
| FROM streamhpc/opencl-sdk-base:ubuntu-18.04-20211119 | ||
| RUN set -ex; \ | ||
| export DEBIAN_FRONTEND=noninteractive ; \ | ||
| # Register ROCm APT repo | ||
| wget --quiet --recursive --no-directories --no-parent "https://repo.radeon.com/amdgpu-install/latest/ubuntu/bionic/" --accept "amdgpu-install-*_all.deb" ; \ | ||
| apt install -y -qq ./amdgpu-install-*_all.deb libnuma-dev initramfs-tools ; \ | ||
| apt update -qq; \ | ||
| rm ./amdgpu-install-*_all.deb ; \ | ||
| # Install OpenCL package only (and dependency) | ||
| amdgpu-install -y --usecase=opencl |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,45 @@ | ||
| FROM ubuntu:18.04 AS apt-installs | ||
| RUN set -ex; \ | ||
| export DEBIAN_FRONTEND=noninteractive ; \ | ||
| apt update -qq; \ | ||
| # install wget to download repository keys and CMake tarballs | ||
| # install software-properties-common for the apt-add-repository command | ||
| apt install -y -qq wget software-properties-common ; \ | ||
| # Canonical hosts recent GCC compilers in ubuntu-toolchain-r/test | ||
| apt-add-repository -y ppa:ubuntu-toolchain-r/test ; \ | ||
| # LLVM hosts most toolchain in separate repos. We only register those absent from ubuntu-toolchain-r/test | ||
| wget -q -O - https://apt.llvm.org/llvm-snapshot.gpg.key | apt-key add - ; \ | ||
| apt-add-repository -y 'deb [arch=amd64] https://apt.llvm.org/bionic/ llvm-toolchain-bionic-11 main' ; \ | ||
| apt-add-repository -y 'deb [arch=amd64] https://apt.llvm.org/bionic/ llvm-toolchain-bionic-12 main' ; \ | ||
| apt-add-repository -y 'deb [arch=amd64] https://apt.llvm.org/bionic/ llvm-toolchain-bionic-13 main' ; \ | ||
| # install ninja, GCC 7-10, LLVM 8-13 and build-essential to get linkers, etc. | ||
| # install git to download dependencies | ||
| # install ruby to run CMock | ||
| # install libidn11 which CMake 3.1.3 only depends on | ||
| # install ca-certificates to `git clone` via HTTPS | ||
| # install SFML dependencies | ||
| # libx11-dev libxrandr-dev libxi-dev | ||
| # mesa-common-dev for gl.h | ||
| # libgl1-mesa-dev for libGL.so | ||
| # libglu1-mesa-dev for glu.h | ||
| # libudev-dev | ||
| # install Vcpkg dependencies | ||
| # curl zip unzip tar | ||
| apt install -y -qq build-essential g++-7 g++-8 g++-9 g++-10 clang-8 clang-9 clang-10 clang-11 clang-12 clang-13 ninja-build git ruby libidn11 ca-certificates libx11-dev libxrandr-dev libxi-dev mesa-common-dev libgl1-mesa-dev libglu1-mesa-dev libudev-dev curl zip unzip tar | ||
|
|
||
| # install CMake minimum (3.0.2 (Headers, ICD Loader), 3.1.3 (CLHPP), 3.10.3 (SDK)) and latest (3.21.2) | ||
| RUN mkdir -p /opt/Kitware/CMake ; \ | ||
| wget -c https://github.com/Kitware/CMake/releases/download/v3.0.2/cmake-3.0.2-Linux-i386.tar.gz -O - | tar -xz --directory /opt/Kitware/CMake ; \ | ||
| mv /opt/Kitware/CMake/cmake-3.0.2-Linux-i386 /opt/Kitware/CMake/3.0.2 ; \ | ||
| wget -c https://github.com/Kitware/CMake/releases/download/v3.1.3/cmake-3.1.3-Linux-x86_64.tar.gz -O - | tar -xz --directory /opt/Kitware/CMake ; \ | ||
| mv /opt/Kitware/CMake/cmake-3.1.3-Linux-x86_64 /opt/Kitware/CMake/3.1.3 ; \ | ||
| wget -c https://github.com/Kitware/CMake/releases/download/v3.10.3/cmake-3.10.3-Linux-x86_64.tar.gz -O - | tar -xz --directory /opt/Kitware/CMake ; \ | ||
| mv /opt/Kitware/CMake/cmake-3.10.3-Linux-x86_64 /opt/Kitware/CMake/3.10.3 ; \ | ||
| wget -c https://github.com/Kitware/CMake/releases/download/v3.21.2/cmake-3.21.2-linux-x86_64.tar.gz -O - | tar -xz --directory /opt/Kitware/CMake ; \ | ||
| mv /opt/Kitware/CMake/cmake-3.21.2-linux-x86_64 /opt/Kitware/CMake/3.21.2 | ||
|
|
||
| # install Vcpkg | ||
| RUN git clone --depth 1 https://github.com/Microsoft/vcpkg.git /opt/Microsoft/vcpkg ; \ | ||
| /opt/Microsoft/vcpkg/bootstrap-vcpkg.sh ; \ | ||
| # install SFML, TCLAP, GLM | ||
| /opt/Microsoft/vcpkg/vcpkg install sfml tclap glm glew |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,30 @@ | ||
| # - Find Stb | ||
| # Find the Stb headers | ||
| # | ||
| # Stb_INCLUDE_DIR - where to find the TCLAP headers | ||
| # Stb_FOUND - True if TCLAP is found | ||
|
|
||
| if (Stb_INCLUDE_DIR) | ||
| # already in cache, be silent | ||
| set (Stb_FIND_QUIETLY TRUE) | ||
| endif (Stb_INCLUDE_DIR) | ||
|
|
||
| # find the headers | ||
| find_path (Stb_INCLUDE_PATH stb_image.h | ||
| PATHS | ||
| ${Stb_DIR} | ||
| PATH_SUFFIXES | ||
| include | ||
| include/stb | ||
| ) | ||
|
|
||
| # handle the QUIETLY and REQUIRED arguments and set Stb_FOUND to | ||
| # TRUE if all listed variables are TRUE | ||
| include (FindPackageHandleStandardArgs) | ||
| find_package_handle_standard_args (Stb "Stb (https://github.com/nothings/stb) could not be found. Set Stb_INCLUDE_PATH to point to the headers adding '-D Stb_INCLUDE_PATH=/path/to/stb' to the cmake command." Stb_INCLUDE_PATH) | ||
|
|
||
| if (Stb_FOUND) | ||
| set (Stb_INCLUDE_DIR ${Stb_INCLUDE_PATH}) | ||
| endif (Stb_FOUND) | ||
|
|
||
| mark_as_advanced(Stb_INCLUDE_PATH) |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,28 @@ | ||
| # - Find TCLAP | ||
| # Find the TCLAP headers | ||
| # | ||
| # TCLAP_INCLUDE_DIR - where to find the TCLAP headers | ||
| # TCLAP_FOUND - True if TCLAP is found | ||
|
|
||
| if (TCLAP_INCLUDE_DIR) | ||
| # already in cache, be silent | ||
| set (TCLAP_FIND_QUIETLY TRUE) | ||
| endif (TCLAP_INCLUDE_DIR) | ||
|
|
||
| # find the headers | ||
| find_path (TCLAP_INCLUDE_PATH tclap/CmdLine.h | ||
| PATHS | ||
| ${CMAKE_SOURCE_DIR}/include | ||
| ${CMAKE_INSTALL_PREFIX}/include | ||
| ) | ||
|
|
||
| # handle the QUIETLY and REQUIRED arguments and set TCLAP_FOUND to | ||
| # TRUE if all listed variables are TRUE | ||
| include (FindPackageHandleStandardArgs) | ||
| find_package_handle_standard_args (TCLAP "TCLAP (http://tclap.sourceforge.net/) could not be found. Set TCLAP_INCLUDE_PATH to point to the headers adding '-DTCLAP_INCLUDE_PATH=/path/to/tclap' to the cmake command." TCLAP_INCLUDE_PATH) | ||
|
|
||
| if (TCLAP_FOUND) | ||
| set (TCLAP_INCLUDE_DIR ${TCLAP_INCLUDE_PATH}) | ||
| endif (TCLAP_FOUND) | ||
|
|
||
| mark_as_advanced(TCLAP_INCLUDE_PATH) |
Submodule OpenCL-CLHPP
updated
16 files
| +321 −0 | .github/workflows/linux.yml | |
| +159 −0 | .github/workflows/macos.yml | |
| +0 −42 | .github/workflows/presubmit.yml | |
| +213 −0 | .github/workflows/windows.yml | |
| +98 −28 | CMakeLists.txt | |
| +55 −18 | README.md | |
| +1 −1 | examples/CMakeLists.txt | |
| +30 −33 | examples/src/CMakeLists.txt | |
| +7 −1 | include/CL/opencl.hpp | |
| +0 −5 | include/CMakeLists.txt | |
| +89 −20 | tests/CMakeLists.txt | |
| +42 −0 | tests/pkgconfig/bare/CMakeLists.txt | |
| +51 −0 | tests/pkgconfig/pkgconfig.cpp | |
| +38 −0 | tests/pkgconfig/sdk/CMakeLists.txt | |
| +23 −0 | tests/strip_defines.cmake | |
| +0 −18 | tests/strip_defines.py |
Submodule OpenCL-ICD-Loader
updated
22 files
| +0 −20 | .appveyor.yml | |
| +268 −0 | .github/workflows/linux.yml | |
| +96 −0 | .github/workflows/macos.yml | |
| +0 −24 | .github/workflows/presubmit.yml | |
| +186 −0 | .github/workflows/windows.yml | |
| +83 −33 | CMakeLists.txt | |
| +49 −46 | README.md | |
| +97 −97 | loader/icd_dispatch_generated.c | |
| +7 −7 | loader/windows/icd_windows.c | |
| +2 −2 | scripts/icd_dispatch_generated.c.mako | |
| +25 −1 | test/CMakeLists.txt | |
| +3 −1 | test/driver_stub/CMakeLists.txt | |
| +3 −0 | test/layer/CMakeLists.txt | |
| +1 −1 | test/layer/icd_print_layer.c | |
| +6 −1 | test/loader_test/CMakeLists.txt | |
| +3 −2 | test/loader_test/icd_test_match.c | |
| +0 −1 | test/loader_test/test_buffer_object.c | |
| +1 −1 | test/loader_test/test_create_calls.c | |
| +2 −0 | test/log/CMakeLists.txt | |
| +41 −0 | test/pkgconfig/bare/CMakeLists.txt | |
| +56 −0 | test/pkgconfig/pkgconfig.c | |
| +40 −0 | test/pkgconfig/sdk/CMakeLists.txt |
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I know Ubuntu isn't universal, but it might be helpful to put this in the readme. Or at least the ones that aren't there by default on a desktop install.
I had to install these to get the SDK to build:
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These packages are mentioned in the OpenCL-Guide in the Linux guide here. These are only required if one builds the samples and not otherwise.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm guessing this is only needed if the package dependencies aren't satisfied via vcpkg?
Regardless, I agree it would be worth including this as an alternative in the README. Installing via the OS package manager will likely be smoother than vcpkg (getting it working on Ubuntu wasn't trivial) and this is the mechanism we're testing in CI.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One can install them either way. It's user preference.