[PATCH] D140530: [RISCV] Add integer scalar instructions to isAssociativeAndCommutative

[Hsiangkai Wang via Phabricator via llvm-commits]

HsiangKai added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVInstrInfo.cpp:1365
+
+  switch (Opc) {
+  default:
----------------
kito-cheng wrote:
> extract this switch to a new function `isAssociativeAndCommutativeIntOpc` and then use that in `RISCVInstrInfo::hasReassociableSibling`?
I moved FRM checking to `hasReassociableSibling`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D140530