<div dir="ltr"><div>Hi Nico</div><div> </div><div>I'm just looking at your chrono / thread / mutex stuff. Here:</div><div> </div><div><a href="http://lists.cs.uiuc.edu/pipermail/cfe-commits/Week-of-Mon-20131007/090709.html">http://lists.cs.uiuc.edu/pipermail/cfe-commits/Week-of-Mon-20131007/090709.html</a></div>
<div> </div><div>First, sorry I only just noticed it! I'm pretty happy to see something like this land!</div><div> </div><div>The chrono patch.</div><div> </div><div>Short version: LGTM.</div><div>Long: There's possibly more to this than you think for it to be "perfect" but I don't know if we need perfect yet / at all.</div>
<div>I'll send you my own research notes on it and you can see what I mean. Then you might (or might not) want to add more to this chrono patch, but the patch as it stands LGTM. Howard, obviously means you can also forget my own "not implemented" patch.</div>
<div> </div><div>pthreads.</div><div> </div><div>I've been playing with pthreads myself, I ported a "third party" library myself to test with because I hadn't noticed your patch until now.</div><div>I have it compiling on my machine with libcxx in but haven't run the code yet. Initially when I saw your patch I thought I'd wasted my time (not that it took long) but actually I think it's been very useful so I can comment better on your patch when I've had a bit more time to look at it.</div>
<div> </div><div>I think a lot of the changes you made are complimentary to the changes I made i.e. they would be exactly the same or help my port.</div><div>I'll apply your patch this weekend and report back. But your patch LGTM. I can submit any revisions I think might be useful later.</div>
<div> </div></div>