Aligning implementation/specification error codes - phase 4 - #1993
Open
Roman-Manevich wants to merge 9 commits into
Open
Aligning implementation/specification error codes - phase 4#1993Roman-Manevich wants to merge 9 commits into
Roman-Manevich wants to merge 9 commits into
Conversation
Roman-Manevich
changed the base branch from
master
to
asl-error-codes-phase-3
September 5, 2026 22:53
Roman-Manevich
force-pushed
the
asl-error-codes-phase-4
branch
3 times, most recently
from
September 5, 2026 23:43
3ec2709 to
6ac171a
Compare
Roman-Manevich
requested review from
HadrienRenaud and
hrutvik
and removed request for
HadrienRenaud
September 6, 2026 15:31
Roman-Manevich
marked this pull request as ready for review
September 6, 2026 15:32
| | BadField of string * ty | ||
| | MissingField of string list * ty | ||
| | BadSlices of error_handling_time * slice list * int | ||
| | BadIndex of error_handling_time * int * int |
Collaborator
There was a problem hiding this comment.
worth a record I think
Collaborator
Author
There was a problem hiding this comment.
I prefer records, but why this specific one and not all other constructors with multiple fields?
| File static-evaluation-non-literal.asl, line 5, characters 13 to 24: | ||
| constant C = R { x = 1 }; | ||
| ^^^^^^^^^^^ | ||
| ASL Type error (TE_SEF): Static evaluation of expression R { x = 1 } failed. |
Collaborator
There was a problem hiding this comment.
Oh I had forgotten we had this limitation. Could we put it on a ticket please?
Collaborator
Author
There was a problem hiding this comment.
I'm not sure what you want the ticket to say.
Roman-Manevich
force-pushed
the
asl-error-codes-phase-4
branch
from
September 7, 2026 09:12
6ac171a to
6589fe9
Compare
hrutvik
approved these changes
Sep 7, 2026
Roman-Manevich
force-pushed
the
asl-error-codes-phase-4
branch
from
September 7, 2026 21:52
6589fe9 to
d963482
Compare
Roman-Manevich
force-pushed
the
asl-error-codes-phase-4
branch
from
September 7, 2026 21:58
d963482 to
bd22e37
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Classify unsupported parameter expressions
Expressions that cannot define subprogram parameters now report
TE_BSPDthroughBadParameterExpr, matchingparams_of_exprinasl.spec.Classify invalid collection field types
Non-bitvector collection fields now report
TE_UTthroughcheck_structure_bits, matching the per-fieldcheck_structure_labelchecks inasl.spec.Classify unsupported parameter types
Types not permitted as subprogram argument or return types now report
TE_BSPDthroughBadParameterType;asl.specand a documented regression now cover pending-constrained integers, and the obsoleteUnsupportedTydiagnostic is removed.Classify out-of-bounds indices
Out-of-bounds native vector accesses now report
DE_BI, orTE_SEFduring static evaluation, throughBadIndexinstead of the uncodedMismatchTypediagnostic.Classify arbitrary negative-length arrays
Parameterized arrays whose length evaluates negatively now report
DE_AETthroughArbitraryEmptyType, matchingEArbitraryinasl.spec.Classify static evaluation failures
Static evaluation that does not produce a literal now reports
TE_SEFthroughStaticEvaluationFailure, matchingstatic_evalinasl.spec.Classify implementation integer overflow
Integers exceeding OCaml
intlimits now report a structured internalImplementationIntegerOverflowdiagnostic instead of the uncodedUnsupportedExpr.