<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><br><div><div>On Apr 8, 2014, at 5:18 PM, Reid Kleckner wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div dir="ltr">I thought this came up before and the answer was "it's a feature, not a bug", but maybe Nick remembers.</div></blockquote><div>Yes. We want LLVM to use the standard/documented names.</div><div><br></div><div><div>I thought LLVM code was supposed to only include LLVM headers. That assures that the code will build on all platforms. </div><div><br></div><div>At some point (in rare instances) you have OS specific code (e.g. llvm/lib/Support/Unix/Process.inc), and that code does the careful dance with how to include OS and LLVM headers.</div><div><br></div></div><div>-Nick</div><div><br></div><br><blockquote type="cite"><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Apr 8, 2014 at 3:12 PM, Akira Hatanaka <span dir="ltr"><<a href="mailto:ahatanak@gmail.com" target="_blank">ahatanak@gmail.com</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">The attached patch renames enums in Support/MachO.h. This is needed because some of the enum names are taken directly from system header files and including both MachO.h and some internal versions of those system header files causes name collisions.<div>
<div><br></div><div>With this patch, we can stop using the following workaround that was required to build lldb:</div><div> <br></div><div><a href="http://lists.cs.uiuc.edu/pipermail/lldb-commits/Week-of-Mon-20140331/010905.html" target="_blank">http://lists.cs.uiuc.edu/pipermail/lldb-commits/Week-of-Mon-20140331/010905.html</a><br>
<div><br></div></div></div><div>Please review.</div></div>
<br>_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">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>
<br></blockquote></div><br></div>
</blockquote></div><br></body></html>