[PATCH] D127761: [RISCV] Split out transfer function explicitly in VSETVLI insertion dataflow [nfc]

Philip Reames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 14 09:29:31 PDT 2022


reames created this revision.
reames added reviewers: craig.topper, frasercrmck, kito-cheng.
Herald added subscribers: sunshaoce, VincentWu, luke957, StephenFan, vkmr, evandro, luismarques, apazos, sameer.abuasal, s.egerton, Jim, benna, psnobl, jocewei, PkmX, the_o, brucehoult, MartinMosbeck, rogfer01, edward-jones, zzheng, jrtc27, shiva0217, niosHD, sabuasal, bollu, simoncook, johnrusso, rbar, asb, hiraditya, arichardson, mcrosier.
Herald added a project: All.
reames requested review of this revision.
Herald added subscribers: pcwang-thead, eopXD, MaskRay.
Herald added a project: LLVM.

In an effort to make this code easier to read and extend, this splits out helper functions for the transfer function of the data flow.  Due to the other results computed during the phases, we can't completely abstract away everything, but we can abstract the actual state transitions.

The motivation here is the following upcoming changes:

- The fault first load patch - already approved, this will be rebased over - adds another case into the transferAfter path.
- An upcoming patch to fold the local prepass back into the main algorithm greatly complicates the transferBefore logic.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D127761

Files:
  llvm/lib/Target/RISCV/RISCVInsertVSETVLI.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D127761.436818.patch
Type: text/x-patch
Size: 6391 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220614/b067e191/attachment.bin>


More information about the llvm-commits mailing list