[PATCH] D41701: [DAG] Teach BaseIndexOffset to correctly handle with indexed operations

[Eli Friedman via Phabricator via llvm-commits]

efriedma added a comment.

I would prefer to have a regression test, but it's okay if you can't come up with a way to write one.



================
Comment at: llvm/lib/CodeGen/SelectionDAG/SelectionDAGAddressAnalysis.cpp:90
+    else // If unknown, give up now.
+      return BaseIndexOffset(Base, Index, Offset, IsIndexSignExt);
+  } else if (N->getAddressingMode() == ISD::PRE_DEC) {
----------------
Does this need to be something like "BaseIndexOffset(Base, N->getOffset(), 0, false);"?  Currently, it looks like this code is discarding the increment.


https://reviews.llvm.org/D41701