[llvm] [AArch64] Consider runtime mode when deciding to use SVE for fixed-length vectors. (PR #96081)
Sander de Smalen via llvm-commits
llvm-commits at lists.llvm.org
Thu Jun 20 01:42:49 PDT 2024
================
@@ -412,7 +417,8 @@ AArch64TargetMachine::getSubtargetImpl(const Function &F) const {
StringRef FS = FSAttr.isValid() ? FSAttr.getValueAsString() : TargetFS;
bool HasMinSize = F.hasMinSize();
- bool IsStreaming = F.hasFnAttribute("aarch64_pstate_sm_enabled") ||
+ bool IsStreaming = ForceStreaming ||
+ F.hasFnAttribute("aarch64_pstate_sm_enabled") ||
F.hasFnAttribute("aarch64_pstate_sm_body");
----------------
sdesmalen-arm wrote:
Are you happy with me fixing up the other case in this patch as well? I figured it seems more sensible to check the `bool` first before calling some more compute-intensive function.
https://github.com/llvm/llvm-project/pull/96081
More information about the llvm-commits
mailing list