[llvm] [OpenMP] Remove unused isStrictSubset template (NFC) (PR #202987)

[via llvm-commits]

llvmorg-github-actions[bot] wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-flang-openmp

Author: Aditya Medhane (flash1729)

<details>
<summary>Changes</summary>

The `isStrictSubset` `ArrayRef<T>` template has no callers, so it never instantiates and trips `-Wunused-template`. The `VariantMatchInfo` overload does the work that's actually used, and `isSubset` stays untouched. Removing the dead template.

NFC.

Part of #<!-- -->202945.

---
Full diff: https://github.com/llvm/llvm-project/pull/202987.diff


1 Files Affected:

- (modified) llvm/lib/Frontend/OpenMP/OMPContext.cpp (-9) 


``````````diff
diff --git a/llvm/lib/Frontend/OpenMP/OMPContext.cpp b/llvm/lib/Frontend/OpenMP/OMPContext.cpp
index 6ced5c104c8ef..7443c4c338ada 100644
--- a/llvm/lib/Frontend/OpenMP/OMPContext.cpp
+++ b/llvm/lib/Frontend/OpenMP/OMPContext.cpp
@@ -162,15 +162,6 @@ template <typename T> static bool isSubset(ArrayRef<T> C0, ArrayRef<T> C1) {
   return true;
 }
 
-/// Return true if \p C0 is a strict subset of \p C1. Note that both arrays are
-/// expected to be sorted.
-template <typename T>
-static bool isStrictSubset(ArrayRef<T> C0, ArrayRef<T> C1) {
-  if (C0.size() >= C1.size())
-    return false;
-  return isSubset<T>(C0, C1);
-}
-
 static bool isStrictSubset(const VariantMatchInfo &VMI0,
                            const VariantMatchInfo &VMI1) {
   // If all required traits are a strict subset and the ordered vectors storing

``````````

</details>


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