[all-commits] [llvm/llvm-project] cd10de: [Clang] Remove AArch64TargetInfo::setArchFeatures ...

[Sander de Smalen via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: cd10ded697878b0a10b65fbcd2a8f9387bdbf2b7
      https://github.com/llvm/llvm-project/commit/cd10ded697878b0a10b65fbcd2a8f9387bdbf2b7
  Author: Sander de Smalen <sander.desmalen at arm.com>
  Date:   2025-07-01 (Tue, 01 Jul 2025)

  Changed paths:
    M clang/lib/Basic/Targets/AArch64.cpp
    M clang/lib/Basic/Targets/AArch64.h
    M clang/test/Preprocessor/arm-target-features.c

  Log Message:
  -----------
  [Clang] Remove AArch64TargetInfo::setArchFeatures (#146107)

When compiling with `-march=armv9-a+nosve` we found that Clang still
defines the `__ARM_FEATURE_SVE2` macro, which is explicitly set in
`setArchFeatures` when compiling for armv9-a.

After some experimenting, I found out that the list of features passed
into `AArch64TargetInfo::handleTargetFeatures` has already been expanded
and takes into account `+no[feature]` and has already expanded features
like `armv9-a`.

>From that I conclude that `setArchFeatures` is no longer required.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications