Skip to content

[SOT] Support builtin dispatch for is_compiled_with_onednn#75747

Merged
SigureMo merged 3 commits intoPaddlePaddle:developfrom
co63oc:l12
Oct 12, 2025
Merged

[SOT] Support builtin dispatch for is_compiled_with_onednn#75747
SigureMo merged 3 commits intoPaddlePaddle:developfrom
co63oc:l12

Conversation

@co63oc
Copy link
Copy Markdown
Contributor

@co63oc co63oc commented Oct 11, 2025

PR Category

User Experience

PR Types

Others

Description

replace mkldnn in test_builtin_dispatch.py

@paddle-bot
Copy link
Copy Markdown

paddle-bot Bot commented Oct 11, 2025

你的PR提交成功,感谢你对开源项目的贡献!
请关注后续CI自动化测试结果,详情请参考Paddle-CI手册
Your PR has been submitted. Thanks for your contribution!
Please wait for the result of CI firstly. See Paddle CI Manual for details.

@paddle-bot paddle-bot Bot added the contributor External developers label Oct 11, 2025
res5 = paddle.base.libpaddle.is_compiled_with_custom_device("npu")
res6 = paddle.base.libpaddle.is_compiled_with_ipu()
res7 = paddle.base.libpaddle.is_compiled_with_xpu()
res8 = paddle.base.libpaddle.is_compiled_with_mkldnn()
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

PaddlePaddle 3.3 会移除 is_compiled_with_mkldnn 么?如果只是弃用而不是移除,这里的测试需要保留,同时添加 is_compiled_with_onednn 的测试,否则会影响下游用户的使用,如果确定是移除,那么这里可以替换

以及需要单独在 python/paddle/jit/sot/utils/paddle_api_config.py 添加 is_compiled_with_onednn 才可能跑通这里的单测

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

PaddlePaddle 3.3是弃用,PaddlePaddle 4.0移除,已增加 is_compiled_with_onednn 和 配置

@co63oc
Copy link
Copy Markdown
Contributor Author

co63oc commented Oct 12, 2025

/re-run all-failed

Comment thread test/sot/test_builtin_dispatch.py Outdated
@SigureMo SigureMo changed the title CI测试不review replace mkldnn in test_builtin_dispatch.py [fluid_ops] [SOT] Support builtin dispatch for is_compiled_with_onednn Oct 12, 2025
@co63oc
Copy link
Copy Markdown
Contributor Author

co63oc commented Oct 12, 2025

/re-run all-failed

@SigureMo SigureMo merged commit 08fe857 into PaddlePaddle:develop Oct 12, 2025
98 of 101 checks passed
SigureMo added a commit to cattidea/Paddle that referenced this pull request Oct 14, 2025
…addle#75747)

---------

Co-authored-by: Nyakku Shigure <sigure.qaq@gmail.com>
@co63oc co63oc deleted the l12 branch October 21, 2025 07:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

contributor External developers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants