[Comgr][hotswap] Add bare-bones hotswap transpiler scaffolding#2430
Closed
tgymnich wants to merge 1 commit into
Closed
[Comgr][hotswap] Add bare-bones hotswap transpiler scaffolding#2430tgymnich wants to merge 1 commit into
tgymnich wants to merge 1 commit into
Conversation
3 tasks
Collaborator
Lands the minimal scaffolding
needed to build a `hotswap-transpiler` static library:
* `raiser.{hpp,cpp}` — `raiseToIR()` entry point. The implementation
creates an `llvm::Module` with a single `ret void` AMDGPU kernel
function for any input. ELF ingestion, MC disassembly and the
per-format raise handlers land in subsequent patches.
* `raise_failure.{hpp,cpp}` — structured failure values consumed by
`RaiseResult::failure`.
* `code_object_utils.hpp` — `KernelMeta` struct used in the
`raiseToIR` signature.
* Minimal `CMakeLists.txt` linking only `LLVMCore`, `LLVMSupport`,
`LLVMTargetParser`. The full library/include surface grows
incrementally as later patches need it.
* `tests/raiser_empty_module_test.cpp` (gtest) verifies the
scaffolding contract: an empty input produces a well-formed
module with one `AMDGPU_KERNEL` function whose body is `ret void`.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
tgymnich
force-pushed
the
hotswap-pr-01-scaffolding
branch
from
d6a0046 to
2675e92
Compare
Collaborator
chinmaydd
added
comgr
Author
|
replace by #2438 to enable proper PR stacking. |
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
4 participants
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.
Lands the minimal scaffolding
needed to build a
hotswap-transpilerstatic library:raiser.{hpp,cpp}—raiseToIR()entry point. The implementation creates anllvm::Modulewith a singleret voidAMDGPU kernel function for any input. ELF ingestion, MC disassembly and the per-format raise handlers land in subsequent patches.raise_failure.{hpp,cpp}— structured failure values consumed byRaiseResult::failure.code_object_utils.hpp—KernelMetastruct used in theraiseToIRsignature.CMakeLists.txtlinking onlyLLVMCore,LLVMSupport,LLVMTargetParser. The full library/include surface grows incrementally as later patches need it.tests/raiser_empty_module_test.cpp(gtest) verifies the scaffolding contract: an empty input produces a well-formed module with oneAMDGPU_KERNELfunction whose body isret void.