[PATCH] D24746: AMDGPU/R600: Don't use REGISTER_{LOAD, STORE} ISD nodes
    Matt Arsenault via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Fri Jan  6 11:17:31 PST 2017
    
    
  
arsenm accepted this revision.
arsenm added a reviewer: arsenm.
arsenm added a comment.
This revision is now accepted and ready to land.
LGTM. Can the node definitions be deleted now?
================
Comment at: lib/Target/AMDGPU/R600ISelLowering.cpp:1468
+  if (Ptr.getOpcode() != AMDGPUISD::DWORDADDR) {
+    assert(VT == MVT::i32 || (LoadNode->dumpr(),0));
+    Ptr = DAG.getNode(ISD::SRL, DL, MVT::i32, Ptr, DAG.getConstant(2, DL, MVT::i32));
----------------
The dump should be removed from here
Repository:
  rL LLVM
https://reviews.llvm.org/D24746
    
    
More information about the llvm-commits
mailing list