[PATCH] D137446: [RISCV] Rework isAllUsesReadW in RISCVSExtWRemoval. NFCI

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 4 10:15:24 PDT 2022


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

Instead of storing the uses to check in the worklist, store the
instruction we want to check uses for.

Now we pop and instruction from the worklist, loop over its uses
and check them. If it's something we need to look across, we'll push
it to the worklist.

By doing it this way, we can have access to which operand
of the user is using the instruction. This will allow supporting
store instructions since we'll be able to disambiguate the the value
operand and the pointer operand. We can also improve support for
*add.uw instructions and shift amount uses.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D137446

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

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D137446.473282.patch
Type: text/x-patch
Size: 6707 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20221104/cb6b0fce/attachment.bin>


More information about the llvm-commits mailing list