<div dir="ltr">I tested on Linux and MacOSX.  Since the tests are coming back clean and it is fixing a build error, I'm going to go ahead and check this in.  Feel free to revise as needed if it's not quite right.<div>

<br></div><div>Checked in as r217185.  Looks to be the change for tracking llvm r217172.</div>







<div><br></div><div>-Todd</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Sep 4, 2014 at 11:23 AM, Todd Fiala <span dir="ltr"><<a href="mailto:tfiala@google.com" target="_blank">tfiala@google.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">Does this look about right?  I get build errors on Linux and MacOSX without this change:<div><br></div><div>
<div>diff --git a/source/Plugins/ObjectFile/Mach-O/ObjectFileMachO.cpp b/source/Plugins/ObjectFile/Mach-O/ObjectFi</div>
<div>index 8d7fe04..67be5a5 100644</div><div>--- a/source/Plugins/ObjectFile/Mach-O/ObjectFileMachO.cpp</div><div>+++ b/source/Plugins/ObjectFile/Mach-O/ObjectFileMachO.cpp</div><div>@@ -5026,9 +5026,9 @@ ObjectFileMachO::GetSDKVersion(uint32_t *versions, uint32_t num_versions)</div>

<div>             {</div><div>                 if (m_data.GetU32 (&offset, &lc.version, (sizeof(lc) / sizeof(uint32_t)) - 2))</div><div>                 {</div><div>-                    const uint32_t xxxx = lc.reserved >> 16;</div>

<div>-                    const uint32_t yy = (lc.reserved >> 8) & 0xffu;</div><div>-                    const uint32_t zz = lc.reserved  & 0xffu;</div><div>+                    const uint32_t xxxx = lc.sdk >> 16;</div>

<div>+                    const uint32_t yy = (lc.sdk >> 8) & 0xffu;</div><div>+                    const uint32_t zz = lc.sdk & 0xffu;</div><div>                     if (xxxx)</div><div>                     {</div>

<div>                         m_sdk_versions.push_back(xxxx);</div><span class="HOEnZb"><font color="#888888"><div>-- <br><div dir="ltr"><table cellspacing="0" cellpadding="0" style="color:rgb(136,136,136);font-family:'Times New Roman'">
<tbody><tr style="color:rgb(85,85,85);font-family:sans-serif;font-size:small">
<td nowrap style="border-top-style:solid;border-top-color:rgb(213,15,37);border-top-width:2px">Todd Fiala |</td><td nowrap style="border-top-style:solid;border-top-color:rgb(51,105,232);border-top-width:2px"> Software Engineer |</td>

<td nowrap style="border-top-style:solid;border-top-color:rgb(0,153,57);border-top-width:2px"> <a href="mailto:tfiala@google.com" style="color:rgb(17,85,204)" target="_blank"><span style="color:rgb(34,34,34);background-color:rgb(255,255,204)">tfiala@google.com</span></a> |</td>

<td nowrap style="border-top-style:solid;border-top-color:rgb(238,178,17);border-top-width:2px"><font color="#1155cc"> <a>650-943-3180</a></font></td></tr></tbody></table><br></div>
</div></font></span></div></div>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div dir="ltr"><table cellspacing="0" cellpadding="0" style="color:rgb(136,136,136);font-family:'Times New Roman'"><tbody><tr style="color:rgb(85,85,85);font-family:sans-serif;font-size:small">
<td nowrap style="border-top-style:solid;border-top-color:rgb(213,15,37);border-top-width:2px">Todd Fiala |</td><td nowrap style="border-top-style:solid;border-top-color:rgb(51,105,232);border-top-width:2px"> Software Engineer |</td>
<td nowrap style="border-top-style:solid;border-top-color:rgb(0,153,57);border-top-width:2px"> <a href="mailto:tfiala@google.com" style="color:rgb(17,85,204)" target="_blank"><span style="background-color:rgb(255,255,204);color:rgb(34,34,34);background-repeat:initial initial">tfiala@google.com</span></a> |</td>
<td nowrap style="border-top-style:solid;border-top-color:rgb(238,178,17);border-top-width:2px"><font color="#1155cc"> <a>650-943-3180</a></font></td></tr></tbody></table><br></div>
</div>