[PATCH] D108200: [llvm][sve] Lowering for VLS masked extending loads

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 13 12:52:11 PDT 2021


efriedma added inline comments.


================
Comment at: llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp:10879
 
-  if (!TLI.isLoadExtLegal(ExtLoadType, VT, Ld->getValueType(0)))
+  if (!TLI.isLoadExtLegalOrCustom(ExtLoadType, VT, Ld->getValueType(0)))
     return SDValue();
----------------
Do we need a check here to make sure we don't create illegal masked loads after legalization?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D108200



More information about the llvm-commits mailing list