Skip to content

[Clang] Fix implicit conversion warning for opencl narrowing#5083

Merged
h-sadia merged 2 commits intomainfrom
hsadia/sftmx_clang
May 6, 2026
Merged

[Clang] Fix implicit conversion warning for opencl narrowing#5083
h-sadia merged 2 commits intomainfrom
hsadia/sftmx_clang

Conversation

@h-sadia
Copy link
Copy Markdown
Contributor

@h-sadia h-sadia commented Apr 27, 2026

@h-sadia h-sadia requested a review from a team as a code owner April 27, 2026 21:27
@github-actions github-actions Bot added the platform:gpu-intel Codeowner: @oneapi-src/onednn-gpu-intel label Apr 27, 2026
@h-sadia h-sadia force-pushed the hsadia/sftmx_clang branch from ba74725 to 7cbbf96 Compare April 27, 2026 21:52
@h-sadia
Copy link
Copy Markdown
Contributor Author

h-sadia commented Apr 27, 2026

make test linters

@h-sadia h-sadia changed the title clang: src: gpu: intel: softmax: fix clang warning [Clang] Fix implicit conversion warning for opencl narrowing Apr 27, 2026
Simonsays095
Simonsays095 approved these changes Apr 27, 2026
View reviewed changes
Comment thread src/gpu/intel/softmax/simple.cl
rjoursler
rjoursler reviewed Apr 27, 2026
View reviewed changes
Comment thread src/gpu/intel/include/philox.h Outdated
@h-sadia h-sadia force-pushed the hsadia/sftmx_clang branch 2 times, most recently from 2e217b7 to 8bb43ca Compare April 29, 2026 17:34
@h-sadia
Copy link
Copy Markdown
Contributor Author

h-sadia commented Apr 29, 2026

make test
disable test_device_cpu
disable test_device_gpu
disable build_gpu_runtime_ocl
disable build_gpu_runtime_sycl
disable build_cpu_runtime_omp
disable build_cpu_runtime_sycl
disable build_cpu_runtime_tbb

@h-sadia h-sadia force-pushed the hsadia/sftmx_clang branch from bd8c429 to aec787a Compare April 29, 2026 22:30
@h-sadia
Copy link
Copy Markdown
Contributor Author

h-sadia commented Apr 29, 2026

make test
disable test_device_cpu
disable test_device_gpu
disable build_gpu_runtime_ocl
disable build_gpu_runtime_sycl
disable build_cpu_runtime_omp
disable build_cpu_runtime_sycl
disable build_cpu_runtime_tbb

@h-sadia h-sadia force-pushed the hsadia/sftmx_clang branch 4 times, most recently from 4acd758 to 8acd700 Compare May 1, 2026 05:38
@h-sadia
Copy link
Copy Markdown
Contributor Author

h-sadia commented May 1, 2026

make test
set test_scope=NIGHTLY
disable test_device_cpu
enable arch_gpu_xe-hpc
enable arch_gpu_xe-hpg-atsm
enable arch_gpu_xe-hpg-dg2
enable arch_gpu_xe-lp
enable arch_gpu_xe-lpg
enable arch_gpu_xe-lpg+
enable arch_gpu_xe2-hpg-bmg
enable arch_gpu_xe2-lpg
enable arch_gpu_xe3-lpg

@h-sadia h-sadia force-pushed the hsadia/sftmx_clang branch from 8acd700 to e460a40 Compare May 2, 2026 00:39
@h-sadia
Copy link
Copy Markdown
Contributor Author

h-sadia commented May 2, 2026

make test
set test_scope=NIGHTLY
disable test_device_cpu
enable arch_gpu_xe-hpc
enable arch_gpu_xe-hpg-atsm
enable arch_gpu_xe-hpg-dg2
enable arch_gpu_xe-lp
enable arch_gpu_xe-lpg
enable arch_gpu_xe-lpg+
enable arch_gpu_xe2-hpg-bmg
enable arch_gpu_xe2-lpg
enable arch_gpu_xe3-lpg

@h-sadia h-sadia force-pushed the hsadia/sftmx_clang branch 2 times, most recently from ba81193 to c0a4fe3 Compare May 4, 2026 23:40
@h-sadia
Copy link
Copy Markdown
Contributor Author

h-sadia commented May 4, 2026

make test
set test_scope=NIGHTLY
disable test_device_cpu
enable arch_gpu_xe-hpc
enable arch_gpu_xe-hpg-atsm
enable arch_gpu_xe-hpg-dg2
enable arch_gpu_xe-lp
enable arch_gpu_xe-lpg
enable arch_gpu_xe-lpg+
enable arch_gpu_xe2-hpg-bmg
enable arch_gpu_xe2-lpg
enable arch_gpu_xe3-lpg

@h-sadia h-sadia merged commit 52c36ba into main May 6, 2026
12 of 14 checks passed
@h-sadia h-sadia deleted the hsadia/sftmx_clang branch May 6, 2026 01:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Simonsays095 Simonsays095 Simonsays095 approved these changes

@hidefromkgb hidefromkgb hidefromkgb approved these changes

@rjoursler rjoursler rjoursler approved these changes

Assignees

No one assigned

Labels

platform:gpu-intel Codeowner: @oneapi-src/onednn-gpu-intel

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@h-sadia @hidefromkgb @rjoursler @Simonsays095