[clang] [clang] [TargetCXXABI] Fix -Wextra-semi warning (NFC) (PR #96982)

via cfe-commits cfe-commits at lists.llvm.org
Thu Jun 27 15:53:37 PDT 2024


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


More information about the cfe-commits mailing list