[PATCH] D142214: [MC] Do not copy MCInstrDescs. NFC.
    Sergei Barannikov via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Fri Jan 20 07:53:45 PST 2023
    
    
  
barannikov88 added inline comments.
================
Comment at: llvm/lib/Target/SPIRV/MCTargetDesc/SPIRVMCCodeEmitter.cpp:64
 static bool hasType(const MCInst &MI, const MCInstrInfo &MII) {
-  MCInstrDesc MCDesc = MII.get(MI.getOpcode());
+  MCInstrDesc &MCDesc = MII.get(MI.getOpcode());
   // If we define an output, and have at least one other argument.
----------------
Doesn't this trigger compilation error (missing `const`)?
Repository:
  rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D142214/new/
https://reviews.llvm.org/D142214
    
    
More information about the llvm-commits
mailing list