[PATCH] D41505: [DAG] Teach findBaseOffset to interpret indexes of indexed memory operations

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 11 03:08:07 PST 2018


RKSimon added inline comments.


================
Comment at: llvm/lib/CodeGen/SelectionDAG/SelectionDAGAddressAnalysis.cpp:135
     }
+    // If we get here break out of the loop.
     break;
----------------
niravd wrote:
> RKSimon wrote:
> > Maybe add a LLVM_FALLTHROUGH and default case here?
> We'd still need to have a trailing break after the switch (or a do-loop-again varaible) to drop out of the while loop because of the other 2 cases. 
You're right - my mistake.


https://reviews.llvm.org/D41505





More information about the llvm-commits mailing list