Skip to content

[sycl-free-function] Refactor RoiPoolKernels by free function - #5389

Open
NeoZhangJianyu wants to merge 1 commit into
mainfrom
jianyuzh_sycl_free_roi_pool_kernels
Open

NeoZhangJianyu wants to merge 1 commit into
mainfrom
jianyuzh_sycl_free_roi_pool_kernels

Conversation

@NeoZhangJianyu

Copy link
Copy Markdown
Contributor

Refactor RoiPoolKernels by free function
UT is passed locally.
rename all kernel functions' name to be with suffix "_kernel_impl".

@github-actions github-actions Bot added disable_e2e Disable all e2e test jobs for the PR disable_distributed Disable distributed UT test jobs for the PR labels Sep 16, 2026
@tye1

tye1 commented Sep 21, 2026

Copy link
Copy Markdown

@torchxpubot check-ut

@torchxpubot

Copy link
Copy Markdown
Contributor

Replying to this comment by @tye1

UT Result Check: PR #5389

New Failures

No new failures detected.

The run reported 1 failure and 2 errors out of 5706 test cases (4254 passed, 1449 skipped), but none of them appear in the authoritative new-failure list — they all match existing known issues.

Recommendation

Safe to merge. The only changed file is src/ATen/native/xpu/sycl/RoiPoolKernels.cpp, and the run produced no new failures beyond the known-issues baseline. Note that this PR adds no tests, so the kernel change is covered only by existing test coverage.


Generated by ut-check job.

@tye1
tye1 force-pushed the jianyuzh_sycl_free_roi_pool_kernels branch from a8f0d26 to b3339d5 Compare September 23, 2026 08:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

disable_distributed Disable distributed UT test jobs for the PR disable_e2e Disable all e2e test jobs for the PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants