vbvictor wrote: > Shouldn't Clang's `-Wextra-semi` and `-Wextra-semi-stmt` be enabled and > enforced across LLVM code base?
Currently we only have https://github.com/llvm/llvm-project/blob/main/llvm/cmake/modules/HandleLLVMOptions.cmake#L845 For now I don't know the process of enabling new options, I suppose it should go via RFC? If I add to cmake ``` add_flag_if_supported("-Wextra-semi" EXTRA_SEMI_FLAG) add_flag_if_supported("-Wextra-semi-stmt" EXTRA_SEMI_STMT_FLAG) ``` It start to diagnose a lot of places, so It may take some time to clean up everything. https://github.com/llvm/llvm-project/pull/176926 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
