LGTM<span></span><br><br>On Friday, February 13, 2015, Matthias Braun <<a href="mailto:mbraun@apple.com">mbraun@apple.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word"><div>ping</div><br><div><blockquote type="cite"><div>On Jan 26, 2015, at 10:25 AM, Matthias Braun <<a href="javascript:_e(%7B%7D,'cvml','mbraun@apple.com');" target="_blank">mbraun@apple.com</a>> wrote:</div><br><div><div style="word-wrap:break-word"><div>Thanks for the review. I updated the messages-2 patch as suggested.</div><div><br></div><div>- Matthias</div><div><br></div><div></div></div><span><0001-Avoid-using-NSSize-which-is-not-available-on-iOS.patch></span><div style="word-wrap:break-word"><div></div><br><div><blockquote type="cite"><div>On Jan 23, 2015, at 4:16 PM, Daniel Dunbar <<a href="javascript:_e(%7B%7D,'cvml','daniel@zuster.org');" target="_blank">daniel@zuster.org</a>> wrote:</div><br><div><div dir="ltr">The patch to messages-2.m should probably just substitute an inline definition of a size class, IIRC (and its been a long time) the test is trying to explicitly check a particular structure layout, so it would be better to just hardcode the definition as:<div>--</div><div>typedef struct {</div><div>  float width, height;</div><div>} MySize;</div><div>--</div><div>since this test was originally intended for testing the 32-bit structure layout, I think.</div><div><br></div><div> - Daniel</div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jan 23, 2015 at 2:55 PM, Matthias Braun <span dir="ltr"><<a href="javascript:_e(%7B%7D,'cvml','mbraun@apple.com');" target="_blank">mbraun@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">The attached patches fix two problems when running test-suite/SingleSource/UnitTests/ObjC with the iOS SDK.<br>
<span><font color="#888888"><br>
- Matthias<br>
<br>
</font></span></blockquote></div><br></div>
</div></blockquote></div><br></div>_______________________________________________<br>llvm-commits mailing list<br><a href="javascript:_e(%7B%7D,'cvml','llvm-commits@cs.uiuc.edu');" target="_blank">llvm-commits@cs.uiuc.edu</a><br><a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br></div></blockquote></div><br></div></blockquote>