[PATCH] D57008: [AMDGPU] With XNACK, cannot clause a load with result coalesced with operand
    Stanislav Mekhanoshin via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Mon Jan 21 10:05:35 PST 2019
    
    
  
rampitec added inline comments.
================
Comment at: lib/Target/AMDGPU/SIFormMemoryClauses.cpp:126
+    unsigned ResReg = ResMO.getReg();
+    for (const MachineOperand &MO : MI.operands()) {
+      if (!MO.isReg() || MO.isDef())
----------------
MI.uses() maybe? Then you do not need to check for isDef() and have less to scan.
Repository:
  rL LLVM
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D57008/new/
https://reviews.llvm.org/D57008
    
    
More information about the llvm-commits
mailing list