thakis added a comment. Yes, that's my understanding. (I'm not super familiar with this code either.)
The line was added in the change that added -Wimplicit-fallthrough (84837d5b5aa0d), and all the -Wimplicit-fallthrough tests still pass with this. Chances are it was unnecessary back then too. (`addStmt` is from ea9fcffe035d, way older.) CHANGES SINCE LAST ACTION https://reviews.llvm.org/D111570/new/ https://reviews.llvm.org/D111570 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits