[Bug preprocessor/109415] New: No predefined macros to differentiate between ARM Cortex-M33 and Cortex-M55

[Bug preprocessor/109415] New: No predefined macros to differentiate between ARM Cortex-M33 and Cortex-M55

[Vedant.VijayYevale at infineon dot com]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=109415

            Bug ID: 109415
           Summary: No predefined macros to differentiate between ARM
                    Cortex-M33 and Cortex-M55
           Product: gcc
           Version: 11.3.1
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: preprocessor
          Assignee: unassigned at gcc dot gnu.org
          Reporter: Vedant.VijayYevale at infineon dot com
  Target Milestone: ---

there is no reliable way to differentiate between CM33 and CM55.
For both CM33 and CM55 __ARM_ARCH_8M_MAIN__  is defined.
there should be updated compiler predefined macro for CM55.

[Bug target/109415] No predefined macros to differentiate between ARM Cortex-M33 and Cortex-M55

[pinskia at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=109415

--- Comment #1 from Andrew Pinski <pinskia at gcc dot gnu.org> ---
I don't see why there should be a define here? The command line will be
different between the two so you could just stick a -DXYZ somewhere or handle
it in a header file.

[Bug target/109415] No predefined macros to differentiate between ARM Cortex-M33 and Cortex-M55

[pinskia at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=109415

--- Comment #2 from Andrew Pinski <pinskia at gcc dot gnu.org> ---
https://github.com/ARM-software/acle Does not define one either ...
That is it does not define ARMv8.1-m vs ARMv8-m difference either ...

[Bug target/109415] No predefined macros to differentiate between ARM Cortex-M33 and Cortex-M55

[Vedant.VijayYevale at infineon dot com]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=109415

--- Comment #3 from vedant <Vedant.VijayYevale at infineon dot com> ---
in arm compiler i have found __ARM_ARCH_8M_MAIN__ and __ARM_ARCH_8_1M_MAIN__
for CM33 and CM55 respectively.

[Bug target/109415] No predefined macros to differentiate between ARM Cortex-M33 and Cortex-M55

[pinskia at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=109415

--- Comment #4 from Andrew Pinski <pinskia at gcc dot gnu.org> ---
Depending on the feature you are testing for, you could use some other macros.

e.g. __ARM_FEATURE_PAUTH, __ARM_FEATURE_BTI, __ARM_FEATURE_CMSE .

[Bug target/109415] No predefined macros to differentiate between ARM Cortex-M33 and Cortex-M55

[pinskia at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=109415

--- Comment #5 from Andrew Pinski <pinskia at gcc dot gnu.org> ---
(In reply to vedant from comment #3)
> in arm compiler i have found __ARM_ARCH_8M_MAIN__ and __ARM_ARCH_8_1M_MAIN__
> for CM33 and CM55 respectively.

As far as I can tell those macros are not documented in
https://github.com/ARM-software/acle . You might want to file a bug with that
first.

[Bug target/109415] No predefined macros to differentiate between ARM Cortex-M33 and Cortex-M55

[pinskia at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=109415

--- Comment #6 from Andrew Pinski <pinskia at gcc dot gnu.org> ---
Also I noticed the ACLE is confusing for any of the -m profile stuff and mostly
just references the -a profile too.

[Bug target/109415] No predefined macros to differentiate between ARM Cortex-M33 and Cortex-M55

[Vedant.VijayYevale at infineon dot com]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=109415

--- Comment #7 from vedant <Vedant.VijayYevale at infineon dot com> ---
in ARM and IAR as well ARMv8.1-m vs ARMv8-m have different macros. it is needed
to maintain code compatibility in different build ecosystems. instead of
specifying -DXYZ everywhere

[Bug target/109415] No predefined macros to differentiate between ARM Cortex-M33 and Cortex-M55

[rearnsha at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=109415

--- Comment #8 from Richard Earnshaw <rearnsha at gcc dot gnu.org> ---
The __ARM_ARCH_...__ macros turned out to be a very bad design decision. Each
new architecture needs a new macro that older compilers (and software) will not
know about.  The ACLE approach is far more sensible and GCC has mostly adopted
that now.  I personally consider the existing __ARM_ARCH_...__ macros to be
deprecated, though I don't think the manual actually says this yet.

There is a known bug in GCC.  ACLE says that __ARM_ARCH should have the value
<arch_major>*100 + <arch_minor> for architectures after arm-v8, but we don't
implement that yet (there may already be a PR about this) and report
__ARM_ARCH=8 for all existing armv8.xxx variants.

[Bug target/109415] No predefined macros to differentiate between ARM Cortex-M33 and Cortex-M55

[rearnsha at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=109415

Richard Earnshaw <rearnsha at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|---                         |DUPLICATE

--- Comment #9 from Richard Earnshaw <rearnsha at gcc dot gnu.org> ---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99312

*** This bug has been marked as a duplicate of bug 99312 ***