Pull caching up to BuildConfigurationValue$Key$Codec. Almost all BuildOptions$DiffForReconstruction serializations are reached in this way, so we get better efficiency. Also, it was already a custom codec, so less new handrolling.

Also use serialization framework for FragmentClassSet, instead of doing serialization directly. Default FragmentClassSet should be a constant, so it will serialize down to a byte or three. Future changes can make all the classes constants as well, if we're getting misses on them.

PiperOrigin-RevId: 196546279

1 files changed, 1 insertions, 0 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/analysis/config/BuildConfiguration.java b/src/main/java/com/google/devtools/build/lib/analysis/config/BuildConfiguration.java
index 93646419fb..e63c5db18f 100644
--- a/src/main/java/com/google/devtools/build/lib/analysis/config/BuildConfiguration.java
+++ b/src/main/java/com/google/devtools/build/lib/analysis/config/BuildConfiguration.java
@@ -109,6 +109,7 @@ public class BuildConfiguration implements BuildConfigurationApi {
    * Sorts fragments by class name. This produces a stable order which, e.g., facilitates consistent
    * output from buildMnemonic.
    */
+  @AutoCodec
   public static final Comparator<Class<? extends Fragment>> lexicalFragmentSorter =
       Comparator.comparing(Class::getName);