[PATCH] D105576: [AArch64][SME] Add system registers and related instructions

Cullen Rhodes via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 19 02:51:33 PDT 2022


c-rhodes added inline comments.


================
Comment at: llvm/lib/Target/AArch64/AsmParser/AArch64AsmParser.cpp:6528
+      return Match_Success;
+    break;
   }
----------------
RKSimon wrote:
> @c-rhodes Coverity is warning that ExpectedVal is still uninitialized when we break here - but should this be "return Match_InvalidOperand;" anyhow?
> @c-rhodes Coverity is warning that ExpectedVal is still uninitialized when we break here - but should this be "return Match_InvalidOperand;" anyhow?

Apologies for late reply, it should, thanks for spotting and fixing!


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D105576/new/

https://reviews.llvm.org/D105576



More information about the llvm-commits mailing list