[PATCH] D63191: [lld][ELF] Check length of subsection in .ARM.attributes

Rui Ueyama via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 13 01:55:42 PDT 2019


ruiu accepted this revision.
ruiu added a comment.
This revision is now accepted and ready to land.

LGTM with the following changes.



================
Comment at: tools/lld/test/ELF/bad-arm-attributes.s:1
+# RUN: llvm-mc -filetype=obj -triple=arm-unknown-linux %s -o %t
+# RUN: ld.lld %t -o %tout 2>&1 | FileCheck %s
----------------
Please move this to tools/lld/test/ELF/invalid directory.


================
Comment at: tools/lld/test/ELF/bad-arm-attributes2.s:1
+# RUN: llvm-mc -filetype=obj -triple=arm-unknown-linux %s -o %t
+# RUN: ld.lld %t -o %tout 2>&1 | FileCheck %s
----------------
Ditto


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

https://reviews.llvm.org/D63191





More information about the llvm-commits mailing list