[NFC][RISCV] Move calculations of SDNode policy operand idx to a separate function
ClosedPublic
Actions

Authored by asi-sc on Sep 19 2022, 5:43 AM.

Details

Reviewers

reames
craig.topper
frasercrmck
arcbbb

Commits

rG3cd503f18161: [NFC][RISCV] Move calculations of SDNode policy operand idx to a separate…

Summary

Since there is no guaranteed correspondence of SDNode and MI operands, we need
getters simular to RISCVII::get*OpNum for SDNodes.

More uses of getVecPolicyOpIdx will be added in D130895.

Diff Detail

Repository: rG LLVM Github Monorepo

Event Timeline

asi-sc created this revision.Sep 19 2022, 5:43 AM

Herald added a project: Restricted Project. · View Herald TranscriptSep 19 2022, 5:43 AM

Herald added subscribers: sunshaoce, VincentWu, vkmr and 28 others. · View Herald Transcript

asi-sc requested review of this revision.Sep 19 2022, 5:43 AM

Herald added a project: Restricted Project. · View Herald TranscriptSep 19 2022, 5:43 AM

Herald added subscribers: llvm-commits, pcwang-thead, eopXD, MaskRay. · View Herald Transcript

asi-sc added reviewers: reames, craig.topper.Sep 19 2022, 5:45 AM

Herald added a subscriber: StephenFan. · View Herald TranscriptSep 19 2022, 5:45 AM

asi-sc added reviewers: frasercrmck, arcbbb.Sep 19 2022, 5:48 AM

Harbormaster completed remote builds in B187477: Diff 461191.Sep 19 2022, 6:25 AM

asi-sc mentioned this in D130895: [RISCV] Make VL choosing for a splat-like VMV based on its users.Sep 19 2022, 8:40 AM

LGTM

This revision is now accepted and ready to land.Sep 19 2022, 6:05 PM

LGTM

@arcbbb, could you please submit the patch on my behalf? Author: Anton Sidorenko <anton.sidorenko@syntacore.com>
Thanks.

Closed by commit rG3cd503f18161: [NFC][RISCV] Move calculations of SDNode policy operand idx to a separate… (authored by asi-sc, committed by arcbbb). · Explain WhySep 20 2022, 10:37 AM

This revision was automatically updated to reflect the committed changes.

arcbbb added a commit: rG3cd503f18161: [NFC][RISCV] Move calculations of SDNode policy operand idx to a separate….

Revision Contents
Changeset List

Path

Size

llvm/

lib/

Target/

RISCV/

RISCVISelDAGToDAG.cpp

25 lines

Diff 461612

View Options

llvm/lib/Target/RISCV/RISCVISelDAGToDAG.cpp

[NFC][RISCV] Move calculations of SDNode policy operand idx to a separate functionClosedPublicActions