[PATCH] D93044: [llvm-readobj/elf] - AArch64: Handle AARCH64_VARIANT_PCS for GNUStyle

George Rimar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 14 23:48:02 PST 2020


grimar added inline comments.


================
Comment at: llvm/test/tools/llvm-readobj/ELF/aarch64-symbols-stother.test:12
+# LLVM:      Other [ (0xC0)
+# LLVM-NEXT: ]
+
----------------
grimar wrote:
> This output looks a bit wrong.
> See the output from `mips-symbols-stother.test`:
> 
> ```
> #      MIPS-LLVM:Name: foo
> #      MIPS-LLVM:Other [
> # MIPS-LLVM-NEXT:  STO_MIPS_MICROMIPS (0x80)
> # MIPS-LLVM-NEXT:  STO_MIPS_OPTIONAL (0x4)
> # MIPS-LLVM-NEXT:  STO_MIPS_PIC (0x20)
> # MIPS-LLVM-NEXT:  STO_MIPS_PLT (0x8)
> # MIPS-LLVM-NEXT:]
> 
> #      MIPS-LLVM:Name: bar
> #      MIPS-LLVM:Other [
> # MIPS-LLVM-NEXT:  STO_MIPS_MIPS16 (0xF0)
> # MIPS-LLVM-NEXT:]
> 
> ```
> 
> I think you need to update `LLVMStyle<ELFT>::printSymbol` too.
This wasn't addressed?
You did the change in `GNUStyle<ELFT>::printSymbol`, it is responsible for `llvm-readelf` output.
If you want to change the `llvm-readobj`, then you need to update the `LLVMStyle<ELFT>::printSymbol` too.


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

https://reviews.llvm.org/D93044



More information about the llvm-commits mailing list