[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