[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,
----------------
paulwalker-arm wrote:

I'm not too keen on `ParsedVF` being an optional return when the parsing is successful.  What about having this be a plain `ElementCount` with `x` returning `ElementCount::Scalable(0)`?

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


More information about the llvm-commits mailing list