[PATCH] D41658: [RISCV] Encode RISCV specific ELF e_flags to RISCV Binary by RISCVTargetStreamer

[Shiva Chen via Phabricator via llvm-commits]

shiva0217 added inline comments.


================
Comment at: lib/Target/RISCV/MCTargetDesc/RISCVTargetStreamer.h:22
+
+// This part is for ELF object output
+class RISCVTargetELFStreamer : public RISCVTargetStreamer {
----------------
asb wrote:
> Nitpick: full-stop after comment.
> 
> Does the comment add value? Maybe just delete it.
Remove the redundant comment


================
Comment at: test/MC/RISCV/elf-flags.s:1
+# RUN: llvm-mc -mattr=+c -filetype=obj < %s | llvm-readobj -file-headers - | FileCheck --check-prefix=ExtC %s
+
----------------
asb wrote:
> Need to set the target for llvm-mc.
> 
> Should probably have RUN lines for both riscv32 and riscv64 as many of the other test/MC/RISCV tests do.
Run the test case for riscv32 and riscv64 target.


Repository:
  rL LLVM

https://reviews.llvm.org/D41658