[PATCH] D93798: [CSKY 4/n] Add basic CSKYAsmParser and CSKYInstPrinter
    Zixuan Wu via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Thu Apr  1 22:22:46 PDT 2021
    
    
  
zixuan-wu added inline comments.
================
Comment at: llvm/lib/Target/CSKY/AsmParser/CSKYAsmParser.cpp:124
+  bool isImm() const override { return Kind == Immediate; }
+  bool isMem() const override { return false; }
+
----------------
MaskRay wrote:
> Is `isMem` unimplemented? If yes, deferring to a subsequent patch is better.
It's implement of pure virtual function, I am afraid it can not ignore.
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D93798/new/
https://reviews.llvm.org/D93798
    
    
More information about the llvm-commits
mailing list