This is an archive of the discontinued LLVM Phabricator instance.

[WebAssembly] Use getCalleeOpNo utility function (NFC)
ClosedPublic

Authored by aheejin on Aug 28 2018, 10:11 AM.

Reviewers

Commits

Repository

rL LLVM

Build Status

Buildable 22009
Build 22009: arc lint + arc unit

LGTM

This revision is now accepted and ready to land.Aug 28 2018, 10:30 AM

This revision was automatically updated to reflect the committed changes.

Path

Size

lib/

Target/

WebAssembly/

16 lines

Show First 20 Lines • Show All 229 Lines • ▼ Show 20 Lines	if (MI.hasUnmodeledSideEffects()) {
default:		default:
Effects = true;		Effects = true;
break;		break;
}		}
}		}

// Analyze calls.		// Analyze calls.
if (MI.isCall()) {		if (MI.isCall()) {
switch (MI.getOpcode()) {		unsigned CalleeOpNo = WebAssembly::getCalleeOpNo(MI);
case WebAssembly::CALL_VOID:		QueryCallee(MI, CalleeOpNo, Read, Write, Effects, StackPointer);
case WebAssembly::CALL_INDIRECT_VOID:
QueryCallee(MI, 0, Read, Write, Effects, StackPointer);
break;
case WebAssembly::CALL_I32: case WebAssembly::CALL_I64:
case WebAssembly::CALL_F32: case WebAssembly::CALL_F64:
case WebAssembly::CALL_INDIRECT_I32: case WebAssembly::CALL_INDIRECT_I64:
case WebAssembly::CALL_INDIRECT_F32: case WebAssembly::CALL_INDIRECT_F64:
QueryCallee(MI, 1, Read, Write, Effects, StackPointer);
break;
default:
llvm_unreachable("unexpected call opcode");
}
}		}
}		}

// Test whether Def is safe and profitable to rematerialize.		// Test whether Def is safe and profitable to rematerialize.
static bool ShouldRematerialize(const MachineInstr &Def, AliasAnalysis &AA,		static bool ShouldRematerialize(const MachineInstr &Def, AliasAnalysis &AA,
const WebAssemblyInstrInfo *TII) {		const WebAssemblyInstrInfo *TII) {
return Def.isAsCheapAsAMove() && TII->isTriviallyReMaterializable(Def, &AA);		return Def.isAsCheapAsAMove() && TII->isTriviallyReMaterializable(Def, &AA);
}		}
▲ Show 20 Lines • Show All 630 Lines • Show Last 20 Lines