<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Aug 5, 2015 at 10:56 AM, Hans Wennborg <span dir="ltr"><<a href="mailto:hans@chromium.org" target="_blank">hans@chromium.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Wed, Aug 5, 2015 at 9:12 AM, Hans Wennborg <<a href="mailto:hans@chromium.org">hans@chromium.org</a>> wrote:<br>
> On Tue, Aug 4, 2015 at 12:35 AM, İsmail Dönmez <<a href="mailto:ismail@donmez.ws">ismail@donmez.ws</a>> wrote:<br>
>> Hi,<br>
>><br>
>> On Mon, Aug 3, 2015 at 10:51 PM, Reid Kleckner <<a href="mailto:reid@kleckner.net">reid@kleckner.net</a>> wrote:<br>
>>>  bool ReadFromFile(fd_t fd, void *buff, uptr buff_size, uptr *bytes_read,<br>
>>>                    error_t *error_p) {<br>
>>> -  UNIMPLEMENTED();<br>
>>> +  CHECK(fd != kInvalidFd);<br>
>>> +  bool success = ::ReadFile(fd, buff, buff_size, bytes_read, nullptr);<br>
>>> +  if (!success && error_p)<br>
>>> +    *error_p = GetLastError();<br>
>>> +  return success;<br>
>>>  }<br>
>><br>
>> This seems to fail on VS2015:<br>
>><br>
>> ..\projects\compiler-rt\lib\sanitizer_common\sanitizer_win.cc(496):<br>
>> error C2664: 'BOOL<br>
>> ReadFile(HANDLE,LPVOID,DWORD,LPDWORD,LPOVERLAPPED)': cannot convert<br>
>> argument 4 from '__sanitizer::uptr *' to 'LPDWORD'<br>
>> ..\projects\compiler-rt\lib\sanitizer_common\sanitizer_win.cc(496):<br>
>> note: Types pointed to are unrelated; conversion requires<br>
>> reinterpret_cast, C-style cast or function-style cast<br>
><br>
> It's failing here too:<br>
><br>
> <a href="http://build.chromium.org/p/chromium.fyi/builders/ClangToTWin/builds/2235/steps/gclient%20runhooks/logs/stdio" rel="noreferrer" target="_blank">http://build.chromium.org/p/chromium.fyi/builders/ClangToTWin/builds/2235/steps/gclient%20runhooks/logs/stdio</a><br>
<br>
</span>Checked in fix attempt in r244077. Reid, please take a look so I<br>
didn't mess anything up :-)</blockquote><div><br></div><div>Thanks for the fix! None of the sanitizers work on win64, so this didn't pop up on lab.llvm and I didn't test it locally.</div></div></div></div>