[PATCH] D78422: [LLD][ELF][ARM] Fix ARM Exidx order for non monotonic section order

Peter Smith via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 22 14:09:37 PDT 2020


psmith marked an inline comment as done.
psmith added a comment.

Thanks, and apologies for forgetting George's comment request. I'll upload a new version and commit tomorrow.



================
Comment at: lld/test/ELF/arm-exidx-script-order.s:1
+// RUN: llvm-mc --arm-add-build-attributes --triple=armv7a-linux-gnueabihf -filetype=obj %s -o %t.o
+// RUN: echo "SECTIONS { \
----------------
MaskRay wrote:
> `// REQUIRES: ARM`
I think it has to be all lower case (all other REQUIRES are lower case). If I use ARM then llvm-lit won't recognise it.


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

https://reviews.llvm.org/D78422





More information about the llvm-commits mailing list