[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