[PATCH] D143018: [DAGCombiner][NFC] Factor out ByteProvider

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 15 12:09:53 PDT 2023


RKSimon added a comment.

LGTM with one minor - does anyone else have any comments?



================
Comment at: llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp:8950
+  assert(isa<LoadSDNode>(FirstByteProvider->Src.value()));
+  LoadSDNode *FirstLoad = cast<LoadSDNode>(FirstByteProvider->Src.value());
 
----------------
RKSimon wrote:
> auto *
auto *FirstLoad


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D143018



More information about the llvm-commits mailing list