This is an archive of the discontinued LLVM Phabricator instance.

[CostModel][RISCV] Model code size cost for reduction
ClosedPublic

Authored by arcbbb on Jan 18 2023, 6:07 PM.

Details

Summary

Since code-size cost doesn't scale linearly with LMUL,
this change is to separate it from throughput.

Diff Detail

Event Timeline

arcbbb created this revision.Jan 18 2023, 6:07 PM
Herald added a project: Restricted Project. · View Herald TranscriptJan 18 2023, 6:07 PM
arcbbb requested review of this revision.Jan 18 2023, 6:07 PM
reames accepted this revision.Feb 24 2023, 7:44 AM

LGTM w/change applied.

Sorry for the delay, I missed this when first posted.

llvm/lib/Target/RISCV/RISCVTargetTransformInfo.cpp
1021

This should still be LT.first - 1

1054

Same here

This revision is now accepted and ready to land.Feb 24 2023, 7:44 AM
arcbbb updated this revision to Diff 502493.Mar 5 2023, 5:34 PM
arcbbb updated this revision to Diff 502495.Mar 5 2023, 5:54 PM
  • Update tests
This revision was landed with ongoing or failed builds.Mar 5 2023, 5:59 PM
This revision was automatically updated to reflect the committed changes.