[PATCH] D85488: AMDGPU: Avoid explicitly listing all the memory nodes
    Matt Arsenault via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Fri Aug  7 10:08:21 PDT 2020
    
    
  
arsenm added inline comments.
================
Comment at: llvm/lib/Target/AMDGPU/SIISelLowering.cpp:10579
+    if (!DCI.isBeforeLegalize()) {
+      if (MemSDNode *MemNode = dyn_cast<MemSDNode>(N))
+        return performMemSDNodeCombine(MemNode, DCI);
----------------
rampitec wrote:
> Are AMDGPUISD nodes also cast to MemSDNode?
Yes, all the ones in the range used for memory nodes
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D85488/new/
https://reviews.llvm.org/D85488
    
    
More information about the llvm-commits
mailing list