[clang] [llvm] [AArch64] Decouple feature dependency expansion. (PR #94279)

via cfe-commits cfe-commits at lists.llvm.org
Mon Jun 3 13:35:08 PDT 2024


github-actions[bot] wrote:

<!--LLVM CODE FORMAT COMMENT: {clang-format}-->


:warning: C/C++ code formatter, clang-format found issues in your code. :warning:

<details>
<summary>
You can test this locally with the following command:
</summary>

``````````bash
git-clang-format --diff 875e91131847e3133d414c11c8ad721b52a287ee 2f15ae2082c996918f11076ef765914434dd615c -- clang/include/clang/AST/ASTContext.h clang/lib/AST/ASTContext.cpp clang/lib/Basic/Targets/AArch64.cpp clang/lib/Basic/Targets/AArch64.h clang/test/CodeGen/aarch64-cpu-supports-target.c clang/test/CodeGen/aarch64-sme-intrinsics/aarch64-sme-attrs.cpp clang/test/CodeGen/aarch64-targetattr.c clang/test/CodeGen/attr-target-version.c clang/test/Sema/aarch64-neon-target.c clang/test/Sema/builtin-cpu-supports.c llvm/include/llvm/TargetParser/AArch64TargetParser.h llvm/lib/TargetParser/AArch64TargetParser.cpp
``````````

</details>

<details>
<summary>
View the diff from clang-format here.
</summary>

``````````diff
diff --git a/clang/lib/AST/ASTContext.cpp b/clang/lib/AST/ASTContext.cpp
index 842b94b4a6..6b7cfbe487 100644
--- a/clang/lib/AST/ASTContext.cpp
+++ b/clang/lib/AST/ASTContext.cpp
@@ -13667,7 +13667,7 @@ QualType ASTContext::getCorrespondingSignedFixedPointType(QualType Ty) const {
 // Given a list of FMV features, add each of their backend features to the list.
 static void
 getFMVBackendFeaturesFor(const llvm::SmallVectorImpl<StringRef> &FMVFeatStrings,
-                         std::vector<std::string>& BackendFeats) {
+                         std::vector<std::string> &BackendFeats) {
   for (StringRef F : FMVFeatStrings) {
     if (auto FMVExt = llvm::AArch64::parseArchExtension(F)) {
       SmallVector<StringRef, 8> Feats;
diff --git a/llvm/include/llvm/TargetParser/AArch64TargetParser.h b/llvm/include/llvm/TargetParser/AArch64TargetParser.h
index 4966c215c0..0a3a3dd695 100644
--- a/llvm/include/llvm/TargetParser/AArch64TargetParser.h
+++ b/llvm/include/llvm/TargetParser/AArch64TargetParser.h
@@ -578,8 +578,7 @@ struct ExtensionSet {
 
   // Convert the set of enabled extension to an LLVM feature list, appending
   // them to Features.
-  template <typename T>
-  void toLLVMFeatureList(std::vector<T> &Features) const {
+  template <typename T> void toLLVMFeatureList(std::vector<T> &Features) const {
     if (BaseArch && !BaseArch->ArchFeature.empty())
       Features.emplace_back(T(BaseArch->ArchFeature));
 

``````````

</details>


https://github.com/llvm/llvm-project/pull/94279


More information about the cfe-commits mailing list