[PATCH] D29394: [DAGCombiner] Support non-zero offset in load combine

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 8 03:35:04 PST 2017


RKSimon added a comment.

One minor



================
Comment at: lib/CodeGen/SelectionDAG/DAGCombiner.cpp:4596
          "memory, so there must be at least one load which produces the value");
   assert(Base && "Base address of the accessed memory location must be set");
 
----------------
Add an assert that ensures FirstOffset has a sane value.


https://reviews.llvm.org/D29394





More information about the llvm-commits mailing list