-
Notifications
You must be signed in to change notification settings - Fork 212
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
Alias type for List and ADT types #2063
Labels
Comments
I found a bug that may be related to the discussion here.
In the above, the last clause defining |
Also, the following example is not working:
|
We have similar issues with ADTs:
|
This was referenced Mar 2, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The current version of Souffle prohibits alias types for lists and ADTs. For example,
and
will fail.
It is a simple change in the type system. We need a special case. In method
TypeDeclarationChecker::checkUnionType
the case,can be extended to
for this purpose. Also, I would like to remark that it might be better to introduce an equivalence type class rather than using the union-type class.
However, the I/O system needs fixing as well. I.e., the root type must be passed on (i.e. the actual List / ADT) so that the I/O system can handle the complex type.
The text was updated successfully, but these errors were encountered: