Skip to content

Add fasta benchmark - #457

Merged
ksh8281 merged 1 commit into
Samsung:mainfrom
hyraxbyerax:feat/fasta-benchmark
Jul 27, 2026
Merged

Add fasta benchmark#457
ksh8281 merged 1 commit into
Samsung:mainfrom
hyraxbyerax:feat/fasta-benchmark

Conversation

@hyraxbyerax

Copy link
Copy Markdown
Contributor

Summary

Add the fasta benchmark and register it in the benchmark suite.

Testing

  • Benchmark passed
  • WASI tests passed

Comment thread test/wasmBenchmarker/ctests/fasta.c Outdated
#define IC 29573U
#define SEED 42U

#define INPUT_SIZE 1000U

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please increase INPUT_SIZE to 250,000 (current value is quite small)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I will apply this change, run some tests, and update the code soon!

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've made the requested changes based on your feedback. Thank you for your review!

@clover2123

Copy link
Copy Markdown
Collaborator

Add a license description like below
(starting with Copyright (c) 2026-present Samsung Electronics Co)

/*
* Copyright (c) 2023-present Samsung Electronics Co., Ltd
*
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
* You may obtain a copy of the License at
*
* http://www.apache.org/licenses/LICENSE-2.0
*
* Unless required by applicable law or agreed to in writing, software

@hyraxbyerax
hyraxbyerax force-pushed the feat/fasta-benchmark branch from 7ce5a30 to df99cc7 Compare July 27, 2026 09:29
Comment thread test/wasmBenchmarker/ctests/fasta.c Outdated
@@ -0,0 +1,251 @@
/*
* Copyright (c) 2023-present Samsung Electronics Co., Ltd

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

2026, not 2023

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The requested changes have been made. Thank you.

@hyraxbyerax
hyraxbyerax force-pushed the feat/fasta-benchmark branch 2 times, most recently from 0e288fb to 089fcc1 Compare July 27, 2026 10:11
@hyraxbyerax
hyraxbyerax force-pushed the feat/fasta-benchmark branch from 089fcc1 to dfd7708 Compare July 27, 2026 10:49
@ksh8281
ksh8281 merged commit f704bda into Samsung:main Jul 27, 2026
89 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants