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

Assertion `V.getNode() && "Getting TableId on SDValue()"' failed. #1767

Closed
Vsevolod-Livinskij opened this issue May 27, 2020 · 3 comments · Fixed by #1770
Closed

Assertion `V.getNode() && "Getting TableId on SDValue()"' failed. #1767

Vsevolod-Livinskij opened this issue May 27, 2020 · 3 comments · Fixed by #1770
Assignees
Labels
LLVM bug Bug in LLVM YARPGen Found by YARPGen (random testing)

Comments

@Vsevolod-Livinskij
Copy link
Contributor

ISPC fails with

>$ispc --target=avx2-i32x16 -o func.o func.ispc
ispc: /llvm-10.0/llvm/lib/CodeGen/SelectionDAG/LegalizeTypes.h:137: llvm::DAGTypeLegalizer::TableId llvm::DAGTypeLegalizer::getTableId(llvm::SDValue): Assertion `V.getNode() && "Getting TableId on SDValue()"' failed.

Reproducer:

extern uniform bool arr_4 [17];
export void test(uniform int64 var_1, uniform bool var_15) {
    foreach(i_0 = var_1...16U)
        arr_4 [i_0] = !(varying bool) reduce_min(180) ? 5923697687793782713ULL : (varying bool) var_15;
}

ISPC version:
1.14.0dev (build commit 763adc5 @ 20200526, LLVM 10.0.0)

@DeepakRajendrakumaran DeepakRajendrakumaran self-assigned this May 27, 2020
@dbabokin dbabokin added the YARPGen Found by YARPGen (random testing) label May 27, 2020
@DeepakRajendrakumaran
Copy link
Collaborator

This is a llvm codegen crash with O2 opt. It's fine with O0. Might possibly be a llvm thing. Have to confirm.

@DeepakRajendrakumaran
Copy link
Collaborator

This has already been fixed in llvm trunk - https://reviews.llvm.org/D76994

@dbabokin you want me to go ahead and port the patch?

@dbabokin
Copy link
Collaborator

Yes.

DeepakRajendrakumaran added a commit to DeepakRajendrakumaran/ispc that referenced this issue May 28, 2020
dbabokin pushed a commit that referenced this issue May 28, 2020
dbabokin pushed a commit to dbabokin/ispc that referenced this issue May 30, 2020
dbabokin pushed a commit that referenced this issue May 30, 2020
@dbabokin dbabokin added the LLVM bug Bug in LLVM label Jun 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
LLVM bug Bug in LLVM YARPGen Found by YARPGen (random testing)
Projects
None yet
3 participants