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

compiler: not implemented handling of *ast.IndexListExpr in substituteTypeArgs function #141

Closed
iamrajiv opened this issue Feb 11, 2024 · 2 comments

Comments

@iamrajiv
Copy link

Description

When I run examples/scrape, I am getting the below error:

➜  scrape git:(perf) ✗ make
./coroc .
2024/02/11 06:42:14.710485 reading, parsing and type-checking
2024/02/11 06:42:15.271370 building SSA program
2024/02/11 06:42:15.477581 building callgraph using vta algorithm
2024/02/11 06:42:15.849120 collecting generic instances
2024/02/11 06:42:15.849958 finding yield points
2024/02/11 06:42:15.849980 coloring functions
2024/02/11 06:42:15.850092 compiling package main
2024/02/11 06:42:15.851580 compiling function main main
2024/02/11 06:42:15.851687 compiling function literal main
panic: not implemented: *ast.IndexListExpr [recovered]
	panic: not implemented: *ast.IndexListExpr

goroutine 1 [running]:
...
...
...
@iamrajiv
Copy link
Author

iamrajiv commented Feb 11, 2024

Hi @achille-roussel,

After investigation, I found that that panic is happening because the substituteTypeArgs function in the types.go file is not handling the case for *ast.IndexListExpr properly.

To fix this issue, we need to add a case for *ast.IndexListExpr in the substituteTypeArgs function.

@iamrajiv iamrajiv changed the title compiler: Panic due to unimplemented feature with *ast.IndexListExpr compiler: not implemented handling of *ast.IndexListExpr in substituteTypeArgs function Feb 11, 2024
@chriso
Copy link
Contributor

chriso commented Jun 17, 2024

Fixed by #148

@chriso chriso closed this as completed Jun 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants