Re: r314079 - Fix implicit-fallthrough warning by adding missing break

2017-09-25 Thread David Blaikie via cfe-commits
Is there also missing test coverage here? On Sun, Sep 24, 2017 at 8:19 AM Simon Pilgrim via cfe-commits < cfe-commits@lists.llvm.org> wrote: > Author: rksimon > Date: Sun Sep 24 08:17:46 2017 > New Revision: 314079 > > URL: http://llvm.org/viewvc/llvm-project?rev=314079&view=rev > Log: > Fix impl

r314079 - Fix implicit-fallthrough warning by adding missing break

2017-09-24 Thread Simon Pilgrim via cfe-commits
Author: rksimon Date: Sun Sep 24 08:17:46 2017 New Revision: 314079 URL: http://llvm.org/viewvc/llvm-project?rev=314079&view=rev Log: Fix implicit-fallthrough warning by adding missing break Modified: cfe/trunk/lib/CrossTU/CrossTranslationUnit.cpp Modified: cfe/trunk/lib/CrossTU/CrossTransla