I'll take a look later today.<br><div class="gmail_quote">On Wed, Dec 17, 2014 at 6:59 AM Aidan Dodds <<a href="mailto:Aidan@codeplay.com">Aidan@codeplay.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi zturner, clayborg, emaste, domipheus,<br>
<br>
The visual studio 2013 x64 build is failing due to some type differences between 32bit and 64bit builds.<br>
<br>
The CONTEXT structure used in RegisterContextWindows_x86 changes its fields based on the build.<br>
Currently I have used #ifdefs to alternate between the different registers for each build, is this an acceptable solution?<br>
<br>
It seems however that HostThreadWindows is largely 32bit specific, so should there rather be a dedicated x64 version?<br>
<br>
In HostThreadWindows its seems that ULONG_PTR also changes its size based on the machine type, however the argument for ExitThread() doesnt change in a simmilar way.<br>
I have created a proxy function to wrap ExitThread() so it can again be used as a function pointer.<br>
<br>
If these changes are acceptable, would someone be able to commit these fixes since I dont currently have commit access.<br>
<br>
Thanks,<br>
Aidan<br>
<br>
<a href="http://reviews.llvm.org/D6704" target="_blank">http://reviews.llvm.org/D6704</a><br>
<br>
Files:<br>
  source/Host/windows/<u></u>HostThreadWindows.cpp<br>
  source/Plugins/Process/<u></u>Windows/<u></u>RegisterContextWindows_x86.cpp<br>
<br>
EMAIL PREFERENCES<br>
  <a href="http://reviews.llvm.org/settings/panel/emailpreferences/" target="_blank">http://reviews.llvm.org/<u></u>settings/panel/<u></u>emailpreferences/</a><br>
</blockquote></div>