[PATCH] D64827: [Xtensa 2/10] Add Xtensa ELF definitions.

Andrei Safronov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 25 17:40:07 PDT 2021


andreisfr marked 2 inline comments as done.
andreisfr added inline comments.


================
Comment at: llvm/test/tools/llvm-readobj/ELF/reloc-types-elf-xtensa.test:1
+## Test that llvm-readobj/llvm-readelf shows proper relocation type
+## names and values for xtensa target.
----------------
jhenderson wrote:
> jhenderson wrote:
> > No need for "elf" to be in the file name, since it's in the ELF subdirectory.
> "elf" is still in the file name. Please remove it.
I changed file name to reloc-types-xtensa.test


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

https://reviews.llvm.org/D64827



More information about the llvm-commits mailing list