Optimize an assertion which was showing up in profiles. (Even better might be

to remove the call from RuleConfiguredTarget.getProvider() and instead check the
types when the providers ImmutableMap is built.)

--
MOS_MIGRATED_REVID=125389561

1 files changed, 5 insertions, 7 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/analysis/AnalysisUtils.java b/src/main/java/com/google/devtools/build/lib/analysis/AnalysisUtils.java
index c350cb239a..40fdf9b6b1 100644
--- a/src/main/java/com/google/devtools/build/lib/analysis/AnalysisUtils.java
+++ b/src/main/java/com/google/devtools/build/lib/analysis/AnalysisUtils.java
@@ -26,7 +26,6 @@ import com.google.devtools.build.lib.collect.nestedset.NestedSetBuilder;
 import com.google.devtools.build.lib.collect.nestedset.Order;
 import com.google.devtools.build.lib.packages.BuildType;
 import com.google.devtools.build.lib.packages.TriState;
-import com.google.devtools.build.lib.util.Preconditions;
 import com.google.devtools.build.lib.vfs.PathFragment;
 
 /**
@@ -129,12 +128,11 @@ public final class AnalysisUtils {
    * Checks that the given provider class either refers to an interface or to a value class.
    */
   public static <T extends TransitiveInfoProvider> void checkProvider(Class<T> clazz) {
-    if (!clazz.isInterface()) {
-      Preconditions.checkArgument(
-          !clazz.getSimpleName().startsWith("AutoValue_") || clazz.getSuperclass() == null,
-          "%s is generated by @AutoValue - you should use %s instead",
-          clazz.getSimpleName(),
-          clazz.getSuperclass().getSimpleName());
+    // Write this check in terms of getName() rather than getSimpleName(); the latter is expensive.
+    if (!clazz.isInterface() && clazz.getName().contains(".AutoValue_")) {
+      // We must have a superclass due to the generic bound above.
+      throw new IllegalArgumentException(
+          clazz + " is generated by @AutoValue; use " + clazz.getSuperclass() + " instead");
     }
   }
 }