Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[Wait for #2727] [ GPU/OpenCL ] enable X86 Opencl with ENABLE_FP16=false #2783

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

[Wait for #2727] [ GPU/OpenCL ] enable X86 Opencl with ENABLE_FP16=false #2783

wants to merge 4 commits into from
+967 −811

Commits on Nov 4, 2024

  1. [ GPGPU ] enable X86 OpenCL with some fixes 

    This PR enables meson build with opencl for X86.
Also, fix some unittest case errors and types.

Resolves:

**Self evaluation:**
1. Build test:	 [X]Passed [ ]Failed [ ]Skipped
2. Run test:	 [X]Passed [ ]Failed [ ]Skipped

Signed-off-by: jijoong.moon <[email protected]>
    @jijoongmoon @EunjuYang
    jijoongmoon authored and EunjuYang committed Nov 4, 2024
    Configuration menu
    Copy the full SHA
    07d1af3 View commit details
    Browse the repository at this point in the history

  2. [ GPU/Unit test ] Disable setPropertiesInvalid_n_gpu test 

    - This commit is related to nnstreamer#2774.
- The test `setPropertiesInvalid_n_gpu` was failed with
LayerPropertySemantics, which tests negative case for some layers not
implemented for GPU.
- This test should be enabled later.

Self evaluation:

Build test: [X]Passed [ ]Failed [ ]Skipped
Run test: [X]Passed [ ]Failed [ ]Skipped

Signed-off-by: Eunju Yang <[email protected]>
    @EunjuYang
    EunjuYang committed Nov 4, 2024
    Configuration menu
    Copy the full SHA
    38b6ad5 View commit details
    Browse the repository at this point in the history

  3. [ Minor ] minor fix in rebase (enable opencl x86) 

    - add missing #endif
- add memory free code, causing possible memory-leak
- enable opencl x86

Signed-off-by: Eunju Yang <[email protected]>
    @EunjuYang
    EunjuYang committed Nov 4, 2024
    Configuration menu
    Copy the full SHA
    b79273c View commit details
    Browse the repository at this point in the history

  4. [ GPU ] separate FP16-related functions to support ENABLE_FP16=FALSE 

    - This commit resolves nnstreamer#2776
- This commit separate FP16-related functions from _cl.cpp file.
- Minor bug is fixed

Signed-off-by: Eunju Yang <[email protected]>
    @EunjuYang
    EunjuYang committed Nov 4, 2024
    Configuration menu
    Copy the full SHA
    a043d8f View commit details
    Browse the repository at this point in the history