[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