<div class="socmaildefaultfont" dir="ltr" style="font-family:Arial, Helvetica, sans-serif;font-size:10.5pt" ><div dir="ltr" >Hi Alex,</div>
<div dir="ltr" > </div>
<div dir="ltr" >I'm able to confirm locally this commit did break that test on several of the build bots. Can you have a look.</div>
<div dir="ltr" > </div>
<div dir="ltr" >Thanks<br>Sean</div>
<div dir="ltr" > </div>
<blockquote data-history-content-modified="1" dir="ltr" style="border-left:solid #aaaaaa 2px; margin-left:5px; padding-left:5px; direction:ltr; margin-right:0px" >----- Original message -----<br>From: Peter Collingbourne via llvm-commits <llvm-commits@lists.llvm.org><br>Sent by: "llvm-commits" <llvm-commits-bounces@lists.llvm.org><br>To: Alex Shlyapnikov <alekseys@google.com><br>Cc: llvm-commits <llvm-commits@lists.llvm.org><br>Subject: Re: [compiler-rt] r330458 - [Sanitizer] Internal Printf string precision argument + padding.<br>Date: Fri, Apr 20, 2018 3:49 PM<br>
<div dir="ltr" >Could this have caused this test failure?
<div><a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__lab.llvm.org-3A8011_builders_clang-2Dppc64le-2Dlinux-2Dlnt_builds_11600&d=DwMFaQ&c=jf_iaSHvJObTbx-siA1ZOg&r=vE89RddfSAMwAmW9D-c51sqQCl8tXHenTEyBrBeQeeA&m=44j85w_M0UtdfV9zXiUc1cz71_CmTtBnXRN6TbwI9w4&s=WOYEvuKj2KZu3HxHogFRQCE8Affd3GERRdU53Ne8erE&e=" target="_blank" >http://lab.llvm.org:8011/builders/clang-ppc64le-linux-lnt/builds/11600</a></div>
<div> </div>
<div>Peter</div></div>
<div>
<div>On Fri, Apr 20, 2018 at 11:03 AM, Alex Shlyapnikov via llvm-commits <span dir="ltr" ><<a href="mailto:llvm-commits@lists.llvm.org" target="_blank" >llvm-commits@lists.llvm.org</a>></span> wrote:
<blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex" >Author: alekseyshl<br>Date: Fri Apr 20 11:03:10 2018<br>New Revision: 330458<br><br>URL: <a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__llvm.org_viewvc_llvm-2Dproject-3Frev-3D330458-26view-3Drev&d=DwMFaQ&c=jf_iaSHvJObTbx-siA1ZOg&r=vE89RddfSAMwAmW9D-c51sqQCl8tXHenTEyBrBeQeeA&m=44j85w_M0UtdfV9zXiUc1cz71_CmTtBnXRN6TbwI9w4&s=lxId1HemfQ5GrW9BpDVvqiAQvCXvOha2MaGtuTfvDAc&e=" rel="noreferrer" target="_blank" >http://llvm.org/viewvc/llvm-<wbr>project?rev=330458&view=rev</a><br>Log:<br>[Sanitizer] Internal Printf string precision argument + padding.<br><br>Summary:<br>Example:<br> Printf("%.*s", 5, "123");<br>should yield:<br> '123 '<br><br>In case Printf's requested string precision is larger than the string<br>argument, the resulting string should be padded up to the requested<br>precision.<br><br>For the simplicity sake, implementing right padding only.<br><br>Reviewers: eugenis<br><br>Subscribers: kubamracek, delcypher, #sanitizers, llvm-commits<br><br>Differential Revision: <a href="https://urldefense.proofpoint.com/v2/url?u=https-3A__reviews.llvm.org_D45844&d=DwMFaQ&c=jf_iaSHvJObTbx-siA1ZOg&r=vE89RddfSAMwAmW9D-c51sqQCl8tXHenTEyBrBeQeeA&m=44j85w_M0UtdfV9zXiUc1cz71_CmTtBnXRN6TbwI9w4&s=1JWiVkL4I1OpNJ7Tmg0vIJfAhySHUnuIiBBHZ03Ea7M&e=" rel="noreferrer" target="_blank" >https://reviews.llvm.org/<wbr>D45844</a><br><br>Modified:<br> compiler-rt/trunk/lib/<wbr>sanitizer_common/sanitizer_<wbr>printf.cc<br> compiler-rt/trunk/lib/<wbr>sanitizer_common/tests/<wbr>sanitizer_printf_test.cc<br><br>Modified: compiler-rt/trunk/lib/<wbr>sanitizer_common/sanitizer_<wbr>printf.cc<br>URL: <a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__llvm.org_viewvc_llvm-2Dproject_compiler-2Drt_trunk_lib_sanitizer-5Fcommon_sanitizer-5Fprintf.cc-3Frev-3D330458-26r1-3D330457-26r2-3D330458-26view-3Ddiff&d=DwMFaQ&c=jf_iaSHvJObTbx-siA1ZOg&r=vE89RddfSAMwAmW9D-c51sqQCl8tXHenTEyBrBeQeeA&m=44j85w_M0UtdfV9zXiUc1cz71_CmTtBnXRN6TbwI9w4&s=IFA6iXTsxuFg_XK7hML_0U6MwAli8UPyQHvFogchclo&e=" rel="noreferrer" target="_blank" >http://llvm.org/viewvc/llvm-<wbr>project/compiler-rt/trunk/lib/<wbr>sanitizer_common/sanitizer_<wbr>printf.cc?rev=330458&r1=<wbr>330457&r2=330458&view=diff</a><br>==============================<wbr>==============================<wbr>==================<br>--- compiler-rt/trunk/lib/<wbr>sanitizer_common/sanitizer_<wbr>printf.cc (original)<br>+++ compiler-rt/trunk/lib/<wbr>sanitizer_common/sanitizer_<wbr>printf.cc Fri Apr 20 11:03:10 2018<br>@@ -105,6 +105,8 @@ static int AppendString(char **buff, con<br> break;<br> result += AppendChar(buff, buff_end, *s);<br> }<br>+ while (result < precision)<br>+ result += AppendChar(buff, buff_end, ' ');<br> return result;<br> }<br><br><br>Modified: compiler-rt/trunk/lib/<wbr>sanitizer_common/tests/<wbr>sanitizer_printf_test.cc<br>URL: <a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__llvm.org_viewvc_llvm-2Dproject_compiler-2Drt_trunk_lib_sanitizer-5Fcommon_tests_sanitizer-5Fprintf-5Ftest.cc-3Frev-3D330458-26r1-3D330457-26r2-3D330458-26view-3Ddiff&d=DwMFaQ&c=jf_iaSHvJObTbx-siA1ZOg&r=vE89RddfSAMwAmW9D-c51sqQCl8tXHenTEyBrBeQeeA&m=44j85w_M0UtdfV9zXiUc1cz71_CmTtBnXRN6TbwI9w4&s=iw8w9Bs5nJgxe1QZNAMZaZTdrtSPXLnsQWN2xTlUUGI&e=" rel="noreferrer" target="_blank" >http://llvm.org/viewvc/llvm-<wbr>project/compiler-rt/trunk/lib/<wbr>sanitizer_common/tests/<wbr>sanitizer_printf_test.cc?rev=<wbr>330458&r1=330457&r2=330458&<wbr>view=diff</a><br>==============================<wbr>==============================<wbr>==================<br>--- compiler-rt/trunk/lib/<wbr>sanitizer_common/tests/<wbr>sanitizer_printf_test.cc (original)<br>+++ compiler-rt/trunk/lib/<wbr>sanitizer_common/tests/<wbr>sanitizer_printf_test.cc Fri Apr 20 11:03:10 2018<br>@@ -146,8 +146,13 @@ TEST(Printf, Precision) {<br> EXPECT_EQ(3U, len);<br> EXPECT_STREQ("123", buf);<br> len = internal_snprintf(buf, sizeof(buf), "%.*s", 6, "12345");<br>- EXPECT_EQ(5U, len);<br>- EXPECT_STREQ("12345", buf);<br>+ EXPECT_EQ(6U, len);<br>+ EXPECT_STREQ("12345 ", buf);<br>+ // CHeck that precision does not overflow the smaller buffer, although<br>+ // 10 chars is requested, it stops at the buffer size, 8.<br>+ len = internal_snprintf(buf, 8, "%.*s", 10, "12345");<br>+ EXPECT_EQ(10U, len); // The required size reported.<br>+ EXPECT_STREQ("12345 ", buf);<br> }<br><br> } // namespace __sanitizer<br><br><br>______________________________<wbr>_________________<br>llvm-commits mailing list<br><a href="mailto:llvm-commits@lists.llvm.org" target="_blank" >llvm-commits@lists.llvm.org</a><br><a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__lists.llvm.org_cgi-2Dbin_mailman_listinfo_llvm-2Dcommits&d=DwMFaQ&c=jf_iaSHvJObTbx-siA1ZOg&r=vE89RddfSAMwAmW9D-c51sqQCl8tXHenTEyBrBeQeeA&m=44j85w_M0UtdfV9zXiUc1cz71_CmTtBnXRN6TbwI9w4&s=d3_I3Nd6o1aLgPk-G3iqFkx4zT62kc-ySwpYPzzY-XY&e=" rel="noreferrer" target="_blank" >http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/llvm-commits</a></blockquote></div>
<div> </div>--
<div data-smartmail="gmail_signature" ><div dir="ltr" >--
<div>Peter</div></div></div></div>
<div><font size="2" face="Default Monospace,Courier New,Courier,monospace" >_______________________________________________<br>llvm-commits mailing list<br>llvm-commits@lists.llvm.org<br><a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__lists.llvm.org_cgi-2Dbin_mailman_listinfo_llvm-2Dcommits&d=DwIGaQ&c=jf_iaSHvJObTbx-siA1ZOg&r=vE89RddfSAMwAmW9D-c51sqQCl8tXHenTEyBrBeQeeA&m=44j85w_M0UtdfV9zXiUc1cz71_CmTtBnXRN6TbwI9w4&s=d3_I3Nd6o1aLgPk-G3iqFkx4zT62kc-ySwpYPzzY-XY&e=" target="_blank" >https://urldefense.proofpoint.com/v2/url?u=http-3A__lists.llvm.org_cgi-2Dbin_mailman_listinfo_llvm-2Dcommits&d=DwIGaQ&c=jf_iaSHvJObTbx-siA1ZOg&r=vE89RddfSAMwAmW9D-c51sqQCl8tXHenTEyBrBeQeeA&m=44j85w_M0UtdfV9zXiUc1cz71_CmTtBnXRN6TbwI9w4&s=d3_I3Nd6o1aLgPk-G3iqFkx4zT62kc-ySwpYPzzY-XY&e=</a></font></div></blockquote>
<div dir="ltr" > </div></div><BR>