More refactor work on single-/multi- architecture accessor methods of AppleConfiguration.

Additionally, tweak single-architecture ios-platform logic such that ios_multi_cpus is checked before ios_cpu.

There are two contexts to note:
  1. Single-architecture logic, (generally post-split), unaware of its own platform type aside
    from configuration. This retrieves platform type from the --apple_platform_type configuration value.
     a. getSingleArchPlatform() for Platform retrieval
     b. getSingleArchitecture() for architecture retrieval
  2. Multi-architecture logic, which should be aware of its own platform type, and passes it into
    configuration accessors.
     a. getMultiArchPlatform(PlatformType)
     b. getMultiArchitectures(PlatformType)

All callers are migrated to these methods, though some still pass IOS platform type even though
they may need to be refactored to support additional platform types later.

--
MOS_MIGRATED_REVID=124370652

1 files changed, 2 insertions, 1 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/rules/objc/BinaryLinkingTargetFactory.java b/src/main/java/com/google/devtools/build/lib/rules/objc/BinaryLinkingTargetFactory.java
index 7ee2cc8fb6..1f98901f9f 100644
--- a/src/main/java/com/google/devtools/build/lib/rules/objc/BinaryLinkingTargetFactory.java
+++ b/src/main/java/com/google/devtools/build/lib/rules/objc/BinaryLinkingTargetFactory.java
@@ -32,6 +32,7 @@ import com.google.devtools.build.lib.packages.RuleClass.ConfiguredTargetFactory.
 import com.google.devtools.build.lib.rules.RuleConfiguredTargetFactory;
 import com.google.devtools.build.lib.rules.apple.AppleConfiguration;
 import com.google.devtools.build.lib.rules.apple.Platform;
+import com.google.devtools.build.lib.rules.apple.Platform.PlatformType;
 import com.google.devtools.build.lib.rules.objc.CompilationSupport.ExtraLinkArgs;
 import com.google.devtools.build.lib.rules.objc.ObjcCommon.ResourceAttributes;
 import com.google.devtools.build.lib.rules.objc.ProtoSupport.TargetType;
@@ -151,7 +152,7 @@ abstract class BinaryLinkingTargetFactory implements RuleConfiguredTargetFactory
             .validateAttributes();
 
         xcTestAppProvider = Optional.of(releaseBundlingSupport.xcTestAppProvider());
-        if (appleConfiguration.getBundlingPlatform() == Platform.IOS_SIMULATOR) {
+        if (appleConfiguration.getMultiArchPlatform(PlatformType.IOS) == Platform.IOS_SIMULATOR) {
           Artifact runnerScript = intermediateArtifacts.runnerScript();
           Artifact ipaFile = ruleContext.getImplicitOutputArtifact(ReleaseBundlingSupport.IPA);
           releaseBundlingSupport.registerGenerateRunnerScriptAction(runnerScript, ipaFile);