[PATCH] D79545: [VE] Implements minimum MC layer for VE (3/4)

Kazushi Marukawa via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 7 01:41:38 PDT 2020


kaz7 planned changes to this revision.
kaz7 marked an inline comment as done.
kaz7 added a comment.

Thank you for your comments and examples for regression tests.  I'll add them.



================
Comment at: llvm/include/llvm/BinaryFormat/ELFRelocs/VE.def:36
+ELF_RELOC(R_VE_DTPOFF64,     23)
+// ELF_RELOC(R_VE_TPOFF64,     24)
+ELF_RELOC(R_VE_TLS_GD_HI32,  25)
----------------
jhenderson wrote:
> Why are some commented out?
I left them commented out because documents left them as commented out.  This time, I put everything defined in manual.  I can leave items what I use.  In order to implement test for all of them, I have to do so...

https://www.hpc.nec/documents/guide/pdfs/VE-ABI_v2.1.pdf   section 4.4.1
https://www.hpc.nec/documents/guide/pdfs/VE-tls_v1.2.pdf   section 5.1


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D79545





More information about the llvm-commits mailing list