[all-commits] [llvm/llvm-project] 1d6d07: [AArch64] Remove FeatureUseScalarIncVL

[David Green via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 1d6d073fbbaebbde6891501fe20f02a0ea345131
      https://github.com/llvm/llvm-project/commit/1d6d073fbbaebbde6891501fe20f02a0ea345131
  Author: David Green <david.green at arm.com>
  Date:   2024-11-10 (Sun, 10 Nov 2024)

  Changed paths:
    M llvm/lib/Target/AArch64/AArch64Features.td
    M llvm/lib/Target/AArch64/AArch64Subtarget.cpp
    M llvm/lib/Target/AArch64/AArch64Subtarget.h
    M llvm/test/CodeGen/AArch64/sve-intrinsics-counting-elems-i32.ll
    M llvm/test/CodeGen/AArch64/sve-intrinsics-counting-elems.ll
    M llvm/test/CodeGen/AArch64/sve-vl-arith.ll

  Log Message:
  -----------
  [AArch64] Remove FeatureUseScalarIncVL

FeatureUseScalarIncVL is a tuning feature, used to control whether addvl or
add+cnt is used. It was previously added as a dependency for FeatureSVE2, an
architecture feature but this can be seen as a layering violation. The main
disadvantage is that -use-scalar-inc-vl cannot be used without disabling sve2
and all dependant features.

This patch now replaces that with an option that if unset defaults to hasSVE ||
hasSME, but is otherwise overriden by the option. The hope is that no cpus will
rely on the tuning feature (or we can readdit if needed.



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