[PATCH] D34857: [COFF, ARM64] Add initial relocation types

Mandeep Singh Grang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Jul 16 17:01:39 PDT 2017


mgrang marked 3 inline comments as done.
mgrang added a comment.

I have addressed comments, and my previous patch was accepted by @compnerd. So unless someone has any more comments I will go ahead and commit this.



================
Comment at: test/MC/AArch64/coff-relocations.s:2
+; RUN: llvm-mc -triple aarch64-windows -filetype obj -o - %s | \
+; RUN: llvm-readobj -r - | FileCheck %s
+
----------------
compnerd wrote:
> Don't think that you need the second `RUN:` here.
Without the second RUN, this line is reported as malformed. I checked that a lot of the other tests also use a similar format of multiple RUNs.


Repository:
  rL LLVM

https://reviews.llvm.org/D34857





More information about the llvm-commits mailing list