Skip to content

Conversation

felipe-iar
Copy link

This PR adds IAR C-STAT support in the Zephyr Static Code Analysis (SCA) framework.

Reference: <LANG>_ICSTAT

CC: @danielfl-iar, @micaelbo, @RobinKastberg

Copy link

github-actions bot commented Sep 2, 2025

Hello @felipe-iar, and thank you very much for your first pull request to the Zephyr project!
Our Continuous Integration pipeline will execute a series of checks on your Pull Request commit messages and code, and you are expected to address any failures by updating the PR. Please take a look at our commit message guidelines to find out how to format your commit messages, and at our contribution workflow to understand how to update your Pull Request. If you haven't already, please make sure to review the project's Contributor Expectations and update (by amending and force-pushing the commits) your pull request if necessary.
If you are stuck or need help please join us on Discord and ask your question there. Additionally, you can escalate the review when applicable. 😊

Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds IAR C-STAT static code analysis tool support to the Zephyr Static Code Analysis (SCA) framework. IAR C-STAT is a comprehensive static analysis tool for C/C++ source code that can find errors and vulnerabilities while supporting various coding standards.

  • Adds a new SCA variant for IAR C-STAT that integrates with the existing Zephyr SCA framework
  • Provides configurable analysis parameters including rulesets, threading, and database options
  • Creates documentation explaining installation, usage, and configuration options

Reviewed Changes

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

File Description
doc/develop/sca/index.rst Adds IAR to the list of supported SCA tools
doc/develop/sca/iar.rst Complete documentation for IAR C-STAT integration including installation, usage, and configuration
cmake/sca/iar/sca.cmake CMake implementation that configures IAR C-STAT with customizable parameters and integrates with CMake's ICSTAT support

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@felipe-iar felipe-iar force-pushed the iar-add-sca-cstat branch 5 times, most recently from c9fd748 to e1a7240 Compare September 3, 2025 07:55
bjorniuppsala
bjorniuppsala previously approved these changes Sep 3, 2025
Copy link
Contributor

@pdgendt pdgendt left a comment

Choose a reason for hiding this comment

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

Is there an example of how the output looks?

This commit adds support for IAR C-STAT Static Analysis
in the Zephyr's SCA Framework.

By specifying -DZEPHYR_SCA_VARIANT=iar in west, a SQLite
database file will be generated under build/sca/iar.

Signed-off-by: Felipe Torrezan <felipe.torrezan@iar.com>
Copy link

sonarqubecloud bot commented Sep 3, 2025

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

Successfully merging this pull request may close these issues.

7 participants