<html><head><meta http-equiv="Content-Type" content="text/html charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;">Hi Rafael,<div><br></div><div>This breaks a buildbot:</div><div><a href="http://lab.llvm.org:8013/builders/clang-x86_64-darwin11-nobootstrap-RAincremental/builds/4327">http://lab.llvm.org:8013/builders/clang-x86_64-darwin11-nobootstrap-RAincremental/builds/4327</a></div><div><pre style="font-family: 'Courier New', courier, monotype; font-size: medium;"><span class="stdout">******************** TEST 'LLVM :: Object/archive-format.test' FAILED ********************
Script:
--
mkdir -p /Users/buildslave/zorg/buildbot/smooshlab/slave-0.8/build.clang-x86_64-darwin11-nobootstrap-RAincremental/clang-build/test/Object/Output/archive-format.test.tmp
cd /Users/buildslave/zorg/buildbot/smooshlab/slave-0.8/build.clang-x86_64-darwin11-nobootstrap-RAincremental/clang-build/test/Object/Output/archive-format.test.tmp
echo bar > 0123456789abcde
echo zed > 0123456789abcdef
rm -f test.a
/Users/buildslave/zorg/buildbot/smooshlab/slave-0.8/build.clang-x86_64-darwin11-nobootstrap-RAincremental/clang-build/Release+Asserts/bin/llvm-ar rc test.a 0123456789abcde 0123456789abcdef
cat test.a | /Users/buildslave/zorg/buildbot/smooshlab/slave-0.8/build.clang-x86_64-darwin11-nobootstrap-RAincremental/clang-build/Release+Asserts/bin/FileCheck -strict-whitespace /Users/buildslave/zorg/buildbot/smooshlab/slave-0.8/build.clang-x86_64-darwin11-nobootstrap-RAincremental/llvm/test/Object/archive-format.test
--
Exit Code: 1
Command Output (stderr):
--
/Users/buildslave/zorg/buildbot/smooshlab/slave-0.8/build.clang-x86_64-darwin11-nobootstrap-RAincremental/llvm/test/Object/archive-format.test:19:13: error: expected string not found in input
CHECK-NEXT: 0123456789abcde/{{.*}} 1000 1000 100664 4 `
^
<stdin>:4:1: note: scanning from here
0123456789abcde/1373668379 501 20 100644 4 `
^
--
********************</span></pre><div><br></div><div><br></div><div apple-content-edited="true">
<div style="color: rgb(0, 0, 0); font-family: Helvetica; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-align: -webkit-auto; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; ">-Quentin</div>
</div>
<br><div><div>On Jul 12, 2013, at 3:22 PM, Rafael Espindola <<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div style="font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;">Author: rafael<br>Date: Fri Jul 12 17:22:34 2013<br>New Revision: 186216<br><br>URL:<span class="Apple-converted-space"> </span><a href="http://llvm.org/viewvc/llvm-project?rev=186216&view=rev">http://llvm.org/viewvc/llvm-project?rev=186216&view=rev</a><br>Log:<br>Fix a off by one error about which members need to use the string table.<br><br>Added:<br> llvm/trunk/test/Object/archive-format.test<br>Modified:<br> llvm/trunk/tools/llvm-ar/llvm-ar.cpp<br><br>Added: llvm/trunk/test/Object/archive-format.test<br>URL:<span class="Apple-converted-space"> </span><a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Object/archive-format.test?rev=186216&view=auto">http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Object/archive-format.test?rev=186216&view=auto</a><br>==============================================================================<br>--- llvm/trunk/test/Object/archive-format.test (added)<br>+++ llvm/trunk/test/Object/archive-format.test Fri Jul 12 17:22:34 2013<br>@@ -0,0 +1,22 @@<br>+Test the exact archive format. In particular, test which file names use the<br>+string table or not.<br>+<br>+REQUIRES: shell<br>+<br>+RUN: mkdir -p %t<br>+RUN: cd %t<br>+<br>+RUN: echo bar > 0123456789abcde<br>+RUN: echo zed > 0123456789abcdef<br>+<br>+RUN: rm -f test.a<br>+RUN: llvm-ar rc test.a 0123456789abcde 0123456789abcdef<br>+RUN: cat test.a | FileCheck -strict-whitespace %s<br>+<br>+CHECK: !<arch><br>+CHECK-NEXT: // 18 `<br>+CHECK-NEXT: 0123456789abcdef/<br>+CHECK-NEXT: 0123456789abcde/{{.*}} 1000 1000 100664 4 `<br>+CHECK-NEXT: bar<br>+CHECK-NEXT: /0 {{.*}} 1000 1000 100664 4 `<br>+CHECK-NEXT: zed<br><br>Modified: llvm/trunk/tools/llvm-ar/llvm-ar.cpp<br>URL:<span class="Apple-converted-space"> </span><a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/llvm-ar/llvm-ar.cpp?rev=186216&r1=186215&r2=186216&view=diff">http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/llvm-ar/llvm-ar.cpp?rev=186216&r1=186215&r2=186216&view=diff</a><br>==============================================================================<br>--- llvm/trunk/tools/llvm-ar/llvm-ar.cpp (original)<br>+++ llvm/trunk/tools/llvm-ar/llvm-ar.cpp Fri Jul 12 17:22:34 2013<br>@@ -419,7 +419,7 @@ StringRef NewArchiveIterator::getNew() c<br>template <typename T><br>void addMember(std::vector<NewArchiveIterator> &Members,<br> std::string &StringTable, T I, StringRef Name) {<br>- if (Name.size() < 15) {<br>+ if (Name.size() < 16) {<br> NewArchiveIterator NI(I, Twine(Name) + "/");<br> Members.push_back(NI);<br> } else {<br><br><br>_______________________________________________<br>llvm-commits mailing list<br><a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br><a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a></div></blockquote></div><br></div></body></html>