Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[schema] Add CLANG_TI compiler #339

Merged
merged 1 commit into from
Dec 18, 2024
Merged

[schema] Add CLANG_TI compiler #339

merged 1 commit into from
Dec 18, 2024

Conversation

brondani
Copy link
Collaborator

@brondani brondani force-pushed the add-clangti-compiler branch from 3823761 to e73f293 Compare December 18, 2024 14:15
@brondani brondani force-pushed the add-clangti-compiler branch from e73f293 to 48ce00d Compare December 18, 2024 14:30
@brondani brondani requested a review from jkrech December 18, 2024 14:42
jkrech
jkrech approved these changes Dec 18, 2024
View reviewed changes
Copy link
Member

@jkrech jkrech left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM - it is consistent with the existing CLANG and as it is a Texas Instruments specific variant appending TI this makes sense in this context even if CMake is using a different compiler identifier.

@jkrech jkrech merged commit c4177fe into main Dec 18, 2024
3 checks passed
@jkrech jkrech deleted the add-clangti-compiler branch December 18, 2024 14:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jkrech jkrech jkrech approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@brondani @jkrech