[PATCH] D61866: [RISCV] Rename RISCVELFStreamer.{cpp,h} to RISCVTargetELFStreamer.{cpp,h}

Alex Bradbury via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 6 05:39:13 PDT 2019


asb requested changes to this revision.
asb added a comment.
This revision now requires changes to proceed.
Herald added a subscriber: Jim.

I think this is less confusing, but there's more to be gained by having a similar naming scheme to the other backends which right now all use FooELFStreamer.{cpp,h}.

I you feel strongly about this change, I'd consider making it for all in-tree backends.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D61866





More information about the llvm-commits mailing list