Remove a superfluous "return null" from JvmConfigurationLoader.

RedirectChaser needs to load the target to check if it's a bind/alias rule so it's not possible that it's not loaded yet by the time we get there.

--
PiperOrigin-RevId: 144429298
MOS_MIGRATED_REVID=144429298

1 files changed, 0 insertions, 3 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/rules/java/JvmConfigurationLoader.java b/src/main/java/com/google/devtools/build/lib/rules/java/JvmConfigurationLoader.java
index 6d9c42da5b..942d2abca3 100644
--- a/src/main/java/com/google/devtools/build/lib/rules/java/JvmConfigurationLoader.java
+++ b/src/main/java/com/google/devtools/build/lib/rules/java/JvmConfigurationLoader.java
@@ -90,9 +90,6 @@ public final class JvmConfigurationLoader implements ConfigurationFragmentFactor
         return null;
       }
       Target javaHomeTarget = lookup.getTarget(label);
-      if (javaHomeTarget == null) {
-        return null;
-      }
       if ((javaHomeTarget instanceof Rule) &&
           "filegroup".equals(((Rule) javaHomeTarget).getRuleClass())) {
         RawAttributeMapper javaHomeAttributes = RawAttributeMapper.of((Rule) javaHomeTarget);