Skip to content

[libdevice](fix) Remove duplicated deprecated function - #1889

Open
Levein7 wants to merge 1 commit into
triton-lang:main-devfrom
Levein7:update-libdevice
Open

[libdevice](fix) Remove duplicated deprecated function#1889
Levein7 wants to merge 1 commit into
triton-lang:main-devfrom
Levein7:update-libdevice

Conversation

@Levein7

@Levein7 Levein7 commented Sep 1, 2026

Copy link
Copy Markdown
Contributor
  1. Remove duplicated "_deprecated" function in libdevice.py
  2. Update pre-built libdevice binary file
    Corresponding libdevice repo commit:
    https://gitcode.com/Ascend/libdevice/commit/3ed2b0b8

Copilot AI lite review requested due to automatic review settings September 1, 2026 02:36
@github-actions github-actions Bot added python Changes to Python runtime or bindings ascend-backend Changes to the Ascend NPU backend labels Sep 1, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR removes the locally defined _deprecated decorator from the Ascend CANN libdevice.py and switches the deprecated libdevice wrappers to reuse the shared implementation from cann/extension/_utils.py.

Changes:

  • Removed the duplicate _deprecated implementation from third_party/ascend/language/cann/libdevice.py.
  • Added a late import of _deprecated from third_party/ascend/language/cann/extension/_utils.py to decorate deprecated libdevice wrapper functions.
  • Updated deprecated wrapper decorators to pass fully-qualified fn_name and replacement strings.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread third_party/ascend/language/cann/libdevice.py Outdated
@Levein7
Levein7 force-pushed the update-libdevice branch 3 times, most recently from aebf871 to 72ed11d Compare September 1, 2026 10:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ascend-backend Changes to the Ascend NPU backend python Changes to Python runtime or bindings

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants