don't truly truncate the heap.

This commit is contained in:
Mark Thom
2017-12-08 21:38:18 -07:00
parent cf6a47272a
commit cee2d5a9c4
5 changed files with 129 additions and 103 deletions

View File

@@ -10,7 +10,7 @@ use std::collections::{HashMap, VecDeque};
use std::fmt; use std::fmt;
use std::io::Error as IOError; use std::io::Error as IOError;
use std::num::{ParseFloatError}; use std::num::{ParseFloatError};
use std::ops::{Add, AddAssign, Div, Sub, Mul, Neg}; use std::ops::{Add, AddAssign, Div, Index, IndexMut, Sub, Mul, Neg};
use std::str::Utf8Error; use std::str::Utf8Error;
use std::vec::Vec; use std::vec::Vec;
@@ -957,7 +957,65 @@ impl AddAssign<usize> for CodePtr {
} }
} }
pub type Heap = Vec<HeapCellValue>; pub struct Heap {
heap: Vec<HeapCellValue>,
pub h: usize
}
impl Heap {
pub fn new() -> Self {
Heap { heap: vec![], h : 0 }
}
pub fn with_capacity(cap: usize) -> Self {
Heap { heap: vec![HeapCellValue::Str(0); cap], h: 0 }
}
pub fn push(&mut self, val: HeapCellValue) {
let h = self.h;
if h < self.heap.len() {
self.heap[h] = val;
} else {
self.heap.push(val);
}
self.h += 1;
}
pub fn truncate(&mut self, h: usize) {
self.h = h;
}
pub fn len(&self) -> usize {
self.heap.len()
}
pub fn append(&mut self, vals: Vec<HeapCellValue>) {
for val in vals.into_iter() {
self.push(val);
}
}
pub fn clear(&mut self) {
self.heap.clear();
self.h = 0;
}
}
impl Index<usize> for Heap {
type Output = HeapCellValue;
fn index(&self, index: usize) -> &Self::Output {
&self.heap[index]
}
}
impl IndexMut<usize> for Heap {
fn index_mut(&mut self, index: usize) -> &mut Self::Output {
&mut self.heap[index]
}
}
pub type Registers = Vec<Addr>; pub type Registers = Vec<Addr>;

View File

@@ -66,8 +66,7 @@ impl CodeOffsets {
}; };
} }
fn second_level_index<Index>(indices: HashMap<Index, ThirdLevelIndex>, fn second_level_index<Index>(indices: HashMap<Index, ThirdLevelIndex>, prelude: &mut CodeDeque)
prelude: &mut CodeDeque)
-> HashMap<Index, IntIndex> -> HashMap<Index, IntIndex>
where Index: Eq + Hash where Index: Eq + Hash
{ {
@@ -117,8 +116,7 @@ impl CodeOffsets {
flattened_index flattened_index
} }
fn switch_on_constant(con_ind: HashMap<Constant, ThirdLevelIndex>, fn switch_on_constant(con_ind: HashMap<Constant, ThirdLevelIndex>, prelude: &mut CodeDeque)
prelude: &mut CodeDeque)
-> IntIndex -> IntIndex
{ {
let con_ind = Self::second_level_index(con_ind, prelude); let con_ind = Self::second_level_index(con_ind, prelude);
@@ -150,8 +148,7 @@ impl CodeOffsets {
} }
} }
fn switch_on_structure(str_ind: HashMap<(Atom, usize), ThirdLevelIndex>, fn switch_on_structure(str_ind: HashMap<(Atom, usize), ThirdLevelIndex>, prelude: &mut CodeDeque)
prelude: &mut CodeDeque)
-> IntIndex -> IntIndex
{ {
let str_ind = Self::second_level_index(str_ind, prelude); let str_ind = Self::second_level_index(str_ind, prelude);
@@ -203,6 +200,7 @@ impl CodeOffsets {
} }
} }
pub fn add_indices(self, code: &mut Code, mut code_body: Code) pub fn add_indices(self, code: &mut Code, mut code_body: Code)
{ {
if self.no_indices() { if self.no_indices() {

View File

@@ -368,12 +368,14 @@ pub fn eval<'a, 'b: 'a>(wam: &'a mut Machine, tl: &'b TopLevel) -> EvalSession<'
Err(e) => return EvalSession::ParserError(e) Err(e) => return EvalSession::ParserError(e)
}; };
print_code(&compiled_pred);
wam.add_predicate(clauses, compiled_pred) wam.add_predicate(clauses, compiled_pred)
}, },
&TopLevel::Fact(ref fact) => { &TopLevel::Fact(ref fact) => {
let mut cg = CodeGenerator::<DebrayAllocator>::new(); let mut cg = CodeGenerator::<DebrayAllocator>::new();
let compiled_fact = cg.compile_fact(fact); let compiled_fact = cg.compile_fact(fact);
print_code(&compiled_fact);
wam.add_fact(fact, compiled_fact) wam.add_fact(fact, compiled_fact)
}, },
&TopLevel::Rule(ref rule) => { &TopLevel::Rule(ref rule) => {
@@ -384,6 +386,7 @@ pub fn eval<'a, 'b: 'a>(wam: &'a mut Machine, tl: &'b TopLevel) -> EvalSession<'
Err(e) => return EvalSession::ParserError(e) Err(e) => return EvalSession::ParserError(e)
}; };
print_code(&compiled_rule);
wam.add_rule(rule, compiled_rule) wam.add_rule(rule, compiled_rule)
}, },
&TopLevel::Query(ref query) => { &TopLevel::Query(ref query) => {
@@ -394,6 +397,7 @@ pub fn eval<'a, 'b: 'a>(wam: &'a mut Machine, tl: &'b TopLevel) -> EvalSession<'
Err(e) => return EvalSession::ParserError(e) Err(e) => return EvalSession::ParserError(e)
}; };
print_code(&compiled_query);
wam.submit_query(compiled_query, cg.take_vars()) wam.submit_query(compiled_query, cg.take_vars())
} }
} }

View File

@@ -22,7 +22,6 @@ enum MachineMode {
} }
struct MachineState { struct MachineState {
h: usize,
s: usize, s: usize,
p: CodePtr, p: CodePtr,
b: usize, b: usize,
@@ -40,7 +39,7 @@ struct MachineState {
tr: usize, tr: usize,
hb: usize, hb: usize,
block: usize, // an offset into the OR stack. block: usize, // an offset into the OR stack.
ball: (usize, Heap), // heap boundary, and a term copy ball: (usize, Vec<HeapCellValue>), // heap boundary, and a term copy
interms: Vec<Number> // intermediate numbers. interms: Vec<Number> // intermediate numbers.
} }
@@ -71,16 +70,16 @@ impl<'a> IndexMut<usize> for DuplicateTerm<'a> {
// the ordinary, heap term copier, used by duplicate_term. // the ordinary, heap term copier, used by duplicate_term.
impl<'a> CopierTarget for DuplicateTerm<'a> { impl<'a> CopierTarget for DuplicateTerm<'a> {
fn source(&self) -> usize { fn source(&self) -> usize {
self.state.h self.state.heap.h
} }
fn threshold(&self) -> usize { fn threshold(&self) -> usize {
self.state.h self.state.heap.h
} }
fn push(&mut self, hcv: HeapCellValue) { fn push(&mut self, hcv: HeapCellValue) {
self.state.heap.push(hcv); self.state.heap.push(hcv);
self.state.h += 1; self.state.heap.h += 1;
} }
fn store(&self, a: Addr) -> Addr { fn store(&self, a: Addr) -> Addr {
@@ -444,7 +443,7 @@ impl Machine {
fn fail<'a>(&mut self) -> EvalSession<'a> fn fail<'a>(&mut self) -> EvalSession<'a>
{ {
if self.ms.ball.1.len() > 0 { if self.ms.ball.1.len() > 0 {
let h = self.ms.h; let h = self.ms.heap.h;
self.ms.copy_and_align_ball_to_heap(); self.ms.copy_and_align_ball_to_heap();
EvalSession::QueryFailureWithException(self.print_term(&Addr::HeapCell(h))) EvalSession::QueryFailureWithException(self.print_term(&Addr::HeapCell(h)))
@@ -620,8 +619,7 @@ macro_rules! try_or_fail {
impl MachineState { impl MachineState {
fn new() -> MachineState { fn new() -> MachineState {
MachineState { h: 0, MachineState { s: 0,
s: 0,
p: CodePtr::default(), p: CodePtr::default(),
b: 0, b: 0,
b0: 0, b0: 0,
@@ -629,7 +627,7 @@ impl MachineState {
num_of_args: 0, num_of_args: 0,
cp: CodePtr::default(), cp: CodePtr::default(),
fail: false, fail: false,
heap: Vec::with_capacity(256), heap: Heap::with_capacity(256),
mode: MachineMode::Write, mode: MachineMode::Write,
and_stack: AndStack::new(), and_stack: AndStack::new(),
or_stack: OrStack::new(), or_stack: OrStack::new(),
@@ -1155,21 +1153,19 @@ impl MachineState {
match self.store(addr.clone()) { match self.store(addr.clone()) {
Addr::HeapCell(hc) => { Addr::HeapCell(hc) => {
let h = self.h; let h = self.heap.h;
self.heap.push(HeapCellValue::Lis(h+1)); self.heap.push(HeapCellValue::Lis(h+1));
self.bind(Ref::HeapCell(hc), Addr::HeapCell(h)); self.bind(Ref::HeapCell(hc), Addr::HeapCell(h));
self.h += 1;
self.mode = MachineMode::Write; self.mode = MachineMode::Write;
}, },
Addr::StackCell(fr, sc) => { Addr::StackCell(fr, sc) => {
let h = self.h; let h = self.heap.h;
self.heap.push(HeapCellValue::Lis(h+1)); self.heap.push(HeapCellValue::Lis(h+1));
self.bind(Ref::StackCell(fr, sc), Addr::HeapCell(h)); self.bind(Ref::StackCell(fr, sc), Addr::HeapCell(h));
self.h += 1;
self.mode = MachineMode::Write; self.mode = MachineMode::Write;
}, },
Addr::Lis(a) => { Addr::Lis(a) => {
@@ -1196,14 +1192,13 @@ impl MachineState {
} }
}, },
Addr::HeapCell(_) | Addr::StackCell(_, _) => { Addr::HeapCell(_) | Addr::StackCell(_, _) => {
self.heap.push(HeapCellValue::Str(self.h + 1)); let h = self.heap.h;
self.heap.push(HeapCellValue::NamedStr(arity, name.clone()));
let h = self.h; self.heap.push(HeapCellValue::Str(h + 1));
self.heap.push(HeapCellValue::NamedStr(arity, name.clone()));
self.bind(addr.as_ref().unwrap(), Addr::HeapCell(h)); self.bind(addr.as_ref().unwrap(), Addr::HeapCell(h));
self.h += 2;
self.mode = MachineMode::Write; self.mode = MachineMode::Write;
}, },
_ => self.fail = true _ => self.fail = true
@@ -1225,7 +1220,6 @@ impl MachineState {
}, },
MachineMode::Write => { MachineMode::Write => {
self.heap.push(HeapCellValue::Con(c.clone())); self.heap.push(HeapCellValue::Con(c.clone()));
self.h += 1;
} }
}; };
@@ -1236,11 +1230,10 @@ impl MachineState {
MachineMode::Read => MachineMode::Read =>
self[reg] = self.heap[self.s].as_addr(self.s), self[reg] = self.heap[self.s].as_addr(self.s),
MachineMode::Write => { MachineMode::Write => {
let h = self.h; let h = self.heap.h;
self.heap.push(HeapCellValue::Ref(Ref::HeapCell(h))); self.heap.push(HeapCellValue::Ref(Ref::HeapCell(h)));
self[reg] = Addr::HeapCell(self.h); self[reg] = Addr::HeapCell(h);
self.h += 1;
} }
}; };
@@ -1256,14 +1249,13 @@ impl MachineState {
}, },
MachineMode::Write => { MachineMode::Write => {
let addr = self.deref(self[reg].clone()); let addr = self.deref(self[reg].clone());
let h = self.h; let h = self.heap.h;
if let Addr::HeapCell(hc) = addr { if let Addr::HeapCell(hc) = addr {
if hc < h { if hc < h {
let val = self.heap[hc].clone(); let val = self.heap[hc].clone();
self.heap.push(val);
self.h += 1; self.heap.push(val);
self.s += 1; self.s += 1;
return; return;
@@ -1272,8 +1264,6 @@ impl MachineState {
self.heap.push(HeapCellValue::Ref(Ref::HeapCell(h))); self.heap.push(HeapCellValue::Ref(Ref::HeapCell(h)));
self.bind(Ref::HeapCell(h), addr); self.bind(Ref::HeapCell(h), addr);
self.h += 1;
} }
}; };
@@ -1290,7 +1280,6 @@ impl MachineState {
MachineMode::Write => { MachineMode::Write => {
let heap_val = self.store(self[reg].clone()); let heap_val = self.store(self[reg].clone());
self.heap.push(HeapCellValue::from(heap_val)); self.heap.push(HeapCellValue::from(heap_val));
self.h += 1;
} }
}; };
@@ -1301,13 +1290,11 @@ impl MachineState {
MachineMode::Read => MachineMode::Read =>
self.s += n, self.s += n,
MachineMode::Write => { MachineMode::Write => {
let h = self.h; let h = self.heap.h;
for i in h .. h + n { for i in h .. h + n {
self.heap.push(HeapCellValue::Ref(Ref::HeapCell(i))); self.heap.push(HeapCellValue::Ref(Ref::HeapCell(i)));
} }
self.h += n;
} }
}; };
} }
@@ -1384,11 +1371,12 @@ impl MachineState {
&QueryInstruction::PutConstant(_, ref constant, reg) => &QueryInstruction::PutConstant(_, ref constant, reg) =>
self[reg] = Addr::Con(constant.clone()), self[reg] = Addr::Con(constant.clone()),
&QueryInstruction::PutList(_, reg) => &QueryInstruction::PutList(_, reg) =>
self[reg] = Addr::Lis(self.h), self[reg] = Addr::Lis(self.heap.h),
&QueryInstruction::PutStructure(_, ref name, arity, reg) => { &QueryInstruction::PutStructure(_, ref name, arity, reg) => {
let h = self.heap.h;
self.heap.push(HeapCellValue::NamedStr(arity, name.clone())); self.heap.push(HeapCellValue::NamedStr(arity, name.clone()));
self[reg] = Addr::Str(self.h); self[reg] = Addr::Str(h);
self.h += 1;
}, },
&QueryInstruction::PutUnsafeValue(n, arg) => { &QueryInstruction::PutUnsafeValue(n, arg) => {
let e = self.e; let e = self.e;
@@ -1397,13 +1385,12 @@ impl MachineState {
if addr.is_protected(e) { if addr.is_protected(e) {
self.registers[arg] = self.store(addr); self.registers[arg] = self.store(addr);
} else { } else {
let h = self.h; let h = self.heap.h;
self.heap.push(HeapCellValue::Ref(Ref::HeapCell(h))); self.heap.push(HeapCellValue::Ref(Ref::HeapCell(h)));
self.bind(Ref::HeapCell(h), addr); self.bind(Ref::HeapCell(h), addr);
self.registers[arg] = self.heap[h].as_addr(h); self.registers[arg] = self.heap[h].as_addr(h);
self.h += 1;
} }
}, },
&QueryInstruction::PutValue(norm, arg) => &QueryInstruction::PutValue(norm, arg) =>
@@ -1417,58 +1404,46 @@ impl MachineState {
self.registers[arg] = self[norm].clone(); self.registers[arg] = self[norm].clone();
}, },
RegType::Temp(_) => { RegType::Temp(_) => {
let h = self.h; let h = self.heap.h;
self.heap.push(HeapCellValue::Ref(Ref::HeapCell(h))); self.heap.push(HeapCellValue::Ref(Ref::HeapCell(h)));
self[norm] = Addr::HeapCell(h); self[norm] = Addr::HeapCell(h);
self.registers[arg] = Addr::HeapCell(h); self.registers[arg] = Addr::HeapCell(h);
self.h += 1;
} }
}; };
}, },
&QueryInstruction::SetConstant(ref constant) => { &QueryInstruction::SetConstant(ref constant) => {
self.heap.push(HeapCellValue::Con(constant.clone())); self.heap.push(HeapCellValue::Con(constant.clone()));
self.h += 1;
}, },
&QueryInstruction::SetLocalValue(reg) => { &QueryInstruction::SetLocalValue(reg) => {
let addr = self.deref(self[reg].clone()); let addr = self.deref(self[reg].clone());
let h = self.h; let h = self.heap.h;
if let Addr::HeapCell(hc) = addr { if let Addr::HeapCell(hc) = addr {
if hc < h { if hc < h {
self.heap.push(HeapCellValue::from(addr)); self.heap.push(HeapCellValue::from(addr));
self.h += 1;
return; return;
} }
} }
self.heap.push(HeapCellValue::Ref(Ref::HeapCell(h))); self.heap.push(HeapCellValue::Ref(Ref::HeapCell(h)));
self.bind(Ref::HeapCell(h), addr); self.bind(Ref::HeapCell(h), addr);
self.h += 1;
}, },
&QueryInstruction::SetVariable(reg) => { &QueryInstruction::SetVariable(reg) => {
let h = self.h; let h = self.heap.h;
self.heap.push(HeapCellValue::Ref(Ref::HeapCell(h))); self.heap.push(HeapCellValue::Ref(Ref::HeapCell(h)));
self[reg] = Addr::HeapCell(h); self[reg] = Addr::HeapCell(h);
self.h += 1;
}, },
&QueryInstruction::SetValue(reg) => { &QueryInstruction::SetValue(reg) => {
let heap_val = self[reg].clone(); let heap_val = self[reg].clone();
self.heap.push(HeapCellValue::from(heap_val)); self.heap.push(HeapCellValue::from(heap_val));
self.h += 1;
}, },
&QueryInstruction::SetVoid(n) => { &QueryInstruction::SetVoid(n) => {
let h = self.h; let h = self.heap.h;
for i in h .. h + n { for i in h .. h + n {
self.heap.push(HeapCellValue::Ref(Ref::HeapCell(i))); self.heap.push(HeapCellValue::Ref(Ref::HeapCell(i)));
} }
self.h += n;
} }
} }
} }
@@ -1528,13 +1503,12 @@ impl MachineState {
self.p = CodePtr::DirEntry(59); self.p = CodePtr::DirEntry(59);
} }
fn throw_exception(&mut self, mut hcv: Vec<HeapCellValue>) { fn throw_exception(&mut self, hcv: Vec<HeapCellValue>) {
let h = self.h; let h = self.heap.h;
self.registers[1] = Addr::HeapCell(h); self.registers[1] = Addr::HeapCell(h);
self.h += hcv.len();
self.heap.append(&mut hcv); self.heap.append(hcv);
self.goto_throw(); self.goto_throw();
} }
@@ -1586,7 +1560,7 @@ impl MachineState {
} }
fn copy_and_align_ball_to_heap(&mut self) { fn copy_and_align_ball_to_heap(&mut self) {
let diff = self.ball.0 - self.h; let diff = self.ball.0 - self.heap.h;
for heap_value in self.ball.1.iter().cloned() { for heap_value in self.ball.1.iter().cloned() {
self.heap.push(match heap_value { self.heap.push(match heap_value {
@@ -1598,8 +1572,6 @@ impl MachineState {
_ => heap_value _ => heap_value
}); });
} }
self.h += self.ball.1.len();
} }
fn execute_built_in_instr(&mut self, code_dir: &CodeDir, instr: &BuiltInInstruction) fn execute_built_in_instr(&mut self, code_dir: &CodeDir, instr: &BuiltInInstruction)
@@ -1622,7 +1594,7 @@ impl MachineState {
self.p += 1; self.p += 1;
}, },
&BuiltInInstruction::DuplicateTerm => { &BuiltInInstruction::DuplicateTerm => {
let old_h = self.h; let old_h = self.heap.h;
let a1 = self[temp_v!(1)].clone(); let a1 = self[temp_v!(1)].clone();
let a2 = self[temp_v!(2)].clone(); let a2 = self[temp_v!(2)].clone();
@@ -1652,7 +1624,7 @@ impl MachineState {
}, },
&BuiltInInstruction::GetBall => { &BuiltInInstruction::GetBall => {
let addr = self.store(self.deref(self[temp_v!(1)].clone())); let addr = self.store(self.deref(self[temp_v!(1)].clone()));
let h = self.h; let h = self.heap.h;
if self.ball.1.len() > 0 { if self.ball.1.len() > 0 {
self.copy_and_align_ball_to_heap(); self.copy_and_align_ball_to_heap();
@@ -1673,7 +1645,7 @@ impl MachineState {
}, },
&BuiltInInstruction::SetBall => { &BuiltInInstruction::SetBall => {
let addr = self[temp_v!(1)].clone(); let addr = self[temp_v!(1)].clone();
self.ball.0 = self.h; self.ball.0 = self.heap.h;
{ {
let mut duplicator = DuplicateBallTerm::new(self); let mut duplicator = DuplicateBallTerm::new(self);
@@ -1865,7 +1837,7 @@ impl MachineState {
self.b, self.b,
self.p + 1, self.p + 1,
self.tr, self.tr,
self.h, self.heap.h,
self.b0, self.b0,
self.num_of_args); self.num_of_args);
@@ -1876,7 +1848,7 @@ impl MachineState {
self.or_stack[b][i] = self.registers[i].clone(); self.or_stack[b][i] = self.registers[i].clone();
} }
self.hb = self.h; self.hb = self.heap.h;
self.p += l; self.p += l;
}, },
&IndexedChoiceInstruction::Retry(l) => { &IndexedChoiceInstruction::Retry(l) => {
@@ -1901,8 +1873,7 @@ impl MachineState {
self.trail.truncate(self.tr); self.trail.truncate(self.tr);
self.heap.truncate(self.or_stack[b].h); self.heap.truncate(self.or_stack[b].h);
self.h = self.or_stack[b].h; self.hb = self.heap.h;
self.hb = self.h;
self.p += l; self.p += l;
}, },
@@ -1925,14 +1896,12 @@ impl MachineState {
self.tr = self.or_stack[b].tr; self.tr = self.or_stack[b].tr;
self.trail.truncate(self.tr); self.trail.truncate(self.tr);
self.h = self.or_stack[b].h; self.heap.truncate(self.or_stack[b].h);
self.heap.truncate(self.h);
self.b = self.or_stack[b].b; self.b = self.or_stack[b].b;
self.or_stack.pop(); self.or_stack.pop();
self.hb = self.h; self.hb = self.heap.h;
self.p += l; self.p += l;
}, },
}; };
@@ -1951,7 +1920,7 @@ impl MachineState {
self.b, self.b,
self.p + offset, self.p + offset,
self.tr, self.tr,
self.h, self.heap.h,
self.b0, self.b0,
self.num_of_args); self.num_of_args);
@@ -1962,7 +1931,7 @@ impl MachineState {
self.or_stack[b][i] = self.registers[i].clone(); self.or_stack[b][i] = self.registers[i].clone();
} }
self.hb = self.h; self.hb = self.heap.h;
self.p += 1; self.p += 1;
}, },
&ChoiceInstruction::RetryMeElse(offset) => { &ChoiceInstruction::RetryMeElse(offset) => {
@@ -1987,8 +1956,7 @@ impl MachineState {
self.trail.truncate(self.tr); self.trail.truncate(self.tr);
self.heap.truncate(self.or_stack[b].h); self.heap.truncate(self.or_stack[b].h);
self.h = self.or_stack[b].h; self.hb = self.heap.h;
self.hb = self.h;
self.p += 1; self.p += 1;
}, },
@@ -2011,14 +1979,13 @@ impl MachineState {
self.tr = self.or_stack[b].tr; self.tr = self.or_stack[b].tr;
self.trail.truncate(self.tr); self.trail.truncate(self.tr);
self.h = self.or_stack[b].h; self.heap.truncate(self.or_stack[b].h);
self.heap.truncate(self.h);
self.b = self.or_stack[b].b; self.b = self.or_stack[b].b;
self.or_stack.pop(); self.or_stack.pop();
self.hb = self.h; self.hb = self.heap.h;
self.p += 1; self.p += 1;
} }
} }
@@ -2068,7 +2035,6 @@ impl MachineState {
} }
fn reset(&mut self) { fn reset(&mut self) {
self.h = 0;
self.hb = 0; self.hb = 0;
self.e = 0; self.e = 0;
self.b = 0; self.b = 0;