Skip to content

Commit

Permalink
PR #21191: [xla:cpu] Fix missing header in oneDNN ACL build
Browse files Browse the repository at this point in the history
Imported from GitHub PR #21191

Fixes build error
```
Compiling src/cpu/jit_utils/jit_utils.cpp failed: (Exit 1): clang failed: error executing command (from target @mkl_dnn_acl_compatible//:mkl_dnn_acl) /usr/lib/llvm-14/bin/clang -U_FORTIFY_SOURCE -fstack-protector -Wall -Wthread-safety -Wself-assign -Wunused-but-set-parameter -Wno-free-nonheap-object -fcolor-diagnostics -fno-omit-frame-pointer -g0 ... (remaining 126 arguments skipped)

Use --sandbox_debug to see verbose messages from the sandbox and retain the sandbox build root for debugging
external/mkl_dnn_acl_compatible/src/cpu/jit_utils/jit_utils.cpp:34:10: fatal error: 'common/ittnotify/jitprofiling.h' file not found
#include "common/ittnotify/jitprofiling.h"
         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
1 error generated.
INFO: Elapsed time: 524.121s, Critical Path: 452.78s
INFO: 543 processes: 45 internal, 498 linux-sandbox.
FAILED: Build did NOT complete successfully
```
Build step:
bazel build --config=mkl_aarch64_threadpool --test_output=all --spawn_strategy=sandboxed //xla/...
Copybara import of the project:

--
23e8fad by Crefeda Rodrigues <[email protected]>:

[xla:cpu] Fix missing headers in oneDNN ACL build

Signed-off-by: Crefeda Rodrigues <[email protected]>

Merging this change closes #21191

FUTURE_COPYBARA_INTEGRATE_REVIEW=#21191 from cfRod:aarch64-xla-build 23e8fad
PiperOrigin-RevId: 713943675
  • Loading branch information
cfRod authored and Google-ML-Automation committed Jan 10, 2025
1 parent 69e993c commit 00d98b7
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions third_party/tsl/third_party/mkl_dnn/mkldnn_acl.BUILD
Original file line number Diff line number Diff line change
Expand Up @@ -167,6 +167,7 @@ cc_library(
"include/**/*",
"include/*",
"src/common/*.hpp",
"src/common/**/*.h",
"src/cpu/**/*.hpp",
"src/cpu/*.hpp",
"src/cpu/aarch64/xbyak_aarch64/**/*.h",
Expand Down

0 comments on commit 00d98b7

Please sign in to comment.