[PATCH] D134638: [Clang][LoongArch] Add inline asm support for constraints k/m/ZB/ZC

[Fangrui Song via Phabricator via llvm-commits]

MaskRay added inline comments.


================
Comment at: llvm/lib/Target/LoongArch/LoongArchISelDAGToDAG.cpp:92
+  case InlineAsm::Constraint_m: {
+    SDValue Base, Offset;
+    Base = Op;
----------------
Prefer initializing the variable when it is declared


================
Comment at: llvm/lib/Target/LoongArch/LoongArchISelDAGToDAG.cpp:113
+  case InlineAsm::Constraint_ZC: {
+    SDValue Base, Offset;
+    Base = Op;
----------------
Prefer initializing the variable when it is declared


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D134638