[PATCH] D25846: scripts: Document existing problems with the symbolizer build script.

[Peter Collingbourne via llvm-commits]

This revision was automatically updated to reflect the committed changes.
Closed by commit rL285013: scripts: Document existing problems with the symbolizer build script. (authored by pcc).

Changed prior to commit:
  https://reviews.llvm.org/D25846?vs=75361&id=75640#toc

Repository:
  rL LLVM

https://reviews.llvm.org/D25846

Files:
  compiler-rt/trunk/lib/sanitizer_common/symbolizer/scripts/build_symbolizer.sh


Index: compiler-rt/trunk/lib/sanitizer_common/symbolizer/scripts/build_symbolizer.sh
===================================================================
--- compiler-rt/trunk/lib/sanitizer_common/symbolizer/scripts/build_symbolizer.sh
+++ compiler-rt/trunk/lib/sanitizer_common/symbolizer/scripts/build_symbolizer.sh
@@ -18,6 +18,12 @@
 #
 # The script will modify the output directory which is given as the first
 # argument to the script.
+#
+# FIXME: We should really be using a simpler approach to building this object
+# file, and it should be available as a regular cmake rule. Conceptually, we
+# want to be doing "ld -r" followed by "objcopy -G" to create a relocatable
+# object file with only our entry points exposed. However, this does not work at
+# present, see PR30750.
 
 SCRIPT_DIR="$(cd "$(dirname "$0")" && pwd)"
 SRC_DIR=$(readlink -f $SCRIPT_DIR/..)


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D25846.75640.patch
Type: text/x-patch
Size: 871 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20161024/92d7958c/attachment.bin>