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

Paul Walker via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 20 10:39:23 PST 2023


================
@@ -66,24 +66,26 @@ ParseRet tryParseMask(StringRef &MangledName, bool &IsMasked) {
 /// vector length. On success, the `<vlen>` token is removed from
 /// the input string `ParseString`.
 ///
-ParseRet tryParseVLEN(StringRef &ParseString, unsigned &VF, bool &IsScalable) {
+ParseRet tryParseVLEN(StringRef &ParseString,
+                      std::optional<unsigned> &ParsedVF) {
   if (ParseString.consume_front("x")) {
-    // Set VF to 0, to be later adjusted to a value grater than zero
-    // by looking at the signature of the vector function with
-    // `getECFromSignature`.
-    VF = 0;
-    IsScalable = true;
+    // We can't determine the VF of a scalable vector by looking at the vlen
----------------
paulwalker-arm wrote:

The encoding is ISA specific and currently only supported when the SVE ISA is specified and thus everything else should result in a parse failure?

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


More information about the llvm-commits mailing list