[llvm] [llvm] Guard CGTestPlugin folder under LLVM_ENABLE_PLUGINS uniformly (NFC) (PR #225362)

[Stefan Gränitz via llvm-commits]

https://github.com/weliveindetail created https://github.com/llvm/llvm-project/pull/225362

We use `LLVM_ENABLE_PLUGINS` as condition to run the unit-tests in CGPluginTest, but we used to built the required plugin under `NOT WIN32 AND NOT CYGWIN`. These only happened to be equivalent. This patch guards the entire folder under `LLVM_ENABLE_PLUGINS` and also adds a respective REQUIRES in the related LIT test.

>From cca2c5e3e0b7d62c7beee366ded2d921daf7a52e Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Stefan=20Gr=C3=A4nitz?= <stefan.graenitz at gmail.com>
Date: Tue, 22 Sep 2026 10:58:15 +0200
Subject: [PATCH] [llvm] Guard CGTestPlugin folder under LLVM_ENABLE_PLUGINS
 uniformly (NFC)

---
 llvm/test/Other/codegen-plugin-loading.ll     |  2 +-
 .../CGPluginTest/Plugin/CMakeLists.txt        | 22 +++++++++----------
 .../CodeGen/CGPluginTest/PluginTest.cpp       | 10 ---------
 llvm/unittests/CodeGen/CMakeLists.txt         |  5 ++++-
 4 files changed, 15 insertions(+), 24 deletions(-)

diff --git a/llvm/test/Other/codegen-plugin-loading.ll b/llvm/test/Other/codegen-plugin-loading.ll
index cfbd239470ccb..0e2af7757696c 100644
--- a/llvm/test/Other/codegen-plugin-loading.ll
+++ b/llvm/test/Other/codegen-plugin-loading.ll
@@ -1,5 +1,5 @@
 ; RUN: llc -load-pass-plugin %llvmshlibdir/CGTestPlugin%pluginext %s -o - | FileCheck %s
-; REQUIRES: native, system-linux, llvm-dylib
+; REQUIRES: native, system-linux, llvm-dylib, plugins
 
 ; CHECK: CodeGen Test Pass running on main
 define void @main() {
diff --git a/llvm/unittests/CodeGen/CGPluginTest/Plugin/CMakeLists.txt b/llvm/unittests/CodeGen/CGPluginTest/Plugin/CMakeLists.txt
index 6cd2e15d2968c..8c1699e007dbc 100644
--- a/llvm/unittests/CodeGen/CGPluginTest/Plugin/CMakeLists.txt
+++ b/llvm/unittests/CodeGen/CGPluginTest/Plugin/CMakeLists.txt
@@ -1,14 +1,12 @@
-if (NOT WIN32 AND NOT CYGWIN)
-  unset(LLVM_LINK_COMPONENTS)
-  add_llvm_library(CGTestPlugin MODULE BUILDTREE_ONLY
-    CodeGenTestPass.cpp
-    Plugin.cpp
+unset(LLVM_LINK_COMPONENTS)
+add_llvm_library(CGTestPlugin MODULE BUILDTREE_ONLY
+  CodeGenTestPass.cpp
+  Plugin.cpp
 
-    DEPENDS
-    intrinsics_gen
-    vt_gen
-    )
+  DEPENDS
+  intrinsics_gen
+  vt_gen
+  )
 
-  set_target_properties(CGTestPlugin PROPERTIES FOLDER "Tests")
-  add_dependencies(CGPluginTest CGTestPlugin)
-endif ()
+set_target_properties(CGTestPlugin PROPERTIES FOLDER "Tests")
+add_dependencies(CGPluginTest CGTestPlugin)
diff --git a/llvm/unittests/CodeGen/CGPluginTest/PluginTest.cpp b/llvm/unittests/CodeGen/CGPluginTest/PluginTest.cpp
index 3e8be11c3433f..260f69c54e6d2 100644
--- a/llvm/unittests/CodeGen/CGPluginTest/PluginTest.cpp
+++ b/llvm/unittests/CodeGen/CGPluginTest/PluginTest.cpp
@@ -37,11 +37,6 @@ class CGPluginTests : public testing::Test {
 } // namespace llvm
 
 TEST_F(CGPluginTests, LoadPlugin) {
-#if !defined(LLVM_ENABLE_PLUGINS)
-  // Skip the test if plugins are disabled.
-  GTEST_SKIP();
-#endif
-
   auto PluginPath{std::string{"CGTestPlugin"} + LLVM_PLUGIN_EXT};
 
   std::string Error;
@@ -51,11 +46,6 @@ TEST_F(CGPluginTests, LoadPlugin) {
 }
 
 TEST_F(CGPluginTests, ExecuteCallback) {
-#if !defined(LLVM_ENABLE_PLUGINS)
-  // Skip the test if plugins are disabled.
-  GTEST_SKIP();
-#endif
-
   volatile bool CallbackExecuted = false;
   volatile bool MPassExecuted = false;
 
diff --git a/llvm/unittests/CodeGen/CMakeLists.txt b/llvm/unittests/CodeGen/CMakeLists.txt
index c192672e1a093..ea8028f4cf979 100644
--- a/llvm/unittests/CodeGen/CMakeLists.txt
+++ b/llvm/unittests/CodeGen/CMakeLists.txt
@@ -59,6 +59,9 @@ add_llvm_unittest(CodeGenTests
   )
 
 add_subdirectory(GlobalISel)
-add_subdirectory(CGPluginTest)
+
+if (LLVM_ENABLE_PLUGINS)
+  add_subdirectory(CGPluginTest)
+endif()
 
 target_link_libraries(CodeGenTests PRIVATE LLVMTestingSupport)