[PATCH] D115859: [LoongArch 2/6] Add ELF machine flag and relocs for upcoming LoongArch target

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 3 16:56:34 PST 2022


MaskRay added a subscriber: grimar.
MaskRay added inline comments.


================
Comment at: llvm/test/tools/llvm-readobj/ELF/reloc-types-elf-loongarch64.test:1
+## Test that llvm-readobj/llvm-readelf shows proper relocation type
+## names and values for loongarch64 target.
----------------
SixWeining wrote:
> jhenderson wrote:
> > You can drop "elf" from the title of this test, since it's already in the `ELF` subdirectory.
> Good suggestion. But we just follow other existing files naming method in the same directory. Maybe we can change all these files in a seperate patch later.
I will do the file renames for other `reloc-types-elf-*` tests (D71203) @grimar 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D115859



More information about the llvm-commits mailing list