Skip to content

core/alignment.h: Fix 'MBEDTLS_GCC_VERSION' is not defined error#800

Open
AmyChan-Intel wants to merge 1 commit into
Mbed-TLS:developmentfrom
AmyChan-Intel:development
Open

core/alignment.h: Fix 'MBEDTLS_GCC_VERSION' is not defined error#800
AmyChan-Intel wants to merge 1 commit into
Mbed-TLS:developmentfrom
AmyChan-Intel:development

Conversation

@AmyChan-Intel
Copy link
Copy Markdown

Only check 'MBEDTLS_GCC_VERSION' when 'MBEDTLS_COMPILER_IS_GCC' is defined

Description

Only check 'MBEDTLS_GCC_VERSION' when 'MBEDTLS_COMPILER_IS_GCC' is defined

PR checklist

Please remove the segment/s on either side of the | symbol as appropriate, and add any relevant link/s to the end of the line.
If the provided content is part of the present PR remove the # symbol.

  • changelog provided | not required because:
  • framework PR provided Mbed-TLS/mbedtls-framework# | not required
  • TF-PSA-Crypto development PR provided # | not required because:
  • TF-PSA-Crypto 1.1 PR provided # | not required because:
  • mbedtls development PR provided Mbed-TLS/mbedtls# | not required because:
  • mbedtls 4.1 PR provided Mbed-TLS/mbedtls# | not required because:
  • mbedtls 3.6 PR provided Mbed-TLS/mbedtls# | not required because:
  • tests provided | not required because:

Notes for the submitter

Please refer to the contributing guidelines, especially the
checklist for PR contributors.

Help make review efficient:

  • Multiple simple commits
    • please structure your PR into a series of small commits, each of which does one thing
  • Avoid force-push
    • please do not force-push to update your PR - just add new commit(s)
  • See our Guidelines for Contributors for more details about the review process.

Only check 'MBEDTLS_GCC_VERSION' when 'MBEDTLS_COMPILER_IS_GCC' is defined

Signed-off-by: Amy Chan <amy.chan@intel.com>
Copy link
Copy Markdown
Contributor

@yanesca yanesca left a comment

Choose a reason for hiding this comment

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

LGTM

@minosgalanakis minosgalanakis added bug Something isn't working component-platform Portability layer and build scripts labels Jun 4, 2026
@minosgalanakis minosgalanakis moved this to Triage in in Community Jun 4, 2026
@minosgalanakis minosgalanakis added size-s Estimated task size: small (~2d) size-xs Estimated task size: extra small (a few hours at most) and removed size-s Estimated task size: small (~2d) labels Jun 4, 2026
@minosgalanakis minosgalanakis added priority-low Low priority - this may not receive review soon needs-review Every commit must be reviewed by at least two team members needs-reviewer This PR needs someone to pick it up for review needs-ci Needs to pass CI tests labels Jun 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working component-platform Portability layer and build scripts needs-ci Needs to pass CI tests needs-review Every commit must be reviewed by at least two team members needs-reviewer This PR needs someone to pick it up for review priority-low Low priority - this may not receive review soon size-xs Estimated task size: extra small (a few hours at most)

Projects

Status: Triage in
Status: In Development

Development

Successfully merging this pull request may close these issues.

3 participants