ITStudio_OUC/OxideJS-New
0
1
Fork 0
mirror of https://github.com/pjh456/OxideJS.git synced 2026-10-23 18:21:13 +08:00
Code Issues Packages Projects Releases Wiki Activity

fix: 顶层 var 全局属性求值前创建（声明定时语义）+ 无初始化 var 声明不再抹槽值

Browse Source
This commit is contained in:
彭俊皓
2026-09-09 22:00:29 +08:00
parent bd1427b116
commit 2457f283fe
6 changed files with 115 additions and 16 deletions
Download Patch File Download Diff File Expand all files Collapse all files
crates/oxide_compiler/tests/compile_stmt_tests.rs
+2 -1
View File
@@ -14,7 +14,8 @@ fn compile_source(source: &str) -> CompiledModule {
fn compile_var_declaration() {
let module = compile_source("var x = 42;");
assert!(!module.bytecode.is_empty());
assert_eq!(module.constants[0], Constant::Int(42));
// 常量池下标不固定（声明实例化序言可能先入池 Undefined），只断言成员存在。
assert!(module.constants.contains(&Constant::Int(42)));
}
#[test]
crates/oxide_compiler/tests/symbol_table_tests.rs
+2 -1
View File
@@ -14,7 +14,8 @@ fn compile_source(source: &str) -> CompiledModule {
fn symbol_table_declare_and_lookup() {
let module = compile_source("var x = 42;");
assert!(!module.bytecode.is_empty());
assert_eq!(module.constants[0], Constant::Int(42));
// 常量池下标不固定（声明实例化序言可能先入池 Undefined），只断言成员存在。
assert!(module.constants.contains(&Constant::Int(42)));
}
#[test]
crates/oxide_emit/src/emit.rs
+13
View File
@@ -1555,6 +1555,8 @@ impl Emitter {
/// 随后由 `oxide_ir::lower::lower` 降为字节码。
/// `repl_persist` 为 true 时脚本顶层 let/const 也写全局对象（REPL 跨轮次持久）；
/// `is_eval_script` 为 true 时顶层 var/function 声明落全局属性 configurable:true。
/// 顶层 var 的全局对象属性在求值开始前统一创建（值 undefined），声明语句
/// 保持赋值语义——声明语句出现之前的读取（typeof/反射/自引用）即见绑定。
pub fn emit_program(
&self, program: &oxide_parser::Program, repl_persist: bool, is_eval_script: bool,
) -> Result<IRFunction, String> {
@@ -1603,6 +1605,17 @@ impl Emitter {
}
}
// 全局声明实例化序言：脚本求值前为顶层 var 名创建全局对象属性（值 undefined），
// 使声明语句执行前的读取（typeof、反射、自引用）可经全局对象见绑定；同名
// 函数声明的属性由首个 sub-pass 以函数值覆盖，声明语句保持值更新语义。
let gdi_var_names = self.collect_var_binding_names(&program.body);
if !gdi_var_names.is_empty() {
let undef_reg = self.emit_undefined(&mut ctx);
for name in &gdi_var_names {
self.emit_global_prop_write(name, undef_reg, &mut ctx);
}
}
// 首个 sub-pass：发函数声明（hoisting），保证任何代码运行前函数对象已就绪。
for stmt in &program.body {
if matches!(stmt, Statement::FunctionDeclaration(_)) {
crates/oxide_emit/src/stmt/declaration/var.rs
+43 -14
View File
@@ -48,37 +48,66 @@ impl Emitter {
let tmp = ctx.alloc_reg();
ctx.inst(Inst::load_const(Operand::Reg(tmp), idx));
let var_reg = ctx.alloc_reg();
let target_reg = if matches!(decl.kind, VariableDeclarationKind::Var) {
// 无初始化 var 是纯声明而非赋值：绑定已预先存在（提升引用或先前写入）
// 时保留槽值，仅首次声明把 undefined 物化进槽。
let (target_reg, already_bound) = if matches!(decl.kind, VariableDeclarationKind::Var) {
match ctx.declare(bi.name.as_str(), var_reg, decl.kind, is_const) {
Ok(()) => var_reg,
Err(_) => ctx.lookup(bi.name.as_str()).unwrap_or(var_reg),
Ok(()) => (var_reg, false),
Err(_) => (ctx.lookup(bi.name.as_str()).unwrap_or(var_reg), true),
}
} else {
// let/const 无初始化器：复用本块预声明槽位，否则新声明
// （for 头等未预声明路径）。
let var_reg = if let Some(reg) = ctx.scopes.symbols.consume_predeclared_slot(bi.name.as_str()) {
let target = if let Some(reg) = ctx.scopes.symbols.consume_predeclared_slot(bi.name.as_str()) {
reg
} else {
let r = ctx.alloc_reg();
ctx.declare(bi.name.as_str(), r, decl.kind, is_const)?;
r
};
var_reg
(target, false)
};
if let Some(&cell_idx) = ctx.captured_bindings.get(bi.name.as_str()) {
ctx.inst(Inst::new(
OpCode::MAKE_CELL,
Operand::Reg(tmp),
Operand::Imm(cell_idx as u16),
Operand::None,
));
} else {
// 被捕获绑定的 cell 在函数入口已实例化（undefined）；声明语句
// 不赋值，仅首次声明刷新初值，已绑定的 cell 保留现值。
if !already_bound {
ctx.inst(Inst::new(
OpCode::MAKE_CELL,
Operand::Reg(tmp),
Operand::Imm(cell_idx as u16),
Operand::None,
));
}
} else if !already_bound {
ctx.inst(Inst::new(OpCode::STORE_VAR, Operand::Reg(target_reg), Operand::Reg(tmp), Operand::None));
}
ctx.init_var(bi.name.as_str());
// 脚本顶层 var 无初始化：仍须写全局对象属性（值为 undefined）。
// 脚本顶层 var：全局对象属性同步绑定当前值（首次声明即 undefined，
// 先前已写入则保留写入值，声明不是覆盖性赋值）。被捕获绑定的值
// 存在 cell 而非槽，从 cell 同步。
if ctx.is_global_scope && matches!(decl.kind, VariableDeclarationKind::Var) {
self.emit_global_prop_write(bi.name.as_str(), tmp, ctx);
let src = if let Some(&cell_idx) = ctx.captured_bindings.get(bi.name.as_str()) {
let r = ctx.alloc_reg();
if let Some((binding, _)) = ctx.scopes.symbols.lookup_any_binding(bi.name.as_str()) {
ctx.inst(Inst::new(
OpCode::CELL_GET,
Operand::Reg(r),
Operand::Reg(binding.reg),
Operand::Imm(cell_idx as u16),
));
} else {
ctx.inst(Inst::new(
OpCode::CELL_GET,
Operand::Reg(r),
Operand::None,
Operand::Imm(cell_idx as u16),
));
}
r
} else {
target_reg
};
self.emit_global_prop_write(bi.name.as_str(), src, ctx);
}
r = Some(var_reg);
}
crates/oxide_vm/tests/eval_script_global_tests.rs
+11
View File
@@ -80,6 +80,17 @@ fn script_top_level_var_still_configurable_false() {
assert!(r.is_bool() && r.as_bool(), "脚本顶层 var 应保持 configurable:false，实际 {:?}", r);
}
// ── eval 路径定时：属性在 eval 体求值开始前创建（configurable:true 描述符） ──
#[test]
fn eval_var_property_created_before_eval_body_runs() {
let r = eval("eval(\"var d = (function(){ var p = Object.getOwnPropertyDescriptor(globalThis, 'x'); return p !== undefined && p.configurable === true; })(); var x; d\")").unwrap();
assert!(
r.is_bool() && r.as_bool(),
"eval 体内声明语句前全局属性应已存在且 configurable:true，实际 {:?}",
r
);
}
// ── 严格 eval：未声明写抛 ReferenceError（Step 2 语义经 eval 路径） ──
#[test]
fn strict_eval_undeclared_write_throws_reference_error() {
crates/oxide_vm/tests/global_var_binding_tests.rs
+44
View File
@@ -98,3 +98,47 @@ fn descriptor_attributes_match_spec() {
let d = eval_string("var x = 1; JSON.stringify(Object.getOwnPropertyDescriptor(globalThis, 'x'))");
assert_eq!(d, r#"{"value":1,"writable":true,"enumerable":true,"configurable":false}"#);
}
#[test]
fn var_property_exists_before_declaration_statement() {
// 全局声明实例化：属性在脚本求值开始即创建，声明语句执行前的
// 反射/读取即可见绑定（值 undefined、脚本模式 configurable:false）。
// 断言值经临时全局承载——尾部 var 声明语句会覆盖程序完成值。
eval_truthy(
"(function(){ seen = Object.prototype.hasOwnProperty.call(globalThis, 'gv'); })(); var gv; seen === true",
);
eval_truthy("(function(){ seen = (typeof gv === 'undefined'); })(); var gv; seen === true");
eval_truthy("(function(){ var p = Object.getOwnPropertyDescriptor(globalThis, 'gv'); seen = (p.configurable === false && p.value === undefined); })(); var gv; seen === true");
}
#[test]
fn var_no_init_preserves_prior_write() {
eval_truthy("x = 1; var x; globalThis.x === 1");
eval_truthy("x = 1; var x; x === 1");
}
#[test]
fn var_no_init_preserves_prior_write_captured() {
// 被捕获 var 的值在 cell：声明语句从 cell 同步全局属性，不抹掉先前写入。
eval_truthy("x = 1; var x; (function(){ return x; })() === 1");
eval_truthy("x = 1; var x; globalThis.x === 1");
}
#[test]
fn var_self_reference_reads_binding_value() {
eval_truthy("var x = x; typeof x === 'undefined'");
eval_truthy("x = 7; var x = x; x === 7");
}
#[test]
fn function_callable_before_its_declaration() {
eval_truthy("(function(){ return f() === 42; })(); function f(){ return 42; }");
}
#[test]
fn var_no_init_does_not_clobber_function_binding() {
eval_truthy("function f(){ return 1; } var f; typeof f === 'function' && f() === 1");
// 不比较 globalThis.f === f 同一性：属性读路径存在预存 codegen 缺陷，
// 行为契约是声明后属性仍为原函数对象（可调用、返回值正确）。
eval_truthy("function f(){ return 1; } var f; typeof globalThis.f === 'function' && globalThis.f() === 1");
}