[PATCH] D111939: [RISCV] Add missing e_flag R_RISCV_TSO

Jessica Clarke via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Oct 16 07:21:46 PDT 2021


jrtc27 added a comment.

Commit message needs s/R_/EF_/



================
Comment at: llvm/test/Object/RISCV/elf-flags.yaml:26
+  Flags:           [ EF_RISCV_RVC, EF_RISCV_FLOAT_ABI_DOUBLE, EF_RISCV_RVE, EF_RISCV_TSO ]
\ No newline at end of file

----------------
Please fix


================
Comment at: llvm/tools/llvm-readobj/ELFDumper.cpp:1566-1571
+    ENUM_ENT(EF_RISCV_RVC, "RVC"),
+    ENUM_ENT(EF_RISCV_FLOAT_ABI_SINGLE, "single-float ABI"),
+    ENUM_ENT(EF_RISCV_FLOAT_ABI_DOUBLE, "double-float ABI"),
+    ENUM_ENT(EF_RISCV_FLOAT_ABI_QUAD, "quad-float ABI"),
+    ENUM_ENT(EF_RISCV_RVE, "RVE"),
+    ENUM_ENT(EF_RISCV_TSO, "TSO")};
----------------
Keep formatting consistent with the surroundings


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D111939



More information about the llvm-commits mailing list