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
[SR-9741] [AD] Deserialization failure of AD-generated struct types #52172
Comments
Do I need some extra stuff that's not in head to reproduce this? When I try to compile it, I get workspace/mergefun.swift:2:31: error: use of undeclared type 'Layer'
public struct Conv2D<Scalar>: Layer
^~~~~ This is the same error message that Gogul found and fixed in lldb, though Gogul only made lldb changes to fix that, so non-lldb invocations should not be affected by the problem or the fix. If this isn't too difficult, it's probably less disruptive to fix-forward rather than reverting the merge, because the merge has been in for a few days and people have been working off of it. |
Latest HEAD should have `Layer` defined already. That said, you can also try compiling https://github.com/rxwei/DeepLearning using `swift build`. That would fail as well. |
I may not have the absolute latest head. I'll try pulling that and rebuilding and seeing if I can reproduce, and then I'll see if I can figure out a fix-forward. @dan-zheng says he's working on a revert of the merge in the meantime. |
Comment by Gogul Balakrishnan (JIRA) @rxwei & @dan-zheng, I was not able to reproduce this on my linux box. |
I finally finished a clean build at head. I can also compile the code posted in this bug successfully. (On my gLinux) |
Have you tried `swift build`? I think running swiftc doesn't deserialize things. |
Oh, running `swift build` in https://github.com/rxwei/DeepLearning (at commit 731ce402ce0bd7459b898b112deb89379bbda893) does indeed trigger this error message for me. |
We did some further digging and verified that it's not caused by the merge. It may actually be related to AD generics support #22023 We haven't tested `swift build` between that point and the merge. |
Additional Detail from JIRA
md5: b1b015f4fa75ca65cf786d107843db57
Issue Description:
The text was updated successfully, but these errors were encountered: