<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;" class="">Your ringing endorsement, coupled with the fact that the CI tests are passing, means this change is <b class="">approved</b> for 2.2. Joe, please feel free to cherry-pick the change to swift-2.2-branch.<div class=""><br class=""></div><div class="">Thanks!</div><div class="">- Joe<br class=""><div class=""><br class=""><div><blockquote type="cite" class=""><div class="">On Feb 9, 2016, at 2:43 PM, Slava Pestov <<a href="mailto:spestov@apple.com" class="">spestov@apple.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div class="">I don't claim to have any kind of deep understanding of IRGen, but this change makes sense to me.<br class=""><br class="">Slava<br class=""><br class=""><blockquote type="cite" class="">On Feb 9, 2016, at 1:12 PM, Joe Pamer <<a href="mailto:jpamer@apple.com" class="">jpamer@apple.com</a>> wrote:<br class=""><br class="">Thanks, Joe! The PR looks good to me, but would someone more familiar with IRGen mind taking a quick look? (Slava?)<br class=""><br class="">- Joe<br class=""><br class=""><blockquote type="cite" class="">On Feb 9, 2016, at 1:08 PM, Joe Groff <<a href="mailto:jgroff@apple.com" class="">jgroff@apple.com</a>> wrote:<br class=""><br class="">Hey Joe, I'm testing a fix for a crash regression reported on Swift 2.2 as <a href="rdar://problem/24568361" class="">rdar://problem/24568361</a>. I'd like to get this cherry-picked into the 2.2 branch.<br class=""><br class=""><a href="https://github.com/apple/swift/pull/1245" class="">https://github.com/apple/swift/pull/1245</a><br class=""><br class="">-Joe<br class=""><br class=""></blockquote><br class=""></blockquote><br class=""></div></div></blockquote></div><br class=""></div></div></body></html>