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

Paul Walker via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 21 03:50:31 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,
----------------
paulwalker-arm wrote:

Sorry, my suggest here is bogus because you cannot create a zero length scalable vector.  What I'm really asking for is whether it's possible to return some kind of token that is later used to contract the ElementCount once all other information is available.  Perhaps this is just a local std::pair that rather resembles an ElementCount?

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


More information about the llvm-commits mailing list