Conversation
…jit (RFC sgl-project#29630) Relocates the shared JIT build/runtime infrastructure (compile pipeline, arch detection, deps, common utils) from sglang.jit_kernel.utils to an internal sglang.kernels._jit package. Behavior unchanged: - KERNEL_PATH already resolves via find_spec('sglang.jit_kernel'), so csrc/ and include/ (the per-op C++ sources + shared headers) and every JIT operator stay in sglang.jit_kernel; the build root is unaffected. - ~90 'sglang.jit_kernel.utils' import sites rewritten to 'sglang.kernels._jit'. - __main__ build CLI stays at sglang.jit_kernel (invocation unchanged). Operators are not touched here; they migrate group-by-group in Phase 4. Namespace import stays metadata-only (invariant CLEAN); 68 kernels tests pass; lint clean. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
BBuf
requested review from
Edwardf0t1,
Fridge003,
HaiShaw,
Qiaolin-Yu,
Ying1123,
ch-wan,
hebiao064,
hnyls2002,
ispobock,
kpham-sgl,
merrymercy,
mickqian and
yingluosanqian
as code owners
July 19, 2026 03:45
Contributor
|
Warning You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again! |
BBuf
requested review from
DarkSharpness,
HydraQYH,
celve and
yuan-luo
as code owners
July 19, 2026 03:45
github-actions
Bot
added
quant
…rnorm (RFC sgl-project#29630) Moves the JIT rmsnorm/qknorm operator wrappers from sglang.jit_kernel.norm to sglang.kernels.ops.layernorm._jit_norm and repoints the group's forward_jit to the local module. A compat shim at sglang.jit_kernel.norm re-exports everything (RFC Phase 4 step 8) so existing in-tree/external import paths keep working; removed in Phase 5. csrc stays in sglang.jit_kernel (KERNEL_PATH unchanged). Stacked on sgl-project#31666. Invariant CLEAN; 68 kernels tests pass; lint clean. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
BBuf
force-pushed
the
bbuf/kernels-phase4-layernorm
branch
from
July 21, 2026 13:07
f53ad85 to
488d7e2
Compare
BBuf
added
run-ci-extra
bypass-maintenance
bypass-fastfail
enable-retry
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Collaborator
Author
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 free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
RFC #29630 Phase 4 for the layernorm group (stacked on #31666).
Moves the JIT rmsnorm/qknorm wrappers
sglang.jit_kernel.norm→sglang.kernels.ops.layernorm._jit_norm; the group'sforward_jitnow imports the local module. A compat shim atsglang.jit_kernel.normre-exports everything (public + private) so in-tree/external callers keep working (RFC Phase 4 step 8; removed in Phase 5).csrc/stays insglang.jit_kernel—KERNEL_PATHresolves viafind_specso the build is unaffected.minimax_m3(rocm-triton gemma, HIP; shared with moe) intentionally not moved here.🤖 Generated with Claude Code
CI States
Latest PR Test (Base): ⏳ Run #29840555839
Latest PR Test (Extra): ❌ Run #29840554879