Skip to content
This repository was archived by the owner on Apr 6, 2026. It is now read-only.

Update the build.toml format in preparation for Metal#143

Closed
danieldk wants to merge 1 commit into
mainfrom
framework-in-general
Closed

Update the build.toml format in preparation for Metal#143
danieldk wants to merge 1 commit into
mainfrom
framework-in-general

Conversation

@danieldk

Copy link
Copy Markdown
Member

Changes:

  • kernel.<name>.language is removed. Instead the compute framework is set in kernel.general.compute-framework.
  • torch.universal is removed. A kernel is now universal when compute-framework is set to universal.
  • If a kernel can be hipified, set kernel.<name>.supports_hipify to true (requires the compute framework to be cuda).

build2toml still parses the old format and adds a new update-build subcommand to upgrade a build.toml. The Nix tooling does not support the old format, but will suggest to run build2toml update-build.

Changes:

* `kernel.<name>.language` is removed. Instead the compute framework
  is set in `kernel.general.compute-framework`.
* `torch.universal` is removed. A kernel is now universal when
  `compute-framework` is set to `universal`.
* If a kernel can be hipified, set `kernel.<name>.supports_hipify`
  to `true` (requires the compute framework to be `cuda`).

`build2toml` still parses the old format and adds a new `update-build`
subcommand to upgrade a `build.toml`. The Nix tooling does not support
the old format, but will suggest to run `build2toml update-build`.
@danieldk danieldk force-pushed the framework-in-general branch from c608fec to 370341a Compare May 27, 2025 10:58
@danieldk danieldk marked this pull request as ready for review May 27, 2025 11:12
@danieldk danieldk marked this pull request as draft May 27, 2025 13:30
@danieldk

Copy link
Copy Markdown
Member Author

Closing in favor of #144.

@danieldk danieldk closed this May 27, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant