diff options
| author | Sebastiano Tronto <sebastiano@tronto.net> | 2023-10-31 19:01:46 +0100 |
|---|---|---|
| committer | Sebastiano Tronto <sebastiano@tronto.net> | 2023-10-31 19:03:41 +0100 |
| commit | bc2e6b8a8d5a6eccd2c2ac433e8a307ee356f14e (patch) | |
| tree | 84b576621069d13c9acfbaa6c8a98d11f16f6914 /README.md | |
| parent | 80ec600701aae54350679bbf5e0ce874c915506e (diff) | |
| download | nissy-core-bc2e6b8a8d5a6eccd2c2ac433e8a307ee356f14e.tar.gz nissy-core-bc2e6b8a8d5a6eccd2c2ac433e8a307ee356f14e.zip | |
Small change to benchmark script + TODO comments
Diffstat (limited to 'README.md')
| -rw-r--r-- | README.md | 26 |
1 files changed, 26 insertions, 0 deletions
| @@ -16,6 +16,32 @@ $ make test | |||
| 16 | * inverse_move | 16 | * inverse_move |
| 17 | * inverse_trans | 17 | * inverse_trans |
| 18 | 18 | ||
| 19 | ### Simple additions | ||
| 20 | |||
| 21 | * Write to AVX2-src format | ||
| 22 | * move() that takes a string (alg) as input | ||
| 23 | |||
| 24 | ### Changes | ||
| 25 | |||
| 26 | * write one function (static inline or public?) for each move (and trans) | ||
| 27 | * only call the specific functions in performance-critical steps (i.e. solve | ||
| 28 | - if the functions are static inline, all performance-critical steps must | ||
| 29 | be internal to cube.c) | ||
| 30 | * this also enables skipping some unnecessary work (e.g. flip edges, twist | ||
| 31 | corners) for many moves (and mirror transformations) | ||
| 32 | * add benchmarks with moves / trans called directly instead of via the | ||
| 33 | generic function | ||
| 34 | * keep generic move and trans functions with big switch case | ||
| 35 | * bring back constants into cube.c, and maybe also moves (TBD: what to | ||
| 36 | do with architecture-specific code? leave in separate files like now, | ||
| 37 | use just one file for each architecture...) | ||
| 38 | |||
| 39 | ### Documentation and interface | ||
| 40 | |||
| 41 | * remove the constant #define's from cube.h (write a comment instead) | ||
| 42 | * reconsider content of cube.h, remove some stuff | ||
| 43 | * remove doc folder, inline documentation as comments in cube.h or cube.c | ||
| 44 | |||
| 19 | ### AVX2 | 45 | ### AVX2 |
| 20 | 46 | ||
| 21 | * fix base get_ and set_ macros (constant arguments?) | 47 | * fix base get_ and set_ macros (constant arguments?) |
