Add TI Arm Clang Compiler Support to Core Tests#277
Add TI Arm Clang Compiler Support to Core Tests#277smmasongt wants to merge 13 commits intoARM-software:mainfrom
Conversation
Update README.md
| super().__init__('CLANG_TI', **args) | ||
|
|
||
| def get_cc(self): | ||
| return os.path.join(self.get_root(), 'tiarmclang') |
There was a problem hiding this comment.
tiarmclang needs to be installed during the workflow and registered via CLANG_TI_TOOLCHAIN_5_0_0=<path>/bin environment variable.
There was a problem hiding this comment.
I verified this locally on MacOS and it works.
There was a problem hiding this comment.
Updated documentation to clarify instructions for environment variables.
CMSIS/Core/Test/README.md
Outdated
| - [CMSIS-Toolbox 2.10.0](https://artifacts.keil.arm.com/cmsis-toolbox/2.10.0/)* | ||
| - [CMake 3.28.4](https://cmake.org/download/)* | ||
| - [Ninja 1.12.0](https://github.com/ninja-build/ninja/releases)* |
There was a problem hiding this comment.
Core Tests do not rely on CMSIS-Toolbox, CMake, and Ninja.
There was a problem hiding this comment.
Removed unnecessary references.
|
@smmasongt, you may want to discuss distributing your toolchain via vcpkg like we do. This would make it easier for users to install the toolchain properly. |
|
@JonatanAntoni I'll work on removing those unnecessary dependencies tonight. I misunderstood the build flow and thought the toolchain environment variables were used indirectly by CMSIS Toolbox. |
I'll reach out to the Arm maintainers to see if they would be willing to add it to vcpkg. I'm not affiliated with TI so hopefully there won't be any license issues with rehosting the installers. |
Ah, thanks for stating this clear. TI would be expected to run their own vcpkg registry. Re-hosting would not be required once the download links are public. Its just a matter of providing some |
2 participants
Builds on previous work in CMSIS-Toolbox to add TI Arm Clang Compiler support to CMSIS under designator
CLANG_TI.Clang_TIoption to Core Unit Tests scripts.Clang Compiler 20.1.0which pointed to 404 website. Moved link from LLVM Embedded Toolchain to new Arm Toolchain for Embedded repo.Note: Cortex-R support is dependent on pull request #45 and future header updates.