[PATCH] D26149: [DAGCombiner] Match load by bytes idiom and fold it into a single load

[Jun Sha via Phabricator via llvm-commits]

Joshua-401 added a comment.
Herald added subscribers: steven.zhang, pengfei.
Herald added a project: LLVM.

Did you consider doing the load combine at IR level?

> Matching the pattern above is easier at SelectionDAG level since address reassociation has already happened and the fact that the loads are adjacent is clear. Understanding that these loads are adjacent at IR level would have involved looking through geps/zexts/adds while looking at the addresses.




Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D26149/new/

https://reviews.llvm.org/D26149