[llvm] [SVE] Don't require lookup when demangling vector function mappings (PR #72260)

Paul Walker via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 22 09:22:23 PST 2023


================
@@ -374,6 +424,24 @@ std::optional<VFInfo> VFABI::tryDemangleForVFABI(StringRef MangledName,
   if (Parameters.empty())
     return std::nullopt;
 
+  // Figure out the number of lanes in vectors for this function variant. This
+  // is easy for fixed length, as the vlen encoding just gives us the value
+  // directly. However, if the vlen mangling indicated that this function
+  // variant expects scalable vectors, then we need to figure out the minimum
+  // based on the widest scalar types in vector arguments.
+  std::optional<ElementCount> EC;
+  if (ParsedVF.second) {
+    // Scalable VF, need to work out the minimum from the element types
+    // in the scalar function arguments.
----------------
paulwalker-arm wrote:

As above, although personally I'd just remove this comment as the function name tells the reader all they need to know.

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


More information about the llvm-commits mailing list