Hello Sam, This commit broke one of our builders: http://lab.llvm.org:8011/builders/ubuntu-gcc7.1-werror/builds/4517
. . . FAILED: /usr/local/gcc-7.1/bin/gcc-7.1 -DGTEST_HAS_RTTI=0 -D_DEBUG -D_GNU_SOURCE -D__STDC_CONSTANT_MACROS -D__STDC_FORMAT_MACROS -D__STDC_LIMIT_MACROS -Itools/clang/tools/c-index-test -I/home/buildslave/am1i-slv2/ubuntu-gcc7.1-werror/llvm/tools/clang/tools/c-index-test -I/home/buildslave/am1i-slv2/ubuntu-gcc7.1-werror/llvm/tools/clang/include -Itools/clang/include -Iinclude -I/home/buildslave/am1i-slv2/ubuntu-gcc7.1-werror/llvm/include -fPIC -Werror -Werror=date-time -Wall -W -Wno-unused-parameter -Wwrite-strings -Wno-missing-field-initializers -pedantic -Wno-long-long -Wno-comment -ffunction-sections -fdata-sections -O3 -UNDEBUG -std=gnu89 -MD -MT tools/clang/tools/c-index-test/CMakeFiles/c-index-test.dir/c-index-test.c.o -MF tools/clang/tools/c-index-test/CMakeFiles/c-index-test.dir/c-index-test.c.o.d -o tools/clang/tools/c-index-test/CMakeFiles/c-index-test.dir/c-index-test.c.o -c /home/buildslave/am1i-slv2/ubuntu-gcc7.1-werror/llvm/tools/clang/tools/c-index-test/c-index-test.c /home/buildslave/am1i-slv2/ubuntu-gcc7.1-werror/llvm/tools/clang/tools/c-index-test/c-index-test.c: In function ‘CursorToText’: /home/buildslave/am1i-slv2/ubuntu-gcc7.1-werror/llvm/tools/clang/tools/c-index-test/c-index-test.c:741:1: error: control reaches end of non-void function [-Werror=return-type] } ^ cc1: all warnings being treated as errors Please have a look? Thanks Galina On Tue, Jan 16, 2018 at 4:54 AM, Sam McCall via cfe-commits < cfe-commits@lists.llvm.org> wrote: > Author: sammccall > Date: Tue Jan 16 04:54:28 2018 > New Revision: 322549 > > URL: http://llvm.org/viewvc/llvm-project?rev=322549&view=rev > Log: > Squash -Wcovered-switch-default wairning > > Modified: > cfe/trunk/tools/c-index-test/c-index-test.c > > Modified: cfe/trunk/tools/c-index-test/c-index-test.c > URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/tools/c- > index-test/c-index-test.c?rev=322549&r1=322548&r2=322549&view=diff > ============================================================ > ================== > --- cfe/trunk/tools/c-index-test/c-index-test.c (original) > +++ cfe/trunk/tools/c-index-test/c-index-test.c Tue Jan 16 04:54:28 2018 > @@ -729,8 +729,7 @@ static CXString CursorToText(CXCursor Cu > return clang_getCursorSpelling(Cursor); > case DisplayType_DisplayName: > return clang_getCursorDisplayName(Cursor); > - case DisplayType_Pretty: > - default: { > + case DisplayType_Pretty: { > CXString text; > CXPrintingPolicy Policy = clang_getCursorPrintingPolicy(Cursor); > ModifyPrintingPolicyAccordingToEnv(Policy); > > > _______________________________________________ > 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
