[PATCH] D77117: [RISCV] Split RISCVISelDAGToDAG.cpp to RISCVISelDAGToDAG.h and RISCVISelDAGToDAG.cpp

[Luís Marques via Phabricator via llvm-commits]

luismarques added a comment.

In D77117#1952272 <https://reviews.llvm.org/D77117#1952272>, @shiva0217 wrote:

> For the downstream RISCV maintenance, it would be easier to inherent the RISCVISelDAGToDAG and only override the method need to customize for the custom provider ISA extension without touching RISCVISelDAGToDAG.cpp which may cause conflict when upgrading the downstream LLVM version.
>  I think it could be helpful for other RISCV downstream maintenance. Could it be the sufficient reason to split out the header?


That sounds reasonable.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D77117