Skip to content

Code cleanup: namespace qualification and header tidy - #47

Open
shsahiti wants to merge 5 commits into
mainfrom
code-cleanup
Open

shsahiti wants to merge 5 commits into
mainfrom
code-cleanup

Conversation

@shsahiti

@shsahiti shsahiti commented Oct 17, 2026 •

Copy link
Copy Markdown
Contributor

Split out of #9 (cleanup half), rebased onto current main.

  • Drops redundant xpu:: prefixes across linear_algebra.hpp, memory.hpp, random.hpp, soa.hpp, keeping the required ones
  • Tidies sincos in math.hpp
  • memory.hpp: drops redundant inline on templates, routes byte counts through bytes<T>(), adds single-element make_unique<T>(value), early-returns in assume_aligned for unpadded types
  • Rebase fixups: uses main's XPU_CUDA_CALLABLE / XPU_RESTRICT / XPU_DEVICE_ONLY and xstd::assume_aligned; restores the %zu allocation-failure message (the draft had a {:d} format in fprintf plus an #include <print> that nvcc can't compile)

Tested: CPU build, 11/11 ctest pass (g++-15). CUDA 13.3 build compiles.

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.

1 participant