Better error messages for failed subeffect constraints
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 5/5
- Tempo stimato
- Più di una settimana
- Idoneità per principianti
- 35/100
- Tipo di issue
- Funzionalità
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Attiva
- Ambito
- compilers
Direzione di ricerca
Start by reading the effect constraint generation, effect-provenance graph, and lax-equation unification paths described in the issue; the payload does not name specific files or tests. Trace how subeffect constraints from lambdas, local definitions, casts, and holes are represented, then check how a failed constraint reaches the diagnostic. Done means the error identifies the actual and expected effects and traces failed lax equations back to their originating subeffect constraint.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
A subeffect constraint s ⊑ t (the body's effect is a subset of the lambda's effect) has no provenance of its own. When it cannot be solved, it is reported as if it were the equality t ~ s, using the provenance and the MismatchedEffects error of an equality. When the effect-provenance graph finds no path, the leftover lax equation s ∩ ¬t ~ ∅ names neither side, so the message gets worse still.
With subeffecting on by default (#13579), every lambda and local def, checked effect cast and hole produces such a constraint, so these messages will become common.
Proposal:
- Give subeffect constraints a provenance of their own, e.g.
ExpectSubEff(expected, actual, loc), and have the effect-provenance graph treat it as a flow edge through the lambda's effect variable. - Report a failed subeffect constraint with a dedicated error that names the effects of the actual effect that the expected effect does not allow.
- When effect unification fails on the lax equation, trace it back to the subeffect constraint it came from instead of reporting the raw set equation.
#13566 is a first attempt; it is closed in favour of this issue.
🤖 Generated with Claude Code
- Lingua principale
- Flix
- Stelle
- 2.7k
- Fork
- 194
- Merge medio
- 7h
- PR unite (30g)
- 137
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Nessun modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di flix/flix
-
bug
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 72/100
flix/flix#9715 · 2 commenti · 1 reazione ·
I maintainer di solito rispondono entro 1 giorno
-
Effect lock: three package changes that eff-check does not seeForse già presa Una pull request collegata a questa issue è aperta o già unita. Apertabug
Difficoltà 5/5 Più di una settimana Idoneità per principianti 38/100
I maintainer di solito rispondono entro 1 giorno
-
A: codegen
Difficoltà 4/5 3-5 giorni Idoneità per principianti 28/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 42/100
I maintainer di solito rispondono entro 1 giorno
-
bug
Difficoltà 4/5 3-5 giorni Idoneità per principianti 55/100
flix/flix#13459 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
Issue simili
-
bot-triaged module: inductor oncall: pt2 topic: fuzzer triaged
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
I maintainer di solito rispondono entro 1 giorno
-
bug needs-triage
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
microsoft/DirectXShaderCompiler#9087 ·
I maintainer di solito rispondono entro 1 giorno
-
confirmed generated by fuzzer llvm:codegen miscompilation
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
llvm/llvm-project#237380 ·
I maintainer di solito rispondono entro 1 giorno
-
Source comments still name the removed set_observer_thresholds and set_observer_scale builtinsApertaarea:observer area:trace-tape found-by:code-review kind:docs-drift
Difficoltà 2/5 1-3 ore Idoneità per principianti 85/100
InauguralSystems/EigenScript#2212 ·
I maintainer di solito rispondono entro 1 giorno
-
bug medium priority openmp tokenizer wrong result
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
I maintainer di solito rispondono entro 1 giorno