Skip to content

fix(integration-tests): Remove __ocml call from batchnorm reference - #13386

Merged
arsenm merged 1 commit into
developfrom
users/arsenm/integration-tests-remove-ocml
Oct 9, 2026
Merged

arsenm merged 1 commit into
developfrom
users/arsenm/integration-tests-remove-ocml

Conversation

@arsenm

@arsenm arsenm commented Oct 9, 2026

Copy link
Copy Markdown
Contributor

__ocml functions are compiler implementation details and shouldn't be used outside the compiler. Use the public hrsqrt for the _Float16 rsqrt overload, similarly to how the public rsqrt/rsqrtf are used for double/float.

Generated ISA is unchanged.

JIRA ID: AIRUNTIME-629

…rence

__ocml functions are compiler implementation details and shouldn't be
used outside the compiler. Use the public hrsqrt for the _Float16 rsqrt
overload, similarly to how the public rsqrt/rsqrtf are used for double/float.

Generated ISA is unchanged.

JIRA ID: AIRUNTIME-629

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
@therock-pr-bot

therock-pr-bot Bot commented Oct 9, 2026 •

Copy link
Copy Markdown

✅ All Checks Passed — Ready for Review

Check Status Details
📝 PR Description ✅ Pass —
⛔ Forbidden Files ✅ Pass —
🧪 Unit Test ⚠️ Warning Error: Source/code files changed without an accompanying unit test.
Expected: add at least one test file named like test_<name>.py / test_<name>.cpp (or <name>_test.*).
Current: code file(s) changed: dnn-providers/integration-tests/gpu-ref/kernels/batchnorm/GpuRefBatchnormFwdInf.cpp; no test file found
🔎 pre-commit ✅ Pass —
🚫 Draft PR 🔜 To Be Enabled —
🚩 Feature Flag 🔜 To Be Enabled —
📊 Code Coverage 🔜 To Be Enabled —
🤖 therock-pr-bot ✅ Pass —

🎉 All checks passed! This PR is ready for review.

📖 Need help? See the Policy FAQ for details on every check and how to fix failures.

🙋 Wish to Override Policy?

@therock-pr-bot

therock-pr-bot Bot commented Oct 9, 2026

Copy link
Copy Markdown

🎉 All checks passed! This PR is ready for review.

@averinevg averinevg left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@arsenm
arsenm merged commit 7c7779c into develop Oct 9, 2026
151 of 165 checks passed
@arsenm
arsenm deleted the users/arsenm/integration-tests-remove-ocml branch October 9, 2026 16:23
@assistant-librarian

Copy link
Copy Markdown
Contributor

TheRock Submodule Bump Activity

Newest first

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants