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

DPC++ compiler crashes during SYCL-CTS test build. #10129

Closed
bader opened this issue Jun 29, 2023 · 0 comments · Fixed by #10211
Closed

DPC++ compiler crashes during SYCL-CTS test build. #10129

bader opened this issue Jun 29, 2023 · 0 comments · Fixed by #10211
Assignees
Labels
bug Something isn't working compiler Compiler related issue confirmed CTS Impacts Khronos SYCL CTS

Comments

@bader
Copy link
Contributor

bader commented Jun 29, 2023

I see failure in pre-commit for KhronosGroup/SYCL-CTS#741.
From: https://github.com/KhronosGroup/SYCL-CTS/actions/runs/5406893408/jobs/9824294810

sycl-post-link: /__w/llvm/llvm/src/llvm/tools/sycl-post-link/SpecConstants.cpp:506: llvm::Instruction* {anonymous}::emitSpecConstant(unsigned int, llvm::Type*, llvm::Instruction*, llvm::Constant*): Assertion `DefaultValue && "default value of spec constant is expected to be known"' failed.
PLEASE submit a bug report to https://github.com/intel/llvm/issues and include the crash backtrace.
Stack dump:
0.	Program arguments: /opt/sycl/bin/sycl-post-link -split=auto -emit-param-info -symbols -emit-exported-symbols -split-esimd -lower-esimd -O3 -spec-const=rt -device-globals -o /tmp/accessor_default_values_core-c0a0db.table /tmp/accessor_default_values_core-bcf2c7.bc
Stack dump without symbol names (ensure you have llvm-symbolizer in your PATH or set the environment var `LLVM_SYMBOLIZER_PATH` to point to it):
0  sycl-post-link 0x0000565301a04640
1  sycl-post-link 0x0000565301a024b4
2  libc.so.6      0x00007fb5e99db520
3  libc.so.6      0x00007fb5e9a2fa7c pthread_kill + 300
4  libc.so.6      0x00007fb5e99db476 raise + 22
5  libc.so.6      0x00007fb5e99c17f3 abort + 211
6  libc.so.6      0x00007fb5e99c171b
7  libc.so.6      0x00007fb5e99d2e96
8  sycl-post-link 0x00005653017ca288
9  sycl-post-link 0x00005653017c9af4
10 sycl-post-link 0x00005653017c9d62
11 sycl-post-link 0x00005653017cc997
12 sycl-post-link 0x00005653017a3e86
13 sycl-post-link 0x0000565301930649
14 sycl-post-link 0x00005653017b2718
15 sycl-post-link 0x000056530172e005
16 libc.so.6      0x00007fb5e99c2d90
17 libc.so.6      0x00007fb5e99c2e40 __libc_start_main + 128
18 sycl-post-link 0x00005653017a33e5
clang++: error: unable to execute command: Aborted (core dumped)
clang++: error: sycl-post-link command failed due to signal (use -v to see invocation)
clang version 17.0.0 (https://github.com/intel/llvm 2ddb42c40c6737ce9faaecfdc9c5f684eba237e4)
Target: x86_64-unknown-linux-gnu
Thread model: posix
InstalledDir: /opt/sycl/bin
clang++: note: diagnostic msg: Error generating preprocessed source(s).
ninja: build stopped: subcommand failed.
Error: Process completed with exit code 1.

DPC++ compiler version: 2ddb42c.

@bader bader added bug Something isn't working CTS Impacts Khronos SYCL CTS compiler Compiler related issue labels Jun 29, 2023
@AlexeySachkov AlexeySachkov self-assigned this Jul 5, 2023
AlexeySachkov added a commit to AlexeySachkov/llvm that referenced this issue Jul 5, 2023
This patch fixes a regression accidentally introduced in intel#9874
(presumably because we didn't have the right test).

SYCL-CTS produces an LLVM IR pattern, where spec constant initializer
contains less elements than there are in a spec constant type, because
some of those elements are implicitly-created paddings. The patch
updates the pass to handle this situation.

Resolves intel#10129
AlexeySachkov added a commit that referenced this issue Jul 14, 2023
… pass (#10211)

This patch fixes a regression accidentally introduced in #9874
(presumably because we didn't have the right test).

SYCL-CTS produces an LLVM IR pattern, where spec constant initializer
contains less elements than there are in a spec constant type, because
some of those elements are implicitly-created paddings. The patch
updates the pass to handle this situation.

Note: this PR only covers a case when padding is inserted at the end of
a struct.

Resolves #10129
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working compiler Compiler related issue confirmed CTS Impacts Khronos SYCL CTS
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants