feat: add C implementation for math/base/special/fast/uint32-log2

[performant23]

This commit if applied adds the native C implementation to @stdlib/math/base/special/fast/uint32-log2

Description

What is the purpose of this pull request?
Add the native C implementation to @stdlib/math/base/special/fast/uint32-log2 package

Related Issues

Does this pull request have any related issues?

This pull request:

• resolves [RFC]: add C implementation for @stdlib/math/base/special/fast/uint32-log2 #1931

Questions

Any questions for reviewers of this pull request?

No.

Other

Any other information relevant to this pull request? This may include screenshots, references, and/or implementation notes.

No.

Checklist

Please ensure the following tasks are completed before submitting this pull request.

• Read, understood, and followed the contributing guidelines.

---

@stdlib-js/reviewers

[performant23]

Would we need to disable the cppcheck equivalent of no-unused-vars rule here? Cause we'll need to modify the value of x at intermediate stages but we never have to formally return it. If so, where can I find the instruction to disable the rule?

Edit: cc @kgryte and @Planeshifter. Hello, I'd be really grateful to receive your feedback for the query above and also, that will speed up the process since by the time the PR gets reviewed, I can keep the required changes ready!

[Pranavchiku]

Tests fail, please check if something is off in your implementation. Thanks for your work!

  total:     5034
  passing:   2
  failing:   5032
  duration:  482ms

[performant23]

Hi @Pranavchiku, I was mistakenly typecasting the return value - fixed.

Edit: So about the linting error, at the last conditional, we're assigning xc but as soon as the conditional ends, we're returning out and so, since the function doesn't involve loops/recursion, the modified xc remains dormant (also it's a temporary variable inside a function and not an pointer). So, the compiler considers that assignment unnecessary and removing that fixes the error. Request you to take a look!

[kgryte]

Thanks for working on this, @performant23. I left a few comments and suggestions for improvement.

[kgryte]

@performant23 This PR is almost there. We can simplify the addon.c file just a touch more by using another macro.

[kgryte]

LGTM. Thanks, @performant23!