<div dir="ltr">Sure! We should not peek errno unless the context that calls C libraries.</div><br><div class="gmail_quote"><div dir="ltr">On Sun, May 15, 2016 at 1:56 AM Mehdi Amini <<a href="mailto:mehdi.amini@apple.com">mehdi.amini@apple.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
> On May 14, 2016, at 8:36 AM, NAKAMURA Takumi <<a href="mailto:geek4civic@gmail.com" target="_blank">geek4civic@gmail.com</a>> wrote:<br>
><br>
> chapuni added a subscriber: chapuni.<br>
><br>
> ================<br>
> Comment at: lib/Support/CachePruning.cpp:44<br>
> @@ +43,3 @@<br>
> +  if (sys::fs::status(TimestampFile, FileStatus)) {<br>
> +    if (errno == ENOENT) {<br>
> +      // If the timestamp file wasn't there, create one now.<br>
> ----------------<br>
> Don't use errno. Fixed in r269565.<br>
<br>
Thanks!<br>
<br>
Note: this was copy/pasted from clang `void pruneModuleCache()` in lib/Frontend/CompilerInstance.cpp, I imagine we should fix it the same way?<br>
<br>
--<br>
Mehdi<br>
<br>
</blockquote></div>