{"push_id":2421253832,"size":1,"distinct_size":1,"ref":"refs/heads/master","head":"d404863fe0ad901d3f165e23d3e2399f0662f1bf","before":"d789c6ffa610ce0ad34e817d8bb557ead5fe4016","commits":[{"sha":"d404863fe0ad901d3f165e23d3e2399f0662f1bf","author":{"name":"Jonas Toth","email":"5e17962580491efa31aad6c5fa89f7a0aee33605@gmail.com"},"message":"[clang-tidy] Resubmit hicpp-multiway-paths-covered without breaking test\n\nThe original check did break the green buildbot in the sanitizer build.\nIt took a while to redroduce and understand the issue.\n\nThere occured a stackoverflow while parsing the AST. The testcase with\n256 case labels was the problem because each case label added another\nstackframe. It seemed that the issue occured only in 'RelWithDebInfo' builds\nand not in normal sanitizer builds.\n\nTo simplify the matchers the recognition for the different kinds of switch\nstatements has been moved into a seperate function and will not be done with\nASTMatchers. This is an attempt to reduce recursion and stacksize as well.\n\nThe new check removed this big testcase. Covering all possible values is still\nimplemented for bitfields and works there. The same logic on integer types\nwill lead to the issue.\n\nRunning it over LLVM gives the following results:\n\n\nDifferential: https://reviews.llvm.org/D40737\n\n\ngit-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@328107 91177308-0d34-0410-b5e6-96231b3b80d8","distinct":true,"url":"https://api.github.com/repos/llvm-project/clang-tools-extra/commits/d404863fe0ad901d3f165e23d3e2399f0662f1bf"}]}
{"actor":{"display_login":"chapuni"}}