stillwater-sc / universal / 24968310082

84%

master: 84%

Build Type

push

github

Committed by web-flow

Commit Message

fix(tools): code hygiene for MSVC (bigobj, narrowing, ASCII docs, posit if-constexpr) (#778)

* Code hygiene for MSVC

* updating README with more recent code

* adding missing headers

* markdown fix

* fix(blas): ASCII-only docs/comments + drop unused ranges include (CodeRabbit)

Three CodeRabbit findings on PR #778:

1. (Major) docs/design/constexpr.md violated the repository ASCII-only
   rule with em-dashes, a right-arrow, and one Latin-1 letter. Bulk
   sed: -- replaces em dash, -> replaces right arrow, e replaces e-acute
   (in "Pade").

2. (Minor) posit_impl.hpp:1456 had a Unicode -> in a comment from the
   recent if-constexpr cleanup. Replaced with ASCII ->.

3. (Critical, already addressed in commit 20292e91) conversions.hpp
   was missing <algorithm>, <iterator>, <vector> includes. The earlier
   commit added them all; this one drops the now-unused <ranges>
   include since the implementation uses std::transform, not the
   ranges adaptor.

Note: the Critical's earlier diff is already on this branch, so
CodeRabbit's snapshot was stale.

Verified ucalc and energy_instrumented_mixed_precision_blas_demo build
clean on gcc with these changes.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Coverage Stats

6 of 7 new or added lines in 2 files covered. (85.71%)

27 existing lines in 4 files now uncovered.

45470 of 53950 relevant lines covered (84.28%)

6361492.67 hits per line