[llvm] [SandboxIR] Add setOperand() and RAUW,RUWIf,RUOW (PR #98410)

Eric Christopher via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 11 14:20:05 PDT 2024


echristo wrote:

That makes sense. A next patch to add the "delegate for X work" comments
might be good just to make sure that the code is straightforward for
reviews. Thoughts?

On Thu, Jul 11, 2024 at 2:15 PM vporpo ***@***.***> wrote:

> Ah yes, we have not added any of the transaction tracking code yet, so the
> IR will currently delegate everything down to LLVM IR. Once we add the
> tracking code all these functions that modify the IR's state, like RAUW,
> setOperand, moveBefore etc. will include code that tracks the state change.
> The first patches that will add some of this functionality are a couple of
> patches down the road.
>
>> Reply to this email directly, view it on GitHub
> <https://github.com/llvm/llvm-project/pull/98410#issuecomment-2223951410>,
> or unsubscribe
> <https://github.com/notifications/unsubscribe-auth/AACP5DBBSM2MVBY4JYDZGVDZL3YVXAVCNFSM6AAAAABKV2YLJWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMRTHE2TCNBRGA>
> .
> You are receiving this because your review was requested.Message ID:
> ***@***.***>
>


https://github.com/llvm/llvm-project/pull/98410


More information about the llvm-commits mailing list