<div dir="ltr">(A full patch works too if it's easier, as long as it applies cleanly on ToT)</div><br><div class="gmail_quote"><div dir="ltr">On Wed, Nov 16, 2016 at 11:46 AM Zachary Turner <<a href="mailto:zturner@google.com">zturner@google.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr" class="gmail_msg">What revision should I apply this on top of?  It doesn't apply cleanly for me at tip.  I think it's my fault because of the original patch, but now I'm having trouble getting this modified patch to work.  Can you give me a diff that I can apply on top of my original patch, which is applied on a clean repo?  So ToT > ::my original patch:: > ::your set of fixes::, and just give me your set of fixes?</div><br class="gmail_msg"><div class="gmail_quote gmail_msg"><div dir="ltr" class="gmail_msg">On Wed, Nov 16, 2016 at 11:36 AM Zachary Turner <<a href="mailto:zturner@google.com" class="gmail_msg" target="_blank">zturner@google.com</a>> wrote:<br class="gmail_msg"></div><blockquote class="gmail_quote gmail_msg" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr" class="gmail_msg">Sweet.  Glad to see that such a large patch had only minor implications.  Thanks for the help!</div><br class="gmail_msg"><div class="gmail_quote gmail_msg"><div dir="ltr" class="gmail_msg">On Wed, Nov 16, 2016 at 11:28 AM Todd Fiala <<a href="mailto:todd.fiala@gmail.com" class="gmail_msg" target="_blank">todd.fiala@gmail.com</a>> wrote:<br class="gmail_msg"></div><blockquote class="gmail_quote gmail_msg" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">tfiala accepted this revision.<br class="gmail_msg">
tfiala added a comment.<br class="gmail_msg">
This revision is now accepted and ready to land.<br class="gmail_msg">
<br class="gmail_msg">
Here is the adjusted patch that fixes the issue I was seeing on TestTerminal.py.  My first set of changes had a lifetime issue where I needed a const char* that was synthesized on the fly and went away by the time I needed it.<br class="gmail_msg">
<br class="gmail_msg">
LGTM on macOS with this patch applied:<br class="gmail_msg">
F2590224: D26698_macos_build_fixes_v2.diff <<a href="https://reviews.llvm.org/F2590224" rel="noreferrer" class="gmail_msg" target="_blank">https://reviews.llvm.org/F2590224</a>><br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<a href="https://reviews.llvm.org/D26698" rel="noreferrer" class="gmail_msg" target="_blank">https://reviews.llvm.org/D26698</a><br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
</blockquote></div></blockquote></div></blockquote></div>