[PATCH] D144249: [RISCV] Add vendor-defined XTheadMemIdx (Indexed Memory Operations) extension

[Craig Topper via Phabricator via llvm-commits]

craig.topper added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVISelDAGToDAG.cpp:769
+  unsigned Opcode;
+  if (LoadedVT == MVT::i8 && AM == ISD::PRE_INC)
+    Opcode = (Ext == ISD::ZEXTLOAD) ? RISCV::TH_LBUIB : RISCV::TH_LBIB;
----------------
Can we put `AM == ISD::PRE_INC` and `Ext == ISD::ZEXTLOAD` into a variables `IsPre` and `IsZExt`. That should reduce the amount of text here.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D144249