Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit aa88b3d0 authored by Jared Duke's avatar Jared Duke Committed by Gerrit Code Review
Browse files

Reland "Enable R8 full mode by default for apps" 

This is attempt 4 at landing this change. Prior attempts were
reverted due to downstream test breakages not in presubmit.
Those issues have been resolved, and additional manual heavy
presubmit tests were run to ensure stability.

Observed APK savings: ~24MB

This reverts commit 74a5c2ec.

Reason for revert: Fixed ManagedProvisioningTests

Bug: 215530220
Change-Id: I223228a345fa1a3bad52fb3e91c744107e04fc07
parent 74a5c2ec

java/app.go

+1 −0
Original line number Diff line number Diff line
@@ -1030,6 +1030,7 @@ func AndroidAppFactory() android.Module { 


	module.Module.dexProperties.Optimize.EnabledByDefault = true

	module.Module.dexProperties.Optimize.Shrink = proptools.BoolPtr(true)

	module.Module.dexProperties.Optimize.Proguard_compatibility = proptools.BoolPtr(false)



	module.Module.properties.Instrument = true

	module.Module.properties.Supports_static_instrumentation = true

java/dex.go

+2 −2
Original line number Diff line number Diff line
@@ -45,8 +45,8 @@ type DexProperties struct { 
		// Whether to continue building even if warnings are emitted.  Defaults to true.

		Ignore_warnings *bool



		// If true, runs R8 in Proguard compatibility mode (default).

		// Otherwise, runs R8 in full mode.

		// If true, runs R8 in Proguard compatibility mode, otherwise runs R8 in full mode.

		// Defaults to false for apps, true for libraries and tests.

		Proguard_compatibility *bool



		// If true, optimize for size by removing unused code.  Defaults to true for apps,