[PATCH] D87098: [SVE][CodeGen] Fix DAGCombiner::ForwardStoreValueToDirectLoad for scalable vectors

Cullen Rhodes via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 4 02:51:22 PDT 2020


c-rhodes added inline comments.


================
Comment at: llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp:14832
     // Simple case: Direct non-truncating forwarding
-    if (LDType.getSizeInBits() == LDMemType.getSizeInBits())
+    if (LDType.getSizeInBits() == StMemSize)
       return ReplaceLd(LD, ST->getValue(), Chain);
----------------
should this be comparing `LdMemSize`?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D87098



More information about the llvm-commits mailing list