================ @@ -2230,7 +2230,7 @@ bool AMDGPUDAGToDAGISel::SelectSMRDBaseOffset(SDValue Addr, SDValue &SBase, SDValue N0, N1; // Extract the base and offset if possible. - if (CurDAG->isBaseWithConstantOffset(Addr) || Addr.getOpcode() == ISD::ADD) { + if (CurDAG->isBaseWithConstantOffset(Addr) || Addr->isAnyAdd()) { ---------------- arsenm wrote:
The existing code doesn't make sense to me, why is this checking isBaseWithConstantOffset if it isn't parsing out the constant offset? This is just redundant with the isAnyAdd? https://github.com/llvm/llvm-project/pull/145330 _______________________________________________ llvm-branch-commits mailing list llvm-branch-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits