jstenglein added a comment. In D75768#1910303 <https://reviews.llvm.org/D75768#1910303>, @erichkeane wrote:
> Any reason to not put this in -Wall like GCC? I'm not terribly knowledgeable > about the intended guidance for adding to Wmost, so if you can clarify this > decision I'd be grateful. Otherwise I think this patch looks fine. Thanks for the comments. It is already part of -Wall since I put it in -Wmost and -Wall includes -Wmost: def All : DiagGroup<"all", [Most, Parentheses, Switch, SwitchBool, MisleadingIndentation]>; Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D75768/new/ https://reviews.llvm.org/D75768 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
