webmachinelearning / webnn-native

🧠⚙️ Standalone native implementation of the Web Neural Network API
Apache License 2.0
50 stars 21 forks source link

Failed to build oneDNN backend with upgraded oneDNN dependency #371

Open BruceDai opened 2 years ago

BruceDai commented 2 years ago

PR #366 with latest commit@ebf149c and PR #330 with commit@59e6099 caught build errors on both Linux and Windows platforms.

# Linux
ninja: error: '../../third_party/oneDNN/build/src/libdnnl.so.2.2', needed by 'libdnnl.so.2.2', missing and no known rule to make it
# Windows
[160/342] CXX obj/src/webnn/native/sources/GraphDNNL.obj
FAILED: obj/src/webnn/native/sources/GraphDNNL.obj 

Please see detail build error on attachment build_error_oneDNN_win.txt

Before fixing issue, I plan to disable "Auto Roll DEPS / oneDNN" CI to keep PR clean. WDYT @fujunwei, thanks.

vpirogov commented 1 year ago

These commits are from oneDNN v3.0 release. This is a major release introducing API changes in comparison to v2.x releases summarized here.