[PATCH] D128135: [DAGCombiner][ARM][RISCV] Teach ShrinkLoadReplaceStoreWithStore to use truncstore.

[Simon Pilgrim via Phabricator via llvm-commits]

RKSimon accepted this revision.
RKSimon added a comment.
This revision is now accepted and ready to land.

LGTM with one minor



================
Comment at: llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp:17372
   // VT we're shrinking to (i8/i16/i32) is legal or we're still before type
   // legalization (and the target doesn't explicitly think this is a bad idea).
   MVT VT = MVT::getIntegerVT(NumBytes * 8);
----------------
Might be worth extending this comment a little for the new truncstore case


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D128135