<div dir="ltr">Thanks for that.<br><br>Committed as r199762.</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Jan 16, 2014 at 1:51 PM, alexw <span dir="ltr"><<a href="mailto:alexw@nicira.com" target="_blank">alexw@nicira.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hey David,<br>
<br>
Please see the attachment for fix patch.<br>
<br>
Sorry again, I must have run the wrong test the first time.<br>
The test is flawed at the beginning.<br>
<br>
I tested the fix, output:<br>
root@server329:~/llvm/llvm/tools/clang/test/Sema#<br>
~/llvm/llvm/utils/lit/lit.py -v   warn-<br>
thread-safety-analysis.c<br>
lit.py: lit.cfg:196: note: using clang:<br>
'/root/llvm/llvm/Debug+Asserts/bin/clang'<br>
-- Testing: 1 tests, 1 threads --<br>
PASS: Clang :: Sema/warn-thread-safety-analysis.c (1 of 1)<br>
Testing Time: 0.03s<br>
  Expected Passes    : 1<br>
<br>
Thanks,<br>
Alex Wang,<br>
test_fix_wthread_safety_for_c.diff<br>
<<a href="http://clang-developers.42468.n3.nabble.com/file/n4037219/test_fix_wthread_safety_for_c.diff" target="_blank">http://clang-developers.42468.n3.nabble.com/file/n4037219/test_fix_wthread_safety_for_c.diff</a>><br>

<br>
<br>
<br>
--<br>
View this message in context: <a href="http://clang-developers.42468.n3.nabble.com/PATCH-2-2-Thread-safety-Add-test-for-Wthread-safety-check-for-C-tp4033444p4037219.html" target="_blank">http://clang-developers.42468.n3.nabble.com/PATCH-2-2-Thread-safety-Add-test-for-Wthread-safety-check-for-C-tp4033444p4037219.html</a><br>

Sent from the Clang Developers mailing list archive at Nabble.com.<br>
<div class="HOEnZb"><div class="h5">_______________________________________________<br>
cfe-dev mailing list<br>
<a href="mailto:cfe-dev@cs.uiuc.edu">cfe-dev@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev</a><br>
</div></div></blockquote></div><br></div>