Skip to content

explore: support selectable module output types - #129

Merged
guillon merged 2 commits into
xtc-tools:mainfrom
guillon:dev/cguillon/explore-module-type
Sep 4, 2026
Merged

explore: support selectable module output types#129
guillon merged 2 commits into
xtc-tools:mainfrom
guillon:dev/cguillon/explore-module-type

Conversation

@guillon

@guillon guillon commented Sep 4, 2026

Copy link
Copy Markdown
Member

Motivation

Allow exploration to select the generated module format instead of always producing a shared library. Ensure TVM archive linking also works when building outside the current directory.

Description

  • Add --module-type with shlib, arlib, and csrc choices.
  • Map the selected module type to the corresponding compiler option.
  • Run TVM archive linking from the output directory and resolve the packed archive path relative to it.

Commits

  • fb6ddec tvm: fix issue when building in non current directory
  • 290f348 explore: add --module-type option

Discussion

Testing not run (not requested).

TODO:

  • No outstanding TODOs.

@guillon guillon added the enhancement New feature or request label Sep 4, 2026
@guillon guillon self-assigned this Sep 4, 2026
@guillon
guillon enabled auto-merge September 4, 2026 22:20
@guillon
guillon merged commit b8db2dd into xtc-tools:main Sep 4, 2026
8 checks passed
@guillon
guillon deleted the dev/cguillon/explore-module-type branch September 4, 2026 22:34
@codecov

codecov Bot commented Sep 4, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 37.50000% with 5 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/xtc/search/explore.py 28.57% 4 Missing and 1 partial ⚠️

📢 Thoughts on this report? Let us know!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant