[PATCH] D27861: [DAGCombiner] Match load by bytes idiom and fold it into a single load. Attempt #2.

[Artur Pilipenko via Phabricator via llvm-commits]

apilipenko marked 5 inline comments as done.
apilipenko added inline comments.


================
Comment at: lib/CodeGen/SelectionDAG/DAGCombiner.cpp:4503
+    assert(L->hasNUsesOfValue(1, 0) && !L->isVolatile() && !L->isIndexed() &&
+           (L->getExtensionType() == ISD::NON_EXTLOAD) &&
+           "Must be enforced by calculateByteProvider");
----------------
RKSimon wrote:
> Are there any situations where we can use ZEXTLOAD? If so add a TODO comment?
I'm going to add ext loads support in a follow up patch. Left a TODO for now.


https://reviews.llvm.org/D27861