Correctly handle PartialTypeVars in inference #305
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Should resolve the new problem in #282 .
Occasionally, the inference result associated to an SSA is
Core.Compiler.PartialTypeVar
. This means that inference has successfully determined that the SSA will be aTypeVar
. More than that, it has figured out some of the runtime information about the SSA, in a similar sense (I believe) to the way thatCore.Compiler.PartialStruct
knows some runtime information about a givenstruct
. I've asked on slack for a MWE of a function in which this is the result of inference. I'll wait a few hours, but will merge this with the current unit test if I nothing appears.edit: many of the people who would know about this are based in the US. I'll get this merged asap, and make a new PR if an MWE becomes available.