[cmake] reorganize CMake harness/targets

[evaleev]

• can install/export TAPP targets for proper consumption from outside packages

This pull request refactors the build and packaging system for the TAPP project to provide a modern, modular CMake setup with proper package export and consumption support. It splits the reference implementation into its own directory, introduces a CMake package config for easy downstream usage, and updates naming conventions throughout. The changes also add tests for consuming TAPP both from the build and install trees, ensuring the package can be used as intended.

CMake build and packaging improvements:

• Refactored the main CMakeLists.txt to use consistent naming (TAPP_*), moved the reference implementation to reference_implementation/CMakeLists.txt, and updated install directory variables for modularity and clarity. [1] [2] [3]
• Added CMake package config and export files (cmake/tapp-config.cmake.in) for downstream projects to easily find and use TAPP components (api, reference). This includes versioning and component checking. [1] [2]

Installation and export enhancements:

• Updated installation commands to export targets and headers for both the API and reference implementation, supporting modular component installs. [1] [2]
• Set up install interface include directories and export names for targets (tapp::api, tapp::reference). [1] [2]

Testing and consumption improvements:

• Added a new test project (test/consume) and corresponding workflow steps in .github/workflows/cmake.yml to verify that TAPP can be consumed from both the build and install trees, ensuring package usability for downstream users. [1] [2] [3]

Miscellaneous adjustments:

• Updated option names and linking logic to match the new structure (e.g., TAPP_BUILD_EXERCISE instead of TAPP_REFERENCE_BUILD_EXERCISE).
• Fixed linking of tblis-static for tests when TBLIS is enabled.

These changes make the TAPP project easier to build, install, and integrate into other CMake-based projects, while maintaining compatibility and modularity.

[Copilot]

Pull request overview

This PR refactors TAPP’s CMake build/packaging to export consumable targets (tapp::api, tapp::reference) and reorganizes the reference implementation, aiming to support downstream find_package(tapp) usage.

Changes:

• Split build logic into api/ and reference_implementation/ subprojects with consistent TAPP_* variables and exported targets.
• Added a CMake package config (cmake/tapp-config.cmake.in) and target export/install rules.
• Added a new consumer smoke-test project under test/consume.

Reviewed changes

Copilot reviewed 7 out of 9 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
CMakeLists.txt	Renames version variables, sets install dirs, adds package export/config generation.
api/CMakeLists.txt	Defines tapp-api as exported tapp::api and installs headers/target.
reference_implementation/CMakeLists.txt	Creates exported tapp::reference, links to API, adds optional TBLIS bindings and install rules.
cmake/tapp-config.cmake.in	Implements find_package(tapp COMPONENTS ...) config with component checks.
reference_implementation/tblis_bindings/tblis_bind.h	Declares exported TBLIS binding entry points.
reference_implementation/tblis_bindings/tblis_bind.cpp	Implements TBLIS binding logic (including tensor reduction/compare helpers).
test/consume/CMakeLists.txt	Adds a standalone project intended to validate downstream consumption.
test/consume/smoke.c	Minimal runtime smoke test exercising handle create/destroy via installed headers/targets.
.github/workflows/cmake.yml	Adjusts sanitizer flag handling during CMake configure.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Pull request overview

Copilot reviewed 7 out of 9 changed files in this pull request and generated 1 comment.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[SpaceyLake]

It looks good to me! But I am not experienced enough to feel like I understand it all. I think it would be good if @janbrandejs also looked at it.

[evaleev]

It looks good to me! But I am not experienced enough to feel like I understand it all. I think it would be good if @janbrandejs also looked at it.

we probably want to consider this before any additional PRs as cuTENSOR and others would have to be revised in response to this ...

[janbrandejs]

i need to check if the (unofficial) tblis as executor is not broken by this change

[janbrandejs]

all my checks passed and the proposed cmake restructuring makes good sense