Discourage LLMs from touching fdctl#10014
Conversation
Greptile SummaryDocumentation-only PR that steers contributors (and LLMs) away from the legacy Frankendancer surface (
No source code, build rules, or runtime behavior are touched. One minor latent inconsistency worth flagging for a future docs pass (not introduced by this PR): Confidence Score: 5/5
|
| Filename | Overview |
|---|---|
| CLAUDE.md | Tightens guidance for LLMs: removes the carve-out that pointed contributors at Firedancer equivalents (firedancer-dev, discof) and now simply tells them to avoid Frankendancer-specific parts (fdctl, fddev, discoh). No code impact. |
| CONTRIBUTING.md | Updates compiler compatibility section: bumps minimum GCC from 8.5 (2024-Jul) to GCC 11 (June 2026), renames the primary build target from "Frankendancer (fdctl)" to "firedancer", and adds "atomic 32 byte load/store" to the Haswell feature set. Drops the list of experimental targets (musl, macOS, FreeBSD, SVM, arm64, ppc64le, sBPFv1/v2). Docs-only. |
| contrib/containers/README.md | Replaces make -j all fdctl fddev with make -j in the container build instructions, aligning with the move away from explicit Frankendancer targets. |
| doc/build-system.md | Updates the example ldd output to reference build/native/gcc/bin/firedancer instead of bin/fdctl, with a corresponding refreshed library list (adds libmvec.so.1, drops librt/libdl/libpthread). Documentation refresh only. |
Last reviewed commit: 7672c1b
Performance Measurements ⏳
|
There was a problem hiding this comment.
Pull request overview
This PR updates repository documentation to steer tooling/LLM guidance away from Frankendancer (fdctl/fddev) and toward Firedancer (firedancer), and refreshes build instructions/examples accordingly.
Changes:
- Replace
fdctlreferences in build-system docs withfiredancer(e.g.,lddexample output). - Update contribution/build compatibility text and supported target wording.
- Simplify container build instructions to use the default
make -jtarget set.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
doc/build-system.md |
Switches the dynamic-linking example from fdctl to firedancer. |
CONTRIBUTING.md |
Updates compiler compatibility and target/CPU feature-set documentation for the firedancer target. |
contrib/containers/README.md |
Simplifies build command to make -j in container instructions. |
CLAUDE.md |
Tightens guidance to avoid Frankendancer-specific components. |
| As of June 2026, Firedancer builds on GNU/Linux sysroots with GCC 11 or | ||
| newer. Clang and CBMC are also supported build environments. |
| The "firedancer" build target only targets x86_64 with a Haswell like | ||
| minimum feature set (AVX2, FMA, atomic 32 byte load/store). |
| Compile desired targets: | ||
|
|
||
| ```bash | ||
| make -j all fdctl fddev | ||
| make -j |
No description provided.