[PATCH] D127477: [RISCV] move `isFaultFirstLoad` into `RISCVInstrInfo`

Yeting Kuo via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 10 02:45:41 PDT 2022

Previous message: [PATCH] D127477: [RISCV] move `isFaultFirstLoad` into `RISCVInstrInfo`
Next message: [PATCH] D127477: [RISCV] move `isFaultFirstLoad` into `RISCVInstrInfo`
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

fakepaper56 added a comment.

LGTM. It's fine to me that not declare isFaultFirstLoad as a static function of class RISCVInstrInfo, although I prefer this method.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D127477

Previous message: [PATCH] D127477: [RISCV] move `isFaultFirstLoad` into `RISCVInstrInfo`
Next message: [PATCH] D127477: [RISCV] move `isFaultFirstLoad` into `RISCVInstrInfo`
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

More information about the llvm-commits mailing list