r214089 - Thread Safety Analysis: Replace the old and broken SExpr with the new

[Delesley Hutchins]

So is recommended practice just to rely on the buildbots to catch the
warnings?  I don't have a windows machine with MSVC to test.

  -DeLesley


On Mon, Jul 28, 2014 at 12:35 PM, Chandler Carruth <chandlerc at google.com> wrote:
> On Mon, Jul 28, 2014 at 12:08 PM, Delesley Hutchins <delesley at google.com>
> wrote:
>>
>> Thanks!  Is there any way to get MSVC warnings like this in clang, so
>> I can see them before commit?
>
>
> Clang has the analogous warning, but it dosen't require an unreachable to
> silence it. The fact that this warning fires on code like this is a bug in
> GCC and MSVC's warning IMO, but the GCC folks (and I must assume the MSVC
> folks) disagree.



-- 
DeLesley Hutchins | Software Engineer | delesley at google.com | 505-206-0315