<div dir="ltr">Hi Renato,<div><br></div><div>I'm aware of the failure, and I'll commit *something* to get the build unbroken today.</div><div><br></div><div>The issue here is that "file_time_type" AKA system_clock::time_point has a much larger range than std::time_t on 32 bit builds,</div><div>so "file_time_type::max()" overflows in the conversion. I'm trying to figure out how best to handle this. A couple of options are:</div><div><br></div><div>1) Teach the tests about this overflow and properly convert unrepresentative times to 'numeric_limits<time_t>::max()' in the implementation.</div><div><br></div><div>2) Make file_type_type a std::chrono::time_point using std::time_ t as the underlying representation.</div><div>This way 'file_time_type::max()' will always be representative as a 'time_t'. However this limits</div><div>the resolution of 'file_time_type' to seconds instead of the current microseconds.</div><div><br></div><div>Ill figure out which is preferable and implement that.</div><div><br></div><div>/Eric</div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Jun 18, 2016 at 5:33 AM, Renato Golin <span dir="ltr"><<a href="mailto:renato.golin@linaro.org" target="_blank">renato.golin@linaro.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 18 June 2016 at 05:10, Eric Fiselier via cfe-commits<br>
<span class=""><<a href="mailto:cfe-commits@lists.llvm.org">cfe-commits@lists.llvm.org</a>> wrote:<br>
> Author: ericwf<br>
> Date: Fri Jun 17 23:10:23 2016<br>
> New Revision: 273078<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=273078&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=273078&view=rev</a><br>
> Log:<br>
> Fix 3 bugs in filesystem tests and implementation.<br>
<br>
</span>Hi Eric,<br>
<br>
Thanks for he fixes, but just FYI, there's still one remaining...<br>
<br>
<a href="http://lab.llvm.org:8011/builders/libcxx-libcxxabi-arm-linux/builds/1077" rel="noreferrer" target="_blank">http://lab.llvm.org:8011/builders/libcxx-libcxxabi-arm-linux/builds/1077</a><br>
<br>
cheers,<br>
--renato<br>
</blockquote></div><br></div>