[PATCH] D78545: [RISCV] Make Lower* functions public for downstream maintenance

Shiva Chen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 28 02:39:14 PDT 2020


shiva0217 added a comment.

Although most of the targets declare these functions as private, I think it might worth changing to public. So the various downstream RISCV users may able to inherit the class and customize the functions for their ISA and return the basic class functions to utilize the original infrastructure. Any thoughts?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78545





More information about the llvm-commits mailing list