More efficient overlay check in PcodeEmit

This commit is contained in:
caheckman 2019-10-02 12:09:45 -04:00
parent bf4a17b67d
commit 40c54939c6

View file

@ -63,7 +63,6 @@ public abstract class PcodeEmit {
private long uniquemask; private long uniquemask;
private long uniqueoffset; private long uniqueoffset;
private AddressSpace overlayspace = null; private AddressSpace overlayspace = null;
private AddressSpace overlayedspace = null;
/** /**
* Pcode emitter constructor for empty or unimiplemented instructions * Pcode emitter constructor for empty or unimiplemented instructions
@ -89,8 +88,8 @@ public abstract class PcodeEmit {
AddressSpace myspace = startAddress.getAddressSpace(); AddressSpace myspace = startAddress.getAddressSpace();
if (myspace.isOverlaySpace()) { if (myspace.isOverlaySpace()) {
overlayspace = myspace; overlayspace = myspace;
overlayedspace = ((OverlayAddressSpace) myspace).getOverlayedSpace(); startAddress = ((OverlayAddressSpace) myspace).getOverlayedSpace().getAddress(
startAddress = overlayedspace.getAddress(startAddress.getOffset()); startAddress.getOffset());
} }
this.fallOffset = fallOffset; this.fallOffset = fallOffset;
this.uniqueFactory = uniqueFactory; this.uniqueFactory = uniqueFactory;
@ -730,6 +729,7 @@ public abstract class PcodeEmit {
} }
void checkOverlays(int opcode, VarnodeData[] in, int isize, VarnodeData out) { void checkOverlays(int opcode, VarnodeData[] in, int isize, VarnodeData out) {
if (overlayspace != null) {
if (uniqueFactory == null) { if (uniqueFactory == null) {
return; return;
} }
@ -741,7 +741,6 @@ public abstract class PcodeEmit {
in[0].offset = space.getBaseSpaceID(); in[0].offset = space.getBaseSpaceID();
} }
} }
if (overlayspace != null) {
for (int i = 0; i < isize; ++i) { for (int i = 0; i < isize; ++i) {
VarnodeData v = in[0]; VarnodeData v = in[0];
if (v.space.equals(overlayspace)) { if (v.space.equals(overlayspace)) {