This is an archive of the discontinued LLVM Phabricator instance.

[AArch64] Only run macro fusion for CPUs with any fusion support.
ClosedPublic

Authored by fhahn on Jul 3 2017, 10:03 AM.

Download Raw Diff

Details

Reviewers

evandro
t.p.northover
javed.absar

Commits

rG15be1ac9ab2e: [AArch64] Only run macro fusion for CPUs with any fusion support.
rL307851: [AArch64] Only run macro fusion for CPUs with any fusion support.

Diff Detail

Event Timeline

fhahn created this revision.Jul 3 2017, 10:03 AM

Herald added subscribers: kristof.beyls, rengolin, aemerson. · View Herald TranscriptJul 3 2017, 10:03 AM

fhahn added a parent revision: D34958: [AArch64] Add AArch64Subtarget::isFusion function..Jul 3 2017, 10:04 AM

LGTM

It would be great if someone could accept this patch in Phabricator, now that D34958 has landed :)

LGTM

This revision is now accepted and ready to land.Jul 12 2017, 2:31 PM

fhahn closed this revision.Jul 12 2017, 2:41 PM

Revision Contents

Path

Size

lib/

Target/

AArch64/

AArch64TargetMachine.cpp

4 lines

Diff 105106

lib/Target/AArch64/AArch64TargetMachine.cpp

Show First 20 Lines • Show All 271 Lines • ▼ Show 20 Lines	public:
}		}

AArch64TargetMachine &getAArch64TargetMachine() const {		AArch64TargetMachine &getAArch64TargetMachine() const {
return getTM<AArch64TargetMachine>();		return getTM<AArch64TargetMachine>();
}		}

ScheduleDAGInstrs *		ScheduleDAGInstrs *
createMachineScheduler(MachineSchedContext *C) const override {		createMachineScheduler(MachineSchedContext *C) const override {
		const AArch64Subtarget &ST = C->MF->getSubtarget<AArch64Subtarget>();
ScheduleDAGMILive *DAG = createGenericSchedLive(C);		ScheduleDAGMILive *DAG = createGenericSchedLive(C);
DAG->addMutation(createLoadClusterDAGMutation(DAG->TII, DAG->TRI));		DAG->addMutation(createLoadClusterDAGMutation(DAG->TII, DAG->TRI));
DAG->addMutation(createStoreClusterDAGMutation(DAG->TII, DAG->TRI));		DAG->addMutation(createStoreClusterDAGMutation(DAG->TII, DAG->TRI));
		if (ST.hasFusion())
DAG->addMutation(createAArch64MacroFusionDAGMutation());		DAG->addMutation(createAArch64MacroFusionDAGMutation());
return DAG;		return DAG;
}		}

ScheduleDAGInstrs *		ScheduleDAGInstrs *
createPostMachineScheduler(MachineSchedContext *C) const override {		createPostMachineScheduler(MachineSchedContext *C) const override {
const AArch64Subtarget &ST = C->MF->getSubtarget<AArch64Subtarget>();		const AArch64Subtarget &ST = C->MF->getSubtarget<AArch64Subtarget>();
if (ST.hasFusion()) {		if (ST.hasFusion()) {
// Run the Macro Fusion after RA again since literals are expanded from		// Run the Macro Fusion after RA again since literals are expanded from
▲ Show 20 Lines • Show All 203 Lines • Show Last 20 Lines