Please use GitHub pull requests for new patches. Phabricator shutdown timeline

[RISCV] Split scheduler class for integer min/max reduction from other integer reductions. NFC
ClosedPublic
Actions

Authored by craig.topper on Jul 12 2023, 12:31 PM.

Details

Reviewers

michaelmaitland
monkchiang
wangpc

Commits

rGd43ccffdc7f5: [RISCV] Split scheduler class for integer min/max reduction from other integer…

Summary

Our downstream needs to give different scheduling for min/max from
other reductions.

Diff Detail

Repository: rG LLVM Github Monorepo

Event Timeline

craig.topper created this revision.Jul 12 2023, 12:31 PM

Herald added a project: Restricted Project. · View Herald TranscriptJul 12 2023, 12:31 PM

Herald added subscribers: jobnoorman, luke, VincentWu and 28 others. · View Herald Transcript

craig.topper requested review of this revision.Jul 12 2023, 12:31 PM

Herald added a project: Restricted Project. · View Herald TranscriptJul 12 2023, 12:31 PM

Herald added subscribers: eopXD, MaskRay. · View Herald Transcript

LGTM, can you mark patch as NFC though?

This revision is now accepted and ready to land.Jul 12 2023, 12:35 PM

craig.topper retitled this revision from [RISCV] Split scheduler class for integer min/max reduction from other integer reductions. to [RISCV] Split scheduler class for integer min/max reduction from other integer reductions. NFC.Jul 12 2023, 12:49 PM

This revision was landed with ongoing or failed builds.Jul 12 2023, 12:58 PM

Closed by commit rGd43ccffdc7f5: [RISCV] Split scheduler class for integer min/max reduction from other integer… (authored by craig.topper). · Explain Why

This revision was automatically updated to reflect the committed changes.

craig.topper added a commit: rGd43ccffdc7f5: [RISCV] Split scheduler class for integer min/max reduction from other integer….

Harbormaster completed remote builds in B244877: Diff 539681.Jul 12 2023, 3:39 PM

wangpc mentioned this in D159029: [RISCV] Correct scheduling information for WriteVIRedMinMaxV in RISCVSchedSiFive7.td..Aug 28 2023, 7:47 PM