[clang] [AArch64] Enable vscale_range with +sme (PR #124466)

Amara Emerson via cfe-commits cfe-commits at lists.llvm.org
Tue Jan 28 10:01:39 PST 2025


================
@@ -708,7 +708,7 @@ AArch64TargetInfo::getVScaleRange(const LangOptions &LangOpts) const {
     return std::pair<unsigned, unsigned>(
         LangOpts.VScaleMin ? LangOpts.VScaleMin : 1, LangOpts.VScaleMax);
 
-  if (hasFeature("sve"))
+  if (hasFeature("sve") || hasFeature("sme"))
----------------
aemerson wrote:

I agree for SME-only platforms it seems fine. But what's the meaning of vscale_range when you have a CPU that has both? At that point do we need to disambiguate the different potential vscale_range's for VL & SVL?

https://github.com/llvm/llvm-project/pull/124466


More information about the cfe-commits mailing list