diff options
author | James Pollard <james@pollard.dev> | 2020-06-12 16:49:59 +0100 |
---|---|---|
committer | James Pollard <james@pollard.dev> | 2020-06-12 16:49:59 +0100 |
commit | 8b96b18187422457d5baeb637923d9ce2e7199be (patch) | |
tree | 85300b441b3f780abfe70ea3457d24bd69f29b39 /src/translation | |
parent | 86f42b92d87020875e2a7ef4ba40de12d261685f (diff) | |
download | vericert-8b96b18187422457d5baeb637923d9ce2e7199be.tar.gz vericert-8b96b18187422457d5baeb637923d9ce2e7199be.zip |
Fix addressing bug.
Diffstat (limited to 'src/translation')
-rw-r--r-- | src/translation/HTLgen.v | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/src/translation/HTLgen.v b/src/translation/HTLgen.v index a891d6c..96ac8df 100644 --- a/src/translation/HTLgen.v +++ b/src/translation/HTLgen.v @@ -256,8 +256,7 @@ Definition translate_eff_addressing (a: Op.addressing) (args: list reg) : mon ex (* Stack arrays/referenced variables *) | Op.Ainstack a, nil => (* We need to be sure that the base address is aligned *) let a := Integers.Ptrofs.unsigned a in (* FIXME: Assuming stack offsets are +ve; is this ok? *) - if (Z.eq_dec (Z.modulo a 4) 0) then ret (Vlit (ZToValue 32 (a / 4))) - else error (Errors.msg "Veriloggen: eff_addressing misaligned stack offset") + ret (Vlit (ZToValue 32 a)) | _, _ => error (Errors.msg "Veriloggen: eff_addressing instruction not implemented: other") end. @@ -336,14 +335,17 @@ Definition add_branch_instr (e: expr) (n n1 n2: node) : mon unit := Definition translate_arr_access (mem : AST.memory_chunk) (addr : Op.addressing) (args : list reg) (stack : reg) : mon expr := match addr, args with (* TODO: We should be more methodical here; what are the possibilities?*) - | Op.Aindexed off, r1::nil => ret (Vvari stack (boplitz Vadd r1 off)) (* FIXME: Cannot guarantee alignment *) + | Op.Aindexed off, r1::nil => (* FIXME: Cannot guarantee alignment *) + ret (Vvari stack (Vbinop Vadd (boplitz Vdiv r1 4) (Vlit (ZToValue 32 (off / 4))))) | Op.Ascaled scale offset, r1::nil => if ((Z.eqb (Z.modulo scale 4) 0) && (Z.eqb (Z.modulo offset 4) 0)) then ret (Vvari stack (Vbinop Vadd (boplitz Vmul r1 (scale / 4)) (Vlit (ZToValue 32 (offset / 4))))) else error (Errors.msg "Veriloggen: translate_arr_access address misaligned") | Op.Aindexed2scaled scale offset, r1::r2::nil => (* Typical for dynamic array addressing *) if ((Z.eqb (Z.modulo scale 4) 0) && (Z.eqb (Z.modulo offset 4) 0)) - then ret (Vvari stack (Vbinop Vadd (boplitz Vadd r1 (offset / 4)) (boplitz Vmul r2 (scale / 4)))) + then ret (Vvari stack + (Vbinop Vadd (Vbinop Vadd (boplitz Vdiv r1 4) (Vlit (ZToValue 32 (offset / 4)))) + (boplitz Vmul r2 (scale / 4)))) else error (Errors.msg "Veriloggen: translate_arr_access address misaligned") | Op.Ainstack a, nil => (* We need to be sure that the base address is aligned *) let a := Integers.Ptrofs.unsigned a in (* FIXME: Assuming stack offsets are +ve; is this ok? *) |