ReVanced/smali
1
0
Fork 0
mirror of https://github.com/revanced/smali.git synced 2025-05-06 17:34:34 +02:00
Code Issues Packages Projects Releases Wiki Activity

Be more precise about narrowing after an instance-of

Browse Source 
Only do the additional narrowing when all predecessors of the instance-of
are equivalent move instructions.
This commit is contained in:
Ben Gruver 2016-10-16 13:45:30 -07:00
parent 22e85fc3ff
commit a6593575e7
1 changed files with 38 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

36
dexlib2/src/main/java/org/jf/dexlib2/analysis/MethodAnalyzer.java
View File

@ -1216,20 +1216,23 @@ public class MethodAnalyzer {
} }
AnalyzedInstruction prevAnalyzedInstruction = analyzedInstruction.getPredecessors().first(); AnalyzedInstruction prevAnalyzedInstruction = analyzedInstruction.getPredecessors().first();
if (prevAnalyzedInstruction.instruction.getOpcode() == Opcode.INSTANCE_OF) { if (prevAnalyzedInstruction.instruction.getOpcode() == Opcode.INSTANCE_OF) {
Instruction22c instanceOfInstruction = (Instruction22c)prevAnalyzedInstruction.instruction;
if (canNarrowAfterInstanceOf(prevAnalyzedInstruction, analyzedInstruction, classPath)) { if (canNarrowAfterInstanceOf(prevAnalyzedInstruction, analyzedInstruction, classPath)) {
List<Integer> narrowingRegisters = Lists.newArrayList(); List<Integer> narrowingRegisters = Lists.newArrayList();
RegisterType newType = RegisterType.getRegisterType(classPath, RegisterType newType = RegisterType.getRegisterType(classPath,
(TypeReference)((Instruction22c)prevAnalyzedInstruction.instruction).getReference()); (TypeReference)instanceOfInstruction.getReference());
if (instructionIndex > 1) { if (instructionIndex > 1) {
// If we have something like: // If we have something like:
// move-object/from16 v0, p1 // move-object/from16 v0, p1
// instance-of v2, v0, Lblah; // instance-of v2, v0, Lblah;
// if-eqz v2, :target // if-eqz v2, :target
// Then we need to narrow both v0 AND p1 // Then we need to narrow both v0 AND p1, but only if all predecessors of instance-of are a
AnalyzedInstruction prevPrevAnalyzedInstruction = // move-object for the same registers
analyzedInstructions.valueAt(instructionIndex - 2);
int additionalNarrowingRegister = -1;
for (AnalyzedInstruction prevPrevAnalyzedInstruction: prevAnalyzedInstruction.predecessors) {
Opcode opcode = prevPrevAnalyzedInstruction.instruction.getOpcode(); Opcode opcode = prevPrevAnalyzedInstruction.instruction.getOpcode();
if (opcode == Opcode.MOVE_OBJECT || opcode == Opcode.MOVE_OBJECT_16 || if (opcode == Opcode.MOVE_OBJECT || opcode == Opcode.MOVE_OBJECT_16 ||
opcode == Opcode.MOVE_OBJECT_FROM16) { opcode == Opcode.MOVE_OBJECT_FROM16) {
@ -1238,12 +1241,31 @@ public class MethodAnalyzer {
RegisterType originalType = RegisterType originalType =
prevPrevAnalyzedInstruction.getPostInstructionRegisterType( prevPrevAnalyzedInstruction.getPostInstructionRegisterType(
moveInstruction.getRegisterB()); moveInstruction.getRegisterB());
if (originalType.type != null) { if (moveInstruction.getRegisterA() != instanceOfInstruction.getRegisterB()) {
additionalNarrowingRegister = -1;
break;
}
if (originalType.type == null) {
additionalNarrowingRegister = -1;
break;
}
if (isNarrowingConversion(originalType, newType)) { if (isNarrowingConversion(originalType, newType)) {
narrowingRegisters.add( if (additionalNarrowingRegister != -1) {
((TwoRegisterInstruction)prevPrevAnalyzedInstruction.instruction).getRegisterB()); if (additionalNarrowingRegister != moveInstruction.getRegisterB()) {
additionalNarrowingRegister = -1;
break;
}
} else {
additionalNarrowingRegister = moveInstruction.getRegisterB();
} }
} }
} else {
additionalNarrowingRegister = -1;
break;
}
}
if (additionalNarrowingRegister != -1) {
narrowingRegisters.add(additionalNarrowingRegister);
} }
} }