Skip to content

refactor: separate ops from platforms#74

Merged
voltjia merged 9 commits intodevelopfrom
refactor/separate-ops-from-platforms
Apr 29, 2026
Merged

refactor: separate ops from platforms#74
voltjia merged 9 commits intodevelopfrom
refactor/separate-ops-from-platforms

Conversation

@voltjia
Copy link
Copy Markdown
Collaborator

@voltjia voltjia commented Apr 29, 2026

No description provided.

@voltjia voltjia requested a review from a team April 29, 2026 11:36
@voltjia voltjia changed the base branch from master to develop April 29, 2026 11:37
@voltjia voltjia requested a review from Ziminli April 29, 2026 11:38
…sion with framework `runtime.h`

The shared CUDA scaffolding header `cuda/runtime.h` defines the
`CudaRuntime` CRTP base, which inherits from `DeviceRuntime` declared in
the framework-level `runtime.h`. Bringing the framework header in via
`#include "../runtime.h"` violates the Google C++ Style Guide's
prohibition on relative include paths, and rewriting to
`#include "runtime.h"` instead self-references because quoted includes
search the current file's directory first.

Renaming the file to `runtime_.h` matches the trailing-underscore
convention used by every other platform-internal scaffolding header
(`device_.h`, `data_type_.h`, `caster_.h`, etc.) and removes the
collision so `#include "runtime.h"` resolves to the framework header
through the project include path. Update the four sub-platform
`runtime_.h` files that include it accordingly.
@voltjia voltjia merged commit 7ced84c into develop Apr 29, 2026
4 checks passed
@voltjia voltjia deleted the refactor/separate-ops-from-platforms branch April 29, 2026 14:44
@voltjia voltjia restored the refactor/separate-ops-from-platforms branch April 30, 2026 07:53
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.

2 participants