Skip to content

Commit 6142139

Browse files
authored
Merge pull request #161 from dop-amin/re-enable-fixup
CM7: Re-enable address offset fixup for Kyber NTT
2 parents f78627a + 6c212c2 commit 6142139

File tree

2 files changed

+1139
-1032
lines changed

2 files changed

+1139
-1032
lines changed

example.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2060,7 +2060,7 @@ def core(self, slothy):
20602060

20612061
slothy.config.inputs_are_outputs = True
20622062
slothy.config.variable_size = True
2063-
slothy.config.unsafe_address_offset_fixup = False
2063+
slothy.config.unsafe_address_offset_fixup = True
20642064

20652065
### TODO
20662066
# - Experiment with lower split factors
@@ -2076,15 +2076,15 @@ def core(self, slothy):
20762076
slothy.optimize_loop("1", forced_loop_type=Arch_Armv7M.BranchLoop)
20772077

20782078
slothy.config.outputs = ["r14"]
2079-
2079+
slothy.config.unsafe_address_offset_fixup = False
20802080
slothy.fusion_loop("2", ssa=False, forced_loop_type=Arch_Armv7M.BranchLoop)
2081+
slothy.config.unsafe_address_offset_fixup = True
20812082

20822083
slothy.config.timeout = 360
20832084
slothy.config.split_heuristic_optimize_seam = 0
20842085
slothy.config.split_heuristic_repeat = 1
20852086
slothy.config.split_heuristic_factor = 4
20862087
slothy.config.split_heuristic_stepsize = 0.1
2087-
slothy.config.unsafe_address_offset_fixup = True
20882088
slothy.optimize_loop("2", forced_loop_type=Arch_Armv7M.BranchLoop)
20892089

20902090
slothy.config.split_heuristic_optimize_seam = 6

0 commit comments

Comments
 (0)