[PATCH] D34087: [X86] EltsFromConsecutiveLoads - detect split loads without a common load base (PR32940)

Nirav Dave via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 13 05:49:30 PDT 2017


niravd added a comment.

I'm suggesting we move BaseIndexOffset into SelectionDAG so it can be called there as well.

> Are you suggesting we move BaseIndexOffset to a header so that it can be used in SelectionDAG as well or move all uses from DAGCombiner.cpp into SelectionDAG.cpp (load combines, store merge, neighbour chains etc.)?




Repository:
  rL LLVM

https://reviews.llvm.org/D34087





More information about the llvm-commits mailing list