Per-constructor @unboxed is not compared across interfaces or coercions
维护者通常 1 天内回复
评估
- 难度
- 4/5
- 预计耗时
- 3-5 天
- 新手友好度
- 25/100
- Issue 类型
- 缺陷
- 描述清晰度
- 基本清楚
- 活跃度
- 活跃
- 技术栈
- ocaml
- 领域
- compilers
调研方向
The cause lies in Includecore.compare_variants, Variant_coercion.variant_representation_matches, and Printtyp.print_variant_configuration_issue in the compiler. Read those modules to see how variant tags and @unboxed are checked. Done means an untagged constructor only matches an untagged one across interfaces and coercions, with correct error naming.
由索引模型根据 Issue 内容生成。
描述
Whether a constructor is @unboxed (untagged) is not compared across interfaces or in coercions:
module M: {
type t = | @unboxed A(int) | B({x: int})
let a: t
} = {
type t = | A(int) | B({x: int})
let a = A(3)
}
let n = switch M.a { | A(i) => i | B(_) => -1 } // -1: the client expects 3, the implementation built {TAG: "A", _0: 3}
The reverse direction is accepted too. So is a coercion between the two representations:
type a = | @unboxed A(int) | B({x: int})
type b = | A(int) | B({x: int}) | C
let x: b = (A(3): a :> b) // 3 is not a b: matching it gives C
Also, when the @unboxed configurations of two variants differ, the coercion error says the wrong variant is unboxed ("Variant c is unboxed" for the plain one).
Observed on master (d48d577bf6). Cause:
Includecore.compare_variantscompares tags of tagged constructors only, so a tagged and an untagged constructor agree.- The coercion check compares only the declared
@as(Variant_coercion.variant_representation_matches). Printtyp.print_variant_configuration_issueinverts "not".
Expected: an untagged constructor agrees only with an untagged one, in interfaces and coercions alike, and the error names the unboxed variant.
Found by a review of the untagged variant work (#8950).
- 主要语言
- OCaml
- 星标
- 7.5k
- 派生
- 483
- 平均合并
- 20 小时
- 30 天内合并 PR
- 219
环境准备
在浏览器里用你自己的 GitHub 账号启动这个项目的开发容器。
- 没有 Dockerfile 或 Docker Compose 文件
- 有 Pull Request 模板
- 阅读贡献指南
从这里开始
- 先读完整个 Issue,再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 Issue 编号。
rescript-lang/rescript 的其他 Issue
-
难度 2/5 1-3 小时 新手友好度 75/100
rescript-lang/rescript#9037 ·
维护者通常 1 天内回复
-
难度 5/5 一周以上 新手友好度 12/100
rescript-lang/rescript#9044 · 2 个 reaction ·
维护者通常 1 天内回复
-
难度 5/5 一周以上 新手友好度 35/100
rescript-lang/rescript#9006 · 1 个 reaction ·
维护者通常 1 天内回复
-
难度 3/5 1-2 天 新手友好度 70/100
rescript-lang/rescript#8997 ·
维护者通常 1 天内回复
-
难度 3/5 1-2 天 新手友好度 68/100
rescript-lang/rescript#8994 ·
维护者通常 1 天内回复
查看 rescript-lang/rescript 的全部 Issue
相似的 Issue
-
难度 2/5 1-3 小时 新手友好度 68/100
维护者通常 1 天内回复
-
enhancement scope:diagnostics scope:syntax
难度 2/5 1-3 小时 新手友好度 77/100
-
code: refactoring good first issue
难度 2/5 1-3 小时 新手友好度 68/100
idris-lang/Idris2#3887 ·
维护者通常 1 天内回复
-
severity: low
难度 2/5 1-3 小时 新手友好度 72/100
luainkernel/lunatik#2207 ·
维护者通常 1 天内回复
-
backend:X86 generated by fuzzer llvm:globalisel miscompilation
难度 2/5 1-3 小时 新手友好度 72/100
llvm/llvm-project#235468 · 1 条评论 ·
维护者通常 1 天内回复