[llvm] r308912 - test-release.sh: Fix phase2 and phase3 binary comparision

[Tom Stellard via llvm-commits]

Hi Hans,

Is this OK to merge to the 5.0 branch?

-Tom

On 07/24/2017 03:28 PM, Tom Stellard via llvm-commits wrote:
> Author: tstellar
> Date: Mon Jul 24 12:28:30 2017
> New Revision: 308912
> 
> URL: http://llvm.org/viewvc/llvm-project?rev=308912&view=rev
> Log:
> test-release.sh: Fix phase2 and phase3 binary comparision
> 
> Summary:
> scudo_utils.cpp.o from compiler-rt has one of the host 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.
> 
> Reviewers: hans
> 
> Reviewed By: hans
> 
> Subscribers: llvm-commits
> 
> Differential Revision: https://reviews.llvm.org/D34989
> 
> Modified:
>     llvm/trunk/utils/release/test-release.sh
> 
> Modified: llvm/trunk/utils/release/test-release.sh
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/utils/release/test-release.sh?rev=308912&r1=308911&r2=308912&view=diff
> ==============================================================================
> --- llvm/trunk/utils/release/test-release.sh (original)
> +++ llvm/trunk/utils/release/test-release.sh Mon Jul 24 12:28:30 2017
> @@ -562,7 +562,7 @@ for Flavor in $Flavors ; do
>              # 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
> 
> 
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
>