[PATCH] D149743: [RISCV][CodeGen] Support Zdinx on RV32 codegen

luxufan via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 11 00:43:13 PDT 2023


StephenFan added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVExpandPseudoInsts.cpp:279
+    const auto &STI = MF->getSubtarget<RISCVSubtarget>();
+    assert(!STI.enableUnalignedScalarMem());
+
----------------
I don't think we need to assert unaligned scalar memory feature is not enabled. Since `RISCVDAGToDAGISel::SelectAddrRegImm` has ensured that the global value's alignment must be greater than the result of `getOffset()` even when the unaligned scalar memory feature is enabled.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D149743



More information about the llvm-commits mailing list