[PATCH] D134638: [Clang][LoongArch] Add inline asm support for constraints k/m/ZB/ZC
Fangrui Song via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Sep 27 15:22:00 PDT 2022
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
More information about the cfe-commits
mailing list