Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

all: clean graph compiler backend #2966

Merged
merged 1 commit into from
Mar 28, 2025
Merged

all: clean graph compiler backend #2966

merged 1 commit into from
Mar 28, 2025

Conversation

TaoLv
Copy link
Contributor

@TaoLv TaoLv commented Mar 27, 2025

The graph compiler backend has been under deprecation since v3.6 release.
We decide to clean it in v3.8 to reduce the maintenance burden.

MFDNN-13407

@TaoLv TaoLv requested review from a team as code owners March 27, 2025 01:19
@github-actions github-actions bot added documentation A request to change/fix/improve the documentation. Codeowner: @oneapi-src/onednn-doc component:graph-api Codeowner: @oneapi-src/onednn-graph component:tests Codeowner: @oneapi-src/onednn-arch component:build labels Mar 27, 2025
10. LLVM (src/graph/backend/graph_compiler/core/src/util/array_ref.hpp)
==============================================================================
The LLVM Project is under the Apache License v2.0 with LLVM Exceptions:
==============================================================================
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hi @vgvozdeva , please note that the TPP file is changed here. It may affect the BOM file as well. Thanks.

@TaoLv
Copy link
Contributor Author

TaoLv commented Mar 27, 2025

make test

Comment on lines -30 to -31
Boolean is only supported in the Graph Compiler in CPU engines. No
primitives support boolean during primitive computation.
Copy link
Contributor

Choose a reason for hiding this comment

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

If boolean is not supported by any other functionality shall we drop it from the table above as well?

+@uxlfoundation/onednn-arch

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It's supported by graph API to define the causal masks.

@TaoLv TaoLv merged commit 396fdfc into main Mar 28, 2025
22 of 23 checks passed
@TaoLv TaoLv deleted the lvtao/main/clean-gc branch March 28, 2025 15:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component:build component:graph-api Codeowner: @oneapi-src/onednn-graph component:tests Codeowner: @oneapi-src/onednn-arch documentation A request to change/fix/improve the documentation. Codeowner: @oneapi-src/onednn-doc
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants