[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