[PATCH] D53645: Include input section name and output section name in an error message.

[Rui Ueyama via Phabricator via llvm-commits]

ruiu updated this revision to Diff 170887.
ruiu added a comment.

- rebase


https://reviews.llvm.org/D53645

Files:
  lld/ELF/Writer.cpp
  lld/test/ELF/execute-only-mixed-data.s


Index: lld/test/ELF/execute-only-mixed-data.s
===================================================================
--- lld/test/ELF/execute-only-mixed-data.s
+++ lld/test/ELF/execute-only-mixed-data.s
@@ -16,7 +16,7 @@
 // RUN: }" > %t.lds
 // RUN: ld.lld -T%t.lds %t.o -o %t -execute-only 2>&1
 
-// CHECK: -execute-only does not support intermingling data and code
+// CHECK: cannot place {{.*}}:(.rodata.foo) into .text: -execute-only does not support intermingling data and code
 
     br lr
 
Index: lld/ELF/Writer.cpp
===================================================================
--- lld/ELF/Writer.cpp
+++ lld/ELF/Writer.cpp
@@ -1759,7 +1759,8 @@
     if (OS->Flags & SHF_EXECINSTR)
       for (InputSection *IS : getInputSections(OS))
         if (!(IS->Flags & SHF_EXECINSTR))
-          error("-execute-only does not support intermingling data and code");
+          error("cannot place " + toString(IS) + " into " + toString(OS->Name) +
+                ": -execute-only does not support intermingling data and code");
 }
 
 // The linker is expected to define SECNAME_start and SECNAME_end


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D53645.170887.patch
Type: text/x-patch
Size: 1107 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20181024/38f2a0e9/attachment.bin>