llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT--> @llvm/pr-subscribers-clang Author: None (akirchhoff-modular) <details> <summary>Changes</summary> While building a downstream project including `TargetCXXABI.h`, I received a `-Wextra-semi` warning. This PR removes the extra semicolon, fixing the warning. --- Full diff: https://github.com/llvm/llvm-project/pull/96982.diff 1 Files Affected: - (modified) clang/include/clang/Basic/TargetCXXABI.h (+1-1) ``````````diff diff --git a/clang/include/clang/Basic/TargetCXXABI.h b/clang/include/clang/Basic/TargetCXXABI.h index c113a6a048ad4..d204452afbf4b 100644 --- a/clang/include/clang/Basic/TargetCXXABI.h +++ b/clang/include/clang/Basic/TargetCXXABI.h @@ -116,7 +116,7 @@ class TargetCXXABI { return T.isKnownWindowsMSVCEnvironment(); } llvm_unreachable("invalid CXXABI kind"); - }; + } /// Does this ABI generally fall into the Itanium family of ABIs? bool isItaniumFamily() const { `````````` </details> https://github.com/llvm/llvm-project/pull/96982 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
