[llvm] [SPIRV] Do not remove capability from all caps (PR #68194)

via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 30 04:42:50 PDT 2023


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 edebbb46b5033720a52e1a2cc3a2a96efbab88c2 a82a27aa567fcef3c4d6490a13e37595efe3b93f -- llvm/lib/Target/SPIRV/SPIRVModuleAnalysis.cpp llvm/lib/Target/SPIRV/SPIRVModuleAnalysis.h
``````````

</details>

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

``````````diff
diff --git a/llvm/lib/Target/SPIRV/SPIRVModuleAnalysis.h b/llvm/lib/Target/SPIRV/SPIRVModuleAnalysis.h
index 57d0a7416342..5124181b49e2 100644
--- a/llvm/lib/Target/SPIRV/SPIRVModuleAnalysis.h
+++ b/llvm/lib/Target/SPIRV/SPIRVModuleAnalysis.h
@@ -60,11 +60,11 @@ struct Requirements {
 struct RequirementHandler {
 private:
   CapabilityList MinimalCaps;
-  
-  // AllCaps and AvailableCaps are related but different. AllCaps is a subset of 
-  // AvailableCaps. AvailableCaps is the complete set of capabilities that are available
-  // to the current target. AllCaps is the set of capabilities that are required
-  // by the current module.
+
+  // AllCaps and AvailableCaps are related but different. AllCaps is a subset of
+  // AvailableCaps. AvailableCaps is the complete set of capabilities that are
+  // available to the current target. AllCaps is the set of capabilities that
+  // are required by the current module.
   SmallSet<Capability::Capability, 8> AllCaps;
   DenseSet<unsigned> AvailableCaps;
 

``````````

</details>


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


More information about the llvm-commits mailing list