[PATCH] D136629: [AMDGPU] Fix delay alu for VOPD with src2acc
    Jay Foad via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Tue Oct 25 03:23:30 PDT 2022
    
    
  
foad added inline comments.
================
Comment at: llvm/lib/Target/AMDGPU/AsmParser/AMDGPUAsmParser.cpp:8551
     for (unsigned SrcIdx = 0; SrcIdx < SrcOperandsNum; ++SrcIdx) {
-      addOp(InstInfo[CompIdx].getParsedSrcIndex(SrcIdx));
+      auto CInfo = InstInfo[CompIdx];
+      addOp(CInfo.getParsedSrcIndex(SrcIdx, CInfo.hasSrc2Acc()));
----------------
dp wrote:
> This may be moved out of the loop (together with `CInfo.hasSrc2Acc()`).
Maybe `const auto &` here and on the declaration of `InstInfo`?
Repository:
  rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D136629/new/
https://reviews.llvm.org/D136629
    
    
More information about the llvm-commits
mailing list