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 implicit-fallthrough warning by adding missing break > > Modified: > cfe/trunk/lib/CrossTU/CrossTranslationUnit.cpp > > Modified: cfe/trunk/lib/CrossTU/CrossTranslationUnit.cpp > URL: > http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/CrossTU/CrossTranslationUnit.cpp?rev=314079&r1=314078&r2=314079&view=diff > > ============================================================================== > --- cfe/trunk/lib/CrossTU/CrossTranslationUnit.cpp (original) > +++ cfe/trunk/lib/CrossTU/CrossTranslationUnit.cpp Sun Sep 24 08:17:46 2017 > @@ -185,6 +185,7 @@ void CrossTranslationUnitContext::emitCr > case index_error_code::invalid_index_format: > Context.getDiagnostics().Report(diag::err_fnmap_parsing) > << IE.getFileName() << IE.getLineNum(); > + break; > case index_error_code::multiple_definitions: > Context.getDiagnostics().Report(diag::err_multiple_def_index) > << IE.getLineNum(); > > > _______________________________________________ > cfe-commits mailing list > cfe-commits@lists.llvm.org > http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits >
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
