[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