benchmark: replace static memory allocation with dynamic allocation in math/base/special#11056
benchmark: replace static memory allocation with dynamic allocation in math/base/special#11056sanjanadubariya2 wants to merge 3 commits intostdlib-js:developfrom
Conversation
…base/special Modified C benchmarks to use heap allocation (malloc/free) instead of stack-allocated arrays. Relates to stdlib-js#8643.
|
Hello! Thank you for your contribution to stdlib. We noticed that the contributing guidelines acknowledgment is missing from your pull request. Here's what you need to do:
This acknowledgment confirms that you've read the guidelines, which include:
We can't review or accept contributions without this acknowledgment. Thank you for your understanding and cooperation. We look forward to reviewing your contribution! |
|
👋 Hi there! 👋 And thank you for opening your first pull request! We will review it shortly. 🏃 💨 Getting Started
Next Steps
Running Tests LocallyYou can use # Run tests for all packages in the math namespace:
make test TESTS_FILTER=".*/@stdlib/math/.*"
# Run benchmarks for a specific package:
make benchmark BENCHMARKS_FILTER=".*/@stdlib/math/base/special/sin/.*"If you haven't heard back from us within two weeks, please ping us by tagging the "reviewers" team in a comment on this PR. If you have any further questions while waiting for a response, please join our Zulip community to chat with project maintainers and other community members. We appreciate your contribution! Documentation Links |
|
I have read the contributing guidelines. |
You need to have the actual markdown referenced by stdlib_bot in your PR description. That’s why we use the PR template. It also includes a section on AI use disclosure. If you open another PR you’ll see the template. |
Description
This PR addresses the tracking issue #8643. I have replaced static array allocations with dynamic memory allocation (
mallocandfree) in the C benchmarks for themath/base/specialdirectory.Changes
double x[100]) withmalloc.free()before function exit.#include <stdlib.h>to all modified files to support memory management functions.Verification
gccin several subdirectories (e.g.,abs).Closes #8643