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

Hans Wennborg via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 25 08:57:50 PDT 2017


Yes, please.

On Mon, Jul 24, 2017 at 12:34 PM, Tom Stellard <tstellar at redhat.com> wrote:
> 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
>>
>


More information about the llvm-commits mailing list