[PATCH] D26935: [ELF] Allow `ASSERT` in output section descriptions
Meador Inge via llvm-commits
llvm-commits at lists.llvm.org
Tue Nov 22 09:02:57 PST 2016
meadori added inline comments.
================
Comment at: test/ELF/linkerscript/assert.s:33-38
+# RUN: echo "SECTIONS { \
+# RUN: .foo : { \
+# RUN: *(.foo) \
+# RUN: ASSERT(SIZEOF(.foo) == 8, \"true\"); \
+# RUN: } \
+# RUN: }" > %t6.script
----------------
ruiu wrote:
> Can you write this in one line? Look at other tests in this file.
Sure. I noticed the recent changes to the tests in this file and changed the test following this one to be on one line. I left this one alone b/c it was a bit longer. Is it OK to exceed 80 columns?
https://reviews.llvm.org/D26935
More information about the llvm-commits
mailing list