Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Improve Directory Structure of Project #14

Merged
merged 11 commits into from
Dec 13, 2023
Merged

Improve Directory Structure of Project #14

merged 11 commits into from
Dec 13, 2023

Conversation

itzmeanjan
Copy link
Owner

  • Put tests and benchmarks into their own directory, not inside include
  • Refactor tests to use google-test framework for managing and running them
  • Improve the state of the Makefile - much easier to maintain now
  • Add macos target for running the tests on Github Actions CI

@itzmeanjan
move tests, benchmarks and examples to their own directory
45aff08 
Signed-off-by: Anjan Roy <[email protected]>
@itzmeanjan
add .clang-format code style specification
a252f91 
Signed-off-by: Anjan Roy <[email protected]>
@itzmeanjan
update Makefile to be easier to maintain
886dc06 
Took inspiration from https://github.com/itzmeanjan/dilithium/blob/master/Makefile

Signed-off-by: Anjan Roy <[email protected]>
@itzmeanjan
refactor how we benchmark sphincs+ variants
d8ca00a 
Signed-off-by: Anjan Roy <[email protected]>
@itzmeanjan
reformat source tree with new .clang-format style
f2d9bdc 
Just updated "ColumnLimit" to higher value so lines don't get wrapped around

Signed-off-by: Anjan Roy <[email protected]>
@itzmeanjan
reorder how and where __rdtsc instruction is issued
94b6a3f 
Signed-off-by: Anjan Roy <[email protected]>
@itzmeanjan
transition to using google-test for running tests
afec09c 
Signed-off-by: Anjan Roy <[email protected]>
@itzmeanjan
get rid of unnecessary const in template parameters
9650447 
Signed-off-by: Anjan Roy <[email protected]>
@itzmeanjan
update github actions CI script
70bc596 
Signed-off-by: Anjan Roy <[email protected]>
@itzmeanjan
from inline static to static inline
3d731f6 
Signed-off-by: Anjan Roy <[email protected]>
@itzmeanjan
port Known Answer Tests into C++, no more Python wrapper API
6c9e4c1 
Signed-off-by: Anjan Roy <[email protected]>
@itzmeanjan itzmeanjan merged commit 3dcb256 into master Dec 13, 2023
2 checks passed
@itzmeanjan itzmeanjan deleted the change-dir-structure branch December 13, 2023 17:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@itzmeanjan