Uh oh!
There was an error while loading. Please reload this page.
This repository was archived by the owner on Aug 20, 2026. It is now read-only.
chore: add teeny as a submodule (external/teeny) - #11
Merged
balbasty merged 1 commit intoJul 25, 2026
Merged
Conversation
Registers balbasty/teeny (header-only C++17 tensor lib on cuda::std::mdspan) as a submodule at external/teeny, pinned to the teeny integration branch (claude/fastfields-teeny-refactor-js42id). teeny is the substrate the kernels are being ported onto; the impl/lib layers reach it via -I into external/teeny/include (+ its vendored CCCL at external/teeny/external/cccl/libcudacxx/include). No kernel source consumes teeny yet — this only registers the dependency. Mirrors the repo's "gitlink in the index, symlink in the dev tree" pattern. Part of fastfields-lib#21. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_015xcQBMEdA5eQtTsx2wB3AC
balbasty added a commit
that referenced
this pull request
Aug 9, 2026
…74) `MeshDist::build_tree` sorts the faces of a BVH node with std::sort(FaceIterator(faces.data, ...) + begin, FaceIterator(faces.data, ...) + end, cmp); `FaceIterator`'s constructor eagerly builds a `Face`, whose pointer constructor loaded `copy` from `face` -- i.e. it dereferenced the iterator. For the past-the-end iterator (`end == nb_faces` at the top level) that reads D `index_t` values one face beyond the faces buffer. `FaceIterator`'s copy constructor rebuilt the `Face` from the raw pointer too, so every copy std::sort made of the end iterator repeated the over-read. This is reachable from the public API: `dt_mesh(..., signed=true, naive=false)` -> `distance_mesh::sdt` -> `build_tree`. AddressSanitizer aborts the existing cpu-lib mesh test on it: ==25759==ERROR: AddressSanitizer: heap-buffer-overflow READ of size 4 at 0x508000000200 #0 PointMixin<3,int,StaticPoint<3,int>>::copy_(...) mesh_utils.h:124 #1 StaticPoint<3,int>::StaticPoint(AnyConstPoint<3,int> const&) #2 MeshDist<3,float,int,int>::Face::Face(int*, int) mesh.h:673 #3 MeshDist<3,float,int,int>::FaceIterator::FaceIterator(int*,int,int) #4 operator+(FaceIterator const&, int) mesh.h:743 #5 MeshDist<3,float,int,int>::build_tree<...> mesh.h:833 ... #11 ff::cpu::dt_mesh(...) distance.cpp:732 0x508000000200 is located 0 bytes after 96-byte region allocated by ff::cpu::distance_mesh::copy_faces<3,int,int> Fix, without touching the sort machinery's semantics for valid iterators: * `Face(index_t*, offset_t)` no longer loads `copy`. Every dereference of a valid iterator goes through `operator*` -> `load_()`, which still loads, so sorted output is unchanged; only the never- dereferenced end iterator stops reading. * `Face(const Face&)` snapshots `other.copy` instead of re-reading through the shared data pointer. For a loaded `other` these are the same values (that is how std::sort's `__val` temporaries already worked); for the end iterator it avoids re-triggering the read. * `FaceIterator(const FaceIterator&)` delegates to `Face`'s copy constructor rather than rebuilding from the raw pointer. * `copy` is value-initialised so an unloaded `Face` never hands out indeterminate indices when copied. * Drive-by: `Face::load()` built `clone`, loaded it, then returned `*this` -- the const `operator*` therefore returned an unloaded Face. Return `clone`. Verification: * `make -C fastfields-cpu-lib test CXX=clang++` -- all 12 suites PASS, with check counts byte-identical to the pre-fix baseline (mesh: 4622 checks / 0 failures), so this is behaviour-preserving. * ASan+UBSan build of `tests/test_distance_mesh.cpp` + `distance.cpp` (g++ -fsanitize=address,undefined): before, it aborted on the heap-buffer-overflow; after, zero heap-buffer-overflow reports and the suite runs to completion (4622 checks, 0 failures). * `nvcc -x cu` compile of fastfields-cuda-lib/distance.cpp is clean -- the touched block is inside `#ifndef __CUDACC__`, so the device build is unaffected. Claude-Session: https://claude.ai/code/session_016AjQcY78NgbagPSbPJRr6Z Co-authored-by: Claude <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for freeto subscribe to this conversation on GitHub.
Already have an account?
Sign in.
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Registers
balbasty/teeny(header-only C++17 tensor lib oncuda::std::mdspan)as a submodule at
external/teeny, pinned to the teeny integration branch. Theimpl/lib layers reach it via
-I external/teeny/include(+ vendored CCCL). Nokernel source consumes teeny yet — this only registers the dependency. Mirrors
the repo's "gitlink in the index, symlink in the dev tree" pattern.
Base is the teeny integration branch. Closesfastfields/fastfields-lib#114. Part of fastfields-lib#21.
Generated by Claude Code