[PATCH] D149986: AMDGPU: Force sc0 and sc1 on stores for gfx940 and gfx941
    Juan Manuel Martinez CaamaƱo via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Wed May 10 00:53:34 PDT 2023
    
    
  
jmmartinez added inline comments.
================
Comment at: llvm/lib/Target/AMDGPU/SIMemoryLegalizer.cpp:524
+         SIAtomicAddrSpace::NONE)
+      return enableSC0Bit(MI) | enableSC1Bit(MI);
+    return false;
----------------
NIT: Is the use of the bitwise or " | " intended? I'd use the logical or " || " instead.
Repository:
  rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D149986/new/
https://reviews.llvm.org/D149986
    
    
More information about the llvm-commits
mailing list