[PATCH] D123515: [RISCV] Support '.option arch' directive

Aditya Kumar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 25 12:19:33 PDT 2023


hiraditya added a comment.

In D123515#4372698 <https://reviews.llvm.org/D123515#4372698>, @craig.topper wrote:

> Looks like reducing the Streamer interface from 3 down to 1 would require refactoring the code to collect the information before sending it to the streamer. I suggest this be moved to a future patch for expediency of getting this landed since the code has already been reviewed. As long as we're the only target using it, it doesn't seem hard to change. I'm willing to help @StephenFan improve this post-commit.

SGTM, Thanks for offering to help Craig!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D123515



More information about the llvm-commits mailing list