<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class="">Thanks! <div class=""><br class=""><div><blockquote type="cite" class=""><div class="">On Apr 3, 2017, at 9:22 PM, Hugh Bellamy <<a href="mailto:hughbellars@gmail.com" class="">hughbellars@gmail.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div dir="ltr" class="">Ah I see. I've sent in <a href="https://github.com/apple/swift/pull/8529" class="">https://github.com/apple/swift/pull/8529</a><div class=""><br class=""></div><div class="">Thanks</div><div class="">Hugh</div></div><div class="gmail_extra"><br class=""><div class="gmail_quote">On 4 April 2017 at 10:57, mishal_shah <span dir="ltr" class=""><<a href="mailto:mishal_shah@apple.com" target="_blank" class="">mishal_shah@apple.com</a>></span> wrote:<br class=""><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word" class="">We need to cherry-pick this fix (PR#8384) into swift-4.0-branch to avoid python lint failure. <div class=""><div class=""><br class=""></div><div class="">Thanks,</div><div class="">Mishal Shah<div class=""><div class="h5"><br class=""><div class=""><blockquote type="cite" class=""><div class="">On Apr 3, 2017, at 8:17 PM, Hugh Bellamy <<a href="mailto:hughbellars@gmail.com" target="_blank" class="">hughbellars@gmail.com</a>> wrote:</div><br class="m_2874186701482012075Apple-interchange-newline"><div class=""><div dir="ltr" class="">This issue was fixed in <a href="https://github.com/apple/swift/pull/8384" target="_blank" class="">https://github.com/apple/<wbr class="">swift/pull/8384</a>, I think.</div><div class="gmail_extra"><br class=""><div class="gmail_quote">On 4 April 2017 at 06:31, Vedant Kumar <span dir="ltr" class=""><<a href="mailto:vsk@apple.com" target="_blank" class="">vsk@apple.com</a>></span> wrote:<br class=""><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">This doesn't look related to my changes.<br class="">
<br class="">
+ Hugh, since he's at least touched the file.<br class="">
<br class="">
vedant<br class="">
<br class="">
> On Apr 3, 2017, at 4:29 PM, <a href="mailto:no-reply@swift.org" target="_blank" class="">no-reply@swift.org</a> wrote:<br class="">
><br class="">
> [FAILURE] oss-swift-4.0-incremental-RA-o<wbr class="">sx [#71]<br class="">
><br class="">
> Build URL: <a href="https://ci.swift.org/job/oss-swift-4.0-incremental-RA-osx/71/" rel="noreferrer" target="_blank" class="">https://ci.swift.org/job/oss-s<wbr class="">wift-4.0-incremental-RA-osx/71<wbr class="">/</a><br class="">
> Project: oss-swift-4.0-incremental-RA-o<wbr class="">sx<br class="">
> Date of build: Mon, 03 Apr 2017 15:39:24 -0700<br class="">
> Build duration: 49 min<br class="">
> Identified problems:<br class="">
><br class="">
> • Regression test failed: This build failed because a regression test in the test suite FAILed. Below is a list of all errors:<br class="">
> • Indication 1<br class="">
> Tests:<br class="">
><br class="">
> Name: Swift(macosx-x86_64)<br class="">
> Failed: 1 test(s), Passed: 9103 test(s), Total: 9104 test(s)<br class="">
> • Failed: Swift(macosx-x86_64).Python.gy<wbr class="">b.swift<br class="">
> Name: Swift-Unit<br class="">
> Failed: 0 test(s), Passed: 476 test(s), Total: 476 test(s)<br class="">
><br class="">
> Changes<br class="">
><br class="">
> • Commit f683c24ef812b985e0e93c0aaa9620<wbr class="">62677b0af4 by Vedant Kumar:<br class="">
> Revert "[InlineCost] Fix codesize issue caused by r286814"<br class="">
><br class="">
> • edit: test/Transforms/Inline/inline-<wbr class="">fast-math-flags.ll<br class="">
> • edit: test/Transforms/Inline/optimiz<wbr class="">ation-remarks-passed-yaml.ll<br class="">
> • edit: test/Transforms/Inline/inline-<wbr class="">cold.ll<br class="">
> • edit: test/Transforms/Inline/inline-<wbr class="">hot-callee.ll<br class="">
> • edit: lib/Analysis/InlineCost.cpp<br class="">
> • edit: test/Transforms/Inline/inline-<wbr class="">cold-callee.ll<br class="">
> • edit: test/Transforms/Inline/inline-<wbr class="">optsize.ll<br class="">
> • edit: test/Transforms/Inline/alloca-<wbr class="">bonus.ll<br class="">
> • edit: test/Transforms/Inline/ptr-dif<wbr class="">f.ll<br class="">
> • edit: test/Transforms/Inline/inline-<wbr class="">hot-callsite.ll<br class="">
> • edit: test/Transforms/Inline/inline_<wbr class="">unreachable-2.ll<br class="">
><br class="">
> • Commit 57278f9a2b853e79b47aaff61b25a9<wbr class="">52aade98f8 by Vedant Kumar:<br class="">
> Revert "[Inliner] Tweak inline threshold for code size regression"<br class="">
><br class="">
> • edit: test/Transforms/Inline/ptr-dif<wbr class="">f.ll<br class="">
><br class="">
> • Commit e3511601a3c036006fab2becb4802b<wbr class="">8f27055f02 by Vedant Kumar:<br class="">
> Revert "[InlineCost] Remove skew when calculating call costs"<br class="">
><br class="">
> • edit: test/Transforms/Inline/inline_<wbr class="">unreachable-2.ll<br class="">
> • edit: test/Transforms/Inline/ptr-dif<wbr class="">f.ll<br class="">
> • edit: test/Transforms/Inline/optimiz<wbr class="">ation-remarks-passed-yaml.ll<br class="">
> • edit: test/Transforms/Inline/inline-<wbr class="">hot-callsite.ll<br class="">
> • edit: test/Transforms/Inline/inline-<wbr class="">optsize.ll<br class="">
> • edit: test/Transforms/Inline/inline-<wbr class="">cold.ll<br class="">
> • edit: test/Transforms/Inline/alloca-<wbr class="">bonus.ll<br class="">
> • edit: lib/Analysis/InlineCost.cpp<br class="">
> • edit: test/Transforms/Inline/inline-<wbr class="">cold-callee.ll<br class="">
> • edit: test/Transforms/Inline/inline-<wbr class="">hot-callee.ll<br class="">
><br class="">
> • Commit 022afe34b9c86521a7d17bcd75811b<wbr class="">6f1c39daba by Vedant Kumar:<br class="">
> Revert "[InlineCost] Reduce inline thresholds to compensate for cost<br class="">
><br class="">
> • edit: test/Transforms/Inline/inline-<wbr class="">fp.ll<br class="">
> • edit: include/llvm/Analysis/InlineCo<wbr class="">st.h<br class="">
> • edit: test/Transforms/Inline/ephemer<wbr class="">al.ll<br class="">
<br class="">
</blockquote></div><br class=""></div>
</div></blockquote></div><br class=""></div></div></div></div></div></blockquote></div><br class=""></div>
</div></blockquote></div><br class=""></div></body></html>