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

James Clarke via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 6 05:50:08 PDT 2019


jrtc27 added a comment.

Hm, actually, looking more closely, there seems to be inconsistency. Some backends use FooELFStreamer.cpp for an MCELFStreamer, and others for a ELF-specific target streamer. Some, like AArch64, even put both in the same file, so if you don't like this patch, I suppose the recommendation would be to do that?


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