<div dir="ltr">This didn't seem to break anything on Linux and MacOSX Xcode builds.  I'll go ahead and check it in now.<div><br></div><div>-Todd</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">
On Thu, Jul 24, 2014 at 1:33 PM, Greg Clayton <span dir="ltr"><<a href="mailto:gclayton@apple.com" target="_blank">gclayton@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Looks good.<br>
<div class="HOEnZb"><div class="h5"><br>
> On Jul 24, 2014, at 11:40 AM, Keno Fischer <<a href="mailto:kfischer@college.harvard.edu">kfischer@college.harvard.edu</a>> wrote:<br>
><br>
> This works for me:<br>
><br>
> diff --git a/source/Host/common/Host.cpp b/source/Host/common/Host.cpp<br>
> index 275b446..09c392b 100644<br>
> --- a/source/Host/common/Host.cpp<br>
> +++ b/source/Host/common/Host.cpp<br>
> @@ -37,6 +37,9 @@<br>
> #include <mach/mach_init.h><br>
> #include <mach-o/dyld.h><br>
> #include <AvailabilityMacros.h><br>
> +#ifndef CPU_SUBTYPE_X86_64_H<br>
> +#define CPU_SUBTYPE_X86_64_H ((cpu_subtype_t)8)<br>
> +#endif<br>
> #endif<br>
><br>
> #if defined (__linux__) || defined (__FreeBSD__) || defined<br>
> (__FreeBSD_kernel__) || defined (__APPLE__) || defined(__NetBSD__)<br>
><br>
> so just commit that? Not sure if it's all that great to have to do<br>
> this, especially because we already have all the definitions we need<br>
> in SafeMachO.h, but I guess this at least fixes the build.<br>
><br>
> On Wed, Jul 23, 2014 at 5:19 PM, Greg Clayton <<a href="mailto:gclayton@apple.com">gclayton@apple.com</a>> wrote:<br>
>> The CPU is new and only available in newer system headers.<br>
>><br>
>> Keno, please add the following code only in the Host.cpp or Host.mm (since these are some of the few files in the LLDB sources that can include host specific header files) for apple builds:<br>
>><br>
>> #ifndef CPU_SUBTYPE_X86_64_H<br>
>> #define CPU_SUBTYPE_X86_64_H ((cpu_subtype_t)8)<br>
>> #endif<br>
>><br>
>> This is usually in /usr/include/mach/machine.h, but it won't be in older versions of the header.<br>
>><br>
>><br>
>>> On Jul 23, 2014, at 8:43 AM, Keno Fischer <<a href="mailto:kfischer@college.harvard.edu">kfischer@college.harvard.edu</a>> wrote:<br>
>>><br>
>>> While building on OS X I have been sent reports of the following (I<br>
>>> did see it myself at one point as well, but worked around it). I tried<br>
>>> including SafeMachO.h but that caused other problems in the llvm<br>
>>> headers. What's the proper way to get around this other than `#define<br>
>>> CPU_SUBTYPE_X86_64_H 8`?<br>
>>><br>
>>> lldb/source/Host/common/Host.cpp:371:68: error: use of undeclared<br>
>>> identifier 'CPU_SUBTYPE_X86_64_H'<br>
>>>               if (cpusubtype == CPU_SUBTYPE_486 || cpusubtype ==<br>
>>> CPU_SUBTYPE_X86_64_H)<br>
>>><br>
>>> Keno<br>
>>> _______________________________________________<br>
>>> lldb-dev mailing list<br>
>>> <a href="mailto:lldb-dev@cs.uiuc.edu">lldb-dev@cs.uiuc.edu</a><br>
>>> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev</a><br>
>><br>
<br>
_______________________________________________<br>
lldb-dev mailing list<br>
<a href="mailto:lldb-dev@cs.uiuc.edu">lldb-dev@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev</a><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div dir="ltr">-Todd</div>
</div>