This is an archive of the discontinued LLVM Phabricator instance.

[ARM][SchedModels] Convert IsPredicatedPred to MCSchedPredicate (alternative approach)
ClosedPublic

Authored by evgeny777 on Oct 16 2020, 7:39 AM.

Diff Detail

Event Timeline

evgeny777 created this revision.Oct 16 2020, 7:39 AM
andreadb accepted this revision.Oct 17 2020, 2:58 AM

LGTM.

The patch is mostly a mechanical change (i.e. a lot of boilerplate to introduce a new scheduling predicate).

@dmgreen has already verified in D89458 that the ARM changes in the mca test are good.

llvm/lib/MC/MCSchedule.cpp
77

not sure if this is more than 80-cols.

123

same.

llvm/tools/llvm-mca/Views/InstructionInfoView.cpp
106–107

80-col

This revision is now accepted and ready to land.Oct 17 2020, 2:58 AM
dmgreen accepted this revision.Oct 17 2020, 6:05 AM

Sounds good to me. Thanks