Downgrade error to warning when field name is used for applymapping
This is currently blocking the roll into studio-master-dev.
Bug: 128868424
Change-Id: I0f283d0db4c06867153019071523cd8d5b1e3fbe
diff --git a/src/main/java/com/android/tools/r8/naming/ProguardMapMinifier.java b/src/main/java/com/android/tools/r8/naming/ProguardMapMinifier.java
index 4796cb9..ed80964 100644
--- a/src/main/java/com/android/tools/r8/naming/ProguardMapMinifier.java
+++ b/src/main/java/com/android/tools/r8/naming/ProguardMapMinifier.java
@@ -266,11 +266,12 @@
@Override
public boolean breakOnNotAvailable(DexReference source, DexString name) {
- // This is an error where we have renamed a member to an name that exists in a subtype or
- // renamed a field to something that exists in a subclass.
+ // If we renamed a member to a name that exists in a subtype we should warn that potentially
+ // a member lookup may no longer visit its parent.
MemberNaming memberNaming = mappedNames.get(source);
assert source.isDexMethod() || source.isDexField();
- reporter.error(
+ // TODO(b/128868424) Check if we can remove this warning for fields.
+ reporter.warning(
ApplyMappingError.mapToExistingMember(
source.toSourceString(),
name.toString(),
