[PATCH] D34989: test-release.sh: Fix phase2 and phase3 binary comparision
Tom Stellard via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Jul 4 11:10:19 PDT 2017
tstellar created this revision.
scudo_utils.cpp.o from compiler-rt had has one of the compiler's builtin
include paths stored in the .debug_line section. So we need to do
sed 's,Phase1,Phase2,g` on the Phase2 object file so it matches Phase3.
https://reviews.llvm.org/D34989
Files:
utils/release/test-release.sh
Index: utils/release/test-release.sh
===================================================================
--- utils/release/test-release.sh
+++ utils/release/test-release.sh
@@ -562,7 +562,7 @@
# case there are build paths in the debug info. On some systems,
# sed adds a newline to the output, so pass $p3 through sed too.
if ! cmp -s \
- <(env LC_CTYPE=C sed -e 's,Phase2,Phase3,g' $p2) \
+ <(env LC_CTYPE=C sed -e 's,Phase2,Phase3,g' -e 's,Phase1,Phase2,g' $p2) \
<(env LC_CTYPE=C sed -e '' $p3) 16 16; then
echo "file `basename $p2` differs between phase 2 and phase 3"
fi
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D34989.105194.patch
Type: text/x-patch
Size: 692 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170704/881e8043/attachment-0001.bin>
More information about the llvm-commits
mailing list