Refactor action sets for consistency and simplicity

BEGIN_PUBLIC
Refactor action sets for consistency and simplicity

* Remove the "CC" prefix on some of the actions, as we're already within the scope of rules_cc.
* Remove the "all" prefix, since the plural "actions" should already imply this.
* Ensure all action types sets end with "_actions"
* Refactor some action type sets to be constructed from other (eg. link actions = link executables + link dynamic libraries).
* Remove the redundant "all_compile_actions", since it was equivalent to "all_cc_compile_actions"
END_PUBLIC

PiperOrigin-RevId: 617985242
Change-Id: I9a3621a9d613ff604efdc5c86ba52e5c77f06731

Author: Googler

cc/toolchains/actions.bzl

@@ -63,7 +63,7 @@ cc_action_type_set = rule(
 Example:
 
 cc_action_type_set(
-    name = "cc_link_executable_actions",
+    name = "link_executable_actions",
     actions = [
         ":cpp_link_executable",
         ":lto_index_for_executable",

cc/toolchains/actions/BUILD

@@ -139,20 +139,20 @@ cc_action_type(
 )
 
 cc_action_type_set(
-    name = "all_ar_actions",
+    name = "ar_actions",
     actions = [":cpp_link_static_library"],
 )
 
 cc_action_type_set(
-    name = "all_assembly_actions",
+    name = "assembly_actions",
     actions = [
         ":preprocess_assemble",
         ":assemble",
     ],
 )
 
 cc_action_type_set(
-    name = "all_cpp_compile_actions",
+    name = "cpp_compile_actions",
     actions = [
         ":linkstamp_compile",
         ":cpp_compile",
@@ -165,37 +165,24 @@ cc_action_type_set(
 )
 
 cc_action_type_set(
-    name = "all_cc_compile_actions",
+    name = "compile_actions",
     actions = [
-        ":all_cpp_compile_actions",
+        ":cpp_compile_actions",
         ":c_compile",
-        ":preprocess_assemble",
-        ":assemble",
+        ":assembly_actions",
     ],
 )
 
 cc_action_type_set(
-    name = "all_compile_actions",
+    name = "link_actions",
     actions = [
-        ":all_cc_compile_actions",
-        ":all_cpp_compile_actions",
+        ":link_executable_actions",
+        ":dynamic_library_link_actions",
     ],
 )
 
 cc_action_type_set(
-    name = "all_cc_link_actions",
-    actions = [
-        ":cpp_link_executable",
-        ":cpp_link_dynamic_library",
-        ":cpp_link_nodeps_dynamic_library",
-        ":lto_index_for_executable",
-        ":lto_index_for_dynamic_library",
-        ":lto_index_for_nodeps_dynamic_library",
-    ],
-)
-
-cc_action_type_set(
-    name = "cc_link_executable_actions",
+    name = "link_executable_actions",
     actions = [
         ":cpp_link_executable",
         ":lto_index_for_executable",
@@ -206,9 +193,8 @@ cc_action_type_set(
     name = "dynamic_library_link_actions",
     actions = [
         ":cpp_link_dynamic_library",
-        ":cpp_link_nodeps_dynamic_library",
         ":lto_index_for_dynamic_library",
-        ":lto_index_for_nodeps_dynamic_library",
+        ":nodeps_dynamic_library_link_actions",
     ],
 )
 

cc/toolchains/toolchain.bzl

@@ -27,10 +27,10 @@ visibility("public")
 #  work out what actions correspond to what file groups.
 _LEGACY_FILE_GROUPS = {
     "ar_files": [
-        "@rules_cc//cc/toolchains/actions:all_ar_actions",  # copybara-use-repo-external-label
+        "@rules_cc//cc/toolchains/actions:ar_actions",  # copybara-use-repo-external-label
     ],
     "as_files": [
-        "@rules_cc//cc/toolchains/actions:all_assembly_actions",  # copybara-use-repo-external-label
+        "@rules_cc//cc/toolchains/actions:assembly_actions",  # copybara-use-repo-external-label
     ],
     "compiler_files": [
         "@rules_cc//cc/toolchains/actions:cc_flags_make_variable",  # copybara-use-repo-external-label