Address review comment

bazelbuild · Dec 4, 2023 · 52de68e · 52de68e
1 parent bb2b267
commit 52de68e
Showing 1 changed file with 9 additions and 0 deletions.
diff --git a/src/main/java/com/google/devtools/build/lib/packages/Package.java b/src/main/java/com/google/devtools/build/lib/packages/Package.java
@@ -1001,6 +1001,15 @@ default boolean precomputeTransitiveLoads() {
 
     private final List<TargetPattern> registeredExecutionPlatforms = new ArrayList<>();
     private final List<TargetPattern> registeredToolchains = new ArrayList<>();
+
+    /**
+     * Tracks the index within {@link #registeredToolchains} of the first toolchain registered from
+     * the WORKSPACE suffixes rather than the WORKSPACE file (if any).
+     *
+     * <p>This is needed to distinguish between these toolchains during resolution: toolchains
+     * registered in WORKSPACE have precedence over those defined in non-root Bazel modules,
+     * which in turn have precedence over those from the WORKSPACE suffixes.</p>
+     */
     private OptionalInt firstWorkspaceSuffixRegisteredToolchain = OptionalInt.empty();
 
     /**