<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On 4 March 2015 at 18:47, Saleem Abdulrasool <span dir="ltr"><<a href="mailto:compnerd@compnerd.org" target="_blank">compnerd@compnerd.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><span>On Wed, Mar 4, 2015 at 4:38 PM, Nick Lewycky <span dir="ltr"><<a href="mailto:nlewycky@google.com" target="_blank">nlewycky@google.com</a>></span> wrote:<br></span><div class="gmail_extra"><div class="gmail_quote"><span><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">The attached patch resolves a few issues building libcxxabi with gcc:<div><br><div> - GCC doesn't support #pragma mark, only Apple GCC and clang. Wrap the pragma mark's in #if 0 to make gcc ignore them but xcode still see them.</div></div></div></blockquote><div><br></div></span><div>Really?  That is rather amusing.  WFM.  I had tried to pacify the build with -Wno-unknown-pragmas, but that was deemed to be too heavy a hammer.</div><span><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div> - Wrap a bunch of "#pragma clang" lines in #ifdef __clang__.</div></div></div></blockquote><div><br></div></span><div>Hmm..I wonder if we should wrap this in a macro.  Yes, you can't use #pragma, but you can use _Pragma.  However, that is unrelated to this change, which strictly improves things.</div></div></div></div></blockquote><div><br></div><div>David Majnemer pointed out to me in person that I could also use _Pragma for the #pragma mark too, defining it to nothing for GCC and _Pragma for Clang.</div><div><br></div><div>Unfortunately I can't test that. I'm going to check this in as is now, but these review comments should be acted upon.</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><span><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div> - Pacify gcc's -Wparenthesis in a case where it's quite reasonable.</div></div></div></blockquote><div><br></div></span><div>Reasonable.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div>Please review!</div></div></div></blockquote><div><br></div><div>Thanks for fixing this.  LGTM.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><span><font color="#888888"><div>Nick</div></font></span></div></div>
<br>_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu" target="_blank">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
<br></blockquote></div><span><font color="#888888"><br><br clear="all"><div><br></div>-- <br><div>Saleem Abdulrasool<br>compnerd (at) compnerd (dot) org</div>
</font></span></div></div>
</blockquote></div><br></div></div>