[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